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

Michal Privoznik mprivozn at redhat.com
Tue Sep 23 11:12:24 UTC 2014


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(-)

diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index d470b1b..65864d2 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -6594,7 +6594,7 @@ qemuBuildNumaArgStr(virQEMUDriverConfigPtr cfg,
             continue;
         }
 
-        if (def->cpu && def->cpu->ncells) {
+        if (def->cpu->ncells) {
             /* Fortunately, we allow only guest NUMA nodes to be continuous
              * starting from zero. */
             pos = def->cpu->ncells - 1;
-- 
1.8.5.5




More information about the libvir-list mailing list