Differential D48933

ctld: Fix parsing of text values for DSCP in UCL parser
ClosedPublic
Actions

Authored by jhb on Feb 11 2025, 1:20 AM.

Tags

None

Referenced Files

	F147698310: D48933.id150925.diff
	Fri, Mar 13, 12:36 AM

	Unknown Object (File)
	Thu, Mar 12, 6:55 PM

	Unknown Object (File)
	Thu, Mar 12, 12:58 PM

	Unknown Object (File)
	Wed, Feb 25, 9:50 AM

	Unknown Object (File)
	Wed, Feb 25, 2:36 AM

	Unknown Object (File)
	Wed, Feb 18, 9:13 AM

	Unknown Object (File)
	Feb 8 2026, 2:40 PM

	Unknown Object (File)
	Dec 21 2025, 7:55 AM

View All 78 Files

Subscribers

Details

Reviewers

Commits

rG1b4e894eaeb1: ctld: Fix parsing of text values for DSCP in UCL parser
rG5e6a462277b1: ctld: Fix parsing of text values for DSCP in UCL parser

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 62360
Build 59244: arc lint + arc unit

Event Timeline

jhb created this revision.Feb 11 2025, 1:20 AM

Herald added a subscriber: imp. · View Herald TranscriptFeb 11 2025, 1:20 AM

Harbormaster completed remote builds in B62360: Diff 150822.Feb 11 2025, 1:20 AM

jhb requested review of this revision.Feb 11 2025, 1:20 AM

jhb added a parent revision: D48932: ctld: Use preferred style for evaluating result of strcmp.

jhb added a child revision: D48934: ctld: Add a dedicated API between parse.y and the rest of the program.

The old code certainly looks broken. I guess nobody was using it?

This revision is now accepted and ready to land.Feb 11 2025, 6:35 PM

In D48933#1116308, @asomers wrote:

The old code certainly looks broken. I guess nobody was using it?

Yes, my guess is that the UCL paths weren't well tested.

Closed by commit rG5e6a462277b1: ctld: Fix parsing of text values for DSCP in UCL parser (authored by jhb). · Explain WhyFeb 12 2025, 7:35 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG5e6a462277b1: ctld: Fix parsing of text values for DSCP in UCL parser.

jhb added a commit: rG1b4e894eaeb1: ctld: Fix parsing of text values for DSCP in UCL parser.Jan 27 2026, 6:45 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

ctld/

44 lines

Diff 150822

usr.sbin/ctld/uclparse.c

Loading...