smbd: Don't print cwd before tcon is done

vfs_GetWd depends upon a current tcon in fake_acls, otherwise it will
call openat with an invalid dirfd on a relative pathname.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>

diff --git a/source3/smbd/uid.c b/source3/smbd/uid.c
index e8f51003a1e4118a2354c5a6df4c2f7b9f5f98b3..74ab3d9dd08ae0b94688cce9d9aef2f037c6ac2c 100644 (file)
--- a/source3/smbd/uid.c
+++ b/source3/smbd/uid.c
@@ -453,9 +453,8 @@ static void print_impersonation_info(connection_struct *conn)
                return;
        }
 
-       cwdfname = vfs_GetWd(talloc_tos(), conn);
-       if (cwdfname == NULL) {
-               return;
+       if (conn->tcon_done) {
+               cwdfname = vfs_GetWd(talloc_tos(), conn);
        }
 
        DBG_INFO("Impersonated user: uid=(%d,%d), gid=(%d,%d), cwd=[%s]\n",
@@ -463,7 +462,7 @@ static void print_impersonation_info(connection_struct *conn)
                 (int)geteuid(),
                 (int)getgid(),
                 (int)getegid(),
-                cwdfname->base_name);
+                cwdfname ? cwdfname->base_name : "no cwd");
        TALLOC_FREE(cwdfname);
 }