s3-smbd: Fixed a possible null pointer dereference.

diff --git a/source3/smbd/uid.c b/source3/smbd/uid.c
index dd6af6d9c018950f38ffffd5607a43d98a24b471..b573a6c9209bac4bf084252f1ecc6e4c7a0f4bc8 100644 (file)
--- a/source3/smbd/uid.c
+++ b/source3/smbd/uid.c
@@ -229,7 +229,7 @@ void conn_clear_vuid_cache(connection_struct *conn, uint16_t vuid)
 bool change_to_user(connection_struct *conn, uint16 vuid)
 {
        const struct auth_serversupplied_info *server_info = NULL;
-       user_struct *vuser = get_valid_user_struct(conn->sconn, vuid);
+       user_struct *vuser;
        int snum;
        gid_t gid;
        uid_t uid;
@@ -242,6 +242,8 @@ bool change_to_user(connection_struct *conn, uint16 vuid)
                return(False);
        }
 
+       vuser = get_valid_user_struct(conn->sconn, vuid);
+
        /*
         * We need a separate check in security=share mode due to vuid
         * always being UID_FIELD_INVALID. If we don't do this then