Skip to content

Comments

Add OpenSSL 1.1.1 as submodule for legacy API support#31

Open
eWloYW8 wants to merge 14 commits intoiagox86:masterfrom
eWloYW8:master
Open

Add OpenSSL 1.1.1 as submodule for legacy API support#31
eWloYW8 wants to merge 14 commits intoiagox86:masterfrom
eWloYW8:master

Conversation

@eWloYW8
Copy link

@eWloYW8 eWloYW8 commented Apr 8, 2025

This PR adds OpenSSL 1.1.1 as a Git submodule and updates the build process to ensure compatibility with APIs that have been removed or deprecated in OpenSSL 3.0+, and adds support for MINGW64 in msys2.

Changes Made

  • Added OpenSSL 1.1.1 as a submodule under folder openssl
  • Modified Makefile to:
    • Build OpenSSL 1.1.1 as part of the project
    • Link against the custom-built OpenSSL instead of the system one
  • Support MINGW64 in Windows

Reason

The project relies on several OpenSSL APIs that were removed in 3.0+, such as EVP_MD_CTX_md_data(). This causes the program to crash with a segmentation fault during SM3 tests when linked against the system-installed OpenSSL 3.0+, and similar functionality has not been made available in OpenSSL 3.0+.

Yi Hao and others added 13 commits April 8, 2025 05:44
Some functions used by the project are no longer available in OpenSSL 3.0+.
To ensure compatibility and avoid refactoring, OpenSSL 1.1.1 was added as a submodule.
Makefile was updated to build and link against the submodule version correctly.
Add SM3 to README.md
- Use --depth 1 to perform shallow clone, fetching only the latest commit
- Use --single-branch to limit submodule cloning to a single branch
…te warning about missing jobserver and speed up compiling
@iagox86 iagox86 self-assigned this Apr 18, 2025
@iagox86 iagox86 self-requested a review April 18, 2025 21:46
@eWloYW8 eWloYW8 force-pushed the master branch 8 times, most recently from 8047c83 to d3f0c5c Compare April 29, 2025 11:41
@eWloYW8 eWloYW8 force-pushed the master branch 2 times, most recently from 1b87b53 to 4b2e408 Compare May 3, 2025 10:55
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.

2 participants