From: Simo Sorce <idra@samba.org>
Date: Sun, 13 Jul 2008 02:44:15 +0000 (-0400)
Subject: Fix bug #6628 - "smbpassdb -a" using rid algorithm with tdbsam passdb
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0a1b125ae4260de3cee6a92b19e3977d922a8c11;p=thirdparty%2Fsamba.git

Fix bug #6628 - "smbpassdb -a" using rid algorithm with tdbsam passdb

samu_set_unix() does not use the rid allocator, but forces to use the
algoritmic allocator unconditionally, this is wrong and inconsistent.
Use samu_alloc_rid_unix() instead.
(If we create a new user we should do it in a way consistent with all
other commands like pdbedit -a and net rpc user add.)
(cherry picked from commit e1d4db8f55b25c7e9c408cb334fb89ccfd4c6565)

Signed-off-by: Michael Adam <obnox@samba.org>

This fixes bug #6628.
---

diff --git a/source/passdb/passdb.c b/source/passdb/passdb.c
index e3a3d3ca9e5..125689caad3 100644
--- a/source/passdb/passdb.c
+++ b/source/passdb/passdb.c
@@ -675,7 +675,7 @@ NTSTATUS local_password_change(const char *user_name,
 				return NT_STATUS_NO_MEMORY;
 			}
 
-			result = samu_set_unix( sam_pass, pwd );
+			result = samu_alloc_rid_unix( sam_pass, pwd );
 
 			DEBUGLEVEL = tmp_debug;