Skip to content

Updated code. #17

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.

Sign up for GitHub

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

Jump to bottom
Merged
divegeek merged 14 commits into from
Feb 9, 2021
Merged

Updated code. #17

divegeek merged 14 commits into from
Feb 9, 2021

Conversation

@BKSSMVenkateswarlu
Copy link
Contributor

@BKSSMVenkateswarlu BKSSMVenkateswarlu commented Feb 7, 2021

  1. MasterKey, AttestationKey and PreSharedKey are stored as Key objects instead of byte array
    and they are maintained by SEProvider.
  2. Changed the Keyderivation algorithm from AESGCM to Hmac sign.
  3. Removed commented code and unused code.
  4. Added documentation and copyright

bvenkateswarlu and others added 14 commits February 5, 2021 19:34
store secrets as keyobjects instead as byte arrays.
6ab0667
Updated the documentation comments
64e37fa
Changed the key derivation alogorithm to hmac sign
2699c29 
Root of trust now contains VerifiedBootKey, VerifiedBootKeyHash,
bootState, deviceLocked state.
1. Removed unused functions in KMAESKey.java
b27520b 
2. Renamed the KMSEProvider function names of hmacSign and cmacKdf
@BKSSMVenkateswarlu
Update JavacardKeymaster4Device.cpp
b6f8ce6 
Changed the Javacard keymaster name to JavacardKeymaster4.1Device v1.0
Added Copyright comment
cc53a65
Removed commented code
3fecf13
@BKSSMVenkateswarlu
Merge pull request #47 from BKSSMVenkateswarlu/secrets_as_keyobjects
04b3b70 
Secrets as keyobjects
Added few debug statements
af0de2f 
Added rsaPkcs1Success test in KMFunctionalTest
Fixed the issue with RSA private and public key buffer while copying
530cb67
Added testDesEcbPkcs7PaddingCorrupted test function
d4928a7
Removed unused imports
182ca26
Removed unused/commented code.
c766d68
@BKSSMVenkateswarlu
Merge pull request #48 from BKSSMVenkateswarlu/test_master
4c4c49d 
Latest code.
@divegeek divegeek merged commit e49cff4 into divegeek:master Feb 9, 2021
subrahmanyaman referenced this pull request in subrahmanyaman/JavaCardKeymaster Aug 26, 2021
Merge pull request #17 from BKSSMVenkateswarlu/master
ec9721d 
Updated code.
mdwivedi pushed a commit that referenced this pull request Jan 14, 2022
@subrahmanyaman
Merge pull request #17 from AvinashHedage/keymint_fixes_2312
4ea2a23 
Confirmation token HAL changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@BKSSMVenkateswarlu @divegeek