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

archive_digest: check return value of EVP_DigestInit() #1717

Merged
merged 1 commit into from
Apr 6, 2022

Conversation

evelikov
Copy link
Collaborator

@evelikov evelikov commented Apr 5, 2022

The function returns 0 on error, which is seemingly very common with
OpenSSL 3.0 and rmd160.

Just error check the lot, so we don't get even more random failures with
future releases of OpenSSL.

Fixes #1549

Signed-off-by: Emil Velikov emil.l.velikov@gmail.com

The function returns 0 on error, which is seemingly very common with
OpenSSL 3.0 and rmd160.

Just error check the lot, so we don't get even more random failures with
future releases of OpenSSL.

Fixes libarchive#1549

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
@mmatuska mmatuska merged commit 5be68f1 into libarchive:master Apr 6, 2022
@evelikov evelikov deleted the ssl branch April 6, 2022 23:31
espindula pushed a commit to espindula/br-blfs that referenced this pull request Dec 9, 2022
The test failure with OpenSSL 3 is fixed by upstream:
libarchive/libarchive#1717
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test case for rmd160 failing when used with OpenSSL3
2 participants