Differential D30734

id: sanitize arguments better
ClosedPublic
Actions

Authored by asomers on Jun 11 2021, 4:40 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, May 13, 1:13 AM

	Unknown Object (File)
	Wed, May 13, 1:13 AM

	Unknown Object (File)
	Wed, May 13, 12:59 AM

	Unknown Object (File)
	Wed, May 13, 12:59 AM

	Unknown Object (File)
	Mon, Apr 27, 10:27 AM

	Unknown Object (File)
	Tue, Apr 21, 6:00 AM

	Unknown Object (File)
	Tue, Apr 21, 1:09 AM

	Unknown Object (File)
	Mon, Apr 20, 5:22 PM

Subscribers

Details

Reviewers

Commits

rG5d7ae6fbb117: id: sanitize arguments better
rGbc997ea6b81e: id: sanitize arguments better
rG4a06e9377398: id: sanitize arguments better

Summary

The -[AMc] flags ignore the user argument. Better if id rejects
invocations that include a user argument along with any of those flags.

PR: 256554
MFC after: 2 weeks
Sponsored by: Axcient

Test Plan

manually tested with all command line options

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

asomers created this revision.Jun 11 2021, 4:40 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 11 2021, 4:40 PM

asomers requested review of this revision.Jun 11 2021, 4:40 PM

Harbormaster completed remote builds in B39849: Diff 90744.Jun 11 2021, 4:40 PM

trasz accepted this revision.Jun 15 2021, 8:52 AM

This revision is now accepted and ready to land.Jun 15 2021, 8:52 AM

Closed by commit rG4a06e9377398: id: sanitize arguments better (authored by asomers). · Explain WhyJun 15 2021, 12:52 PM

This revision was automatically updated to reflect the committed changes.

asomers added a commit: rG4a06e9377398: id: sanitize arguments better.

asomers added a commit: rGbc997ea6b81e: id: sanitize arguments better.Jul 3 2021, 11:30 PM

asomers added a commit: rG5d7ae6fbb117: id: sanitize arguments better.Jul 4 2021, 4:15 AM

Revision Contents
Changeset List

Path

Size

usr.bin/

id/

2 lines

Diff 90917

usr.bin/id/id.c

Loading...