#21 Fix static analysers warnings
Closed: fixed 3 years ago by rcritten. Opened 10 years ago by lslebodn.

Static analysers report some warnings for this project

Error: CHECKED_RETURN
Error: DEADCODE
Error: TAINTED_STRING
Error: TOCTOU

Error: CPPCHECK_WARNING:
certmonger-0.68/src/submit-x.c:71: error[uninitvar]: Uninitialized variable: ret
Error: CPPCHECK_WARNING:
certmonger-0.68/src/util.c:127: error[memleak]: Memory leak: tmp


Metadata Update from @lslebodn:
- Issue set to the milestone: 0.75

7 years ago

Unfortunately this got overlooked and the attached patches are missing.

I've run certmonger through a number of rounds of clang and coverity and while there are still some remaining issues to address or mark as false positives it may be safe to assume that least a few of these have already been addressed.

I'm going to mark this as done.

Metadata Update from @rcritten:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

3 years ago

Login to comment on this ticket.

Metadata