[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]