[PATCH 04/25] util: validate return from xmlNodeGetContent before use

Laine Stump laine at redhat.com
Fri Jun 26 23:14:19 UTC 2020


On 6/25/20 6:55 PM, Ján Tomko wrote:
> On a Wednesday in 2020, Laine Stump wrote:
>> There were a few uses of xmlNodeGetContent() that didn't check for
>> NULL before using the result.
>>
>> A NULL return from xmlNodeGetContent() *could* (probably does) mean
>> that there was an Out of Memory condition, but it is unclear from the
>> documentation if that is always the case, or if it could just indicate
>> a missing value in the document, so we don't report an OOM error, but
>> just don't try to use it for, e.g., conversion to an integer.
>
> Is it possible to have an element with "no value"?


I never found anywhere that said "No". But I also never found anywhere 
that says "yes", so I opted for "do no harm" (or something like that).

>
> Even <wwn/> gives me an empty string instead of NULL.


Okay, *that* says "No". So I'll change the patch to always report an OOM 
error.


>
> Jano
>
>>
>> Signed-off-by: Laine Stump <laine at redhat.com>
>> ---
>> src/conf/domain_conf.c | 28 ++++++++++++++--------------
>> 1 file changed, 14 insertions(+), 14 deletions(-)





More information about the libvir-list mailing list