1
0
mirror of https://github.com/gluster/glusterdocs.git synced 2026-02-06 00:48:24 +01:00
Files
glusterdocs/Developer-guide/Fixing-issues-reported-by-tools-for-static-code-analysis.md
Humble Devassy Chirammal e892c9c863 Merge pull request #152 from mscherer/fix_cli
Fix command line (spurious parenthesis)
2016-09-18 18:22:41 +05:30

67 lines
2.3 KiB
Markdown
Raw Permalink Blame History

This file contains invisible Unicode characters
This file contains invisible Unicode characters that are indistinguishable to humans but may be processed differently by a computer. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.
Static Code Analysis Tools
--------------------------
Bug fixes for issues reported by *Static Code Analysis Tools* should
follow [Development Work Flow](./Development-Workflow.md)
### Coverity
GlusterFS is part of [Coverity's](https://scan.coverity.com/) scan
program.
- To see Coverity issues you have to be a member of the GlusterFS
project in Coverity scan website.
- Here is the link to [Coverity scan
website](https://scan.coverity.com/projects/987)
- Go to above link and subscribe to GlusterFS project (as
contributor). It will send a request to Admin for including you in
the Project.
- Once admins for the GlusterFS Coverity scan approve your request,
you will be able to see the defects raised by Coverity.
- [BZ 789278](https://bugzilla.redhat.com/show_bug.cgi?id=789278)
should be used as a umbrella bug for Coverity issues in master
branch unless you are trying to fix a specific bug in Bugzilla.
- While sending patches for fixing Coverity issues please use the
same bug number.
- For 3.6 branch the Coverity tracking bug is
[1122834](https://bugzilla.redhat.com/show_bug.cgi?id=1122834)
- When you decide to work on some issue, please assign it to your name
in the same Coverity website. So that we don't step on each others
work.
- When marking a bug intentional in Coverity scan website, please put
an explanation for the same. So that it will help others to
understand the reasoning behind it.
*If you have more questions please send it to
[gluster-devel](http://www.gluster.org/interact/mailinglists) mailing
list*
### CPP Check
Cppcheck is available in Fedora and EL's EPEL repo
- Install Cppcheck
yum install cppcheck
- Clone GlusterFS code
git clone https://github.com/gluster/glusterfs
- Run Cpp check
cppcheck glusterfs/ 2>cppcheck.log
- [BZ 1091677](https://bugzilla.redhat.com/show_bug.cgi?id=1091677)
should be used for submitting patches to master branch for Cppcheck
reported issues.
### Daily Runs
We now have daily runs of various static source code analysis tools on
the glusterfs sources. There are daily analyses of the master and
on currently supported branches.
Results are posted at
<http://download.gluster.org/pub/gluster/glusterfs/static-analysis/>