Skip to content

validate private keys#4271

Merged
tomholub merged 2 commits intomasterfrom
issue-0000-validate-decrypted-secret-key-packets
Jan 27, 2022
Merged

validate private keys#4271
tomholub merged 2 commits intomasterfrom
issue-0000-validate-decrypted-secret-key-packets

Conversation

@rrrooommmaaa
Copy link
Contributor

@rrrooommmaaa rrrooommmaaa commented Jan 26, 2022

This PR validates all private key packets on decrypt or on parse (if they are unencrypted)

close #0000 // if this PR closes an issue


Tests (delete all except exactly one):

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

Copy link
Collaborator

@tomholub tomholub left a comment

Choose a reason for hiding this comment

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

👍

@tomholub
Copy link
Collaborator

@martgil I think you had issue reproducing the problem. You can ask here. Did you try to put the broken key into inbox and then load it? If it succeeded, then with the above PR it should fail - please triage.

@martgil
Copy link
Collaborator

martgil commented Jan 27, 2022

Did you try to put the broken key into inbox and then load it?

Yes, that's exactly what I did.

If it succeeded, then with the above PR it should fail - please triage.

Yes @tomholub, the private key is blocked and no longer accepted when a user tried to import/add it.

Screen Shot 2022-01-27 at 2 02 52 PM

Screen Shot 2022-01-27 at 2 03 37 PM

The android and ios are currently affected by the bug. Maybe they are not aware of it just yet.
I can file a GitHub issue for each repo if you'd like.

@tomholub
Copy link
Collaborator

@martgil yes, please file the issues there. For Android, ping @IvanPizhenko so that he can maybe create a test case for Paul to look at.

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.

3 participants