[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