Skip to content

20260415-trace-errcodes-fixes-and-tests#10234

Merged
dgarske merged 3 commits intowolfSSL:masterfrom
douzzer:20260415-trace-errcodes-fixes-and-tests
Apr 16, 2026
Merged

20260415-trace-errcodes-fixes-and-tests#10234
dgarske merged 3 commits intowolfSSL:masterfrom
douzzer:20260415-trace-errcodes-fixes-and-tests

Conversation

@douzzer
Copy link
Copy Markdown
Contributor

@douzzer douzzer commented Apr 16, 2026

wolfcrypt/test/test.c: in ed25519_test(), fix RARE_ED_BAD_ENC_E and RARE_ED_BAD_SIG_E macros to use WC_NO_ERR_TRACE() safely;

.github/workflows/trackmemory.yml: add --enable-debug-trace-errcodes to a couple scenarios.

wolfcrypt/src/logging.c and wolfssl/wolfcrypt/logging.h: add wc_backtrace_set_fp().

tests/api/test_aes.c: fix gating for test_wc_AesGcm_MonteCarlo() to exclude WOLFSSL_AFALG and WOLFSSL_DEVCRYPTO.

tested with

wolfssl-multi-test.sh ...
super-quick-check
all-gcc-c99-backtrace
cppcheck-all-debug-backtrace
clang-tidy-all-debug-backtrace
all-afalg
defaults-afalg

douzzer added 3 commits April 15, 2026 21:12
…ARE_ED_BAD_SIG_E macros to use WC_NO_ERR_TRACE() safely;

.github/workflows/trackmemory.yml: add --enable-debug-trace-errcodes to a couple scenarios.
@github-actions
Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

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 #10234

Scan targets checked: wolfcrypt-api_misuse, wolfcrypt-bugs, wolfcrypt-compliance, wolfcrypt-concurrency, wolfcrypt-consttime, wolfcrypt-defaults, wolfcrypt-mutation, wolfcrypt-portability, wolfcrypt-proptest, wolfcrypt-src, wolfcrypt-zeroize

Findings: 1
1 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread wolfcrypt/src/logging.c
@dgarske dgarske merged commit f086e91 into wolfSSL:master Apr 16, 2026
437 of 438 checks passed
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.

4 participants