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

CryptoPkg updates for newer openssl 3.x versions #6185

Merged
merged 8 commits into from
Sep 25, 2024

Conversation

kraxel
Copy link
Member

@kraxel kraxel commented Sep 10, 2024

  • CrtLibSupport: add sleep()
  • CrtLibSupport: fix gettimeofday()
  • CrtLibSupport: factor out EFI_TIME -> time_t calculation to new function
  • CrtLibSupport: add mktime()
  • CrtLibSupport: add timezone
  • CrtLibSupport: add intptr_t
  • openssl: add Library/OpensslLib/openssl to includes, drop e_os.h hack

Will be needed by openssl-3.2.x

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Turn gettimeofday() into a proper function with return value.

Will be needed by openssl-3.2.x

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
No functional change.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Will be needed by openssl-3.2.x

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Will be needed by openssl-3.2.x

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Will be needed by openssl-3.4.x

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Adding $(OPENSSL_PATH)/e_os.h to the list of source files had the effect
that $(OPENSSL_PATH)/ was added to the list of include directories.
With the file being gone in openssl-3.2.x this doesn't work any more.

Add the directory to the [Includes.Common.Private] section instead.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
@kraxel kraxel mentioned this pull request Sep 10, 2024
@kraxel kraxel marked this pull request as ready for review September 10, 2024 19:06
@jyao1
Copy link
Contributor

jyao1 commented Sep 13, 2024

LGTM

@jyao1 jyao1 added the push Auto push patch series in PR if all checks pass label Sep 25, 2024
@mergify mergify bot merged commit 39462fc into tianocore:master Sep 25, 2024
126 checks passed
@kraxel kraxel deleted the devel/openssl-prep branch December 13, 2024 08:19
mdkinney added a commit to mdkinney/edk2 that referenced this pull request Jan 12, 2025
Move local variable init to C statements to follow
coding standard and remove the use of field names in
structure initialization to maximize compiler compatibility.

This issue was introduced by PR tianocore#6185

Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
mergify bot pushed a commit that referenced this pull request Jan 13, 2025
Move local variable init to C statements to follow
coding standard and remove the use of field names in
structure initialization to maximize compiler compatibility.

This issue was introduced by PR #6185

Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
niruiyu pushed a commit to niruiyu/edk2 that referenced this pull request Feb 19, 2025
(cherry picked from commit 4218026)

Move local variable init to C statements to follow
coding standard and remove the use of field names in
structure initialization to maximize compiler compatibility.

This issue was introduced by PR tianocore#6185

Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
(cherry picked from commit 4218026)

Change-Id: I936e3268daef1c75e74d760869d3b82ec46dd3f1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
push Auto push patch series in PR if all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants