summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2008-12-29 22:01:57 +0100
committerVolker Lendecke <vl@samba.org>2008-12-30 11:26:10 +0100
commit12f4d01f4a9a99690a9f3421160bb1cea85aa9ff (patch)
treecbd17e12080f4a9fc5e2fa503ccef8ef57bbdacb
parentcf79461f4e25752750ee6ff14fb8620a6c11ac1d (diff)
downloadsamba-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.c3
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",