FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor


 
 
LinkBack Thread Tools
 
Old 10-10-2011, 08:45 AM
"Fabio M. Di Nitto"
 
Default liblogthread

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.
 
Old 10-10-2011, 08:45 AM
"Fabio M. Di Nitto"
 
Default liblogthread

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.
 
Old 10-10-2011, 02:54 PM
David Teigland
 
Default liblogthread

On Mon, Oct 10, 2011 at 10:45:17AM +0200, Fabio M. Di Nitto wrote:
> This is the first patchset to address some issues spotted by Coverity scan.

look fine
 

Thread Tools




All times are GMT. The time now is 04:25 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org