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
Fix coverity 1516093 tainted scalar #19519
Conversation
|uclen| is created from three byte values, so this seems a bit redundant, but if it makes coverity happy
Second attempt at fixing this issue. |
Is the CIFuzz cancel (really a crash?) relevant? |
24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually. |
Merged to master branch. Thank you for your contribution. |
|uclen| is created from three byte values, so this seems a bit redundant, but if it makes coverity happy Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from #19519)
Thank you! |
Looks as though Coverity is clean now. |
|uclen| is created from three byte values, so this seems a bit redundant, but if it makes coverity happy Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from openssl#19519)
|uclen| is created from three byte values, so this seems a bit redundant, but if it makes coverity happy
Checklist