reply_spnego_kerberos did not set the domain of the user handed to

register_vuid correctly. We ended up with the local netbios name in
substitutions for %D later.

Volker

P.S: Tridge, I can *really* see why you want to get rid of global variables
:-)
(This used to be commit 05bfaa858f8253b02fe0f78f97fb665e6847585d)

diff --git a/source3/smbd/sesssetup.c b/source3/smbd/sesssetup.c
index 80be9da53bfce6006f74abed4f53211e78dd959a..9405c065e182e9af47cc781efeaaa4aac26dfea6 100644 (file)
--- a/source3/smbd/sesssetup.c
+++ b/source3/smbd/sesssetup.c
@@ -262,6 +262,13 @@ static int reply_spnego_kerberos(connection_struct *conn,
                return ERROR_NT(ret);
        }
 
+        /* make_server_info_pw does not set the domain. Without this we end up
+        * with the local netbios name in substitutions for %D. */
+
+        if (server_info->sam_account != NULL) {
+                pdb_set_domain(server_info->sam_account, domain, PDB_SET);
+        }
+
        /* register_vuid keeps the server info */
        sess_vuid = register_vuid(server_info, session_key, nullblob, client);