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.

Sign up for GitHub

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

Jump to bottom

Fix WOLFSSL_NO_TLS12 build error #7915

Merged
merged 1 commit into from
Aug 28, 2024
Merged

Fix WOLFSSL_NO_TLS12 build error #7915

merged 1 commit into from
Aug 28, 2024

Conversation

embhorn
Copy link
Member

@embhorn embhorn commented Aug 28, 2024 •
edited
Loading

Description

Fix build error with

./configure --enable-tls13 --disable-tlsv12 --enable-secure-renegotiation --enable-nullcipher CFLAGS="-DNO_WOLFSSL_CLIENT"

make
/usr/bin/ld: src/.libs/libwolfssl_la-ssl.o: in function `wolfSSL_Rehandshake':
ssl.c:(.text+0x568a): undefined reference to `SendHelloRequest'
/usr/bin/ld: src/.libs/libwolfssl.so.42.2.0: hidden symbol `SendHelloRequest' isn't defined
/usr/bin/ld: final link failed: bad value

Fixes zd18539

Testing

Customer confirmed

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@embhorn embhorn self-assigned this Aug 28, 2024
@embhorn
Copy link
Member Author

embhorn commented Aug 28, 2024

Added config test here:
https://github.com/wolfSSL/testing/pull/775

@embhorn embhorn requested a review from dgarske August 28, 2024 16:35
@embhorn embhorn assigned dgarske and unassigned embhorn Aug 28, 2024
@dgarske dgarske merged commit d4f6b5b into wolfSSL:master Aug 28, 2024
127 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dgarske dgarske dgarske approved these changes

Assignees

@dgarske dgarske

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@embhorn @dgarske