[libvirt] [PATCH 2/6] virDomainFormatSchedDef: Handle case when func returns NULL
Michal Privoznik
mprivozn at redhat.com
Tue May 31 11:05:16 UTC 2016
On 31.05.2016 12:52, Peter Krempa wrote:
> On Tue, May 31, 2016 at 12:33:26 +0200, Michal Privoznik wrote:
>> As unlikely as it might seem, func passed to this function can
>> return NULL. And in some cases it indeed does so:
>> virDomainDefGetVcpuSched and virDomainDefGetIOThreadSched.
>> However, the function I'm fixing blindly dereference value func
>> returned thus leading to SIGSEGV.
>
> Same as in the coverity case complaining about this it won't happen here
> since we are iterating a known list of items that was collected
> beforehand and thus no crashing will happen.
>
Well, having covery reports a false positive is one thing, being unable
to compile is another one. This is what I was afraid of and expressed it
here:
https://www.redhat.com/archives/libvir-list/2016-May/msg02063.html
On the other hand, you use more recent compiler than I do, so you'll hit
these error sooner O:-) IOW, one day we will have to deal with this. Again.
Michal
More information about the libvir-list
mailing list