diff options
author | Volker Lendecke <vl@samba.org> | 2008-12-29 22:01:57 +0100 |
---|---|---|
committer | Volker Lendecke <vl@samba.org> | 2008-12-30 11:26:10 +0100 |
commit | 12f4d01f4a9a99690a9f3421160bb1cea85aa9ff (patch) | |
tree | cbd17e12080f4a9fc5e2fa503ccef8ef57bbdacb | |
parent | cf79461f4e25752750ee6ff14fb8620a6c11ac1d (diff) | |
download | samba-12f4d01f4a9a99690a9f3421160bb1cea85aa9ff.tar.gz samba-12f4d01f4a9a99690a9f3421160bb1cea85aa9ff.tar.bz2 samba-12f4d01f4a9a99690a9f3421160bb1cea85aa9ff.zip |
First part of bugfix for #5933
Ofer Tal <otsmb@shmoop.org> fully correctly noted that we're incrementing
num_validated_vuids twice per session setup, but decrement it only once.
Looking at sesssetup.c we always call register_initial_vuid() before
register_existing_vuid(), so there's no point in incrementing it in
register_existing_vuid().
Jeremy, please check!
-rw-r--r-- | source3/smbd/password.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/source3/smbd/password.c b/source3/smbd/password.c index 68e134a0d6..d729c2e77b 100644 --- a/source3/smbd/password.c +++ b/source3/smbd/password.c @@ -281,9 +281,6 @@ int register_existing_vuid(uint16 vuid, "and will be vuid %u\n", (int)vuser->server_info->utok.uid, vuser->server_info->unix_name, vuser->vuid)); - next_vuid++; - num_validated_vuids++; - if (!session_claim(vuser)) { DEBUG(1, ("register_existing_vuid: Failed to claim session " "for vuid=%d\n", |