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

sys/hashes: support for SHA-{384,512/{224,256}} #20484

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

mguetschow
Copy link
Contributor

Contribution description

This PR adds support for the most common truncated versions of SHA512 (SHA-512/224, SHA-512/256 and SHA-384), following the example of the SHA-224 implementation.

Testing procedure

Respective tests are added to unittests.

Issues/PRs references

First part of #20127

Copy link
Contributor

@Teufelchen1 Teufelchen1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐈‍⬛ Looks good, very few comments!

sys/hashes/sha256.c Outdated Show resolved Hide resolved
sys/hashes/sha384.c Show resolved Hide resolved
sys/hashes/sha512_224.c Outdated Show resolved Hide resolved
sys/hashes/sha512_common.c Show resolved Hide resolved
sys/hashes/sha512_common.c Show resolved Hide resolved
@Teufelchen1 Teufelchen1 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Mar 20, 2024
@mguetschow
Copy link
Contributor Author

Should be ready for another round of review.

@riot-ci
Copy link

riot-ci commented Mar 20, 2024

Murdock results

✔️ PASSED

9300007 sys/hashes: support for SHA-{384,512/{224,256}}

Success Failures Total Runtime
10009 0 10009 10m:41s

Artifacts

@github-actions github-actions bot added the Area: pkg Area: External package ports label Mar 22, 2024
@mguetschow
Copy link
Contributor Author

The last commit is a one-to-one translation of #19969 (comment).

If Murdock runs through, may I squash?

@Teufelchen1
Copy link
Contributor

Yes yes, please squash!

pkg/esp32_sdk: rename sha384_init to avoid name clash
@mguetschow
Copy link
Contributor Author

Done :)

@Teufelchen1 Teufelchen1 added this pull request to the merge queue Mar 25, 2024
Merged via the queue into RIOT-OS:master with commit 4982049 Mar 25, 2024
25 checks passed
@mguetschow mguetschow deleted the hashes-sha512-trunc branch March 25, 2024 13:58
@mguetschow
Copy link
Contributor Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: pkg Area: External package ports Area: sys Area: System Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants