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 crypto submodule #2953

Merged

Conversation

gilles-peskine-arm
Copy link
Contributor

Fixes the failure of check-generates-files in the TLS job on the Mbed Crypto repository.

Missed listing in the previous submodule update:

* Mbed-TLS#321: Replace config.pl by config.py
* Mbed-TLS#322: Update Mbed Crypto with latest Mbed TLS changes as of 2019-11-15
* Mbed-TLS#308: Small performance improvement of mbedtls_mpi_div_mpi()
* Mbed-TLS#324: test_psa_constant_names: support key agreement, better code structure
* Mbed-TLS#320: Link to the PSA crypto portal page from README.md
* Mbed-TLS#293: Always gather MBEDTLS_ENTROPY_BLOCK_SIZE bytes of entropy
* Mbed-TLS#310: Clarify test descriptions in test_suite_memory_buffer_alloc
* Mbed-TLS#307: Add ASN.1 ENUMERATED tag support
* Mbed-TLS#328: Remove dependency of crypto_values.h on crypto_extra.h
* Mbed-TLS#325: Rename psa_asymmetric_{sign_verify} to psa_{sign,verify}_hash

Missed listing in the previous submodule update:

* Mbed-TLS#304: Make sure Asan failures are detected in 'make test'
@gilles-peskine-arm gilles-peskine-arm added mbed TLS team needs-review Every commit must be reviewed by at least two team members, labels Dec 6, 2019
@gilles-peskine-arm
Copy link
Contributor Author

CI is ok: the only failure is an expected API change.

Copy link
Contributor

@yanesca yanesca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated the submodule on development head, there were no crypto changes in the diff.

@AndrzejKurek AndrzejKurek self-requested a review December 12, 2019 09:27
@Patater
Copy link
Contributor

Patater commented Dec 12, 2019

CI failure is only API compatibility with psa_asymmetric_verify() and psa_asymmetric_sign(), which are pre-release PSA functions. Merging.

@Patater Patater removed the needs-review Every commit must be reviewed by at least two team members, label Dec 12, 2019
@Patater Patater merged commit 88d1c05 into Mbed-TLS:development Dec 12, 2019
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.

4 participants