On 08/27/2014 09:51 AM, John Ferlan wrote:
After installing a new version of Coverity - my daily checker has
found
many more (>140) "new" issues. Figured I'd start tackling the list bit
by bit. I didn't dig into when any of these were introduced - with so
many on the list it'd be overwhelming :-)
John Ferlan (10):
parallels: Resolve Coverity USE_AFTER_FREE
xen_common: Resolve Coverity USE_AFTER_FREE
xen_xm: Resolve Coverity USE_AFTER_FREE
storage_driver: Resolve Coverity REVERSE_INULL
domain_conf: Resolve Coverity REVERSE_INULL
qemu_command: Resolve Coverity REVERSE_INULL
domain_conf: Resolve Coverity REVERSE_INULL
qemu_driver: Resolve Coverity CONSTANT_EXPRESSION_RESULT
libxl_migration: Resolve Coverity NULL_RETURNS
qemu_capabilities: Resolve Coverity NULL_RETURNS
src/conf/domain_conf.c | 18 +++++++++---------
src/libxl/libxl_migration.c | 1 -
src/parallels/parallels_network.c | 4 +---
src/qemu/qemu_capabilities.c | 5 ++++-
src/qemu/qemu_command.c | 2 +-
src/qemu/qemu_driver.c | 4 ++--
src/storage/storage_driver.c | 6 ++++--
src/xenconfig/xen_common.c | 5 ++---
src/xenconfig/xen_xm.c | 1 +
9 files changed, 24 insertions(+), 22 deletions(-)
I pushed 1-8 & 10 - will await more feedback on 9. Besides there are
another 140 go to!
Thanks -
John