From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fabio M. Di Nitto Date: Mon, 10 Oct 2011 10:45:17 +0200 Subject: [Cluster-devel] [coverity] liblogthread Message-ID: <1318236321-21955-1-git-send-email-fdinitto@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit This is the first patchset to address some issues spotted by Coverity scan. Original errors/warnings: Analysis summary report: ------------------------ Files analyzed : 1 Total LoC input to cov-analyze : 12453 Functions analyzed : 11 Paths analyzed : 243 New defects found : 8 Total 1 ATOMICITY 3 BUFFER_SIZE_WARNING 1 MISSING_LOCK 1 NULL_RETURNS 1 PW.NOT_COMPATIBLE_WITH_PREVIOUS_DECL 1 SECURE_CODING Elapsed time: 00:00:00 Processing 8 C/C++ errors. 0% 10 20 30 40 50 60 70 80 90 100% |----|----|----|----|----|----|----|----|----|----| *************************************************** Processed 8 C/C++ errors. After fixes: Analysis summary report: ------------------------ Files analyzed : 1 Total LoC input to cov-analyze : 12475 Functions analyzed : 12 Paths analyzed : 250 New defects found : 2 Total 1 ATOMICITY 1 PW.NOT_COMPATIBLE_WITH_PREVIOUS_DECL Elapsed time: 00:00:01 Processing 2 C/C++ errors. 0% 10 20 30 40 50 60 70 80 90 100% |----|----|----|----|----|----|----|----|----|----| *************************************************** Processed 2 C/C++ errors. NOTES: PW.NOT_COMPATIBLE_WITH_PREVIOUS_DECL can be safely ignored as it is an incompatible declaration between system gcc and coverity headers. The remaining ATOMICITY error appears to be bogus.