Skip to content

20260623-WC_16BIT_CPU-WC_ATOMIC_INT_ARG#10763

Merged
philljj merged 1 commit into
wolfSSL:masterfrom
douzzer:20260623-WC_16BIT_CPU-WC_ATOMIC_INT_ARG
Jun 23, 2026
Merged

20260623-WC_16BIT_CPU-WC_ATOMIC_INT_ARG#10763
philljj merged 1 commit into
wolfSSL:masterfrom
douzzer:20260623-WC_16BIT_CPU-WC_ATOMIC_INT_ARG

Conversation

@douzzer

@douzzer douzzer commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

wolfssl/wolfcrypt/wc_port.h and wolfcrypt/src/wc_port.c: for 16 bit portability, use target native int for WC_ATOMIC_INT_ARG, add user overrideability, and adjust WC_INIT_STATE_COUNT_BITS to depend on sizeof(WC_ATOMIC_UINT_ARG). add a wc_static_assert to sanity-check WC_INIT_STATE_STATE_BITS, and use CHAR_BIT opportunistically in the other wc_static_assert to sanity check that CHAR_BIT is at least 8.

fixes ZD#22027

tested with

wolfssl-multi-test.sh ...
check-source-text
clang-tidy-all-sp-all

…ortability,

  use target native int for WC_ATOMIC_INT_ARG, add user overrideability, and
  adjust WC_INIT_STATE_COUNT_BITS to depend on sizeof(WC_ATOMIC_UINT_ARG).  add
  a wc_static_assert to sanity-check WC_INIT_STATE_STATE_BITS, and use CHAR_BIT
  opportunistically in the other wc_static_assert to sanity check that CHAR_BIT
  is at least 8.

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #10763

Scan targets checked: wolfcrypt-bugs, wolfcrypt-rs-bugs, wolfcrypt-src, wolfssl-bugs, wolfssl-src

No new issues found in the changed files. ✅

@douzzer

douzzer commented Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

retest this please

CAVP_AES_CCM
cJSON_Compare against expected file 140-3-known-tests-expected/AES-CCM_474646-response.json
FAIL-ingish...
FAILED!
ret of acvpHelper_check_KAT: 2
Changed ret code from CAVP_RUN_IT_AGAIN to CAVP_FAILURE

@douzzer douzzer added the For This Release Release version 5.9.2 label Jun 23, 2026
@philljj philljj merged commit ad1cd47 into wolfSSL:master Jun 23, 2026
304 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.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants