Automated Code Analysis

cppcheck analysis of xen-unstable_3.3-unstable+hg17961-1.dsc

• ./extras/mini-os/lib/math.c:167 [error] - Division by zero
• ./tools/blktap/drivers/blktapctrl.c:804 [error] - Resource leak: fd
• ./tools/blktap/drivers/block-qcow.c:1179 [possible error] - Resource leak: fd
• ./tools/blktap/drivers/block-qcow2.c:698 [error] - Memory leak: new_l1_table
• ./tools/blktap/drivers/tapdisk.c:605 [error] - Null pointer dereference
• ./tools/console/daemon/utils.c:107 [error] - Resource leak: fd
• ./tools/firmware/extboot/signrom.c:43 [error] - Resource leak: fin
• ./tools/firmware/extboot/signrom.c:43 [error] - Resource leak: fout
• ./tools/firmware/rombios/rombios.c:8998 [error] - Invalid number of character (() when these macros are defined: ''.
• ./tools/firmware/rombios/rombios.c:8998 [error] - Invalid number of character (() when these macros are defined: 'GCC_PROTOS'.
• ./tools/firmware/rombios/rombios.c:8998 [error] - Invalid number of character (() when these macros are defined: 'HVMTEST'.
• ./tools/firmware/rombios/rombios.c:8998 [error] - Invalid number of character (() when these macros are defined: 'NO_PIO32'.
• ./tools/firmware/rombios/rombios.c:8998 [error] - Invalid number of character (() when these macros are defined: 'PLEX86'.
• ./tools/ioemu/block-vvfat.c:723 [error] - Resource leak: dir
• ./tools/ioemu/block-vvfat.c:2215 [error] - Memory leak: cluster
• ./tools/ioemu/dyngen.c:814 [error] - Resource leak: fd
• ./tools/ioemu/hw/fmopl.c:601 [error] - Buffer access out-of-bounds: OPL.AR_TABLE
• ./tools/ioemu/hw/fmopl.c:602 [error] - Buffer access out-of-bounds: OPL.DR_TABLE
• ./tools/ioemu/hw/piix4acpi.c:482 [error] - Uninitialized variable: ret_str
• ./tools/ioemu/loader.c:228 [error] - Resource leak: fd
• ./tools/ioemu/tests/linux-test.c:434 [error] - Memory leak: stack1
• ./tools/ioemu/tests/linux-test.c:434 [error] - Memory leak: stack2
• ./tools/libfsimage/zfs/fsys_zfs.c:1131 [error] - Undefined behaviour: bootpath is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./tools/libxc/xc_domain_save.c:667 [error] - Uninitialized variable: i
• ./tools/libxc/xc_linux.c:190 [possible error] - Dangerous usage of strncat. Tip: the 3rd parameter means maximum number of characters to append
• ./tools/libxc/xc_linux.c:191 [possible error] - Dangerous usage of strncat. Tip: the 3rd parameter means maximum number of characters to append
• ./tools/python/xen/lowlevel/flask/flask.c:58 [error] - Memory leak: buf
• ./tools/security/secpol_tool.c:329 [error] - Resource leak: fd
• ./tools/vtpm_manager/manager/securestorage.c:219 [error] - Resource leak: fh
• ./tools/vtpm_manager/manager/securestorage.c:293 [error] - Resource leak: fh
• ./tools/vtpm_manager/manager/vtsp.c:399 [error] - Uninitialized variable: paramText
• ./tools/vtpm_manager/tcs/contextmgr.c:65 [error] - Memory leak: pBlock
• ./tools/xcutils/readnotes.c:148 [error] - Resource leak: fd
• ./tools/xenstore/xenstored_core.c:1691 [error] - Resource leak: fd
• ./tools/xenstore/xenstored_domain.c:299 [error] - ### Internal error in Cppcheck. Please report it.
• ./tools/xenstore/xenstored_transaction.c:273 [error] - ### Internal error in Cppcheck. Please report it.
• ./tools/xenstore/xenstored_watch.c:203 [error] - ### Internal error in Cppcheck. Please report it.
• ./xen/arch/ia64/linux-xen/process-linux-xen.c:607 [error] - Uninitialized variable: cfm
• ./xen/arch/ia64/linux-xen/sn/kernel/io_init.c:777 [error] - Undefined behaviour: address is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./xen/arch/ia64/tools/privify/privify_elf64.c:48 [error] - Resource leak: from
• ./xen/arch/ia64/tools/privify/privify_elf64.c:61 [error] - Resource leak: to
• ./xen/arch/ia64/vmx/vmx_fault.c:355 [error] - Uninitialized variable: type
• ./xen/arch/x86/oprofile/nmi_int.c:272 [error] - Invalid number of character (() when these macros are defined: ''.
• ./xen/common/gdbstub.c:252 [error] - Uninitialized variable: i

---