From f768b4c3e91d8153c42d6e96ebf6f9254eb81b4e Mon Sep 17 00:00:00 2001 From: Eric Blake Date: Tue, 2 Aug 2011 16:45:23 -0600 Subject: [PATCH] qemu: silence coverity false positives Coverity gets confused by our logic. Add some hints to silence false positives. * src/qemu/qemu_driver.c (qemudDomainGetVcpuPinInfo): Add hint. (qemuDomainGetMemoryParameters): Likewise. --- src/qemu/qemu_driver.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 2044e23f8f..ce19be78e9 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -3432,6 +3432,9 @@ qemudDomainGetVcpuPinInfo(virDomainPtr dom, goto cleanup; } + /* Coverity didn't realize that targetDef must be set if we got here. */ + sa_assert(targetDef); + if (nodeGetInfo(dom->conn, &nodeinfo) < 0) goto cleanup; hostcpus = VIR_NODEINFO_MAXCPUS(nodeinfo); @@ -6139,6 +6142,9 @@ static int qemuDomainGetMemoryParameters(virDomainPtr dom, param->value.ul = 0; param->type = VIR_TYPED_PARAM_ULLONG; + /* Coverity does not realize that if we get here, group is set. */ + sa_assert(group); + switch (i) { case 0: /* fill memory hard limit here */ rc = virCgroupGetMemoryHardLimit(group, &val);