Skip to content

20250418-linuxkm-lkcapi-cleanup#8696

Merged
philljj merged 2 commits intowolfSSL:masterfrom
douzzer:20250418-linuxkm-lkcapi-cleanup
Apr 19, 2025
Merged

20250418-linuxkm-lkcapi-cleanup#8696
philljj merged 2 commits intowolfSSL:masterfrom
douzzer:20250418-linuxkm-lkcapi-cleanup

Conversation

@douzzer
Copy link
Copy Markdown
Contributor

@douzzer douzzer commented Apr 18, 2025

linuxkm: clean up LKCAPI glue:

typography fixes;

static attr on const byte arrays;

refactor km_ecdsa_verify() to use a single heap allocation;

fix linuxkm_test_ecdsa_nist_driver() to copy sig and hash to a heap allocation before passing to sg_set_buf() (avoids unmapping of rodata);

in linuxkm_test_pkcs1_driver() move hash from stack to heap;

add LINUX_VERSION_CODE >= 5.4 assertion in linuxkm/lkcapi_glue.c;

streamline macro logic in linuxkm/lkcapi_sha_glue.c.

unrelated, re #8689:
wolfssl/internal.h: clean up WOLFSSL_TEST_VIS comments and an unneeded attr on GetOutputBuffer().

douzzer added 2 commits April 18, 2025 18:21
typography fixes;

static attr on const byte arrays;

refactor km_ecdsa_verify() to use a single heap allocation;

fix linuxkm_test_ecdsa_nist_driver() to copy sig and hash to a heap allocation before passing to sg_set_buf() (avoids unmapping of rodata);

in linuxkm_test_pkcs1_driver() move hash from stack to heap;

add LINUX_VERSION_CODE >= 5.4 assertion in linuxkm/lkcapi_glue.c;

streamline macro logic in linuxkm/lkcapi_sha_glue.c.
@douzzer douzzer requested a review from philljj April 18, 2025 23:30
@philljj
Copy link
Copy Markdown
Contributor

philljj commented Apr 19, 2025

retest this please

@philljj philljj merged commit 3cdc521 into wolfSSL:master Apr 19, 2025
186 checks passed
@douzzer douzzer added the For This Release Release version 5.9.1 label Apr 20, 2025
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.

2 participants