Skip to content

20260323-InitSuites-ALL#10052

Closed
douzzer wants to merge 1 commit intowolfSSL:masterfrom
douzzer:20260323-InitSuites-ALL
Closed

20260323-InitSuites-ALL#10052
douzzer wants to merge 1 commit intowolfSSL:masterfrom
douzzer:20260323-InitSuites-ALL

Conversation

@douzzer
Copy link
Copy Markdown
Contributor

@douzzer douzzer commented Mar 23, 2026

src/internal.c: in ParseCipherList(), if the list is "ALL", add anonymous and null ciphers too.

fixes regression from 54d6555 in #9841, detected by scripts/openssl.test in super-quick-check.

tested with

./configure --enable-all
WOLFSSL_OPENSSL_TEST=1  scripts/openssl.test

@douzzer douzzer requested a review from wolfSSL-Bot March 23, 2026 20:31
@douzzer douzzer requested a review from julek-wolfssl March 23, 2026 20:33
douzzer added a commit to douzzer/wolfssl that referenced this pull request Mar 23, 2026
@douzzer douzzer mentioned this pull request Mar 23, 2026
Copy link
Copy Markdown
Member

@julek-wolfssl julek-wolfssl left a comment

Choose a reason for hiding this comment

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

This is from the openssl manual page.

ALL

All cipher suites except the eNULL ciphers (which must be explicitly enabled if needed). As of OpenSSL 1.0.0, the ALL cipher suites are sensibly ordered by default.

@douzzer
Copy link
Copy Markdown
Contributor Author

douzzer commented Mar 23, 2026

wrong fix
(see -l ALL in scripts/openssl.test)

@douzzer douzzer closed this Mar 23, 2026
JeremiahM37 pushed a commit to JeremiahM37/wolfssl that referenced this pull request Apr 15, 2026
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