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

Standardize the PSA -> Mbed TLS error translation across Mbed TLS modules #6158

Closed
AndrzejKurek opened this issue Jul 29, 2022 · 0 comments · Fixed by #6829
Closed

Standardize the PSA -> Mbed TLS error translation across Mbed TLS modules #6158

AndrzejKurek opened this issue Jul 29, 2022 · 0 comments · Fixed by #6829
Assignees
Labels
component-psa PSA keystore/dispatch layer (storage, drivers, …) enhancement size-m Estimated task size: medium (~1w)

Comments

@AndrzejKurek
Copy link
Contributor

When working on #6141 (comment), an idea emerged to have a single helper function to translate errors from the PSA format to Mbed TLS format, module-aware.
The new function would accept the module origin as a parameter and would first handle module-specific cases, and if the error doesn't match, a general translation of module-agnostic errors that are uniform, no matter the caller origin would occur.

The work to be done in this issue is as follows:

  • Determine whether such function is possible to be created and mantained;
  • Determine whether it is worth creating such function - won't it uncontrollably grow? Does it really present added value by having some shared errors across modules, or do the areas not overlap?
  • Create such function and remove other, module-specific ones.
@yanesca yanesca added enhancement component-psa PSA keystore/dispatch layer (storage, drivers, …) labels Aug 1, 2022
@mpg mpg added the size-m Estimated task size: medium (~1w) label Sep 5, 2022
@AndrzejKurek AndrzejKurek self-assigned this Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component-psa PSA keystore/dispatch layer (storage, drivers, …) enhancement size-m Estimated task size: medium (~1w)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants