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

CC RSA ALT #1229

Closed
wants to merge 18 commits into from
Closed

CC RSA ALT #1229

wants to merge 18 commits into from

Conversation

NirEkhauz
Copy link
Contributor

Description

Addition/Changes to support the RSA _ALT for Crypto-Cell

Status

READY

Todos

  • Tests
  • Documentation
  • Changelog updated
  • Backported

roeefriedman and others added 14 commits December 17, 2017 15:18
Start Cherry Picking in order to gather all our relevant changes to form the RSA_ALT
    - non-standard modulus size
    - encrypt with private key/decrypt with public key
- Added relevant exclude files
Adapt mbedtls tests to comply with CC restrictions in rsa-verify
Cherry-pick of all CC changes w/o the  .exclude files
JIRA:PJ02447SW-503
CC does not support Signing a non hashed message.
Disabling mbedtls tests that try to sign a message without hashing.

Jira:PJ02447SW-502
Update 'version_features.c' file with new RSA related defines in config.h
Fix config.h file due to wrong define name
Restore config.h #Defines former status due to un-planned changes
Close some #Defines in config.h due to Travis failures
Close all new added #DEFINES in order to re-trigger Travis.
@mbed-ci
Copy link

mbed-ci commented Dec 24, 2017

Automatic CI verification build not done, please verify manually.

@Patater
Copy link
Contributor

Patater commented Jan 22, 2018

Hi @NirEkhauz,

Please verify that #1060 meets your needs and if so, we can close this PR.

Thanks

@simonbutcher
Copy link
Contributor

@NirEkhauz - Is there anything in this PR which isn't present in #1060 which you'd like to submit as a new PR? You seem to have added tests and other things.

@simonbutcher
Copy link
Contributor

Confirmed offline with @NirEkhauz that nothing else needs to be merged, so closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants