Skip to content

20260322-various-fixes#10041

Merged
SparkiDev merged 3 commits intowolfSSL:masterfrom
douzzer:20260322-various-fixes
Mar 23, 2026
Merged

20260322-various-fixes#10041
SparkiDev merged 3 commits intowolfSSL:masterfrom
douzzer:20260322-various-fixes

Conversation

@douzzer
Copy link
Contributor

@douzzer douzzer commented Mar 22, 2026

wolfssl/wolfcrypt/error-crypt.h: add __extension__ to __GNUC__&&!__STRICT_ANSI__ variant of wc_debug_trace_error_codes_enabled(), to inhibit false positive "error: ISO C forbids braced-groups within expressions" with -pedantic.

src/keys.c and src/tls13.c: add WC_NO_ERR_TRACE() annotations for various initializations.

wolfssl/ssl.h, examples/client/client.c, examples/server/server.c, src/bio.c, tests/api.c: add error tracing for WOLFSSL_SHUTDOWN_NOT_DONE.

tests/api.c: in test_wolfSSL_read_write_ex(), use WOLFSSL_SUCCESS rather than 1 for expected-success wolfSSL_shutdown()s, and add note that the wrong value is being returned (the test currently always fails, which is masked by an always-success retval).

wolfcrypt/src/logging.c: fix recursion in wc_backtrace_render().

tested with

wolfssl-multi-test.sh ...
fips-140-3-dev-optest-acvp-sp-asm-trace-errcodes
fips-140-3-dev-all-trace-errcodes
super-quick-check

douzzer added 3 commits March 22, 2026 13:11
…RICT_ANSI__ variant of wc_debug_trace_error_codes_enabled(), to inhibit false positive "error: ISO C forbids braced-groups within expressions" with -pedantic.
…ious initializations.

wolfssl/ssl.h, examples/client/client.c, examples/server/server.c, src/bio.c, tests/api.c: add error tracing for WOLFSSL_SHUTDOWN_NOT_DONE.

tests/api.c: in test_wolfSSL_read_write_ex(), use WOLFSSL_SUCCESS rather than 1 for expected-success wolfSSL_shutdown()s, and add note that the wrong value is being returned (the test currently always fails, which is masked by an always-success retval).
@SparkiDev SparkiDev merged commit 86db2d4 into wolfSSL:master Mar 23, 2026
489 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.

3 participants