Skip to content

Add misc derivations#571

Merged
halderen merged 4 commits into
softhsm:developfrom
realmfoo:misc_derivations
Feb 2, 2021
Merged

Add misc derivations#571
halderen merged 4 commits into
softhsm:developfrom
realmfoo:misc_derivations

Conversation

@realmfoo
Copy link
Copy Markdown
Contributor

Added derive support with CKM_CONCATENATE_DATA_AND_BASE, CKM_CONCATENATE_BASE_AND_DATA and CKM_CONCATENATE_BASE_AND_KEY.

Comment thread src/lib/SoftHSM.cpp Outdated
Comment thread src/lib/SoftHSM.cpp
Comment thread src/lib/SoftHSM.cpp
Comment thread src/lib/test/DeriveTests.cpp
Signed-off-by: realmfoo <konstantin.leboev@gmail.com>
Signed-off-by: realmfoo <konstantin.leboev@gmail.com>
Signed-off-by: realmfoo <konstantin.leboev@gmail.com>
…AND_KEY

Added tests for CKA_EXTRACTABLE and CKA_SENSITIVE attributes.

Signed-off-by: realmfoo <konstantin.leboev@gmail.com>
@realmfoo
Copy link
Copy Markdown
Contributor Author

realmfoo commented Oct 8, 2020

@Aearsis please let me know if you need any additional actions from my end...

@Aearsis
Copy link
Copy Markdown
Contributor

Aearsis commented Oct 8, 2020

@Aearsis please let me know if you need any additional actions from my end...

Don't be mistaken, I'm not a maintainer :) This is more for @halderen or @rijswijk .

@rijswijk
Copy link
Copy Markdown
Contributor

Thanks for the contribution, I reviewed the code, it looks good to me. Asking @halderen if he concurs, and if so, to merge it into develop

@rijswijk rijswijk requested a review from halderen October 15, 2020 06:50
@halderen
Copy link
Copy Markdown
Contributor

halderen commented Feb 2, 2021

Reviewed and contains the checks regarding the sensitive and extractable attributes, that are absolutely needed.

@halderen halderen merged commit b7e2aeb into softhsm:develop Feb 2, 2021
@realmfoo realmfoo deleted the misc_derivations branch February 3, 2021 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants