Re: [libvirt] [PATCH] network: Resolve some issues around vlan copying

2013-01-17 Thread John Ferlan
On 01/16/2013 12:55 PM, Laine Stump wrote: From: John Ferlan jfer...@redhat.com Remove extraneous check for 'netdef' when dereferencing for vlan.nTags. Prior code would already check if netdef was NULL. Coverity complained about a path where the 'vlan' was potentially valid, but a prior

Re: [libvirt] [PATCH] network: Resolve some issues around vlan copying

2013-01-17 Thread Laine Stump
On 01/17/2013 11:05 AM, John Ferlan wrote: On 01/16/2013 12:55 PM, Laine Stump wrote: From: John Ferlan jfer...@redhat.com Remove extraneous check for 'netdef' when dereferencing for vlan.nTags. Prior code would already check if netdef was NULL. Coverity complained about a path where the

[libvirt] [PATCH] network: Resolve some issues around vlan copying

2013-01-16 Thread Laine Stump
From: John Ferlan jfer...@redhat.com Remove extraneous check for 'netdef' when dereferencing for vlan.nTags. Prior code would already check if netdef was NULL. Coverity complained about a path where the 'vlan' was potentially valid, but a prior checks may not have allocated