ARMmbed / ARMmbed/mbed-crypto

Make PSA entropy injection easier to use

Open
#147 3 comments 0 reactions 0 assignees View on GitHub
enhancement Jira status: REOPENED mirrored type: enhancement
Dominant language
C
Stars
108
Forks
95
PR merge metrics
No merged PRs in 30d

Description

### Description

Currently, a user must [define their own NV seed read and write callbacks when they enable PSA entropy injection](https://github.com/ARMmbed/mbed-crypto/blob/development/include/psa/crypto_extra.h#L157-L158). Also, they must ensure that their NV seed read and write callbacks are compatible with how the seed is written from Mbed Crypto. This is bad coupling and makes the feature too hard to use.

Instead, provide default, Mbed-Crypto-compatible NV seed read and write callbacks, automatically register them with the Mbed TLS NV Seed system, and error if any other NV Seed configuration options are set (as only one set of NV Seed callbacks can be present in any given system).

### Issue request type
[ ] Question
[x] Enhancement
[ ] Bug

Contributor guide

Open the contributing guide

Research direction

Start with include/psa/crypto_extra.h and trace how PSA entropy injection currently connects to the Mbed TLS NV Seed system. Review the existing NV seed callback configuration and determine the required conflict checks; the work is done when compatible defaults are registered automatically and other NV Seed options produce an error.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
security
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.