Skip to content
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

key zeroisation fix for p12 #7109

Closed
wants to merge 1 commit into from
Closed

key zeroisation fix for p12 #7109

wants to merge 1 commit into from

Conversation

slontis
Copy link
Member

@slontis slontis commented Sep 4, 2018

No description provided.

@InfoHunter InfoHunter added the approval: done This pull request has the required number of approvals label Sep 4, 2018
@mattcaswell
Copy link
Member

What branches?

@mattcaswell mattcaswell added this to the Assessed milestone Sep 4, 2018
@levitte
Copy link
Member

levitte commented Sep 4, 2018

Just as for the other zeroization PRs, this should be cherry-picked to 1.1.0 as well

@levitte levitte added branch: master Merge to master branch 1.1.0 branch: 1.1.1 Merge to OpenSSL_1_1_1-stable branch labels Sep 4, 2018
levitte pushed a commit that referenced this pull request Sep 4, 2018
@slontis @paulidale
key zeroisation fix for p12
f5cee41 
Reviewed-by: Paul Yang <yang.yang@baishancloud.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from #7109)
levitte pushed a commit that referenced this pull request Sep 4, 2018
@slontis @paulidale
key zeroisation fix for p12
febb025 
Reviewed-by: Paul Yang <yang.yang@baishancloud.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from #7109)

(cherry picked from commit f5cee41)
@paulidale
Copy link
Contributor

Merged to 1.1.0 and master. Thanks all.

@paulidale paulidale closed this Sep 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@InfoHunter InfoHunter InfoHunter approved these changes

@paulidale paulidale paulidale approved these changes

Assignees
No one assigned
Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch branch: 1.1.1 Merge to OpenSSL_1_1_1-stable branch
Projects
None yet
Milestone
Assessed
Development

Successfully merging this pull request may close these issues.
5 participants
@slontis @mattcaswell @levitte @paulidale @InfoHunter