Skip to content

Conversation

@m-gorecki
Copy link
Contributor

This upgrades mbedtls version to v3.62:

  1. mbedtls repository changed to version v3.6.2
  2. Removed tests that are no longer supported
  3. Updated/modified gcm_mynewt API:
    a) Function mbedtls_gcm_update_add is removed, because now
    function mbedtls_gcm_update_ad from mbedtls upstream
    provides the same functionality.
    b) Due to internal mbedtls API changes mbedtls_gcm_setkey_noalloc
    function now needs one argument more, which is keybits.
    This argument specifies length of key in bits and should
    be set to 128, 192 or 256. Other values won't be accepted.
    In earlier version this could be extracted from
    mbedtls_cipher_info_t key_bitlen field, which is no longer
    the case - now it's 4 bit bitfield later processed by
    internal mbedtls macros.
    It probably would be possible to not break this APIs, but
    since a lot of mbedtls APIs where changed in new version
    it was assumed that migration changes in projects using mbedtls
    would be necessary anyway.

@sjanc
Copy link
Contributor

sjanc commented Sep 17, 2025

@vrahane FYI

This upgrades mbedtls version to v3.6.2:
1. mbedtls repository changed to version v3.6.2
2. Removed tests that are no longer supported
3. Updated/modified gcm_mynewt API:
   a) Function mbedtls_gcm_update_add is removed, because now
      function mbedtls_gcm_update_ad from mbedtls upstream
      provides the same functionality.
   b) Due to internal mbedtls API changes mbedtls_gcm_setkey_noalloc
      function now needs one argument more, which is keybits.
      This argument specifies length of key in bits and should
      be set to 128, 192 or 256. Other values won't be accepted.
      In earlier version this could be extracted from
      mbedtls_cipher_info_t key_bitlen field, which is no longer
      the case - now it's 4 bit bitfield later processed by
      internal mbedtls macros.
   It probably would be possible to not break this APIs, but
   since a lot of mbedtls APIs where changed in new version
   it was assumed that migration changes in projects using mbedtls
   would be necessary anyway.
@m-gorecki m-gorecki force-pushed the mbedtls-migr-2 branch 3 times, most recently from 937782a to 8e10037 Compare October 8, 2025 09:52
This adjusts codings style for the latest clang-format requirements
@m-gorecki m-gorecki marked this pull request as ready for review October 8, 2025 15:14
@sjanc
Copy link
Contributor

sjanc commented Oct 23, 2025

#AutoPTS run mynewt SM

@codecoup-tester
Copy link

Scheduled PR #3401 (comment), board: nrf52, estimated start time: 16:48:41, test case count: 64, estimated duration: 0:34:40

Test cases to be runSM/CEN/PROT/BV-01-C
SM/CEN/JW/BV-05-C
SM/CEN/JW/BI-04-C
SM/CEN/JW/BI-01-C
SM/CEN/PKE/BV-01-C
SM/CEN/PKE/BV-04-C
SM/CEN/PKE/BI-01-C
SM/CEN/PKE/BI-02-C
SM/CEN/OOB/BV-01-C
SM/CEN/OOB/BV-03-C
SM/CEN/OOB/BV-09-C
SM/CEN/OOB/BI-01-C
SM/CEN/EKS/BV-01-C
SM/CEN/EKS/BI-01-C
SM/CEN/KDU/BV-05-C
SM/CEN/KDU/BV-06-C
SM/CEN/KDU/BV-10-C
SM/CEN/KDU/BI-04-C
SM/CEN/PIS/BV-02-C
SM/CEN/PIS/BV-03-C
SM/CEN/SCJW/BV-01-C
SM/CEN/SCJW/BV-04-C
SM/CEN/SCJW/BI-01-C
SM/CEN/SCPK/BV-01-C
SM/CEN/SCPK/BV-04-C
SM/CEN/SCPK/BI-01-C
SM/CEN/SCPK/BI-02-C
SM/CEN/SCOB/BV-01-C
SM/CEN/SCOB/BV-04-C
SM/CEN/SCOB/BI-01-C
SM/CEN/SCOB/BI-04-C
SM/PER/PROT/BV-02-C
SM/PER/JW/BV-02-C
SM/PER/JW/BI-03-C
SM/PER/JW/BI-02-C
SM/PER/PKE/BV-02-C
SM/PER/PKE/BV-05-C
SM/PER/PKE/BI-03-C
SM/PER/OOB/BV-02-C
SM/PER/OOB/BV-04-C
SM/PER/OOB/BV-10-C
SM/PER/OOB/BI-02-C
SM/PER/EKS/BV-02-C
SM/PER/EKS/BI-02-C
SM/PER/KDU/BV-01-C
SM/PER/KDU/BV-02-C
SM/PER/KDU/BV-07-C
SM/PER/KDU/BV-08-C
SM/PER/KDU/BI-02-C
SM/PER/KDU/BI-03-C
SM/PER/KDU/BI-04-C
SM/PER/PIS/BV-01-C
SM/PER/PIS/BV-02-C
SM/PER/SCJW/BV-02-C
SM/PER/SCJW/BV-03-C
SM/PER/SCJW/BI-02-C
SM/PER/SCPK/BV-02-C
SM/PER/SCPK/BV-03-C
SM/PER/SCPK/BI-03-C
SM/PER/SCPK/BI-04-C
SM/PER/SCOB/BV-02-C
SM/PER/SCOB/BV-03-C
SM/PER/SCOB/BI-02-C
SM/PER/SCOB/BI-03-C

@codecoup-tester
Copy link

AutoPTS Bot failed:
Reason unknown

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants