Page MenuHomeFreeBSD

random(4): Correct a bare zero to the appropriate enum
ClosedPublic

Authored by cem on Sep 1 2018, 4:45 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 22, 11:51 PM
Unknown Object (File)
Thu, Nov 14, 7:41 PM
Unknown Object (File)
Tue, Nov 5, 9:38 AM
Unknown Object (File)
Oct 1 2024, 1:49 PM
Unknown Object (File)
Sep 30 2024, 9:17 AM
Unknown Object (File)
Sep 22 2024, 5:08 PM
Unknown Object (File)
Sep 16 2024, 3:51 PM
Unknown Object (File)
Sep 7 2024, 11:00 AM
Subscribers

Details

Summary

The convention for updating hc_destination[] is to index with a
random_entropy_source. Zero happens to match RANDOM_CACHED, which is
correct for this source (early random data). Spell the zero value as the
enum name instead of the magic constant.

No functional change.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 19339
Build 18944: arc lint + arc unit

Event Timeline

delphij added a subscriber: delphij.

Looks reasonable.

sys/dev/random/random_harvestq.c
421

@markm -- Why is this explicit_bzero() done here in the loop instead of done below (e.g. line 423)?

This revision is now accepted and ready to land.Sep 1 2018, 5:54 AM
markm added inline comments.
sys/dev/random/random_harvestq.c
421

Over-paranoia, I guess. I wouldn't object to it moving.

sys/dev/random/random_harvestq.c
421

One other caveat is that random_harvestq_fast_process_event -> random_fortuna_process_event() also explicit_bzeros the event in the loop already.

This revision was automatically updated to reflect the committed changes.