Page MenuHomeFreeBSD
Differential D38372

sha512_224: Fix SHA512_224_Final() on little-endian machines.
ClosedPublic
Actions

Authored by des on Feb 3 2023, 3:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 21, 2:52 AM
Unknown Object (File)
Sun, Oct 20, 5:50 PM
Unknown Object (File)
Sun, Oct 20, 5:50 PM
Unknown Object (File)
Sun, Oct 20, 5:50 PM
Unknown Object (File)
Oct 3 2024, 10:52 PM
Unknown Object (File)
Oct 3 2024, 1:24 AM
Unknown Object (File)
Oct 2 2024, 11:13 PM
Unknown Object (File)
Oct 2 2024, 10:19 PM
View All 73 Files
Subscribers
imp

Details

Reviewers
cperciva
Group Reviewers
Klara
Commits
rG316cf82382e0: sha512_224: Fix SHA512_224_Final() on little-endian machines.
rGb9e0160e0788: sha512_224: Fix SHA512_224_Final() on little-endian machines.
rG6680cfe8e0ee: sha512_224: Fix SHA512_224_Final() on little-endian machines.
Summary

PR: 266863
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 49509
Build 46399: arc lint + arc unit

Event Timeline

des created this revision.Feb 3 2023, 3:21 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 3 2023, 3:21 PM
des requested review of this revision.Feb 3 2023, 3:21 PM
Harbormaster completed remote builds in B49509: Diff 116394.Feb 3 2023, 3:21 PM
des added a child revision: D38373: libmd / md5: Add SHA-512/224..Feb 3 2023, 3:21 PM
des added reviewers: Klara, cperciva.Feb 3 2023, 3:24 PM
des added a comment.Feb 3 2023, 3:27 PM

Note that I intended to add assertions to be64enc_vect() and be64dec_vect(), but since the same code is used both in the kernel and in user space, it would have been needlessly complicated.

cperciva accepted this revision.Feb 4 2023, 2:43 PM

You're going to be adding test cases for this in libmd, right?

This revision is now accepted and ready to land.Feb 4 2023, 2:43 PM
des added a comment.Feb 6 2023, 12:11 PM
In D38372#872487, @cperciva wrote:

You're going to be adding test cases for this in libmd, right?

Yes, see D38373. That's how I found the bug.

Closed by commit rG6680cfe8e0ee: sha512_224: Fix SHA512_224_Final() on little-endian machines. (authored by sebastian.huber_embedded-brains.de, committed by des). · Explain WhyFeb 6 2023, 5:04 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG6680cfe8e0ee: sha512_224: Fix SHA512_224_Final() on little-endian machines..
des added a commit: rGb9e0160e0788: sha512_224: Fix SHA512_224_Final() on little-endian machines..Feb 9 2023, 8:41 PM
des added a commit: rG316cf82382e0: sha512_224: Fix SHA512_224_Final() on little-endian machines..Feb 10 2023, 12:24 AM

Revision Contents
Changeset List

PathSize
sys/
crypto/
sha2/
13 lines

Diff 116394

View Options

sys/crypto/sha2/sha512c.c

Loading...