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
check the return value of ossl_bio_new_from_core_bio #17154
Conversation
Changed them. I apologize for my negligence as I just keep the original logic but do not fix them to the correct route such as |
This seems to have been forgotten... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good - although there is now a conflict which needs resolving.
1ac9b06
to
9226cc9
Compare
Ah, you used a merge commit. Unfortunately we do not accept merge commits. You need to rebase on top of the latest master instead. |
450d3e1
to
76ed26a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
24 hours has passed since 'approval: done' was set, but this PR has failing CI tests. Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually. |
There are missing checks of its return value in 8 different spots. Reviewed-by: Matt Caswell <matt@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from #17154)
Squashed, fixed end-of-line whitespace issues, merged to master and 3.0 branches. Thank you for your contribution. |
check the return value of ossl_bio_new_from_core_bio in 8 different spots:
Note: I am not very familiar with these functions, but I had tried to keep their functionalities as the original. For example, I tracked the return value of the subsequent function(such as
i2b_PVK_bio_ex()
inencode_key2ms.c:58
, etc) when an error happens is -1, so I set the return value to -1 when the return value ofossl_bio_new_from_core_bio()
inencode_key2ms.c:57
is NULL.