Skip to content

Regression testing fixes - memory allocation failure testing#10652

Open
SparkiDev wants to merge 1 commit into
wolfSSL:masterfrom
SparkiDev:regression_fixes_25
Open

Regression testing fixes - memory allocation failure testing#10652
SparkiDev wants to merge 1 commit into
wolfSSL:masterfrom
SparkiDev:regression_fixes_25

Conversation

@SparkiDev

Copy link
Copy Markdown
Contributor

Description

crl.c, internal.h: leak of sigParams requiring reorder the struct fields to that it is above memcpy part.
dtl13.c: free the DRLS fragments buffer in Dtls13FreeFsmResources in case fragment is never sent.
ocsp.c: only free cid if locally allocated.
tls.c: make sure ecc_kse is zeroized and can be freed.
tls13.c: set hsHashesEch after init so isn't lost on failure.
evp_pk.c: free key on the BIO error path

Fixed various tests to not leak or crash on memory allocation failure.

Testing

Memory allocation failure testing.

@SparkiDev SparkiDev self-assigned this Jun 10, 2026
@SparkiDev

Copy link
Copy Markdown
Contributor Author

Jenkins: retest this please

crl.c, internal.h: leak of sigParams requiring reorder the struct fields to that it is above memcpy part.
dtl13.c: free the DRLS fragments buffer in Dtls13FreeFsmResources in case fragment is never sent.
ocsp.c: only free cid if locally allocated.
tls.c: make sure ecc_kse is zeroized and can be freed.
tls13.c: set hsHashesEch after init so isn't lost on failure.
evp_pk.c: free key on the BIO error path

Fixed various tests to not leak or crash on memory allocation failure.
@SparkiDev SparkiDev force-pushed the regression_fixes_25 branch from 16bc473 to 6289c62 Compare June 10, 2026 07:34
@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown

MemBrowse Memory Report

gcc-arm-cortex-m4-pq

  • FLASH: .text +64 B (+0.0%, 277,208 B / 1,048,576 B, total: 26% used)

gcc-arm-cortex-m4-rsa-only

  • FLASH: .text -64 B (-0.0%, 322,616 B / 1,048,576 B, total: 31% used)

gcc-arm-cortex-m7-pq

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