Page MenuHomeFreeBSD
Differential D38642

config: error out on malformed env/hint lines
ClosedPublic
Actions

Authored by kevans on Feb 16 2023, 5:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 26, 12:25 PM
Unknown Object (File)
Tue, Dec 23, 5:00 PM
Unknown Object (File)
Thu, Dec 18, 10:30 AM
Unknown Object (File)
Nov 5 2025, 11:57 PM
Unknown Object (File)
Nov 3 2025, 5:07 AM
Unknown Object (File)
Oct 30 2025, 5:34 AM
Unknown Object (File)
Oct 26 2025, 2:57 AM
Unknown Object (File)
Oct 22 2025, 10:07 PM
View All 87 Files
Subscribers
None

Details

Reviewers
imp
des
emaste
Commits
rGda8884202940: config: error out on malformed env/hint lines
Summary

We need these to be assignments, error out if they're not.

While we're here, tag errout as __printflike so that the compiler will
sanity check the format string + args.

CID: 1505293

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 49856
Build 46747: arc lint + arc unit

Event Timeline

kevans requested review of this revision.Feb 16 2023, 5:25 PM
kevans created this revision.
Harbormaster completed remote builds in B49856: Diff 117466.Feb 16 2023, 5:25 PM
kevans added a parent revision: D38641: config: push path() into moveifchanged().Feb 16 2023, 5:25 PM
kevans added a child revision: D38643: config: address a number of Coverity issues.
imp accepted this revision.Feb 16 2023, 5:33 PM

Looks good. I'd be tempted to separate the prinflike thing, but if not that's cool too.

This revision is now accepted and ready to land.Feb 16 2023, 5:33 PM
emaste accepted this revision.Feb 16 2023, 5:52 PM
Closed by commit rGda8884202940: config: error out on malformed env/hint lines (authored by kevans). · Explain WhyFeb 16 2023, 8:38 PM
This revision was automatically updated to reflect the committed changes.
kevans added a commit: rGda8884202940: config: error out on malformed env/hint lines.

Revision Contents
Changeset List

PathSize
usr.sbin/
config/
25 lines

Diff 117466

View Options

usr.sbin/config/mkmakefile.cc

Loading...