Skip to content

20250411-more-libwolfssl_sources_h#8662

Merged
dgarske merged 1 commit intowolfSSL:masterfrom
douzzer:20250411-more-libwolfssl_sources_h
Apr 11, 2025
Merged

20250411-more-libwolfssl_sources_h#8662
dgarske merged 1 commit intowolfSSL:masterfrom
douzzer:20250411-more-libwolfssl_sources_h

Conversation

@douzzer
Copy link
Copy Markdown
Contributor

@douzzer douzzer commented Apr 11, 2025

update several files in wolfcrypt/src/port/arm to include libwolfssl_sources.h;

update wolfcrypt/src/port/af_alg, wolfcrypt/src/port/devcrypto, and wolfcrypt/src/port/kcapi to include libwolfssl_sources.h;

remove a slew of includes across lib sources made redundant by libwolfssl_sources.h.

tested with wolfssl-multi-test.sh ... super-quick-check using fips-dev from https://github.com/wolfSSL/fips/pull/341 and wolfsm from wolfSSL/wolfsm#27, with -DTEST_LIBWOLFSSL_SOURCES_INCLUSION_SEQUENCE.

also used

git grep -E '#include <wolfssl/wolfcrypt/(error-crypt|settings|logging|types|wc_port)\.h>' src wolfcrypt/src/

to find and fix redundant and missing includes, and used

git diff --color=always | egrep -v '#include <wolfssl/wolfcrypt/(error-crypt|settings|logging|types|wc_port)\.h>' | less

to confirm no needed includes were inadvertently removed.

…sources.h;

update wolfcrypt/src/port/af_alg, wolfcrypt/src/port/devcrypto, and wolfcrypt/src/port/kcapi to include libwolfssl_sources.h;

remove a slew of includes across lib sources made redundant by libwolfssl_sources.h.
@douzzer
Copy link
Copy Markdown
Contributor Author

douzzer commented Apr 11, 2025

retest this please (java.io.StreamCorruptedException: invalid stream header: 636F7272)

@dgarske dgarske merged commit 75501fd into wolfSSL:master Apr 11, 2025
186 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