Bug 1044046 - several null pointer deferences and other issues
Summary: several null pointer deferences and other issues
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: libvirt
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Libvirt Maintainers
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-12-17 17:00 UTC by Rahul Sundaram
Modified: 2013-12-17 17:50 UTC (History)
10 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-12-17 17:49:32 UTC
Type: Bug


Attachments (Terms of Use)

Description Rahul Sundaram 2013-12-17 17:00:45 UTC
Description of problem:

I have been playing around with cppcheck and it seems to indicate several potential issues. please check.  note there that cppcheck is capable of doing a lot more extensive checks but more potential for false warnings.  I have reported similar issues against systemd and other programs and you might consider adding a build target

http://cgit.freedesktop.org/systemd/systemd/commit/?id=16f4efb4150c65e3c61adaa8ea512489de49f532


cppcheck . --std=c99 --quiet 

[daemon/remote.c:5202]: (error) Possible null pointer dereference: files
[daemon/remote.c:5254]: (error) Possible null pointer dereference: files
[src/conf/domain_conf.c:4732]: (error) Possible null pointer dereference: seclabels
[src/interface/interface_backend_udev.c:846]: (error) Possible null pointer dereference: slave_list
[src/interface/interface_backend_udev.c:953]: (error) Possible null pointer dereference: member_list
[src/locking/lock_driver_sanlock.c:1022]: (error) Uninitialized variable: res_count
[src/network/bridge_driver.c:817]: (error) syntax error
[src/nwfilter/nwfilter_ebiptables_driver.c:3962]: (error) Possible null pointer dereference: ebtChains
[src/qemu/qemu_driver.c:970]: (error) Possible null pointer dereference: domains
[src/storage/storage_backend_mpath.c:242]: (error) Uninitialized variable: next

Comment 1 Daniel Berrangé 2013-12-17 17:45:27 UTC
After a quick run through this list I believe these are all false positives, which isn't entirely surprising since we're already using Coverity to check this on a nightly basis.

Comment 2 Rahul Sundaram 2013-12-17 17:50:01 UTC
Thanks for the quick response


Note You need to log in before you can comment on or make changes to this bug.