Skip to content

Coverity: fix error handling issues #20132

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
Closed
wants to merge 2 commits into from
Closed

Coverity: fix error handling issues #20132

wants to merge 2 commits into from

Conversation

paulidale
Copy link
Contributor

Fixing Coverity 1520505 and 1520506.

  • documentation is added or updated
  • tests are added or updated

@paulidale paulidale added branch: master Merge to master branch approval: review pending This pull request needs review by a committer approval: otc review pending branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 labels Jan 25, 2023
@paulidale paulidale self-assigned this Jan 25, 2023
@paulidale paulidale added the tests: exempted The PR is exempt from requirements for testing label Jan 25, 2023
@t8m t8m added triaged: bug The issue/pr is/fixes a bug and removed approval: otc review pending labels Jan 25, 2023
@slontis slontis added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Jan 27, 2023
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Jan 28, 2023
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@paulidale
Copy link
Contributor Author

Merged, thanks for the reviewing.

@paulidale paulidale closed this Jan 29, 2023
@paulidale paulidale deleted the cov-25-1-23 branch January 29, 2023 21:36
openssl-machine pushed a commit that referenced this pull request Jan 29, 2023
@paulidale
coverity 1520505: error handling
00407fb 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from #20132)
openssl-machine pushed a commit that referenced this pull request Jan 29, 2023
@paulidale
coverity 1520506: error handling
a4347a9 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from #20132)
openssl-machine pushed a commit that referenced this pull request Jan 29, 2023
@paulidale
coverity 1520505: error handling
2680cd2 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from #20132)

(cherry picked from commit 00407fb)
openssl-machine pushed a commit that referenced this pull request Jan 29, 2023
@paulidale
coverity 1520506: error handling
da6d418 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from #20132)

(cherry picked from commit a4347a9)
openssl-machine pushed a commit that referenced this pull request Jan 29, 2023
@paulidale
coverity 1520505: error handling
4476dca 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from #20132)

(cherry picked from commit 00407fb)
openssl-machine pushed a commit that referenced this pull request Jan 29, 2023
@paulidale
coverity 1520506: error handling
00e8275 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from #20132)

(cherry picked from commit a4347a9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t8m t8m t8m approved these changes

@slontis slontis slontis approved these changes

Assignees

@paulidale paulidale

Labels
approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Merge to master branch branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@paulidale @openssl-machine @t8m @slontis