Differential D53255

random: CTASSERT check sizeof random_source_descr[]
ClosedPublic
Actions

Authored by obrien on Oct 22 2025, 7:25 AM.

Tags

None

Referenced Files

	F165205247: D53255.id165593.diff
	Thu, Aug 6, 7:32 PM

	F165195771: D53255.diff
	Thu, Aug 6, 5:17 PM

	Unknown Object (File)
	Mon, Aug 3, 6:41 PM

	Unknown Object (File)
	Sun, Aug 2, 12:17 PM

	Unknown Object (File)
	Sun, Aug 2, 12:54 AM

	Unknown Object (File)
	Sat, Aug 1, 10:08 PM

	Unknown Object (File)
	Sat, Aug 1, 8:21 AM

	Unknown Object (File)
	Sat, Aug 1, 8:01 AM

Subscribers

Details

Reviewers

Group Reviewers

Commits

rG7d106aaa9364: random: CTASSERT check sizeof random_source_descr[]
rGd53618d58283: random: CTASSERT check sizeof random_source_descr[]
rG8bd9a9e9e4cb: random: CTASSERT check sizeof random_source_descr[]

Summary

Ensure that the number of elements of random_source_descr[]
of matches that of enum random_entropy_source.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 68033
Build 64916: arc lint + arc unit

Event Timeline

obrien created this revision.Oct 22 2025, 7:25 AM

Herald added a subscriber: imp. · View Herald TranscriptOct 22 2025, 7:25 AM

obrien requested review of this revision.Oct 22 2025, 7:25 AM

Harbormaster completed remote builds in B67986: Diff 164731.Oct 22 2025, 7:25 AM

update review tag

Harbormaster completed remote builds in B67987: Diff 164732.Oct 22 2025, 7:26 AM

add checking of fxrng_ent_char[] too

Harbormaster completed remote builds in B68033: Diff 164846.Oct 23 2025, 4:42 AM

typo exiting editor

Harbormaster completed remote builds in B68039: Diff 164853.Oct 23 2025, 6:13 AM

tweak

Harbormaster completed remote builds in B68051: Diff 164890.Oct 23 2025, 6:08 PM

obrien added a reviewer: csprng.Oct 30 2025, 3:38 AM

obrien added a subscriber: csprng.

obrien removed a subscriber: csprng.

Otherwise LGTM

sys/dev/random/random_harvestq.c
684	The second assertion probably makes more sense colocated with the definition of ENTROPYSOURCE or hc_source_mask.

respond to comments

Harbormaster completed remote builds in B68303: Diff 165529.Oct 31 2025, 5:19 AM

obrien added inline comments.Oct 31 2025, 5:21 AM

sys/dev/random/random_harvestq.c
684	Actually I like that better - less duplication. I didn't touch fenestrasX/fx_pool.c and only added the CTASSERT to my changeset when I found so many missing items.

obrien marked an inline comment as done.Oct 31 2025, 5:21 AM

cem accepted this revision.Oct 31 2025, 5:27 AM

This revision is now accepted and ready to land.Oct 31 2025, 5:27 AM

Closed by commit rG8bd9a9e9e4cb: random: CTASSERT check sizeof random_source_descr[] (authored by obrien). · Explain WhyOct 31 2025, 6:02 PM

This revision was automatically updated to reflect the committed changes.

obrien added a commit: rG8bd9a9e9e4cb: random: CTASSERT check sizeof random_source_descr[].

obrien added a commit: rGd53618d58283: random: CTASSERT check sizeof random_source_descr[].Nov 7 2025, 12:08 AM

cperciva added a commit: rG7d106aaa9364: random: CTASSERT check sizeof random_source_descr[].Nov 7 2025, 5:17 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

random/

fenestrasX/

4 lines

random_harvestq.c

6 lines

Diff 164846

sys/dev/random/fenestrasX/fx_pool.c

Loading...

sys/dev/random/random_harvestq.c

Loading...