[libvirt] [PATCH] network: fix crash when starting a network with no <pf> element

[Martin Kletzander]

On Fri, Aug 15, 2014 at 02:50:30AM -0400, Laine Stump wrote:
>On 08/14/2014 05:10 PM, Martin Kletzander wrote:
>> On Thu, Aug 14, 2014 at 12:40:07PM -0400, Laine Stump wrote:
>>>
>>> src/network/bridge_driver.c | 17 +++++++----------
>>> 1 file changed, 7 insertions(+), 10 deletions(-)
>>>
>>> diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
>>> index 1ba4c3d..fc4c73d 100644
>>> --- a/src/network/bridge_driver.c
>>> +++ b/src/network/bridge_driver.c
>>> @@ -2196,6 +2196,9 @@ networkCreateInterfacePool(virNetworkDefPtr
>>> netdef)
>>>     int ret = -1;
>>>     size_t i;
>>>
>>> +    if (!netdef->forward.npfs || netdef->forward.nifs > 0)
>>
>> I'd match these to use both !var (or var != 0), but other than that
>> the patch is fine, ACK with that fixed.
>
>I decided to go with == 0 and > 0. They are closest to what is happening
>in my mind when I think about the condition, so hopefully will be the
>easiest for others to understand.
>
>Thanks!

Thank you for such quick response.  It's still weird though, because
it didn't segfault every start and when it segfaulted, it wasn't on
this line, but in some malloc(), which means we must have screwed up
something pretty bad.  But valgrind suggested there is an invalid read
in here, so let's hope this fixes it :)

Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140815/e22e3e65/attachment-0001.sig>