[libvirt] [PATCH] small fixes in storage_conf.c

Jim Meyering jim at meyering.net
Mon Mar 2 09:13:11 UTC 2009


Ryota Ozaki wrote:
> These patches fix:
> - missing VIR_FREE for the return value of virXPathString
> - wrong error message "missing owner element" where should be "missing
> group element"

Thank you!
Both of those are fine.
I'll apply them today.

FYI, looking at the first one, I spotted another problem.
Since strtol dereferences its 2nd parameter when it's non-NULL,
it must always be initialized.

I'll post that fix separately.




More information about the libvir-list mailing list