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

20241204-WOLF_AGG_DUMMY_MEMBER #8254

Merged

Conversation

douzzer
Copy link
Contributor

@douzzer douzzer commented Dec 5, 2024

wolfssl/wolfcrypt/types.h and wolfssl/wolfcrypt/hash.h: define WOLF_AGG_DUMMY_MEMBER, pivoting on HAVE_EMPTY_AGGREGATES, and use WOLF_AGG_DUMMY_MEMBER in wc_Hashes.

configure.ac: add --enable-fips=cert4718 alias for v5, and make --enable-fips=v5 set FIPS to 5.2.1; set DEF_FAST_MATH and DEF_SP_MATH to "no" when "yes" would conflict with user-supplied arguments.

@douzzer douzzer self-assigned this Dec 5, 2024
@douzzer
Copy link
Contributor Author

douzzer commented Dec 5, 2024

retest this please. (PRB-single-flag.txt_70 --enable-fips=v5 --enable-smallstack failed in setup with "fatal: Not a valid object name: 'v5.2.1-stable'.")

@dgarske dgarske removed their request for review December 5, 2024 18:48
@douzzer douzzer force-pushed the 20241204-WOLF_AGG_DUMMY_MEMBER branch 2 times, most recently from c743847 to cd89b42 Compare December 5, 2024 21:39
@douzzer douzzer assigned wolfSSL-Bot and unassigned douzzer Dec 5, 2024
Copy link
Contributor

@bandi13 bandi13 left a comment

Choose a reason for hiding this comment

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

This looks like a good solution

Copy link
Contributor

@bandi13 bandi13 left a comment

Choose a reason for hiding this comment

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

Have some small suggestion

wolfssl/wolfcrypt/types.h Outdated Show resolved Hide resolved
@bandi13 bandi13 assigned douzzer and unassigned kaleb-himes and wolfSSL-Bot Dec 6, 2024
…ble-fips=v5 set FIPS to 5.2.1; set DEF_FAST_MATH and DEF_SP_MATH to "no" when "yes" would conflict with user-supplied arguments.
…GG_DUMMY_MEMBER, pivoting on HAVE_EMPTY_AGGREGATES, and use WOLF_AGG_DUMMY_MEMBER in wc_Hashes.
… changed in d85c108 -- original value was correct, misdiagnosed by faulty test.
…ak git fetching to keep wolfssl and fips tags distinct, and fetch all needed tags by name to assure availability for checkout. also, hide stdout noise from pushd/popd.
…_AGGREGATES to conform to wolfssl convention -- defined() for true, !defined() for false -- while retaining ability for user override-off by passing in explicit 0 definition.
…llness check around XFREE(ssl->buffers.digest.buffer, ...).
@douzzer douzzer force-pushed the 20241204-WOLF_AGG_DUMMY_MEMBER branch from cd89b42 to 0ad072a Compare December 6, 2024 19:03
@douzzer douzzer requested a review from bandi13 December 6, 2024 19:04
Copy link
Contributor

@bandi13 bandi13 left a comment

Choose a reason for hiding this comment

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

I think this can be simplified.

wolfssl/wolfcrypt/cryptocb.h Show resolved Hide resolved
wolfssl/wolfcrypt/types.h Show resolved Hide resolved
@douzzer douzzer requested a review from bandi13 December 6, 2024 19:36
Copy link
Contributor

@bandi13 bandi13 left a comment

Choose a reason for hiding this comment

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

Discussed offline. May do the suggestions in a future PR.

@douzzer douzzer assigned wolfSSL-Bot and unassigned douzzer Dec 6, 2024
@douzzer
Copy link
Contributor Author

douzzer commented Dec 6, 2024

note, partially addresses ZD#18760

for more on ZD#18760 see #8253 (WIP)

@JacobBarthelmeh JacobBarthelmeh merged commit f764dbe into wolfSSL:master Dec 6, 2024
145 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.

5 participants