Properly reset fields in clean_unrhdr.
ClosedPublic
Actions

Authored by mjoras on Oct 13 2017, 5:17 PM.

Tags

None

Referenced Files

	F83161176: D12662.id33999.diff
	Tue, May 7, 4:54 AM

	Unknown Object (File)
	Thu, May 2, 7:35 AM

	Unknown Object (File)
	Tue, Apr 30, 4:23 PM

	Unknown Object (File)
	Mon, Apr 29, 2:27 PM

	Unknown Object (File)
	Mon, Apr 29, 1:29 PM

	Unknown Object (File)
	Mar 28 2024, 7:22 PM

	Unknown Object (File)
	Mar 28 2024, 12:51 AM

	Unknown Object (File)
	Feb 19 2024, 6:12 AM

View All 27 Files

Subscribers

imp

Details

Reviewers

rstone

Summary

In r324542 I neglected to reset the first and last fields of struct
unrhdr. This causes a tmpfs to fail the unr(9) consistency checks with
DIAGNOSTIC on. Fix this by resetting the fields to be as if init_unrhdr
was called.

Test Plan

Tested using a tmpfs with an extracted ports tree, unmounted with DIAGNOSTIC
on.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 12040
Build 12349: arc lint + arc unit

Event Timeline

mjoras created this revision.Oct 13 2017, 5:17 PM

Herald added a subscriber: imp. · View Herald TranscriptOct 13 2017, 5:17 PM

use init_unrhdr instead of duplicating it

Harbormaster completed remote builds in B12040: Diff 33970.Oct 13 2017, 8:30 PM

Might as well use TAILQ_FOREACH_SAFE while we are here.

Harbormaster completed remote builds in B12053: Diff 33999.Oct 14 2017, 11:58 PM

rstone accepted this revision.Oct 16 2017, 3:40 PM

This revision is now accepted and ready to land.Oct 16 2017, 3:40 PM

mjoras closed this revision.Oct 16 2017, 4:23 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

subr_unit.c

4 lines

Diff 33970

View Options

Properly reset fields in clean_unrhdr.ClosedPublicActions