Differential D45041

sysctl: Make sysctl_ctx_free() a bit safer
ClosedPublic
Actions

Authored by markj on May 1 2024, 12:00 PM.

Tags

None

Referenced Files

	F86767215: D45041.id138040.diff
	Tue, Jun 25, 6:30 AM

	Unknown Object (File)
	Sun, Jun 16, 11:13 AM

	Unknown Object (File)
	Sun, Jun 2, 6:32 PM

	Unknown Object (File)
	May 4 2024, 5:39 PM

	Unknown Object (File)
	May 3 2024, 7:43 PM

	Unknown Object (File)
	May 2 2024, 6:39 PM

	Unknown Object (File)
	May 2 2024, 6:39 PM

	Unknown Object (File)
	May 2 2024, 2:37 PM

Subscribers

Details

Reviewers

thj
emaste

Commits

rG24268f262b41: sysctl: Make sysctl_ctx_free() a bit safer
rGd5eae57088f5: sysctl: Make sysctl_ctx_free() a bit safer

Summary

Clear the list before returning so that it's idempotent when it
succeeds.

While here, use TAILQ_FOREACH_SAFE in the loop which unregisters OIDs.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 57494
Build 54382: arc lint + arc unit

Event Timeline

markj created this revision.May 1 2024, 12:00 PM

Herald added subscribers: olce, imp. · View Herald TranscriptMay 1 2024, 12:00 PM

markj requested review of this revision.May 1 2024, 12:00 PM

Harbormaster completed remote builds in B57494: Diff 137940.May 1 2024, 12:00 PM

thj accepted this revision.May 1 2024, 12:03 PM

This revision is now accepted and ready to land.May 1 2024, 12:03 PM

emaste accepted this revision.May 1 2024, 12:13 PM

Closed by commit rGd5eae57088f5: sysctl: Make sysctl_ctx_free() a bit safer (authored by markj). · Explain WhyMay 2 2024, 8:02 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rGd5eae57088f5: sysctl: Make sysctl_ctx_free() a bit safer.

markj added a commit: rG24268f262b41: sysctl: Make sysctl_ctx_free() a bit safer.May 11 2024, 2:17 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

6 lines

Diff 137940

sys/kern/kern_sysctl.c

Loading...