Skip to content
This repository was archived by the owner on May 24, 2026. It is now read-only.

Jc keymaster 41 android 11 - #24

Merged
mdwivedi merged 6 commits into
divegeek:jc_keymaster_41_android_11from
BKSSMVenkateswarlu:jc_keymaster_41_android_11
Mar 23, 2021
Merged

Jc keymaster 41 android 11#24
mdwivedi merged 6 commits into
divegeek:jc_keymaster_41_android_11from
BKSSMVenkateswarlu:jc_keymaster_41_android_11

Conversation

@BKSSMVenkateswarlu

Copy link
Copy Markdown
Contributor

No description provided.

BKSSMVenkateswarluand others added 3 commits March 11, 2021 18:29
Allow upgrade if any of the boot parameters gets changed.
Signed-off-by: BKSSM Venkateswarlu <venki.manikanta@gmail.com>
KMException.throwIt(KMError.INVALID_ARGUMENT);
}
}
return (byte) 0;

@mdwivedimdwivediMar 19, 2021

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you fail the else if at line 1111 and versions are valid then you will need to ruturn 1 and also if line 1104 is false then you will need to return 0 (by deafult).

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resolved.

@mdwivedi
mdwivedi merged commit e5fd227 into divegeek:jc_keymaster_41_android_11Mar 23, 2021
mdwivedi pushed a commit that referenced this pull request Mar 31, 2021
Merge pull request #24 from BKSSMVenkateswarlu/jc_keymaster_41_androi…
subrahmanyaman referenced this pull request in subrahmanyaman/JavaCardKeymaster Aug 26, 2021
subrahmanyaman referenced this pull request in subrahmanyaman/JavaCardKeymaster Aug 26, 2021
Merge pull request #24 from BKSSMVenkateswarlu/jc_keymaster_41_androi…
mdwivedi pushed a commit that referenced this pull request Jan 14, 2022
Handled attest key purpose validation in generate key and import key
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@BKSSMVenkateswarlu@mdwivedi