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

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

 



On 14.10.2016 04:53, Sławek Kapłoński wrote:
> v2: http://www.redhat.com/archives/libvir-list/2016-October/msg00451.html
> 
> Differences in v3:
> * function to check string moved from src/util/virxml to src/util/virstring
> * validation if name of network contains \n char moved from parsing XML to
>   functions responsible for create/define new networks
> 
> Sławek Kapłoński (2):
>   util: Add function to check if string contains some chars
>   Forbid new-line char in name of new networks
> 
>  src/conf/network_conf.c     | 2 +-
>  src/libvirt_private.syms    | 1 +
>  src/network/bridge_driver.c | 6 ++++++
>  src/util/virstring.c        | 9 +++++++++
>  src/util/virstring.h        | 1 +
>  5 files changed, 18 insertions(+), 1 deletion(-)
> 

Looks more like it. I mean, we are nearly there. See my comments to each
patch.

Michal

--
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]