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.

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

Recent Coverity issues fixes #14928

Closed
wants to merge 4 commits into from
Closed

Recent Coverity issues fixes #14928

wants to merge 4 commits into from

Conversation

t8m
Copy link
Member

@t8m t8m commented Apr 19, 2021

No description provided.

@t8m t8m added approval: review pending This pull request needs review by a committer branch: master Merge to master branch labels Apr 19, 2021
@t8m t8m added this to the 3.0.0 beta1 milestone Apr 19, 2021
@mattcaswell mattcaswell added the triaged: OTC evaluated This issue/pr was triaged by OTC label Apr 20, 2021
Also add possible number truncation check.
@paulidale paulidale 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 Apr 21, 2021
@openssl-machine
Copy link
Collaborator

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.

openssl-machine pushed a commit that referenced this pull request Apr 22, 2021
Fixes Coverity CID 1476284

Also add possible number truncation check.

Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #14928)
openssl-machine pushed a commit that referenced this pull request Apr 22, 2021
Fixes Coverity ID 1476282

Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #14928)
openssl-machine pushed a commit that referenced this pull request Apr 22, 2021
Fixes Coverity ID 1476283

Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #14928)
@t8m
Copy link
Member Author

t8m commented Apr 22, 2021

Merged to master. Thank you for the reviews and comments.

@t8m t8m closed this Apr 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch triaged: OTC evaluated This issue/pr was triaged by OTC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants