diff options
| author |   djm <djm@openbsd.org> | 2010-11-04 02:45:34 +0000 |
|---|---|---|
| committer | djm <djm@openbsd.org> | 2010-11-04 02:45:34 +0000 |
| commit | 2ed20a593b71fd90cf90bb72b4c85dcba33b11da (patch) | |
| tree | a2253541b9a028eb16c061537099b15e4ab8a86f /usr.bin/ssh/sftp-server.c | |
| parent | no need to specify the -t flag to ctags (diff) | |
| download | wireguard-openbsd-2ed20a593b71fd90cf90bb72b4c85dcba33b11da.tar.xz wireguard-openbsd-2ed20a593b71fd90cf90bb72b4c85dcba33b11da.zip | |
umask should be parsed as octal. reported by candland AT xmission.com;
ok markus@
Diffstat (limited to 'usr.bin/ssh/sftp-server.c')
| -rw-r--r-- | usr.bin/ssh/sftp-server.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/usr.bin/ssh/sftp-server.c b/usr.bin/ssh/sftp-server.c index 216348c2363..1e5b0d9d295 100644 --- a/usr.bin/ssh/sftp-server.c +++ b/usr.bin/ssh/sftp-server.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sftp-server.c,v 1.91 2010/01/13 01:40:16 djm Exp $ */ +/* $OpenBSD: sftp-server.c,v 1.92 2010/11/04 02:45:34 djm Exp $ */ /* * Copyright (c) 2000-2004 Markus Friedl. All rights reserved. * @@ -1349,8 +1349,7 @@ sftp_server_main(int argc, char **argv, struct passwd *user_pw) ssize_t len, olen, set_size; SyslogFacility log_facility = SYSLOG_FACILITY_AUTH; char *cp, buf[4*4096]; - const char *errmsg; - mode_t mask; + long mask; extern char *optarg; extern char *__progname; @@ -1383,11 +1382,12 @@ sftp_server_main(int argc, char **argv, struct passwd *user_pw) error("Invalid log facility \"%s\"", optarg); break; case 'u': - mask = (mode_t)strtonum(optarg, 0, 0777, &errmsg); - if (errmsg != NULL) - fatal("Invalid umask \"%s\": %s", - optarg, errmsg); - (void)umask(mask); + errno = 0; + mask = strtol(optarg, &cp, 8); + if (mask < 0 || mask > 0777 || *cp != '\0' || + cp == optarg || (mask == 0 && errno != 0)) + fatal("Invalid umask \"%s\"", optarg); + (void)umask((mode_t)mask); break; case 'h': default: |