diff options
| author | Alan Somers <asomers@FreeBSD.org> | 2021-06-11 16:38:07 +0000 |
|---|---|---|
| committer | Alan Somers <asomers@FreeBSD.org> | 2021-06-15 12:51:16 +0000 |
| commit | 4a06e9377398b34922f8a67d7cb3ea980b95bde8 (patch) | |
| tree | 6257757844a2480a84fd681a7610788dd506ec7e | |
| parent | 7d681ad774f00cf06c4ef910add91e0f8a79f7ae (diff) | |
| download | src-4a06e9377398b34922f8a67d7cb3ea980b95bde8.tar.gz src-4a06e9377398b34922f8a67d7cb3ea980b95bde8.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
MFC after: 2 weeks
Reviewed by: trasz
Sponsored by: Axcient
Differential Revision: https://reviews.freebsd.org/D30734
| -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: |