Re: [PATCH v4 0/3] Forbid new-line char in name of networks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

-- 
Best regards / Pozdrawiam
Sławek Kapłoński
slawek@xxxxxxxxxxxx

On Thu, 20 Oct 2016, Michal Privoznik wrote:

> On 20.10.2016 04:57, Sławek Kapłoński wrote:
> > v3: http://www.redhat.com/archives/libvir-list/2016-October/msg00627.html
> > 
> > Differences in v4:
> > * function to check string moved from src/util/virstring to src/util/virxml
> > 
> > Sławek Kapłoński (3):
> >   util: Add function to check if string contains some illegal chars
> >   Use new util function to check network name
> >   Forbid new-line char in name of new networks
> > 
> >  src/conf/network_conf.c     |  5 +----
> >  src/libvirt_private.syms    |  1 +
> >  src/network/bridge_driver.c |  3 +++
> >  src/util/virxml.c           | 28 ++++++++++++++++++++++++++++
> >  src/util/virxml.h           |  3 +++
> >  5 files changed, 36 insertions(+), 4 deletions(-)
> > 
> 
> I've fixed all the small nits I've found and raised in the review and
> pushed this.
> 
> Congratulations on your first libvirt contribution!

Thanks a lot. I hope it will be not the last one :)

> 
> Michal
> 
> --
> libvir-list mailing list
> libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]