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

Update MbedTLS 3.4.0 -> 3.6.2 and set it as default #6

Closed
wants to merge 1 commit into from

Conversation

gpoulios
Copy link

Summary

Even though there is CONFIG_MBEDTLS_VERSION, this commit is needed for 2 reasons:

  1. The patches: 0001 works almost as-is (with fuzz 2) but 0002 hunk #​4 failed and had to apply it manually. Otherwise, the patches are identical.
  2. The changes in mbedtls_config.h: Following the same approach for the Kconfig as in the original contribution, this commit updates both Kconfig and mbedtls_config.h according to:
mbedtls$ git diff v3.4.0...v3.6.2 -- include/mbedtls/mbedtls_config.h

Impact

None for projects not using MbedTLS.

For projects using MbedTLS, please check your CONFIG_MBEDTLS_* settings for any options affected by this commit, or MbedTLS's git diff v3.4.0...v3.6.2 -- include/mbedtls/mbedtls_config.h. Besides that, numerous security fixes.

Testing

Builds fine and seems to work fine but I haven't performed extensive testing: I've only run tests on RISC-V.

Even though one could in theory simply set CONFIG_MBEDTLS_VERSION="3.6.2",
this commit is needed for 2 reasons:
  1. The patches need to be updated: 0001 works almost as-is (with fuzz 2)
     but 0002 hunk #4 fails and needed adjustment. Otherwise, the patches
     are identical.
  2. mbedtls_config.h needs to be updated: Following the same approach
     for the Kconfig this commit updates both Kconfig and mbedtls_config.h
     according to the changes:
       mbedtls$ git diff v3.4.0...v3.6.2 -- include/mbedtls/mbedtls_config.h

Signed-off-by: George Poulios <[email protected]>
@gpoulios gpoulios closed this Nov 30, 2024
@gpoulios gpoulios removed the request for review from jlaitine November 30, 2024 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant