Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding define for wc_aes_block_size #766

Merged

Conversation

night1rider
Copy link

@night1rider night1rider commented Feb 1, 2025

Found when compiling with wolfSSL built with --enable-opensslcoexist it will lead to compile errors like:

| ../git/wolfssh/internal.h:462:27: error: 'AES_BLOCK_SIZE' undeclared here (not in a function); did you mean 'WC_AES_BLOCK_SIZE'?
|   462 | #define STATIC_BUFFER_LEN AES_BLOCK_SIZE
|       |                           ^~~~~~~~~~~~~~
| ../git/wolfssh/internal.h:475:31: note: in expansion of macro 'STATIC_BUFFER_LEN'
|   475 |     ALIGN16 byte staticBuffer[STATIC_BUFFER_LEN];

Found this issue when trying to make a yocto image that has wolfSSL, wolfSSH, and wolfProvider. wolfProvider needs opensslcoexist according to documentation.

@night1rider night1rider self-assigned this Feb 1, 2025
@night1rider night1rider force-pushed the AES_BLOCK_SIZE_opensslcoexist branch from 0b9ef8f to 797f4ed Compare February 1, 2025 22:35
@night1rider night1rider force-pushed the AES_BLOCK_SIZE_opensslcoexist branch from 797f4ed to bcbddd0 Compare February 1, 2025 23:00
@SparkiDev SparkiDev self-assigned this Feb 3, 2025
@SparkiDev SparkiDev merged commit 759bcbd into wolfSSL:master Feb 3, 2025
74 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