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

	F145794248: D53255.id165593.diff
	Tue, Feb 24, 2:15 PM

	Unknown Object (File)
	Sat, Feb 21, 4:51 AM

	Unknown Object (File)
	Sun, Feb 15, 7:27 PM

	Unknown Object (File)
	Mon, Feb 2, 8:26 PM

	Unknown Object (File)
	Sun, Feb 1, 11:34 AM

	Unknown Object (File)
	Fri, Jan 30, 11:21 PM

	Unknown Object (File)
	Tue, Jan 27, 12:19 PM

	Unknown Object (File)
	Jan 2 2026, 10:38 AM

View All 64 Files

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 Not Applicable

Unit

Tests Not Applicable

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
683	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
683	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/

3 lines

random_harvestq.c

6 lines

Diff 165593

sys/dev/random/fenestrasX/fx_pool.c

Loading...

sys/dev/random/random_harvestq.c

Loading...