[libvirt] [PATCH] umlAutostartDomain: avoid NULL-deref upon virGetLastError failure

Jim Meyering jim at meyering.net
Mon May 17 16:53:38 UTC 2010


Eric Blake wrote:
> On 05/17/2010 09:09 AM, Jim Meyering wrote:
>>> That looks awkward to end with a trailing space.  Might it be better to use:
>>>
>>>  err ? err->message : _("unknown reason")
>>>
>>> At any rate, ACK that this fixes a potential NULL deref.
>>
>> That's a good point.

However, considering there are 20 others just like it,
and in the spirit of keeping this fix minimal, I've pushed it as-is.

>> I considered going one step further and changing
>> virGetLastError so that it never returns NULL,
>> instead returning a pointer to a static buffer where
>> err->message is something like "internal failure".
...




More information about the libvir-list mailing list