diff options
| author | Jeremy Allison <jra@samba.org> | 2001-04-15 23:36:05 +0000 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2001-04-15 23:36:05 +0000 |
| commit | 18f3f5ff9282417b350873fd572e9e0c4d4bcea4 (patch) | |
| tree | b971c1cccaea3192f8ef9f01b453a8bb2350e492 /source3 | |
| parent | 06a50f51845bf639f7abd47cf39638054f255131 (diff) | |
| download | samba-18f3f5ff9282417b350873fd572e9e0c4d4bcea4.tar.gz samba-18f3f5ff9282417b350873fd572e9e0c4d4bcea4.tar.bz2 samba-18f3f5ff9282417b350873fd572e9e0c4d4bcea4.zip | |
Fixed potential bug in "become_guest" pointed out by elrond. Get_Pwnam()
returns a pointer to changable storage so ensure we save the details and
don't use the pointer directly.
Jeremy.
(This used to be commit d9fdaae54ee3a267aebd02ff6058a98aefc084c2)
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/smbd/uid.c | 19 |
1 files changed, 13 insertions, 6 deletions
diff --git a/source3/smbd/uid.c b/source3/smbd/uid.c index 02522a37a2..da4c538319 100644 --- a/source3/smbd/uid.c +++ b/source3/smbd/uid.c @@ -33,19 +33,26 @@ extern struct current_user current_user; BOOL become_guest(void) { static struct passwd *pass=NULL; - - if (!pass) + static uid_t guest_uid = (uid_t)-1; + static gid_t guest_gid = (gid_t)-1; + static fstring guest_name; + + if (!pass) { pass = Get_Pwnam(lp_guestaccount(-1),True); - if (!pass) - return(False); + if (!pass) + return(False); + guest_uid = pass->pw_uid; + guest_gid = pass->pw_gid; + fstrcpy(guest_name, pass->pw_name); + } #ifdef AIX /* MWW: From AIX FAQ patch to WU-ftpd: call initgroups before setting IDs */ - initgroups(pass->pw_name, (gid_t)pass->pw_gid); + initgroups(guest_name, guest_gid); #endif - set_sec_ctx(pass->pw_uid, pass->pw_gid, 0, NULL, NULL); + set_sec_ctx(guest_uid, guest_gid, 0, NULL, NULL); current_user.conn = NULL; current_user.vuid = UID_FIELD_INVALID; |