[libvirt] [PATCH V2] Fix parsing of bond interface XML
Jim Fehlig
jfehlig at suse.com
Fri Mar 22 15:23:11 UTC 2013
Jim Fehlig wrote:
> Eric Blake wrote:
>
>> On 03/21/2013 05:13 PM, Jim Fehlig wrote:
>>
>>
>>> Noticed that parsing bond interface XML containing the miimon element
>>> fails
>>>
>>> <interface type="bond" name="bond0">
>>> ...
>>> <bond mode="active-backup">
>>> <miimon freq="100" carrier="netif"/>
>>> ...
>>> </bond>
>>> </interface>
>>>
>>> This configuration does not contain the optional updelay and downdelay
>>> attributes, but parsing will fail due to returning the result of
>>> virXPathULong (a -1 when the attribute doesn't exist) from
>>> virInterfaceDefParseBond after examining the updelay attribute.
>>>
>>> While fixing this bug, cleanup the function to use virXPathInt instead
>>> of virXPathULong, and store the result directly instead of using a tmp
>>> variable. Using virXPathInt actually fixes a potential silent
>>> truncation bug noted by Eric Blake.
>>>
>>> Also, there is no cleaup in the error label. Remove the label,
>>>
>>>
>> s/cleaup/cleanup/
>>
>>
>
> Will fix.
>
>
>>
>>
>>> returning failure where failure occurs and success if the end of the
>>> function is reached.
>>> ---
>>>
>>>
>>>
>> ACK
>>
>>
>
> But I'm going to wait to push this until I test it! After fiddling with
> the network too much on my test machine, I can no longer reach it. I'll
> have access to that machine tomorrow and will push this after testing.
>
Tested now and pushed.
Thanks,
Jim
More information about the libvir-list
mailing list