Skip to content

20231115-misc-fixits#6973

Merged
JacobBarthelmeh merged 6 commits intowolfSSL:masterfrom
douzzer:20231115-misc-fixits
Nov 15, 2023
Merged

20231115-misc-fixits#6973
JacobBarthelmeh merged 6 commits intowolfSSL:masterfrom
douzzer:20231115-misc-fixits

Conversation

@douzzer
Copy link
Copy Markdown
Contributor

@douzzer douzzer commented Nov 15, 2023

Fixes various issues, mostly identified by multi-test nightly. See commit messages for details.

Tested with wolfssl-multi-test.sh ... super-quick-check

…L_GMTIME gmtime() into gmtime_r(), and always define HAVE_GMTIME_R when defined(WOLFSSL_GMTIME).
…f max_relative_stack, to accommodate compilation settings when subsumed within testsuite.
…ertByIssuer(), MicriumGenerateCookie(), uIPGenerateCookie(), and GNRC_GenerateCookie();

tests/api.c: when NO_SHA, omit test_wolfSSL_CertManagerCheckOCSPResponse() and test_wolfSSL_CheckOCSPResponse() (both use static artifacts with SHA1 name and key hashes).
…AULTS, and enable aesgcm_stream only if enable_aesgcm.
@douzzer douzzer requested a review from anhu November 15, 2023 07:50
Copy link
Copy Markdown
Member

@anhu anhu left a comment

Choose a reason for hiding this comment

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

Everything looks fine to me, but there are failing PRBs.

@anhu anhu assigned douzzer and unassigned anhu Nov 15, 2023
@douzzer
Copy link
Copy Markdown
Contributor Author

douzzer commented Nov 15, 2023

retest this please.

@douzzer douzzer requested a review from anhu November 15, 2023 17:20
@douzzer douzzer assigned anhu and wolfSSL-Bot and unassigned douzzer and anhu Nov 15, 2023
Comment thread src/wolfio.c
configure.ac: fix dependencies for enable_dsa vs enable_sha in enable-all, enable-all-crypto, and ENABLED_DSA setup.
@douzzer
Copy link
Copy Markdown
Contributor Author

douzzer commented Nov 15, 2023

retest this please.

@JacobBarthelmeh JacobBarthelmeh merged commit bb73c23 into wolfSSL:master Nov 15, 2023
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.

4 participants