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
x509_vfy: remove redundant stack allocation #14161
Conversation
Fix CID 1472833 by removing a codepath that attempts to allocate a stack if not already allocated, when the stack was already allocated unconditionally a few lines previously. Interestingly enough, this additional allocation path (and the comment describing the need for it) were added in commit 69664d6, also prompted by Coverity(!). It seems that the intervening (and much more recent) commit d53b437 that allowed sk_X509_dup() to accept a NULL argument allowed the earlier initialization path to unconditionally allocate a stack, rendering this later allocation fully redundant.
I know it is painful, but could you also update the triage fields in coverity please.. |
Update (force) pushed; hopefully the re-review burden is not too big.
Gosh, I will have to remember where my account credentials live. I think I actually have one, though I'm not even 100% sure of that... |
How did you view the issues? You can sign in with github credentials. |
There's an email summary that gets sent periodically to ... oh hey, the email address on my github account. I must have enabled email notification for something, somewhere. |
This pull request is ready to merge |
Fix CID 1472833 by removing a codepath that attempts to allocate a stack if not already allocated, when the stack was already allocated unconditionally a few lines previously. Interestingly enough, this additional allocation path (and the comment describing the need for it) were added in commit 69664d6, also prompted by Coverity(!). It seems that the intervening (and much more recent) commit d53b437 that allowed sk_X509_dup() to accept a NULL argument allowed the earlier initialization path to unconditionally allocate a stack, rendering this later allocation fully redundant. Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Shane Lontis <shane.lontis@oracle.com> (Merged from #14161)
Thanks for fixing. Merged to master. |
Fix CID 1472833 by removing a codepath that attempts to allocate a
stack if not already allocated, when the stack was already allocated
unconditionally a few lines previously.