RIOT icon indicating copy to clipboard operation
RIOT copied to clipboard

sys/psa_crypto: SHA-{384,512/{224,256}}

Open mguetschow opened this issue 10 months ago • 2 comments

Contribution description

Since #20484, RIOT contains software-implementations for the most commonly used truncated versions of SHA-512. This PR adds glue code to enable usage via the PSA Crypto API.

Sidenote: Although the nRF5 SDK documents support for SHA-384, it appears not to be supported on the nrf52840 CRYPTOCELL, as it returns CRYS_HASH_ILLEGAL_OPERATION_MODE_ERROR. This is also confirmed in this thread. They also state that this board actually does not support SHA-512 in hardware either. Maybe it's worth disabling the periph feature for SHA-512 in that case, too?

Testing procedure

make -C examples/psa_crypto all flash term

Issues/PRs references

Second part of #20127, follow-up of #20484

There's also some interdependence with #20545, one of them should be updated after the other has been merged @Einhornhool

Closes #20127

mguetschow avatar Apr 19 '24 11:04 mguetschow

Murdock results

:heavy_check_mark: PASSED

6be95ca482de471d89dba94763df80f24bef1f45 tests/sys/psa_crypto_hashes: sync with content from examples/psa_crypto

Success Failures Total Runtime
10104 0 10105 13m:34s

Artifacts

riot-ci avatar Apr 19 '24 11:04 riot-ci

Actually, let's defer this to after #20545, I'd get some more symlink interdependencies with this.

mguetschow avatar Apr 19 '24 12:04 mguetschow

Actually, let's defer this to after #20545, I'd get some more symlink interdependencies with this.

Done, after #20545 has been merged. Should be ready for review.

mguetschow avatar May 15 '24 08:05 mguetschow

Thanks everyone!

mguetschow avatar May 17 '24 13:05 mguetschow