Bug 759971

Summary: Coverity scan revealed defects
Product: Red Hat Enterprise Linux 7 Reporter: Alex Jia <ajia>
Component: libvirtAssignee: Gunannan Ren <gren>
Status: CLOSED UPSTREAM QA Contact: Virtualization Bugs <virt-bugs>
Severity: high Docs Contact:
Priority: high    
Version: 7.0CC: dallan, dyuan, mzhan, rwu
Target Milestone: rc   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-10-29 15:28:41 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
CoverityScan-libvirt-0.9.7-3.el7
none
CoverityScan-libvirt-0.9.11-1.el7
none
CoverityScan-libvirt-0.9.11.3-1.el7 none

Description Alex Jia 2011-12-05 03:11:16 UTC
Description of problem:
Coverity found some issues by defecting libvirt-0.9.7-3.el7.src.rpm,
for details, please see attachment.


Version-Release number of selected component (if applicable):
libvirt-0.9.7-3.el7.src.rpm

How reproducible:
always

Steps to Reproduce:
1. run coverity scan

  
Actual results:

Analysis summary report:
------------------------
Files analyzed                 : 215
Total LoC input to cov-analyze : 358212
Functions analyzed             : 6277
Paths analyzed                 : 482080
New defects found              : 43 Total
                                  3 CHECKED_RETURN
                                  4 DEADCODE
                                  2 FORWARD_NULL
                                  2 MISSING_BREAK
                                  1 MISSING_RETURN
                                  1 NEGATIVE_RETURNS
                                  2 NULL_RETURNS
                                 12 RESOURCE_LEAK
                                  1 RETURN_LOCAL
                                 14 REVERSE_INULL
                                  1 SIGN_EXTENSION


Expected results:
fixed it.

Additional info:

Comment 1 Alex Jia 2011-12-05 03:12:41 UTC
Created attachment 540616 [details]
CoverityScan-libvirt-0.9.7-3.el7

Comment 2 Alex Jia 2012-04-12 10:57:26 UTC
Coverity on libvirt-0.9.11-1.el7:

Analysis summary report:
------------------------
Files analyzed                 : 230
Total LoC input to cov-analyze : 382347
Functions analyzed             : 6419
Paths analyzed                 : 472339
Defect occurrences found       : 67 Total
                                  8 CHECKED_RETURN
                                  5 DEADCODE
                                  2 FORWARD_NULL
                                  2 MISSING_BREAK
                                  1 MISSING_RETURN
                                  2 NEGATIVE_RETURNS
                                  4 NULL_RETURNS
                                  1 OVERRUN_STATIC
                                 22 RESOURCE_LEAK
                                  1 RETURN_LOCAL
                                 14 REVERSE_INULL
                                  1 SIGN_EXTENSION
                                  4 UNINIT

Comment 3 Alex Jia 2012-04-12 10:58:05 UTC
Created attachment 577033 [details]
CoverityScan-libvirt-0.9.11-1.el7

Comment 4 Alex Jia 2012-05-17 02:42:14 UTC
Coverity on libvirt-0.9.11.3-1.el7:

Analysis summary report:
------------------------
Files analyzed                 : 230
Total LoC input to cov-analyze : 383301
Functions analyzed             : 6421
Paths analyzed                 : 472831
Defect occurrences found       : 57 Total
                                  6 CHECKED_RETURN
                                  5 DEADCODE
                                  1 FORWARD_NULL
                                  2 MISSING_BREAK
                                  1 MISSING_RETURN
                                  1 NEGATIVE_RETURNS
                                  1 NULL_RETURNS
                                  1 OVERRUN_STATIC
                                 20 RESOURCE_LEAK
                                  1 RETURN_LOCAL
                                 14 REVERSE_INULL
                                  1 SIGN_EXTENSION
                                  3 UNINIT

Comment 5 Alex Jia 2012-05-17 02:43:09 UTC
Created attachment 585082 [details]
CoverityScan-libvirt-0.9.11.3-1.el7

Comment 6 Dave Allan 2012-10-29 15:28:41 UTC
Closing as upstream as the release against which this BZ was reported is no longer the current release; we're working on getting quiet coverity runs so that any future commits that cause coverity noise can be noticed immediately.