Skip to content

Fixes crash when providing a pkcs12 file with only one private key in #215

Open
unier wants to merge 1 commit intoyourkarma:masterfrom
unier:master
Open

Fixes crash when providing a pkcs12 file with only one private key in #215
unier wants to merge 1 commit intoyourkarma:masterfrom
unier:master

Conversation

@unier
Copy link

@unier unier commented Sep 24, 2019

Added a patch to make sure the pkcs12 that were opened actually has the number of private - certificate pairs that are needed to continue. If a pkcs12 that was created with only one private key in it and no certificate was tried used, this just crashed when accessing index 0 with the CFArrayGetValueAtIndex

…he number of private - certificate pairs that are needed to continue. If a pkcs12 that was created with only one private key in it and no certificate was tried used, this just crashed when accessing index 0 with the CFArrayGetValueAtIndex
@lolgear
Copy link
Collaborator

lolgear commented Sep 24, 2019

Hi!
Thank you for PR!
Could you also add tests for this case?

@lolgear lolgear mentioned this pull request Jun 8, 2021
8 tasks
@lolgear
Copy link
Collaborator

lolgear commented Jun 8, 2021

@unier
If you don't mind I added this PR via combining it with another PR.
I guess this PR could be closed.
Could you check latest master?

Thanks!

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