Skip to content

Coverity#10418

Open
rlm2002 wants to merge 6 commits intowolfSSL:masterfrom
rlm2002:coverity
Open

Coverity#10418
rlm2002 wants to merge 6 commits intowolfSSL:masterfrom
rlm2002:coverity

Conversation

@rlm2002
Copy link
Copy Markdown
Contributor

@rlm2002 rlm2002 commented May 6, 2026

Description

560796: Dereference before null check - remove check as extension->data would not be NULL at this point
560760: Logically dead code - remove dead length check
560126: Dereference after null check - introduce defaultSz variable to handle s == NULL
560794: Dereference after null check - remove check as ssl->ctx and ssl->ctx->method would not be NULL in normal operation at this point

Also adds unit test for NULL parameters in EVP_DigestFinal() with SHAKE

Testing

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

Checklist

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

@rlm2002 rlm2002 self-assigned this May 6, 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.

1 participant