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

pppd: Fix build without OpenSSL #533

Merged
merged 1 commit into from
Nov 21, 2024
Merged

Conversation

tpaukrt
Copy link
Contributor

@tpaukrt tpaukrt commented Nov 7, 2024

The symbol OPENSSL_VERSION_NUMBER is not defined when pppd is compiled without OpenSSL support, so it evaluates to zero. This results in the following linker error:

crypto.c:241: undefined reference to `ERR_free_strings'

The symbol OPENSSL_VERSION_NUMBER is not defined when pppd is
compiled without OpenSSL support, so it evaluates to zero.
This results in the following linker error:

crypto.c:241: undefined reference to `ERR_free_strings'

Signed-off-by: Tomas Paukrt <[email protected]>
@Neustradamus
Copy link
Member

@paulusmack: What do you think about @tpaukrt PR?

cc: @rascheel.

Linked to:

@paulusmack paulusmack merged commit 5f6eabd into ppp-project:master Nov 21, 2024
31 checks passed
@tpaukrt tpaukrt deleted the fix-build branch November 21, 2024 04:11
@Neustradamus Neustradamus removed the request for review from paulusmack November 21, 2024 05:41
@Neustradamus
Copy link
Member

Thanks to @tpaukrt for your PR and thanks to @paulusmack for merging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants