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.
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
Potential null pointer reference #19429
Potential null pointer reference #19429
Conversation
The proper fix is IMO to fix the bug in the testcase. I.e., put the BN_CTX_start right after the BN_CTX_new() call. |
Yes, I see the same approach for other test cases, but the unclear addition still has the same problem. And I'm going to do it as is customary. |
417d3a2
to
8f7b400
Compare
Please do not add merge commits. Use git rebase against fresh master branch instead. |
cdb32e2
to
173e8db
Compare
Sorry, Git rebase has been redone. |
This pull request is ready to merge |
Merged to master, 3.0, and 3.1 branches. Thank you for your contribution. |
Reviewed-by: Shane Lontis <shane.lontis@oracle.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from #19429)
Reviewed-by: Shane Lontis <shane.lontis@oracle.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from openssl#19429)
BN_CTX_end()-->BN_STACK_pop() may cause a null pointer reference.
Fixes #19428