Coverity Errors |
| Issues for pmlogrewrite |
| ID |
Checker |
File |
Function |
Classification |
| 42 |
FORWARD_NULL |
/src2/git/pcp/src/pmlogrewrite/<stdout> |
yy_get_next_buffer |
IGNORED |
| | Context is flex generated code (from lex.l) where it appears the pointer cannot be NULL |
| 45 |
FORWARD_NULL |
/src2/git/pcp/src/pmlogrewrite/util.c |
__pmLogRename |
FIXED |
| | Commit efee6ca adds /*NOTREACHED*/ annotation |
| 49 |
FORWARD_NULL |
/src2/git/pcp/src/pmlogrewrite/util.c |
__pmLogRemove |
FIXED |
| | See commit efee6ca |
| 53 |
FORWARD_NULL |
/src2/git/pcp/src/pmlogrewrite/util.c |
__pmLogRemove |
FIXED |
| | See commit efee6ca |
| 55 |
FORWARD_NULL |
/src2/git/pcp/src/pmlogrewrite/metric.c |
start_metric |
FIXED |
| | Commit 8fda161 fixes a number of real, small but benign bugs |
| 87 |
NEGATIVE_RETURNS |
/src2/git/pcp/src/pmlogrewrite/pmlogrewrite.c |
writelabel |
FIXED |
| | Commit 86eff78 adds an assert() after the ftell() calls |
| 88 |
NEGATIVE_RETURNS |
/src2/git/pcp/src/pmlogrewrite/pmlogrewrite.c |
writelabel |
FIXED |
| | See commit 86eff78 |
| 89 |
NEGATIVE_RETURNS |
/src2/git/pcp/src/pmlogrewrite/pmlogrewrite.c |
writelabel |
FIXED |
| | See commit 86eff78 |
| 112 |
NEGATIVE_RETURNS |
/src2/git/pcp/src/pmlogrewrite/logio.c |
_pmLogGet |
FIXED |
| | Commit 8b7efb0 adds an assert() after the ftell() call |
| 185 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmlogrewrite/logio.c |
_pmLogGet |
FIXED |
| | Commit 33879bf frees the malloc'd array on the error branches |
| 188 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmlogrewrite/pmlogrewrite.c |
parseargs |
FIXED |
| | Commit f4916b4 |
| 201 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmlogrewrite/logio.c |
_pmLogGet |
FIXED |
| | See commit 33879bf |
| 209 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmlogrewrite/util.c |
__pmLogRemove |
FIXED |
| | Commit 229ef5b adds missing free() and closedir() calls |
| 215 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmlogrewrite/util.c |
__pmLogRemove |
FIXED |
| | See commit 229ef5b |
| 228 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmlogrewrite/util.c |
__pmLogRename |
FIXED |
| | See commit 229ef5b |
| 252 |
REVERSE_INULL |
/src2/git/pcp/src/pmlogrewrite/metric.c |
_pmUnpackDesc |
FIXED |
| | See commit 8fda161 |
| 282 |
SIZEOF_MISMATCH |
/src2/git/pcp/src/pmlogrewrite/pmlogrewrite.c |
parseargs |
UNINSPECTED |
| 283 |
SIZEOF_MISMATCH |
/src2/git/pcp/src/pmlogrewrite/metric.c |
_pmUnpackDesc |
FIXED |
| | See commit 8fda161 |
| 284 |
SIZEOF_MISMATCH |
/src2/git/pcp/src/pmlogrewrite/metric.c |
_pmUnpackDesc |
FIXED |
| | See commit 8fda161 |
| 285 |
SIZEOF_MISMATCH |
/src2/git/pcp/src/pmlogrewrite/indom.c |
start_indom |
FIXED |
| | Commit 8bb1304 |
| 286 |
SIZEOF_MISMATCH |
/src2/git/pcp/src/pmlogrewrite/indom.c |
_pmUnpackInDom |
FIXED |
| | Commit 8bb1304 |
| 306 |
UNREACHABLE |
/src2/git/pcp/src/pmlogrewrite/metric.c |
do_desc |
FIXED |
| | See commit 8fda161 |