Skip to content

20260305 Coverity fixes#9887

Merged
douzzer merged 2 commits intowolfSSL:masterfrom
rlm2002:static_analysis
Mar 11, 2026
Merged

20260305 Coverity fixes#9887
douzzer merged 2 commits intowolfSSL:masterfrom
rlm2002:static_analysis

Conversation

@rlm2002
Copy link
Copy Markdown
Contributor

@rlm2002 rlm2002 commented Mar 5, 2026

Description

351993 Unused value - removed assignment tempBuf = NULL as value is overwritten by loop and is freed before breaking from loop. tempBuf is not accessed after free.

315860 Out-of-bounds write - include RIPEMD_BLOCK_SIZE when checking value of ripemd->buffLen so local[ripemd->buffLen++] = 0x80;` will not attempt to write outside of bounds.

Testing

How did you test?

./configure --enable-all && make check

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@rlm2002 rlm2002 self-assigned this Mar 5, 2026
@rlm2002 rlm2002 marked this pull request as ready for review March 5, 2026 21:00
@philljj philljj added the For This Release Release version 5.9.1 label Mar 6, 2026
@rlm2002
Copy link
Copy Markdown
Contributor Author

rlm2002 commented Mar 6, 2026

retest this please

@rlm2002 rlm2002 assigned wolfSSL-Bot and unassigned rlm2002 Mar 10, 2026
@douzzer douzzer added the Staged Staged for merge pending final test results and review label Mar 11, 2026
@douzzer douzzer merged commit a5bc0cd into wolfSSL:master Mar 11, 2026
592 of 594 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For This Release Release version 5.9.1 Staged Staged for merge pending final test results and review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants