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

Digest testing: improve #8501

Merged
merged 1 commit into from
Feb 25, 2025
Merged

Conversation

SparkiDev
Copy link
Contributor

@SparkiDev SparkiDev commented Feb 24, 2025

Description

Make testing digests consistent.
Add KATs for all digests.
Check unaligned input and output works.
Perform chunking tests for all digests.

Fix Blake2b and Blake2s to checkout parameters in update and final functions.
Fix Shake256 and Shake128 to checkout parameters in absorb and squeeze blocks functions.

Add default digest size enums for Blake2b and Blake2s.

Testing

<clone wolfssl and wolfsm into same directory>
cd wolfsm; ./install.sh
cd ../wolfssl
./autogen.sh
./configure '--disable-shared' '--enable-md5' '--enable-sha' '--enable-shake128' '--enable-shake256' '--enable-blake2' '--enable-blake2s' '--enable-sm3' '--enable-ripemd' '--enable-hashflags' '--enable-opensslall' '--enable-debug'
make
./tests/unit.test --api

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@SparkiDev SparkiDev self-assigned this Feb 24, 2025
@SparkiDev SparkiDev force-pushed the digest_test_rework branch 11 times, most recently from 9c29c5f to b1ad757 Compare February 25, 2025 08:00
Make testing digests consistent.
Add KATs for all digests.
Check unaligned input and output works.
Perform chunking tests for all digests.

Fix Blake2b and Blake2s to checkout parameters in update and final
functions.
Fix Shake256 and Shake128 to checkout parameters in absorb and squeeze
blocks functions.

Add default digest size enums for Blake2b and Blake2s.
@dgarske dgarske merged commit 3557cc7 into wolfSSL:master Feb 25, 2025
178 checks passed
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.

3 participants