Differential D43198

Fix multiple bugs with ctld's UCL parsing
ClosedPublic
Actions

Authored by asomers on Dec 26 2023, 7:08 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Nov 3, 4:02 PM

	Unknown Object (File)
	Sun, Nov 3, 3:47 PM

	Unknown Object (File)
	Thu, Oct 31, 4:03 PM

	Unknown Object (File)
	Mon, Oct 21, 2:37 AM

	Unknown Object (File)
	Mon, Oct 21, 2:30 AM

	Unknown Object (File)
	Mon, Oct 21, 2:30 AM

	Unknown Object (File)
	Fri, Oct 18, 7:48 PM

	Unknown Object (File)
	Wed, Oct 9, 11:02 PM

View All 44 Files

Subscribers

Details

Reviewers

jhb
mav

Commits

rG5a2767d20325: Fix multiple bugs with ctld's UCL parsing
rGc2ec8038a7f1: Fix multiple bugs with ctld's UCL parsing
rG2391e536c01c: Fix multiple bugs with ctld's UCL parsing

Summary

Don't segfault when parsing a misformatted auth-group section
If the config file specifies a chap section within a target but no auth-group, create a new anonymous auth-group. That matches the behavior with non-UCL config files.
Protect some potential segfaults with assertions

PR: 274380
MFC after: 1 week
Sponsored by: Axcient

Test Plan

manually tested

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

asomers created this revision.Dec 26 2023, 7:08 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 26 2023, 7:08 PM

asomers requested review of this revision.Dec 26 2023, 7:08 PM

Harbormaster completed remote builds in B55126: Diff 131882.Dec 26 2023, 7:08 PM

imp added inline comments.Dec 26 2023, 7:35 PM

usr.sbin/ctld/ctld.c
535	Is this a programming error, or a run-time check? If the former, assert is fine. If the latter, then (a) it might not actually check and (b) it will lead to a program termination with an assert. Is that OK here and elsewhere?

asomers added inline comments.Dec 26 2023, 7:38 PM

usr.sbin/ctld/ctld.c
535	Formerly, it failed at runtime. Now it's just there to catch future programming errors. See the call site at line 737.

jhb accepted this revision.Dec 27 2023, 5:43 PM

This revision is now accepted and ready to land.Dec 27 2023, 5:43 PM

Closed by commit rG2391e536c01c: Fix multiple bugs with ctld's UCL parsing (authored by asomers). · Explain WhyDec 27 2023, 5:50 PM

This revision was automatically updated to reflect the committed changes.

asomers added a commit: rG2391e536c01c: Fix multiple bugs with ctld's UCL parsing.

asomers added a commit: rGc2ec8038a7f1: Fix multiple bugs with ctld's UCL parsing.Jan 19 2024, 8:22 PM

asomers added a commit: rG5a2767d20325: Fix multiple bugs with ctld's UCL parsing.Jan 20 2024, 1:54 AM

Revision Contents
Changeset List

Path

Size

usr.sbin/

ctld/

1 line

26 lines

Diff 131915

usr.sbin/ctld/ctld.c

Loading...

usr.sbin/ctld/uclparse.c

Loading...