Bug 959887 - clang static src analysis of glusterfs
Summary: clang static src analysis of glusterfs
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: GlusterFS
Classification: Community
Component: build
Version: mainline
Hardware: i686
OS: Linux
high
medium
Target Milestone: ---
Assignee: raghav
QA Contact:
URL: http://clang-scans.rhcloud.com/gluste...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-05-06 07:59 UTC by John Smith
Modified: 2014-06-18 07:50 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-06-30 15:33:48 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)
the browseable html report of the clang analysis on glusterfs (54.96 MB, application/x-gzip)
2013-05-06 07:59 UTC, John Smith
no flags Details
screen shot of important errors (92.72 KB, image/png)
2013-05-28 06:12 UTC, Amar Tumballi
no flags Details

Description John Smith 2013-05-06 07:59:42 UTC
Created attachment 744009 [details]
the browseable html report of the clang analysis on glusterfs

I ran the the llvm/clang static source code analyzer on glusterfs (git master 'git://forge.gluster.org/glusterfs-core/glusterfs.git') (git commit: Thu May 2, 2013). The attachment is the complete browseable html report.

Comment 1 John Smith 2013-05-12 13:40:36 UTC
Updated the 'url' to point to the browseable html code coverage report.

Comment 2 John Smith 2013-05-24 19:17:38 UTC
Updated the 'url'

Comment 3 Amar Tumballi 2013-05-28 06:12:48 UTC
Created attachment 753754 [details]
screen shot of important errors

Only the errors ticked in the box shown above needs to be addressed, and out of that, we can't ideally make changes to 'contrib/' directory, so consider making changes to all other files.

Comment 4 Vijay Bellur 2013-05-28 12:14:36 UTC
All the security errors are coming in from contrib and they have no bearing in glusterfs context. Hence we can ignore the security warnings in this clang report.

Comment 5 Justin Clift 2013-05-29 00:54:10 UTC
Out of curiosity, the contrib/ code errors seem to be in the fuse stuff we're carrying.

Does anything get compiled from that contrib/ code then distributed by us?


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