Re: [libvirt] [PATCH 1/8] conf: Fix leak in virNodeDeviceDefParseXML

2017-02-27 Thread Michal Privoznik
On 20.02.2017 14:18, John Ferlan wrote: > The 'nodes' is overwritten after the first usage and possibly leaked > if any code in the first set of parsing goes to error. > > Found by Coverity. > > Signed-off-by: John Ferlan > --- > src/conf/node_device_conf.c | 10 --

Re: [libvirt] [PATCH 1/8] conf: Fix leak in virNodeDeviceDefParseXML

2017-02-27 Thread Laine Stump
On 02/20/2017 08:18 AM, John Ferlan wrote: The 'nodes' is overwritten after the first usage and possibly leaked if any code in the first set of parsing goes to error. ACK. -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH 1/8] conf: Fix leak in virNodeDeviceDefParseXML

2017-02-20 Thread John Ferlan
The 'nodes' is overwritten after the first usage and possibly leaked if any code in the first set of parsing goes to error. Found by Coverity. Signed-off-by: John Ferlan --- src/conf/node_device_conf.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff