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

Fix error handling in compute_key, BN_CTX_get can return NULL #2132

Closed
wants to merge 1 commit into from
Closed

Fix error handling in compute_key, BN_CTX_get can return NULL #2132

wants to merge 1 commit into from

Conversation

bernd-edlinger
Copy link
Member

This should fix Issue #2131

@kroeckx kroeckx self-assigned this Jan 6, 2017
@kroeckx kroeckx added branch: 1.0.2 Merge to OpenSSL_1_0_2-stable branch 1.1.0 branch: master Merge to master branch labels Jan 6, 2017
@ekasper ekasper added the approval: done This pull request has the required number of approvals label Jan 13, 2017
@levitte
Copy link
Member

levitte commented Jan 24, 2017

Ping. This seems to have been forgotten...

@richsalz
Copy link
Contributor

just merge it, @levitte :)

levitte pushed a commit that referenced this pull request Jan 24, 2017
@bernd-edlinger @levitte
Fix error handling in compute_key, BN_CTX_get can return NULL
7928ee4 
Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from #2132)
levitte pushed a commit that referenced this pull request Jan 24, 2017
@bernd-edlinger @levitte
Fix error handling in compute_key, BN_CTX_get can return NULL
8957add 
Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from #2132)
(cherry picked from commit 7928ee4)
levitte pushed a commit that referenced this pull request Jan 24, 2017
@bernd-edlinger @levitte
Fix error handling in compute_key, BN_CTX_get can return NULL
75249be 
Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from #2132)
(cherry picked from commit 7928ee4)
@levitte
Copy link
Member

levitte commented Jan 24, 2017

Okie, merged.

master: 7928ee4
1.1.0: 75249be
1.0.2: 8957add

@levitte levitte closed this Jan 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kroeckx kroeckx kroeckx approved these changes

@richsalz richsalz richsalz approved these changes

Assignees

@kroeckx kroeckx

Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch branch: 1.0.2 Merge to OpenSSL_1_0_2-stable branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@bernd-edlinger @levitte @richsalz @kroeckx @ekasper