[libvirt] [PATCH] qemuBuildNumaArgStr: Discard def->cpu check

John Ferlan jferlan at redhat.com
Tue Sep 23 11:21:06 UTC 2014



On 09/23/2014 07:12 AM, Michal Privoznik wrote:
> In the function at one place we check if def->cpu is NULL prior
> to accessing def->cpu->ncells. Then, later in the code,
> def->cpu->ncells is accessed directly, without the check. This
> makes coverity unhappy, because the first check makes it think
> def->cpu can be NULL. However, the function is not called if
> def->cpu is NULL. Therefore, remove the first check and hopefully
> make coverity cheer again.
> 
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
>  src/qemu/qemu_command.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

It did... ACK

Tks -

John




More information about the libvir-list mailing list