diff options
| author | Mattias Andrée <maandree@kth.se> | 2023-06-23 10:40:22 +0200 |
|---|---|---|
| committer | Mattias Andrée <maandree@kth.se> | 2023-06-23 10:40:22 +0200 |
| commit | 901d3533ec3285b58960d0af0937c8e05692ebe0 (patch) | |
| tree | e552881286ad78a0fbfb3aafe9a9c1ed360d53b8 /key2root-addkey.c | |
| parent | Implement key2root-lskeys (diff) | |
| download | key2root-901d3533ec3285b58960d0af0937c8e05692ebe0.tar.gz key2root-901d3533ec3285b58960d0af0937c8e05692ebe0.tar.bz2 key2root-901d3533ec3285b58960d0af0937c8e05692ebe0.tar.xz | |
Add argument checks to key2root-addkey and key2root-rmkey
Signed-off-by: Mattias Andrée <maandree@kth.se>
Diffstat (limited to '')
| -rw-r--r-- | key2root-addkey.c | 22 |
1 files changed, 21 insertions, 1 deletions
diff --git a/key2root-addkey.c b/key2root-addkey.c index 9dab78b..d5ec701 100644 --- a/key2root-addkey.c +++ b/key2root-addkey.c @@ -20,7 +20,11 @@ usage(void) int main(int argc, char *argv[]) { + const char *user; + const char *keyname; + const char *parameters; int allow_replace = 0; + int failed = 0; ARGBEGIN { case 'r': @@ -33,10 +37,26 @@ main(int argc, char *argv[]) if (argc < 2 || argc > 3) usage(); + user = argv[0]; + keyname = argv[1]; + parameters = argv[2]; + + if (!user[0] || user[0] == '.' || strchr(user, '/') || strchr(user, '~')) { + fprintf(stderr, "%s: bad user name specified: %s\n", argv0, user); + failed = 1; + } + if (keyname[strcspn(keyname, " \t\f\n\r\v")]) { + fprintf(stderr, "%s: bad key name specified: %s, includes whitespace\n", argv0, keyname); + failed = 1; + } if (isatty(STDIN_FILENO)) { fprintf(stderr, "%s: standard input must not be a TTY.\n", argv0); - exit(1); + failed = 1; } + if (failed) + return 1; + + /* TODO */ return 0; } |