Skip to content

double free -- should be o_hash#9270

Merged
dgarske merged 1 commit intowolfSSL:masterfrom
effbiae:sm3-free-x2
Oct 6, 2025
Merged

double free -- should be o_hash#9270
dgarske merged 1 commit intowolfSSL:masterfrom
effbiae:sm3-free-x2

Conversation

@effbiae
Copy link
Copy Markdown
Contributor

@effbiae effbiae commented Oct 5, 2025

Description

fix a double free() of hmac->i_hash.sm3

Fixes #9190

Testing

./configure --enable-sm3 --enable-hmac-copy

Checklist

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

@wolfSSL-Bot
Copy link
Copy Markdown

Can one of the admins verify this patch?

@dgarske
Copy link
Copy Markdown
Member

dgarske commented Oct 5, 2025

Okay to test. Contributor agreement on file

@effbiae
Copy link
Copy Markdown
Contributor Author

effbiae commented Oct 6, 2025

i turned on the hostap-vm.yml workflow in my github repo and the same test fails (owe_invalid_assoc_resp)..
any ideas why?
i note wc_Sm3Free() is a nop so the patch should change nothing

@dgarske dgarske merged commit 874633d into wolfSSL:master Oct 6, 2025
267 of 268 checks passed
@effbiae effbiae deleted the sm3-free-x2 branch October 7, 2025 01:04
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.

3 participants