summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDarren Tucker <dtucker@zip.com.au>2003-09-22 12:54:37 +1000
committerDarren Tucker <dtucker@zip.com.au>2003-09-22 12:54:37 +1000
commitfbe3b36ca9386a9b32f05e36450216acaa8bb404 (patch)
tree81098e263d83b96aeadbfee849181e0ea02481f1
parent8daf4b45eab80b2fa4e06f26d09f834bd2e5bb93 (diff)
- (dtucker) [uidswap.c] Don't test restoration of uid on Cygwin since the
OS does not support permanently dropping privileges. Patch from vinschen at redhat.com.
-rw-r--r--ChangeLog5
-rw-r--r--uidswap.c2
2 files changed, 6 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 9058dfc7a..51c5584e4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -16,6 +16,9 @@
16 - (dtucker) [contrib/cygwin/README contrib/cygwin/ssh-host-config] Update 16 - (dtucker) [contrib/cygwin/README contrib/cygwin/ssh-host-config] Update
17 ssh-host-config to match current defaults, bump README version. Patch from 17 ssh-host-config to match current defaults, bump README version. Patch from
18 vinschen at redhat.com. 18 vinschen at redhat.com.
19 - (dtucker) [uidswap.c] Don't test restoration of uid on Cygwin since the
20 OS does not support permanently dropping privileges. Patch from
21 vinschen at redhat.com.
19 22
2020030919 2320030919
21 - (djm) Bug #683: Remove reference to --with-ipv4-default from INSTALL; 24 - (djm) Bug #683: Remove reference to --with-ipv4-default from INSTALL;
@@ -1152,4 +1155,4 @@
1152 - Fix sshd BindAddress and -b options for systems using fake-getaddrinfo. 1155 - Fix sshd BindAddress and -b options for systems using fake-getaddrinfo.
1153 Report from murple@murple.net, diagnosis from dtucker@zip.com.au 1156 Report from murple@murple.net, diagnosis from dtucker@zip.com.au
1154 1157
1155$Id: ChangeLog,v 1.3017 2003/09/22 02:32:00 dtucker Exp $ 1158$Id: ChangeLog,v 1.3018 2003/09/22 02:54:37 dtucker Exp $
diff --git a/uidswap.c b/uidswap.c
index 3f13f9bf4..9e161d0f0 100644
--- a/uidswap.c
+++ b/uidswap.c
@@ -191,10 +191,12 @@ permanently_set_uid(struct passwd *pw)
191 (u_int)pw->pw_gid); 191 (u_int)pw->pw_gid);
192 } 192 }
193 193
194#ifndef HAVE_CYGWIN
194 /* Try restoration of UID if changed (test clearing of saved uid) */ 195 /* Try restoration of UID if changed (test clearing of saved uid) */
195 if (old_uid != pw->pw_uid && 196 if (old_uid != pw->pw_uid &&
196 (setuid(old_uid) != -1 || seteuid(old_uid) != -1)) 197 (setuid(old_uid) != -1 || seteuid(old_uid) != -1))
197 fatal("%s: was able to restore old [e]uid", __func__); 198 fatal("%s: was able to restore old [e]uid", __func__);
199#endif
198 200
199 /* Verify UID drop was successful */ 201 /* Verify UID drop was successful */
200 if (getuid() != pw->pw_uid || geteuid() != pw->pw_uid) { 202 if (getuid() != pw->pw_uid || geteuid() != pw->pw_uid) {