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

RSA decrypt 1a: PK #5762

Closed
2 tasks
mpg opened this issue Apr 21, 2022 · 0 comments · Fixed by #5772
Closed
2 tasks

RSA decrypt 1a: PK #5762

mpg opened this issue Apr 21, 2022 · 0 comments · Fixed by #5772
Assignees
Labels
enhancement size-s Estimated task size: small (~2d)

Comments

@mpg
Copy link
Contributor

mpg commented Apr 21, 2022

Opaque RSA keys introduced in #5625 and extended in #5711 can only do signatures, with mbedtls_pk_sign[_ext](). This task is to extend them to also support decryption (PKCS#1v1.5, not OAEP) with mbedtls_pk_decrypt().

  • Implement pk_opaque_decrypt() in pk_wrap.c and register it in mbedtls_pk_rsa_opaque_info.
  • Make sure it's tested, either by extending and existing test function or writing a new one, as is more convenient. Testing can be done using known values as decryption is deterministic.

Depends on: #5761

@mpg mpg added enhancement Product Backlog size-s Estimated task size: small (~2d) labels Apr 21, 2022
@mpg mpg changed the title RSA decrypt 1: PK RSA decrypt 1a: PK Apr 21, 2022
@superna9999 superna9999 self-assigned this Apr 21, 2022
@superna9999 superna9999 mentioned this issue Apr 22, 2022
1 task
@mpg mpg closed this as completed in #5772 May 11, 2022
mpg added a commit to mpg/mbedtls that referenced this issue Jun 7, 2022
- misc updates about on-going/recent work
- removal of the section about mixed-PSK: being done in Mbed-TLS#5762
- clarifications in some places
- some typo fixes

Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
mpg added a commit to mpg/mbedtls that referenced this issue Jul 4, 2022
- misc updates about on-going/recent work
- removal of the section about mixed-PSK: being done in Mbed-TLS#5762
- clarifications in some places
- some typo fixes

Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement size-s Estimated task size: small (~2d)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants