Skip to content

24062026 Coverity fixes#10773

Open
rlm2002 wants to merge 9 commits into
wolfSSL:masterfrom
rlm2002:coverity
Open

24062026 Coverity fixes#10773
rlm2002 wants to merge 9 commits into
wolfSSL:masterfrom
rlm2002:coverity

Conversation

@rlm2002

@rlm2002 rlm2002 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Description

  • Logically dead code
    • 561219: remove XBADFILE check
    • 561215/561216: remove dead check
  • Untrusted loop bound
    • 561218: Add bounds check to test helper
  • Overflowed return value
    • 561217: move casts and harden function
  • Unchecked return value
  • 561216/561205: Remove call to remove(pubFile/privFile), add (void) cast
  • Assign instead of compare
    • 561214: change ExpectTrue() to assignment
  • Untrusted pointer read
    • 561207: Bound tainted lengths in ECH test helper

Testing

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

@rlm2002 rlm2002 self-assigned this Jun 24, 2026
@rlm2002 rlm2002 marked this pull request as ready for review June 25, 2026 15:11
@github-actions

Copy link
Copy Markdown

retest this please

@rlm2002 rlm2002 assigned wolfSSL-Bot and unassigned rlm2002 Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants