diff options
| author | Alan Somers <asomers@FreeBSD.org> | 2021-06-11 16:38:07 +0000 |
|---|---|---|
| committer | Alan Somers <asomers@FreeBSD.org> | 2021-07-03 23:29:41 +0000 |
| commit | bc997ea6b81eace14f999e782502ae060687db2c (patch) | |
| tree | 4d8e5206f2ac867762d4b1d86e437c418c2d718c | |
| parent | ba56ee36eb966ac43cc51b52db5fbe568bf2ff3f (diff) | |
| download | src-bc997ea6b81eace14f999e782502ae060687db2c.tar.gz src-bc997ea6b81eace14f999e782502ae060687db2c.zip | |
id: sanitize arguments better
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
Reviewed by: trasz
Sponsored by: Axcient
Differential Revision: https://reviews.freebsd.org/D30734
(cherry picked from commit 4a06e9377398b34922f8a67d7cb3ea980b95bde8)
| -rw-r--r-- | usr.bin/id/id.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/usr.bin/id/id.c b/usr.bin/id/id.c index 5abfb655c948..b8988dedd6f6 100644 --- a/usr.bin/id/id.c +++ b/usr.bin/id/id.c @@ -144,6 +144,8 @@ main(int argc, char *argv[]) if (iswhoami && argc > 0) usage(); + if ((cflag || Aflag || Mflag) && argc > 0) + usage(); switch(Aflag + Gflag + Mflag + Pflag + gflag + pflag + uflag) { case 1: |