diff options
author | Dmitry V. Levin <ldv@altlinux.org> | 2017-05-18 12:28:51 +0300 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2017-05-18 12:27:25 +0200 |
commit | 9aa3a11857109297b521d0a8926dd90361b991ed (patch) | |
tree | 69ccd94dd99869f45cdb89fee2c6e8283a985634 | |
parent | f1a97618031701ee9561e95ace4c7a52fd5dbd79 (diff) | |
download | quota-tools-9aa3a11857109297b521d0a8926dd90361b991ed.tar.gz |
quotaops: check setgid/setuid return code
setgid/setuid syscalls may fail for different reasons,
do not ignore these errors.
Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
Signed-off-by: Jan Kara <jack@suse.cz>
-rw-r--r-- | quotaops.c | 6 |
1 files changed, 4 insertions, 2 deletions
@@ -218,8 +218,10 @@ int editprivs(char *tmpfile) int i; sigprocmask(SIG_SETMASK, &omask, NULL); - setgid(getgid()); - setuid(getuid()); + if (setgid(getgid())) + die(1, _("%s failed: %s\n"), "setgid", strerror(errno)); + if (setuid(getuid())) + die(1, _("%s failed: %s\n"), "setuid", strerror(errno)); if (!(ed = getenv("VISUAL"))) if (!(ed = getenv("EDITOR"))) ed = _PATH_VI; |