Bug 1228142
| Summary: | clang-analyzer: adding clang static analysis support | ||
|---|---|---|---|
| Product: | [Community] GlusterFS | Reporter: | Prasanna Kumar Kalever <prasanna.kalever> |
| Component: | scripts | Assignee: | Prasanna Kumar Kalever <prasanna.kalever> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | mainline | CC: | amukherj, bugs |
| Target Milestone: | --- | Keywords: | Triaged |
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | glusterfs-3.9.0 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2017-03-27 18:13:07 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Prasanna Kumar Kalever
2015-06-04 09:57:31 UTC
REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#1) for review on master by Prasanna Kumar Kalever REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#3) for review on master by Prasanna Kumar Kalever REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#4) for review on master by Prasanna Kumar Kalever REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#5) for review on master by Prasanna Kumar Kalever REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#6) for review on master by Prasanna Kumar Kalever REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#7) for review on master by Prasanna Kumar Kalever changing version to mainline in order to retire pre-release. If you know the appropriate, correct version for this bug, please set it. changing version to mainline in order to retire pre-release. If you know the appropriate, correct version for this bug, please set it. changing version to mainline in order to retire pre-release. If you know the appropriate, correct version for this bug, please set it. REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#9) for review on master by Prasanna Kumar Kalever (prasanna.kalever) REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#10) for review on master by Prasanna Kumar Kalever (pkalever) REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#11) for review on master by Prasanna Kumar Kalever (pkalever) REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#12) for review on master by Raghavendra Talur (rtalur) REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#13) for review on master by Prasanna Kumar Kalever (pkalever) This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#14) for review on master by Prasanna Kumar Kalever (pkalever) REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#15) for review on master by Prasanna Kumar Kalever (pkalever) REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#16) for review on master by Prasanna Kumar Kalever (pkalever) REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#17) for review on master by Prasanna Kumar Kalever (pkalever) REVIEW: http://review.gluster.org/11083 (clang-analyzer: adding clang static analysis support) posted (#18) for review on master by Prasanna Kumar Kalever (pkalever) COMMIT: http://review.gluster.org/11083 committed in master by Jeff Darcy (jdarcy) ------ commit bdde0cfa1d5848e59d20c1adc19ee1b687eec66a Author: Prasanna Kumar Kalever <prasanna.kalever> Date: Thu Jun 4 14:55:17 2015 +0530 clang-analyzer: adding clang static analysis support clang static analyzer is a source code analysis tool that finds bugs in C, C++. This patch automates clang analyzes part by integrating it with 'make clang-check' clang-checker.sh: runs clang analyzer with and without the HEAD commit, and shows the bugs introduced by HEAD commit (if any) sample report: $[ glusterfs ]: make clang-check ================ Clang analyzer in progress ================ ... BASELINE BUGS LIST (before applying patch): ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Out of bound array access --> 3 Memory leak --> 1 Unix API --> 24 Dead increment --> 5 Dereference of null pointer --> 1995 Uninitialized argument value --> 2 All Bugs --> 2872 Called function pointer is null null dereference --> 4 Dead initialization --> 49 Dead assignment --> 691 Undefined allocation of 0 bytes CERT MEM0 C CWE --> 5 Argument with nonnull attribute passed null --> 84 Result of operation is garbage or undefined --> 9 TARGET BUGS LIST (after applying patch): ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Out of bound array access --> 3 Memory leak --> 1 Unix API --> 24 Dead increment --> 5 Dereference of null pointer --> 1995 Uninitialized argument value --> 2 All Bugs --> 2875 Called function pointer is null null dereference --> 4 Dead initialization --> 49 Dead assignment --> 694 Undefined allocation of 0 bytes CERT MEM0 C CWE --> 5 Argument with nonnull attribute passed null --> 84 Result of operation is garbage or undefined --> 9 SUMMARY OF CLANG-ANALYZER: ~~~~~~~~~~~~~~~~~~~~~~~~~~ Extra 3 Bug[s] Introduced in: Dead assignment Patch Value given by Clang analyzer '-1' Explore complete results at /home/user/work/glusterfs/baseline/results/index.html /home/user/work/glusterfs/target/results/index.html ================= Done with Clang Analysis ================= Change-Id: I512e33cfc02885a111152fbc1832144261dda1da BUG: 1228142 Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever> Reviewed-on: http://review.gluster.org/11083 Tested-by: Prasanna Kumar Kalever <pkalever> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.com> Tested-by: Gluster Build System <jenkins.com> Smoke: Gluster Build System <jenkins.com> Reviewed-by: Raghavendra Talur <rtalur> Reviewed-by: Kaleb KEITHLEY <kkeithle> This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.9.0, please open a new bug report. glusterfs-3.9.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution. [1] http://lists.gluster.org/pipermail/gluster-users/2016-November/029281.html [2] https://www.gluster.org/pipermail/gluster-users/ |