id: sanitize arguments better
ClosedPublic
Actions

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

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

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 39849
Build 36738: arc lint + arc unit

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/

id.c

2 lines

Diff 90744

View Options

id: sanitize arguments betterClosedPublicActions