From 02f3cab20766f70c3a5128e9c690a89720931637 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Mon, 29 Dec 2008 22:01:57 +0100 Subject: [PATCH] First part of bugfix for #5933 Ofer Tal 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! (cherry picked from commit 228d0c9459e9187d671875eda03aaa1435ab8401) --- source/smbd/password.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/source/smbd/password.c b/source/smbd/password.c index 80eba562c54..e8390bb2e6e 100644 --- a/source/smbd/password.c +++ b/source/smbd/password.c @@ -325,9 +325,6 @@ int register_existing_vuid(uint16 vuid, "and will be vuid %u\n", (int)vuser->uid,vuser->user.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", -- 2.11.4.GIT