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

Add PSA inject entropy config option and testing #69

Closed
4 tasks
ronald-cron-arm opened this issue Sep 28, 2023 · 1 comment
Closed
4 tasks

Add PSA inject entropy config option and testing #69

ronald-cron-arm opened this issue Sep 28, 2023 · 1 comment
Labels
enhancement New feature or request size-s Estimated task size: small (~2d)

Comments

@ronald-cron-arm
Copy link
Contributor

ronald-cron-arm commented Sep 28, 2023

  • Add PSA_CRYPTO_INJECT_ENTROPY config option and its documentation in the section "PSA cryptography core configuration options" of crypto_config.h. Similar to MBEDTLS_PSA_INJECT_ENTROPY in mbedtls. Disabled by default.
  • Add its translation to MBEDTLS_PSA_INJECT_ENTROPY in config_adjust_mbedtls_from_psa_crypto.h. MBEDTLS_NO_DEFAULT_ENTROPY_SOURCES needs to be defined here as well.
  • Add a test_psa_inject_entropy test component similar to the one in mbedtls.
  • Check that the test component do what it intends to do and give evidence of this in this issue or the associated PR for the benefit of reviewers

Depends on:

@ronald-cron-arm ronald-cron-arm added enhancement New feature or request size-s Estimated task size: small (~2d) labels Sep 28, 2023
@github-project-automation github-project-automation bot moved this to PSA repo Q4 - all.sh components in Backlog for Mbed TLS Aug 30, 2024
@gilles-peskine-arm
Copy link

We will remove entropy injection in its current form in TF-PSA-Crypto 1.0. This is part of the RNG redesign which will likely not be completed before the 1.0 release, but since it's an API break we will definitely remove the option before 1.0.

Therefore we shouldn't spend any time making it work in TF-PSA-Crypto.

@gilles-peskine-arm gilles-peskine-arm closed this as not planned Won't fix, can't repro, duplicate, stale Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size-s Estimated task size: small (~2d)
Projects
Status: PSA repo Q4 - all.sh components
Development

No branches or pull requests

2 participants