Coverity Errors |
| Issues for pmie |
| ID |
Checker |
File |
Function |
Classification |
| 64 |
FORWARD_NULL |
/src2/git/pcp/src/pmie/src/show.c |
showSyn |
FIXED |
| | No issue here, commit d23043b adds an assert to be sure, to be sure |
| 78 |
MISSING_BREAK |
/src2/git/pcp/src/pmie/src/eval.c |
findEval |
FIXED |
| | Cases in question are never encountered, so the missing break; was benign. Commit c45540f removes the cases and adds a comment to assist with readability |
| 79 |
MISSING_BREAK |
/src2/git/pcp/src/pmie/src/lexicon.c |
yylex |
FIXED |
| | No issue, as control does not pass to the "missing" break, however commit 2348bc2 makes cosmetic changes to clean up the code in this area |
| 143 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmie/src/syslog.c |
do_syslog_args |
FIXED |
| | Commit f36f18c |
| 207 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmie/src/fun.c |
indom_changed |
IGNORED |
| | No issue, Coverity analysis is wrong as no allocation is done in the numinst <= 0 cases |
| 208 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmie/src/fun.c |
indom_changed |
IGNORED |
| | No issue, Coverity analysis is wrong as no allocation is done in the numinst <= 0 cases |
| 231 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmie/src/pmie.c |
interact |
FIXED |
| | Commit 008b050 |
| 239 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmie/src/pragmatics.c |
initMetric |
IGNORED |
| | No issue, Coverity analysis is wrong as no allocation is done in the sts <= 0 cases |
| 240 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmie/src/pragmatics.c |
initMetric |
IGNORED |
| | No issue, Coverity analysis is wrong as no allocation is done in the sts <= 0 cases |
| 241 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmie/src/pragmatics.c |
initMetric |
IGNORED |
| | No issue, Coverity analysis is wrong as no allocation is done in the sts <= 0 cases |
| 243 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmie/src/pragmatics.c |
reinitMetric |
IGNORED |
| | No issue, Coverity analysis is wrong as no allocation is done in the sts <= 0 cases |
| 244 |
RESOURCE_LEAK |
/src2/git/pcp/src/pmie/src/pragmatics.c |
reinitMetric |
IGNORED |
| | No issue, Coverity analysis is wrong as no allocation is done in the sts <= 0 cases |
| 258 |
REVERSE_INULL |
/src2/git/pcp/src/pmie/src/symbol.c |
symFree |
FIXED |
| | Commit 2f1b769 |
| 297 |
UNINIT |
/src2/git/pcp/src/pmie/src/dstruct.c |
instExpr |
FIXED |
| | Commit ce603bc |
| 320 |
USE_AFTER_FREE |
/src2/git/pcp/src/pmie/src/symbol.c |
symClearTable |
IGNORED |
| | No issue, Coverity analysis is wrong as remBucket() does the right thing™ and st->hdr.next is updated before the caller dereferences it again |
| 329 |
USE_AFTER_FREE |
/src2/git/pcp/src/pmie/src/dstruct.c |
freeExpr |
IGNORED |
| | No issue, Coverity analysis is wrong as m points to an array of Metric structs, so free(m); m++; is OK, if a little odd |
| 330 |
USE_AFTER_FREE |
/src2/git/pcp/src/pmie/src/dstruct.c |
freeExpr |
IGNORED |
| | No issue, Coverity analysis is wrong as m points to an array of Metric structs, so free(m); m++; is OK, if a little odd |
| 334 |
USE_AFTER_FREE |
/src2/git/pcp/src/pmie/src/pragmatics.c |
taskFetch |
IGNORED |
| | No issue, Coverity is confused ... dereference in pmFetch() is (a) guarded by pmDebug & DBG_TRACE_FETCH (so only debug message) and (b) appears to be on a path where *result has been set by a successful fetch |
| 351 |
USE_AFTER_FREE |
/src2/git/pcp/src/pmie/src/syslog.c |
do_syslog_args |
IGNORED |
| | No issue, Coverity analysis is wrong? Needs review |