diff options
| author | Ben Lindstrom <mouring@eviladmin.org> | 2001-04-22 19:08:00 +0200 |
|---|---|---|
| committer | Ben Lindstrom <mouring@eviladmin.org> | 2001-04-22 19:08:00 +0200 |
| commit | ee2786a2a1c295e25b165af1b830fd0ef28985db (patch) | |
| tree | eeb3669dc08dde41013094cba041dd2ef063ef6d /uidswap.c | |
| parent | fix perl test, fix nroff test, fix Makefile to build outside source tree (diff) | |
| download | openssh-ee2786a2a1c295e25b165af1b830fd0ef28985db.tar.xz openssh-ee2786a2a1c295e25b165af1b830fd0ef28985db.zip | |
- markus@cvs.openbsd.org 2001/04/20 16:32:22
[uidswap.c]
set non-privileged gid before uid; tholo@ and deraadt@
Diffstat (limited to 'uidswap.c')
| -rw-r--r-- | uidswap.c | 10 |
1 files changed, 5 insertions, 5 deletions
@@ -12,7 +12,7 @@ */ #include "includes.h" -RCSID("$OpenBSD: uidswap.c,v 1.15 2001/04/08 11:24:33 markus Exp $"); +RCSID("$OpenBSD: uidswap.c,v 1.16 2001/04/20 16:32:22 markus Exp $"); #include "log.h" #include "uidswap.h" @@ -81,7 +81,7 @@ temporarily_use_uid(struct passwd *pw) } /* - * Restores to the original uid. + * Restores to the original (privileged) uid. */ void restore_uid(void) @@ -92,7 +92,7 @@ restore_uid(void) return; if (!temporarily_use_uid_effective) fatal("restore_uid: temporarily_use_uid not effective"); - /* Set the effective uid back to the saved uid. */ + /* Set the effective uid back to the saved privileged uid. */ if (seteuid(saved_euid) < 0) fatal("seteuid %u: %.100s", (u_int) saved_euid, strerror(errno)); if (setgroups(saved_egroupslen, saved_egroups) < 0) @@ -111,8 +111,8 @@ permanently_set_uid(struct passwd *pw) { if (temporarily_use_uid_effective) fatal("restore_uid: temporarily_use_uid effective"); - if (setuid(pw->pw_uid) < 0) - fatal("setuid %u: %.100s", (u_int) pw->pw_uid, strerror(errno)); if (setgid(pw->pw_gid) < 0) fatal("setgid %u: %.100s", (u_int) pw->pw_gid, strerror(errno)); + if (setuid(pw->pw_uid) < 0) + fatal("setuid %u: %.100s", (u_int) pw->pw_uid, strerror(errno)); } |