Skip to content

dtls13: free and null the cipher slot on init failure in Dtls13InitAesCipher and ChaCha equivalent#10360

Open
gasbytes wants to merge 1 commit intowolfSSL:masterfrom
gasbytes:cipher-init-dtls13-fix
Open

dtls13: free and null the cipher slot on init failure in Dtls13InitAesCipher and ChaCha equivalent#10360
gasbytes wants to merge 1 commit intowolfSSL:masterfrom
gasbytes:cipher-init-dtls13-fix

Conversation

@gasbytes
Copy link
Copy Markdown
Contributor

Description

dtls13: free and null the cipher slot on init failure in Dtls13InitAesCipher and ChaCha equivalent

Fixes zd#21720

Testing

./configure --enable-dtls --enable-tls13 --enable-dtls13 &&
make -j8 &&
make check

Checklist

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

@gasbytes gasbytes marked this pull request as ready for review April 30, 2026 14:25
@gasbytes gasbytes self-assigned this Apr 30, 2026
@gasbytes
Copy link
Copy Markdown
Contributor Author

Jenkins retest this please (timeout)

@gasbytes gasbytes force-pushed the cipher-init-dtls13-fix branch from 2c1a3a1 to eace4f2 Compare April 30, 2026 14:59
@gasbytes
Copy link
Copy Markdown
Contributor Author

Jenkins retest this please (timeout)

@gasbytes gasbytes assigned wolfSSL-Bot and unassigned gasbytes Apr 30, 2026
Copy link
Copy Markdown
Member

@julek-wolfssl julek-wolfssl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This gets cleaned up in FreeCiphers. What is the justification to do it here?

@github-actions
Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

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