Attachment 587302 Details for Bug 825903 – CoverityScan on libvirt-0.9.10-21.el6.x86_64

CoverityScan on libvirt-0.9.10-21.el6.x86_64

libvirt-0.9.10-21.el6.err (text/plain), 36.28 KB, created by Alex Jia on 2012-05-29 03:13:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Jia

Created: 2012-05-29 03:13:50 UTC

Size: 36.28 KB

patch

obsolete

>
>Error: CHECKED_RETURN:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:104: example_checked: "pciDeviceListAdd(list, activeDev)" has its value checked in "pciDeviceListAdd(list, activeDev) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:61: example_checked: "pciDeviceListAdd(list, dev)" has its value checked in "pciDeviceListAdd(list, dev) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:462: example_checked: "pciDeviceListAdd(driver->activePciHostdevs, dev)" has its value checked in "pciDeviceListAdd(driver->activePciHostdevs, dev) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:151: example_checked: "pciDeviceListAdd(driver->activePciHostdevs, dev)" has its value checked in "pciDeviceListAdd(driver->activePciHostdevs, dev) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/pci.c:1142: example_checked: "pciDeviceListAdd(inactiveDevs, dev)" has its value checked in "pciDeviceListAdd(inactiveDevs, dev) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:728: check_return: Calling function "pciDeviceListAdd" without checking return value (as is done elsewhere 5 out of 6 times).
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:728: unchecked_value: No check of the return value of "pciDeviceListAdd(driver->inactivePciHostdevs, dev)".
>
>Error: CHECKED_RETURN:
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:845: example_checked: "virAsprintf(configfile, "%s/.libvirt/libvirtd.conf", userdir)" has its value checked in "virAsprintf(configfile, "%s/.libvirt/libvirtd.conf", userdir) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:938: example_assign: Assigning: "ret" = return value from "virAsprintf(&data->mdns_name, "Virtualization Host")".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:948: example_checked: "ret" has its value checked in "ret < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:944: example_assign: Assigning: "ret" = return value from "virAsprintf(&data->mdns_name, "Virtualization Host %s", localhost)".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:948: example_checked: "ret" has its value checked in "ret < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:301: example_checked: "virAsprintf(pidfile, "%s/.libvirt/libvirtd.pid", userdir)" has its value checked in "virAsprintf(pidfile, "%s/.libvirt/libvirtd.pid", userdir) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:794: example_checked: "virAsprintf(&tmp, "%d:file:%s/log/libvirt/libvirtd.log", virLogGetDefaultPriority(), "/var")" has its value checked in "virAsprintf(&tmp, "%d:file:%s/log/libvirt/libvirtd.log", virLogGetDefaultPriority(), "/var") == -1".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/network_conf.c:592: check_return: Calling function "virAsprintf" without checking return value (as is done elsewhere 406 out of 427 times).
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/network_conf.c:592: unchecked_value: No check of the return value of "virAsprintf(&name, dcgettext("libvirt", "Service name is too long, limit is %d bytes", 5), 482)".
>
>Error: CHECKED_RETURN:
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:845: example_checked: "virAsprintf(configfile, "%s/.libvirt/libvirtd.conf", userdir)" has its value checked in "virAsprintf(configfile, "%s/.libvirt/libvirtd.conf", userdir) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:938: example_assign: Assigning: "ret" = return value from "virAsprintf(&data->mdns_name, "Virtualization Host")".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:948: example_checked: "ret" has its value checked in "ret < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:944: example_assign: Assigning: "ret" = return value from "virAsprintf(&data->mdns_name, "Virtualization Host %s", localhost)".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:948: example_checked: "ret" has its value checked in "ret < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:301: example_checked: "virAsprintf(pidfile, "%s/.libvirt/libvirtd.pid", userdir)" has its value checked in "virAsprintf(pidfile, "%s/.libvirt/libvirtd.pid", userdir) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:794: example_checked: "virAsprintf(&tmp, "%d:file:%s/log/libvirt/libvirtd.log", virLogGetDefaultPriority(), "/var")" has its value checked in "virAsprintf(&tmp, "%d:file:%s/log/libvirt/libvirtd.log", virLogGetDefaultPriority(), "/var") == -1".
>/builddir/build/BUILD/libvirt-0.9.10/src/nwfilter/nwfilter_ebiptables_driver.c:2891: check_return: Calling function "virAsprintf" without checking return value (as is done elsewhere 406 out of 427 times).
>/builddir/build/BUILD/libvirt-0.9.10/src/nwfilter/nwfilter_ebiptables_driver.c:2891: unchecked_value: No check of the return value of "virAsprintf(&protostr, "-d 01:80:c2:00:00:00 ")".
>
>Error: CHECKED_RETURN:
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:845: example_checked: "virAsprintf(configfile, "%s/.libvirt/libvirtd.conf", userdir)" has its value checked in "virAsprintf(configfile, "%s/.libvirt/libvirtd.conf", userdir) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:938: example_assign: Assigning: "ret" = return value from "virAsprintf(&data->mdns_name, "Virtualization Host")".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:948: example_checked: "ret" has its value checked in "ret < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:944: example_assign: Assigning: "ret" = return value from "virAsprintf(&data->mdns_name, "Virtualization Host %s", localhost)".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:948: example_checked: "ret" has its value checked in "ret < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:301: example_checked: "virAsprintf(pidfile, "%s/.libvirt/libvirtd.pid", userdir)" has its value checked in "virAsprintf(pidfile, "%s/.libvirt/libvirtd.pid", userdir) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:794: example_checked: "virAsprintf(&tmp, "%d:file:%s/log/libvirt/libvirtd.log", virLogGetDefaultPriority(), "/var")" has its value checked in "virAsprintf(&tmp, "%d:file:%s/log/libvirt/libvirtd.log", virLogGetDefaultPriority(), "/var") == -1".
>/builddir/build/BUILD/libvirt-0.9.10/src/nwfilter/nwfilter_ebiptables_driver.c:2894: check_return: Calling function "virAsprintf" without checking return value (as is done elsewhere 406 out of 427 times).
>/builddir/build/BUILD/libvirt-0.9.10/src/nwfilter/nwfilter_ebiptables_driver.c:2894: unchecked_value: No check of the return value of "virAsprintf(&protostr, "-p 0x%04x ", l3_protocols[protoidx].attr)".
>
>Error: CHECKED_RETURN:
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:845: example_checked: "virAsprintf(configfile, "%s/.libvirt/libvirtd.conf", userdir)" has its value checked in "virAsprintf(configfile, "%s/.libvirt/libvirtd.conf", userdir) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:938: example_assign: Assigning: "ret" = return value from "virAsprintf(&data->mdns_name, "Virtualization Host")".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:948: example_checked: "ret" has its value checked in "ret < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:944: example_assign: Assigning: "ret" = return value from "virAsprintf(&data->mdns_name, "Virtualization Host %s", localhost)".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:948: example_checked: "ret" has its value checked in "ret < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:301: example_checked: "virAsprintf(pidfile, "%s/.libvirt/libvirtd.pid", userdir)" has its value checked in "virAsprintf(pidfile, "%s/.libvirt/libvirtd.pid", userdir) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:794: example_checked: "virAsprintf(&tmp, "%d:file:%s/log/libvirt/libvirtd.log", virLogGetDefaultPriority(), "/var")" has its value checked in "virAsprintf(&tmp, "%d:file:%s/log/libvirt/libvirtd.log", virLogGetDefaultPriority(), "/var") == -1".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_process.c:2683: check_return: Calling function "virAsprintf" without checking return value (as is done elsewhere 406 out of 427 times).
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_process.c:2683: unchecked_value: No check of the return value of "virAsprintf(&msg, "was paused (%s)", virDomainPausedReasonTypeToString(reason))".
>
>Error: CHECKED_RETURN:
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:3955: example_checked: "pthread_sigmask(0, &sigmask, &oldsigmask)" has its value checked in "pthread_sigmask(0, &sigmask, &oldsigmask) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:3486: example_checked: "pthread_sigmask(0, &sigmask, &oldsigmask)" has its value checked in "pthread_sigmask(0, &sigmask, &oldsigmask) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:7057: example_checked: "pthread_sigmask(0, &sigmask, &oldsigmask)" has its value checked in "pthread_sigmask(0, &sigmask, &oldsigmask) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:3160: example_checked: "pthread_sigmask(0, &sigmask, &oldsigmask)" has its value checked in "pthread_sigmask(0, &sigmask, &oldsigmask) < 0".
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:7259: check_return: Calling function "pthread_sigmask" without checking return value (as is done elsewhere 4 out of 5 times).
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:7259: unchecked_value: No check of the return value of "pthread_sigmask(0, &sigmask, &oldsigmask)".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/strerror_r.c:153: dead_error_condition: On this path, the condition "msg" cannot be true.
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/strerror_r.c:151: const: After this line, the value of "msg" is equal to 0.
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/strerror_r.c:151: assignment: Assigning: "msg" = "NULL".
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/strerror_r.c:154: dead_error_line: Execution cannot reach this statement "return safe_copy(buf, bufle...".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/strerror.c:45: dead_error_condition: On this path, the condition "msg" cannot be true.
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/strerror.c:44: const: After this line, the value of "msg" is equal to 0.
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/strerror.c:44: assignment: Assigning: "msg" = "NULL".
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/strerror.c:46: dead_error_line: Execution cannot reach this statement "return (char *)msg;".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2849: dead_error_condition: On this path, the switch value "netType" cannot be "VIR_DOMAIN_NET_TYPE_BRIDGE".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 2.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 3.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 4.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2823: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_CLIENT".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2825: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_MCAST".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2824: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_SERVER".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2849: dead_error_line: Execution cannot reach this statement "case VIR_DOMAIN_NET_TYPE_BR...".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2851: dead_error_condition: On this path, the switch value "netType" cannot be "VIR_DOMAIN_NET_TYPE_DIRECT".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 2.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 3.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 4.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2823: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_CLIENT".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2825: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_MCAST".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2824: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_SERVER".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2851: dead_error_line: Execution cannot reach this statement "case VIR_DOMAIN_NET_TYPE_DI...".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2847: dead_error_condition: On this path, the switch value "netType" cannot be "VIR_DOMAIN_NET_TYPE_ETHERNET".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 2.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 3.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 4.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2823: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_CLIENT".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2825: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_MCAST".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2824: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_SERVER".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2847: dead_error_line: Execution cannot reach this statement "case VIR_DOMAIN_NET_TYPE_ET...".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2852: dead_error_condition: On this path, the switch value "netType" cannot be "VIR_DOMAIN_NET_TYPE_HOSTDEV".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 2.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 3.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 4.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2823: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_CLIENT".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2825: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_MCAST".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2824: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_SERVER".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2852: dead_error_line: Execution cannot reach this statement "case VIR_DOMAIN_NET_TYPE_HO...".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2850: dead_error_condition: On this path, the switch value "netType" cannot be "VIR_DOMAIN_NET_TYPE_INTERNAL".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 2.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 3.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 4.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2823: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_CLIENT".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2825: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_MCAST".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2824: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_SERVER".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2850: dead_error_line: Execution cannot reach this statement "case VIR_DOMAIN_NET_TYPE_IN...".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2853: dead_error_condition: On this path, the switch value "netType" cannot be "VIR_DOMAIN_NET_TYPE_LAST".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 2.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 3.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 4.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2823: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_CLIENT".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2825: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_MCAST".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2824: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_SERVER".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2853: dead_error_begin: Execution cannot reach this statement "case VIR_DOMAIN_NET_TYPE_LAST:".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2848: dead_error_condition: On this path, the switch value "netType" cannot be "VIR_DOMAIN_NET_TYPE_NETWORK".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 2.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 3.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 4.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2823: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_CLIENT".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2825: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_MCAST".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2824: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_SERVER".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2848: dead_error_line: Execution cannot reach this statement "case VIR_DOMAIN_NET_TYPE_NE...".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2846: dead_error_condition: On this path, the switch value "netType" cannot be "VIR_DOMAIN_NET_TYPE_USER".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 2.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 3.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2799: const: After this line, the value of "netType" is equal to 4.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2823: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_CLIENT".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2825: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_MCAST".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2824: equality_cond: Jumping to case "VIR_DOMAIN_NET_TYPE_SERVER".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_command.c:2846: dead_error_line: Execution cannot reach this statement "case VIR_DOMAIN_NET_TYPE_USER:".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/rpc/virnetclient.c:332: dead_error_condition: On this path, the condition "ka" cannot be true.
>/builddir/build/BUILD/libvirt-0.9.10/src/rpc/virnetclient.c:274: const: After this line, the value of "ka" is equal to 0.
>/builddir/build/BUILD/libvirt-0.9.10/src/rpc/virnetclient.c:312: const: After this line, the value of "ka" is equal to 0.
>/builddir/build/BUILD/libvirt-0.9.10/src/rpc/virnetclient.c:274: assignment: Assigning: "ka" = "NULL".
>/builddir/build/BUILD/libvirt-0.9.10/src/rpc/virnetclient.c:312: new_values: Noticing condition "ka = virKeepAliveNew(-1, 0U, client, virNetClientKeepAliveSendCB, virNetClientKeepAliveDeadCB, virNetClientEventFree)".
>/builddir/build/BUILD/libvirt-0.9.10/src/rpc/virnetclient.c:333: dead_error_begin: Execution cannot reach this statement "virKeepAliveStop(ka);".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/lxc/lxc_controller.c:960: dead_error_condition: On this path, the condition "ret == 4" cannot be true.
>/builddir/build/BUILD/libvirt-0.9.10/src/lxc/lxc_controller.c:959: at_most: After this line, the value of "ret" is at most -1.
>/builddir/build/BUILD/libvirt-0.9.10/src/lxc/lxc_controller.c:959: new_values: Noticing condition "ret < 0".
>/builddir/build/BUILD/libvirt-0.9.10/src/lxc/lxc_controller.c:961: dead_error_line: Execution cannot reach this statement "continue;".
>
>Error: DEADCODE:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_driver.c:8172: dead_error_condition: On this path, the condition "group" cannot be true.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_driver.c:8067: const: After this line, the value of "group" is equal to 0.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_driver.c:8067: assignment: Assigning: "group" = "NULL".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_driver.c:8173: dead_error_line: Execution cannot reach this statement "virCgroupFree(&group);".
>
>Error: EVALUATION_ORDER:
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:7159: write_write_order: In "disk = disk = def->disks[i]", "disk" is written in "disk" (the assignment left-hand side) and written in "disk = def->disks[i]" but the order in which the side effects take place is undefined because there is no intervening sequence point.
>
>Error: FORWARD_NULL:
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:16728: var_compare_op: Comparing "from" to null implies that "from" might be null.
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:16729: var_deref_model: Passing null variable "from" to function "__coverity_strcmp", which dereferences it.
>
>Error: MISSING_RETURN:
>/tmp/tmpBOs1EA.c:1: missing_return: Arriving at the end of a function without returning a value.
>
>Error: NEGATIVE_RETURNS:
>/builddir/build/BUILD/libvirt-0.9.10/src/util/buf.c:147: negative_return_fn: Function "virBufferGetIndent(buf, true)" returns a negative number.
>/builddir/build/BUILD/libvirt-0.9.10/src/util/buf.c:89: return_negative_constant: Explicitly returning negative value "-1".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/buf.c:147: var_assign: Assigning: signed variable "indent" = "virBufferGetIndent".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/buf.c:157: negative_returns: "indent" is passed to a parameter that cannot be negative.
>
>Error: NEGATIVE_RETURNS:
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:434: var_tested_neg: Assigning: "iterIndex" = a negative value.
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:473: negative_returns: Using variable "iterIndex" as an index to array "res->iter".
>
>Error: NO_EFFECT:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_driver.c:9627: array_null: Comparing an array to null is not useful: ""qcow2"".
>
>Error: NULL_RETURNS:
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_driver.c:9237: example_checked: "pciDeviceListFind(driver->inactivePciHostdevs, pci)" has its value checked in "pciDeviceListFind(driver->inactivePciHostdevs, pci)".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_driver.c:9267: example_assign: Assigning: "other" = return value from "pciDeviceListFind(driver->activePciHostdevs, pci)".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_driver.c:9268: example_checked: "other" has its value checked in "other".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:103: example_checked: "pciDeviceListFind(driver->activePciHostdevs, dev)" has its value checked in "activeDev = pciDeviceListFind(driver->activePciHostdevs, dev)".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:777: example_assign: Assigning: "activeDev" = return value from "pciDeviceListFind(driver->activePciHostdevs, dev)".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:778: example_checked: "activeDev" has its value checked in "activeDev".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:409: example_checked: "pciDeviceListFind(driver->activePciHostdevs, dev)" has its value checked in "other = pciDeviceListFind(driver->activePciHostdevs, dev)".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:481: returned_null: Function "pciDeviceListFind" returns null (checked 12 out of 13 times).
>/builddir/build/BUILD/libvirt-0.9.10/src/util/pci.c:1616: return_null: Explicitly returning NULL.
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:481: var_assigned: Assigning: "activeDev" = null return value from "pciDeviceListFind".
>/builddir/build/BUILD/libvirt-0.9.10/src/qemu/qemu_hostdev.c:483: dereference: Dereferencing a pointer that might be null "activeDev" when calling "pciDeviceSetUsedBy".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/pci.c:1480: deref_parm: Directly dereferencing parameter "dev".
>
>Error: NULL_RETURNS:
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:761: example_assign: Assigning: "obj" = return value from "virHashLookup(doms->objs, uuidstr)".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:762: example_checked: "obj" has its value checked in "obj".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:14403: example_checked: "virHashLookup(paths, path)" has its value checked in "virHashLookup(paths, path)".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:13012: example_checked: "virHashLookup(doms->objs, uuidstr)" has its value checked in "virHashLookup(doms->objs, uuidstr) != NULL".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:13902: example_checked: "virHashLookup(snapshots->objs, def->name)" has its value checked in "virHashLookup(snapshots->objs, def->name) != NULL".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:606: example_checked: "virHashLookup(table->hashTable, name)" has its value checked in "virHashLookup(table->hashTable, name)".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:390: returned_null: Function "virHashLookup" returns null (checked 35 out of 40 times).
>/builddir/build/BUILD/libvirt-0.9.10/src/util/virhash.c:372: return_null: Explicitly returning NULL.
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:390: var_assigned: Assigning: "varValue" = null return value from "virHashLookup".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:392: dereference: Dereferencing a pointer that might be null "varValue" when calling "virNWFilterVarValueGetNthValue".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:157: deref_parm: Directly dereferencing parameter "val".
>
>Error: NULL_RETURNS:
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:761: example_assign: Assigning: "obj" = return value from "virHashLookup(doms->objs, uuidstr)".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:762: example_checked: "obj" has its value checked in "obj".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:14403: example_checked: "virHashLookup(paths, path)" has its value checked in "virHashLookup(paths, path)".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:13012: example_checked: "virHashLookup(doms->objs, uuidstr)" has its value checked in "virHashLookup(doms->objs, uuidstr) != NULL".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/domain_conf.c:13902: example_checked: "virHashLookup(snapshots->objs, def->name)" has its value checked in "virHashLookup(snapshots->objs, def->name) != NULL".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:606: example_checked: "virHashLookup(table->hashTable, name)" has its value checked in "virHashLookup(table->hashTable, name)".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:398: returned_null: Function "virHashLookup" returns null (checked 35 out of 40 times).
>/builddir/build/BUILD/libvirt-0.9.10/src/util/virhash.c:372: return_null: Explicitly returning NULL.
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:398: var_assigned: Assigning: "tmp" = null return value from "virHashLookup".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:399: dereference: Dereferencing a pointer that might be null "tmp" when calling "virNWFilterVarValueGetNthValue".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:157: deref_parm: Directly dereferencing parameter "val".
>
>Error: OVERRUN_STATIC:
>/builddir/build/BUILD/libvirt-0.9.10/src/locking/lock_driver_sanlock.c:158: overrun-buffer-arg: Overrunning static array ""__LIBVIRT__DISKS__"" of size 19 bytes by passing it to a function which indexes it with argument "48UL" at byte position 47.
>
>Error: RESOURCE_LEAK:
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_conf.c:363: alloc_fn: Calling allocation function "virNWFilterVarAccessParse".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:930: alloc_arg: "virAlloc" allocates memory that is stored into "dest".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:101: alloc_fn: Storage is returned from allocation function "calloc".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:101: var_assign: Assigning: "*((void **)ptrptr)" = "calloc(1UL, size)".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:946: return_alloc: Returning allocated memory "dest".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_conf.c:363: var_assign: Assigning: "varAccess" =  storage returned from "virNWFilterVarAccessParse(var)".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_conf.c:369: noescape: Variable "varAccess" is not freed or pointed-to in function "virNWFilterVarAccessEqual".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_params.c:897:57: noescape: "virNWFilterVarAccessEqual" does not free or save its pointer parameter "b".
>/builddir/build/BUILD/libvirt-0.9.10/src/conf/nwfilter_conf.c:378: leaked_storage: Variable "varAccess" going out of scope leaks the storage it points to.
>
>Error: RESOURCE_LEAK:
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:1893: alloc_arg: Calling allocation function "esxVI_ObjectSpec_Alloc" on "objectSpec".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi_types.generated.c:2065: alloc_arg: "esxVI_Alloc" allocates memory that is stored into "*ptrptr".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:1626: alloc_arg: "virAllocN" allocates memory that is stored into "*ptrptr".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:129: alloc_fn: Storage is returned from allocation function "calloc".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:129: var_assign: Assigning: "*((void **)ptrptr)" = "calloc(count, size)".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:2006: leaked_storage: Variable "objectSpec" going out of scope leaks the storage it points to.
>
>Error: RESOURCE_LEAK:
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:1945: alloc_arg: Calling allocation function "esxVI_PropertySpec_Alloc" on "propertySpec".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi_types.generated.c:2693: alloc_arg: "esxVI_Alloc" allocates memory that is stored into "*ptrptr".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:1626: alloc_arg: "virAllocN" allocates memory that is stored into "*ptrptr".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:129: alloc_fn: Storage is returned from allocation function "calloc".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:129: var_assign: Assigning: "*((void **)ptrptr)" = "calloc(count, size)".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:2006: leaked_storage: Variable "propertySpec" going out of scope leaks the storage it points to.
>
>Error: RESOURCE_LEAK:
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:3913: alloc_arg: Calling allocation function "esxVI_ObjectSpec_Alloc" on "objectSpec".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi_types.generated.c:2065: alloc_arg: "esxVI_Alloc" allocates memory that is stored into "*ptrptr".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:1626: alloc_arg: "virAllocN" allocates memory that is stored into "*ptrptr".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:129: alloc_fn: Storage is returned from allocation function "calloc".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:129: var_assign: Assigning: "*((void **)ptrptr)" = "calloc(count, size)".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:4075: leaked_storage: Variable "objectSpec" going out of scope leaks the storage it points to.
>
>Error: RESOURCE_LEAK:
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:3920: alloc_arg: Calling allocation function "esxVI_PropertySpec_Alloc" on "propertySpec".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi_types.generated.c:2693: alloc_arg: "esxVI_Alloc" allocates memory that is stored into "*ptrptr".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:1626: alloc_arg: "virAllocN" allocates memory that is stored into "*ptrptr".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:129: alloc_fn: Storage is returned from allocation function "calloc".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/memory.c:129: var_assign: Assigning: "*((void **)ptrptr)" = "calloc(count, size)".
>/builddir/build/BUILD/libvirt-0.9.10/src/esx/esx_vi.c:4075: leaked_storage: Variable "propertySpec" going out of scope leaks the storage it points to.
>
>Error: RESOURCE_LEAK:
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:201: open_fn: Calling opening function "open".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:201: var_assign: Assigning: "stdinfd" =  handle returned from "open("/dev/null", 0)".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:205: noescape: Variable "stdinfd" is not closed or saved in function "dup2".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:222: leaked_handle: Handle variable "stdinfd" going out of scope leaks the handle.
>
>Error: RESOURCE_LEAK:
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:203: open_fn: Calling opening function "open".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:203: var_assign: Assigning: "stdoutfd" =  handle returned from "open("/dev/null", 1)".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:207: noescape: Variable "stdoutfd" is not closed or saved in function "dup2".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:209: noescape: Variable "stdoutfd" is not closed or saved in function "dup2".
>/builddir/build/BUILD/libvirt-0.9.10/daemon/libvirtd.c:222: leaked_handle: Handle variable "stdoutfd" going out of scope leaks the handle.
>
>Error: RETURN_LOCAL:
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/careadlinkat.c:99: local_ptr_assign_local: Assigning: "buffer" = "stack_buf" (address of local variable "stack_buf").
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/careadlinkat.c:103: local_ptr_assign_ptr: Assigning: "buf" = "buffer".
>/builddir/build/BUILD/libvirt-0.9.10/gnulib/lib/careadlinkat.c:150: return_local_addr_alias: Returning pointer "buf" which points to local variable "stack_buf".
>
>Error: SIZEOF_MISMATCH:
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:5316: suspicious_sizeof: Passing argument "8UL /* sizeof (cpumap) */" to function "_vshCalloc" and then casting the return value to "unsigned char *" is suspicious.
>
>Error: UNINIT:
>/builddir/build/BUILD/libvirt-0.9.10/src/lxc/lxc_driver.c:1395: var_decl: Declaring variable "fd" without initializer.
>/builddir/build/BUILD/libvirt-0.9.10/src/lxc/lxc_driver.c:1443: uninit_use_in_call: Using uninitialized value "fd" when calling "virFileClose".
>/builddir/build/BUILD/libvirt-0.9.10/src/util/virfile.c:50: read_parm: Reading a parameter value.
>
>Error: UNINIT:
>/builddir/build/BUILD/libvirt-0.9.10/src/lxc/lxc_controller.c:1104: var_decl: Declaring variable "consoles" without initializer.
>/builddir/build/BUILD/libvirt-0.9.10/src/lxc/lxc_controller.c:1237: uninit_use: Using uninitialized value "consoles".
>
>Error: UNINIT:
>/builddir/build/BUILD/libvirt-0.9.10/src/rpc/virnetserver.c:283: var_decl: Declaring variable "sig_action" without initializer.
>/builddir/build/BUILD/libvirt-0.9.10/src/rpc/virnetserver.c:297: uninit_use_in_call: Using uninitialized value "sig_action.sa_flags" when calling "sigaction".
>/builddir/build/BUILD/libvirt-0.9.10/src/rpc/virnetserver.c:297: uninit_use_in_call: Using uninitialized value "sig_action.sa_mask": field "sig_action.sa_mask".__val is uninitialized when calling "sigaction".
>
>Error: UNUSED_VALUE:
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:9611: returned_pointer: Pointer "br_node" returned by "virXPathNode("./bridge", ctxt)" is never used.
>
>Error: UNUSED_VALUE:
>/builddir/build/BUILD/libvirt-0.9.10/tools/virsh.c:9616: returned_pointer: Pointer "if_node" returned by "virXPathNode("./bridge/interface[2]", ctxt)" is never used.

Actions: View

Attachments on bug 825903: 587302 | 602408