Skip to content

20260327-sanitizer-and-tls12_kdf_test-gating-fixes#10095

Merged
lealem47 merged 3 commits intowolfSSL:masterfrom
douzzer:20260327-sanitizer-and-fixes-and-tls12_kdf_test-gating
Mar 27, 2026
Merged

20260327-sanitizer-and-tls12_kdf_test-gating-fixes#10095
lealem47 merged 3 commits intowolfSSL:masterfrom
douzzer:20260327-sanitizer-and-fixes-and-tls12_kdf_test-gating

Conversation

@douzzer
Copy link
Copy Markdown
Contributor

@douzzer douzzer commented Mar 27, 2026

src/conf.c, src/ssl.c, wolfcrypt/src/asn.c, wolfssl/wolfcrypt/asn.h: fixes for invalid-pointer-pair memory errors reported by clang sanitizer with detect_invalid_pointer_pairs=2 in ASAN_OPTIONS.

wolfssl/wolfcrypt/coding.h and wolfssl/wolfcrypt/settings.h: fix feature activation for WOLFSSL_BASE16 (move from coding.h to settings.h).

tested with

wolfssl-multi-test.sh ...
super-quick-check
linuxkm-fips-v5-vanilla-dist-insmod-cust-kernel-3
sanitizer-clang-all-intelasm

with ASAN_OPTIONS="quarantine_size_mb=0:detect_stack_use_after_return=1:check_initialization_order=1:strict_string_checks=1:detect_invalid_pointer_pairs=2"

douzzer added 3 commits March 27, 2026 16:40
…fixes for invalid-pointer-pair memory errors reported by clang sanitizer with detect_invalid_pointer_pairs=2 in ASAN_OPTIONS.
…ure activation for WOLFSSL_BASE16 (move from coding.h to settings.h).
@douzzer douzzer added the For This Release Release version 5.9.1 label Mar 27, 2026
@douzzer douzzer changed the title 20260327-sanitizer-and-fixes-and-tls12_kdf_test-gating 20260327-sanitizer-and-tls12_kdf_test-gating-fixes Mar 27, 2026
Copy link
Copy Markdown

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #10095

Scan targets checked: src, src-bugs, src-compliance, wolfcrypt-bugs, wolfcrypt-src

No new issues found in the changed files. ✅

@douzzer
Copy link
Copy Markdown
Contributor Author

douzzer commented Mar 27, 2026

retest this please

Copy link
Copy Markdown
Contributor

@lealem47 lealem47 left a comment

Choose a reason for hiding this comment

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

Looks good

@lealem47 lealem47 merged commit b7e7e75 into wolfSSL:master Mar 27, 2026
563 of 565 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For This Release Release version 5.9.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants