summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVladimir A. Nazarenko <naszar@ya.ru>2013-10-20 15:57:24 +1100
committerAllan McRae <allan@archlinux.org>2013-10-31 16:20:02 +1000
commit7e767b7e90755f5b8c90b37e68ae14098fba5520 (patch)
treebe942bf3375e71c88e10fc081a22440b6a3678ea
parent8f5dbd56145a19d6f1ce951cec84ef5816f48349 (diff)
downloadpacman-7e767b7e90755f5b8c90b37e68ae14098fba5520.tar.xz
Use getuid instead of geteuid
If someone gives the pacman binary setuid permissions, the geteuid check allows it to start running but subsequently fail. As we do not support setting pacman setuid, use getuid to check permissions instead. FS#37174. Signed-off-by: Vladimir A. Nazarenko <naszar@ya.ru> Signed-off-by: Allan McRae <allan@archlinux.org>
-rw-r--r--src/pacman/pacman.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c
index f22f40f3..3c9457a9 100644
--- a/src/pacman/pacman.c
+++ b/src/pacman/pacman.c
@@ -1031,7 +1031,7 @@ int main(int argc, char *argv[])
size_t i;
struct sigaction new_action, old_action;
const int signals[] = { SIGHUP, SIGINT, SIGTERM, SIGSEGV };
- uid_t myuid = geteuid();
+ uid_t myuid = getuid();
/* Set signal handlers */
/* Set up the structure to specify the new action. */