s3: VFS: Change SMB_VFS_CHDIR to use const struct smb_filename * instead of const...
[sfrench/samba-autobuild/.git] / source3 / smbd / service.c
index da5d763ad8fd3f279fa61da5ec101b3ad5cbe5aa..47cd0fa19d5b922cd5d2568f23d18091933ec01e 100644 (file)
@@ -30,6 +30,8 @@
 #include "auth.h"
 #include "lib/param/loadparm.h"
 #include "messages.h"
+#include "lib/afs/afs_funcs.h"
+#include "lib/util_path.h"
 
 static bool canonicalize_connect_path(connection_struct *conn)
 {
@@ -46,124 +48,32 @@ static bool canonicalize_connect_path(connection_struct *conn)
 /****************************************************************************
  Ensure when setting connectpath it is a canonicalized (no ./ // or ../)
  absolute path stating in / and not ending in /.
- Observent people will notice a similarity between this and check_path_syntax :-).
 ****************************************************************************/
 
 bool set_conn_connectpath(connection_struct *conn, const char *connectpath)
 {
        char *destname;
-       char *d;
-       const char *s = connectpath;
-        bool start_of_name_component = true;
 
        if (connectpath == NULL || connectpath[0] == '\0') {
                return false;
        }
 
-       /* Allocate for strlen + '\0' + possible leading '/' */
-       destname = (char *)SMB_MALLOC(strlen(connectpath) + 2);
-       if (!destname) {
+       destname = canonicalize_absolute_path(conn, connectpath);
+       if (destname == NULL) {
                return false;
        }
-       d = destname;
-
-       *d++ = '/'; /* Always start with root. */
-
-       while (*s) {
-               if (*s == '/') {
-                       /* Eat multiple '/' */
-                       while (*s == '/') {
-                                s++;
-                        }
-                       if ((d > destname + 1) && (*s != '\0')) {
-                               *d++ = '/';
-                       }
-                       start_of_name_component = True;
-                       continue;
-               }
-
-               if (start_of_name_component) {
-                       if ((s[0] == '.') && (s[1] == '.') && (s[2] == '/' || s[2] == '\0')) {
-                               /* Uh oh - "/../" or "/..\0" ! */
-
-                               /* Go past the ../ or .. */
-                               if (s[2] == '/') {
-                                       s += 3;
-                               } else {
-                                       s += 2; /* Go past the .. */
-                               }
-
-                               /* If  we just added a '/' - delete it */
-                               if ((d > destname) && (*(d-1) == '/')) {
-                                       *(d-1) = '\0';
-                                       d--;
-                               }
-
-                               /* Are we at the start ? Can't go back further if so. */
-                               if (d <= destname) {
-                                       *d++ = '/'; /* Can't delete root */
-                                       continue;
-                               }
-                               /* Go back one level... */
-                               /* Decrement d first as d points to the *next* char to write into. */
-                               for (d--; d > destname; d--) {
-                                       if (*d == '/') {
-                                               break;
-                                       }
-                               }
-                               /* We're still at the start of a name component, just the previous one. */
-                               continue;
-                       } else if ((s[0] == '.') && ((s[1] == '\0') || s[1] == '/')) {
-                               /* Component of pathname can't be "." only - skip the '.' . */
-                               if (s[1] == '/') {
-                                       s += 2;
-                               } else {
-                                       s++;
-                               }
-                               continue;
-                       }
-               }
-
-               if (!(*s & 0x80)) {
-                       *d++ = *s++;
-               } else {
-                       size_t siz;
-                       /* Get the size of the next MB character. */
-                       next_codepoint(s,&siz);
-                       switch(siz) {
-                               case 5:
-                                       *d++ = *s++;
-                                       /*fall through*/
-                               case 4:
-                                       *d++ = *s++;
-                                       /*fall through*/
-                               case 3:
-                                       *d++ = *s++;
-                                       /*fall through*/
-                               case 2:
-                                       *d++ = *s++;
-                                       /*fall through*/
-                               case 1:
-                                       *d++ = *s++;
-                                       break;
-                               default:
-                                       break;
-                       }
-               }
-               start_of_name_component = false;
-       }
-       *d = '\0';
-
-       /* And must not end in '/' */
-       if (d > destname + 1 && (*(d-1) == '/')) {
-               *(d-1) = '\0';
-       }
 
        DEBUG(10,("set_conn_connectpath: service %s, connectpath = %s\n",
-               lp_servicename(SNUM(conn)), destname ));
-
-       string_set(&conn->connectpath, destname);
-       SAFE_FREE(destname);
+               lp_servicename(talloc_tos(), SNUM(conn)), destname ));
+
+       talloc_free(conn->connectpath);
+       conn->connectpath = destname;
+       /* Ensure conn->cwd is initialized - start as conn->connectpath. */
+       TALLOC_FREE(conn->cwd);
+       conn->cwd = talloc_strdup(conn, conn->connectpath);
+       if (!conn->cwd) {
+               return false;
+       }
        return true;
 }
 
@@ -171,9 +81,10 @@ bool set_conn_connectpath(connection_struct *conn, const char *connectpath)
  Load parameters specific to a connection/service.
 ****************************************************************************/
 
-bool set_current_service(connection_struct *conn, uint16 flags, bool do_chdir)
+bool set_current_service(connection_struct *conn, uint16_t flags, bool do_chdir)
 {
        int snum;
+       enum remote_arch_types ra_type;
 
        if (!conn)  {
                last_conn = NULL;
@@ -184,12 +95,22 @@ bool set_current_service(connection_struct *conn, uint16 flags, bool do_chdir)
 
        snum = SNUM(conn);
 
-       if (do_chdir &&
-           vfs_ChDir(conn,conn->connectpath) != 0 &&
-           vfs_ChDir(conn,conn->origpath) != 0) {
-                DEBUG(((errno!=EACCES)?0:3),("chdir (%s) failed, reason: %s\n",
-                         conn->connectpath, strerror(errno)));
-               return(False);
+       {
+               struct smb_filename connectpath_fname = {
+                       .base_name = conn->connectpath
+               };
+               struct smb_filename origpath_fname = {
+                       .base_name = conn->origpath
+               };
+
+               if (do_chdir &&
+                   vfs_ChDir(conn, &connectpath_fname) != 0 &&
+                   vfs_ChDir(conn, &origpath_fname) != 0) {
+                       DEBUG(((errno!=EACCES)?0:3),
+                               ("chdir (%s) failed, reason: %s\n",
+                               conn->connectpath, strerror(errno)));
+                       return(False);
+               }
        }
 
        if ((conn == last_conn) && (last_flags == flags)) {
@@ -199,28 +120,35 @@ bool set_current_service(connection_struct *conn, uint16 flags, bool do_chdir)
        last_conn = conn;
        last_flags = flags;
 
-       /* Obey the client case sensitivity requests - only for clients that support it. */
-       switch (lp_casesensitive(snum)) {
-               case Auto:
-                       {
-                               /* We need this uglyness due to DOS/Win9x clients that lie about case insensitivity. */
-                               enum remote_arch_types ra_type = get_remote_arch();
-                               if ((ra_type != RA_SAMBA) && (ra_type != RA_CIFSFS)) {
-                                       /* Client can't support per-packet case sensitive pathnames. */
-                                       conn->case_sensitive = False;
-                               } else {
-                                       conn->case_sensitive = !(flags & FLAG_CASELESS_PATHNAMES);
-                               }
-                       }
-                       break;
-               case True:
-                       conn->case_sensitive = True;
-                       break;
-               default:
-                       conn->case_sensitive = False;
-                       break;
-       }
-       return(True);
+       /*
+        * Obey the client case sensitivity requests - only for clients that
+        * support it. */
+       switch (lp_case_sensitive(snum)) {
+       case Auto:
+               /*
+                * We need this uglyness due to DOS/Win9x clients that lie
+                * about case insensitivity. */
+               ra_type = get_remote_arch();
+               if (conn->sconn->using_smb2) {
+                       conn->case_sensitive = false;
+               } else if ((ra_type != RA_SAMBA) && (ra_type != RA_CIFSFS)) {
+                       /*
+                        * Client can't support per-packet case sensitive
+                        * pathnames. */
+                       conn->case_sensitive = false;
+               } else {
+                       conn->case_sensitive =
+                                       !(flags & FLAG_CASELESS_PATHNAMES);
+               }
+       break;
+       case True:
+               conn->case_sensitive = true;
+               break;
+       default:
+               conn->case_sensitive = false;
+               break;
+       }
+       return true;
 }
 
 /****************************************************************************
@@ -242,13 +170,13 @@ static NTSTATUS share_sanity_checks(const struct tsocket_address *remote_address
        }
 
        if (!lp_snum_ok(snum) ||
-           !allow_access(lp_hostsdeny(snum), lp_hostsallow(snum),
+           !allow_access(lp_hosts_deny(snum), lp_hosts_allow(snum),
                          rhost, raddr)) {
                return NT_STATUS_ACCESS_DENIED;
        }
 
        if (dev[0] == '?' || !dev[0]) {
-               if (lp_print_ok(snum)) {
+               if (lp_printable(snum)) {
                        fstrcpy(dev,"LPT1:");
                } else if (strequal(lp_fstype(snum), "IPC")) {
                        fstrcpy(dev, "IPC");
@@ -257,9 +185,12 @@ static NTSTATUS share_sanity_checks(const struct tsocket_address *remote_address
                }
        }
 
-       strupper_m(dev);
+       if (!strupper_m(dev)) {
+               DEBUG(2,("strupper_m %s failed\n", dev));
+               return NT_STATUS_INVALID_PARAMETER;
+       }
 
-       if (lp_print_ok(snum)) {
+       if (lp_printable(snum)) {
                if (!strequal(dev, "LPT1:")) {
                        return NT_STATUS_BAD_DEVICE_TYPE;
                }
@@ -272,7 +203,7 @@ static NTSTATUS share_sanity_checks(const struct tsocket_address *remote_address
        }
 
        /* Behave as a printer if we are supposed to */
-       if (lp_print_ok(snum) && (strcmp(dev, "A:") == 0)) {
+       if (lp_printable(snum) && (strcmp(dev, "A:") == 0)) {
                fstrcpy(dev, "LPT1:");
        }
 
@@ -299,7 +230,7 @@ static NTSTATUS find_forced_group(bool force_user,
        bool user_must_be_member = False;
        gid_t gid;
 
-       groupname = talloc_strdup(talloc_tos(), lp_force_group(snum));
+       groupname = lp_force_group(talloc_tos(), snum);
        if (groupname == NULL) {
                DEBUG(1, ("talloc_strdup failed\n"));
                result = NT_STATUS_NO_MEMORY;
@@ -312,7 +243,7 @@ static NTSTATUS find_forced_group(bool force_user,
        }
 
        groupname = talloc_string_sub(talloc_tos(), groupname,
-                                     "%S", lp_servicename(snum));
+                                     "%S", lp_servicename(talloc_tos(), snum));
        if (groupname == NULL) {
                DEBUG(1, ("talloc_string_sub failed\n"));
                result = NT_STATUS_NO_MEMORY;
@@ -395,7 +326,7 @@ static NTSTATUS create_connection_session_info(struct smbd_server_connection *sc
                if (!lp_guest_ok(snum)) {
                        DEBUG(2, ("guest user (from session setup) "
                                  "not permitted to access this share "
-                                 "(%s)\n", lp_servicename(snum)));
+                                 "(%s)\n", lp_servicename(talloc_tos(), snum)));
                        return NT_STATUS_ACCESS_DENIED;
                }
        } else {
@@ -406,7 +337,7 @@ static NTSTATUS create_connection_session_info(struct smbd_server_connection *sc
                                  "permitted to access this share "
                                  "(%s)\n",
                                  session_info->unix_info->unix_name,
-                                 lp_servicename(snum)));
+                                 lp_servicename(talloc_tos(), snum)));
                        return NT_STATUS_ACCESS_DENIED;
                }
        }
@@ -421,7 +352,7 @@ static NTSTATUS create_connection_session_info(struct smbd_server_connection *sc
 }
 
 /****************************************************************************
-  set relavent user and group settings corresponding to force user/group
+  Set relevant user and group settings corresponding to force user/group
   configuration for the given snum.
 ****************************************************************************/
 
@@ -429,7 +360,7 @@ NTSTATUS set_conn_force_user_group(connection_struct *conn, int snum)
 {
        NTSTATUS status;
 
-       if (*lp_force_user(snum)) {
+       if (*lp_force_user(talloc_tos(), snum)) {
 
                /*
                 * Replace conn->session_info with a completely faked up one
@@ -441,7 +372,7 @@ NTSTATUS set_conn_force_user_group(connection_struct *conn, int snum)
                struct auth_session_info *forced_serverinfo;
                bool guest;
 
-               fuser = talloc_string_sub(conn, lp_force_user(snum), "%S",
+               fuser = talloc_string_sub(conn, lp_force_user(talloc_tos(), snum), "%S",
                                          lp_const_servicename(snum));
                if (fuser == NULL) {
                        return NT_STATUS_NO_MEMORY;
@@ -479,7 +410,7 @@ NTSTATUS set_conn_force_user_group(connection_struct *conn, int snum)
         * any groupid stored for the connecting user.
         */
 
-       if (*lp_force_group(snum)) {
+       if (*lp_force_group(talloc_tos(), snum)) {
 
                status = find_forced_group(
                        conn->force_user, snum, conn->session_info->unix_info->unix_name,
@@ -502,31 +433,43 @@ NTSTATUS set_conn_force_user_group(connection_struct *conn, int snum)
        return NT_STATUS_OK;
 }
 
-/****************************************************************************
-  Setup the share access mask for a connection.
-****************************************************************************/
-
-static void create_share_access_mask(connection_struct *conn, int snum)
+static NTSTATUS notify_init_sconn(struct smbd_server_connection *sconn)
 {
-       const struct security_token *token = conn->session_info->security_token;
-
-       share_access_check(token,
-                       lp_servicename(snum),
-                       MAXIMUM_ALLOWED_ACCESS,
-                       &conn->share_access);
+       NTSTATUS status;
 
-       if (security_token_has_privilege(token, SEC_PRIV_SECURITY)) {
-               conn->share_access |= SEC_FLAG_SYSTEM_SECURITY;
+       if (sconn->notify_ctx != NULL) {
+               return NT_STATUS_OK;
        }
-       if (security_token_has_privilege(token, SEC_PRIV_RESTORE)) {
-               conn->share_access |= (SEC_RIGHTS_PRIV_RESTORE);
+
+       sconn->notify_ctx = notify_init(sconn, sconn->msg_ctx, sconn->ev_ctx,
+                                       sconn, notify_callback);
+       if (sconn->notify_ctx == NULL) {
+               return NT_STATUS_NO_MEMORY;
        }
-       if (security_token_has_privilege(token, SEC_PRIV_BACKUP)) {
-               conn->share_access |= (SEC_RIGHTS_PRIV_BACKUP);
+
+       status = messaging_register(sconn->msg_ctx, sconn,
+                                   MSG_SMB_NOTIFY_CANCEL_DELETED,
+                                   smbd_notify_cancel_deleted);
+       if (!NT_STATUS_IS_OK(status)) {
+               DBG_DEBUG("messaging_register failed: %s\n",
+                         nt_errstr(status));
+               TALLOC_FREE(sconn->notify_ctx);
+               return status;
        }
-       if (security_token_has_privilege(token, SEC_PRIV_TAKE_OWNERSHIP)) {
-               conn->share_access |= (SEC_STD_WRITE_OWNER);
+
+       status = messaging_register(sconn->msg_ctx, sconn,
+                                   MSG_SMB_NOTIFY_STARTED,
+                                   smbd_notifyd_restarted);
+       if (!NT_STATUS_IS_OK(status)) {
+               DBG_DEBUG("messaging_register failed: %s\n",
+                         nt_errstr(status));
+               messaging_deregister(sconn->msg_ctx,
+                                    MSG_SMB_NOTIFY_CANCEL_DELETED, sconn);
+               TALLOC_FREE(sconn->notify_ctx);
+               return status;
        }
+
+       return NT_STATUS_OK;
 }
 
 /****************************************************************************
@@ -534,16 +477,16 @@ static void create_share_access_mask(connection_struct *conn, int snum)
   connecting user if appropriate.
 ****************************************************************************/
 
-static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
+static NTSTATUS make_connection_snum(struct smbXsrv_connection *xconn,
                                        connection_struct *conn,
                                        int snum, struct user_struct *vuser,
                                        const char *pdev)
 {
+       struct smbd_server_connection *sconn = xconn->client->sconn;
        struct smb_filename *smb_fname_cpath = NULL;
        fstring dev;
        int ret;
        bool on_err_call_dis_hook = false;
-       bool claimed_connection = false;
        uid_t effuid;
        gid_t effgid;
        NTSTATUS status;
@@ -581,25 +524,37 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
                      ( lp_enable_asu_support() && strequal(dev,"ADMIN$")) );
 
        /* Case options for the share. */
-       if (lp_casesensitive(snum) == Auto) {
+       if (lp_case_sensitive(snum) == Auto) {
                /* We will be setting this per packet. Set to be case
                 * insensitive for now. */
                conn->case_sensitive = False;
        } else {
-               conn->case_sensitive = (bool)lp_casesensitive(snum);
+               conn->case_sensitive = (bool)lp_case_sensitive(snum);
        }
 
-       conn->case_preserve = lp_preservecase(snum);
-       conn->short_case_preserve = lp_shortpreservecase(snum);
+       conn->case_preserve = lp_preserve_case(snum);
+       conn->short_case_preserve = lp_short_preserve_case(snum);
 
        conn->encrypt_level = lp_smb_encrypt(snum);
+       if (conn->encrypt_level > SMB_SIGNING_OFF) {
+               if (lp_smb_encrypt(-1) == SMB_SIGNING_OFF) {
+                       if (conn->encrypt_level == SMB_SIGNING_REQUIRED) {
+                               DBG_ERR("Service [%s] requires encryption, but "
+                                       "it is disabled globally!\n",
+                                       lp_servicename(talloc_tos(), snum));
+                               status = NT_STATUS_ACCESS_DENIED;
+                               goto err_root_exit;
+                       }
+                       conn->encrypt_level = SMB_SIGNING_OFF;
+               }
+       }
 
        conn->veto_list = NULL;
        conn->hide_list = NULL;
        conn->veto_oplock_list = NULL;
        conn->aio_write_behind_list = NULL;
 
-       conn->read_only = lp_readonly(SNUM(conn));
+       conn->read_only = lp_read_only(SNUM(conn));
 
        status = set_conn_force_user_group(conn, snum);
        if (!NT_STATUS_IS_OK(status)) {
@@ -610,13 +565,13 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
 
        {
                char *s = talloc_sub_advanced(talloc_tos(),
-                                       lp_servicename(SNUM(conn)),
+                                       lp_servicename(talloc_tos(), SNUM(conn)),
                                        conn->session_info->unix_info->unix_name,
                                        conn->connectpath,
                                        conn->session_info->unix_token->gid,
                                        conn->session_info->unix_info->sanitized_username,
                                        conn->session_info->info->domain_name,
-                                       lp_pathname(snum));
+                                       lp_path(talloc_tos(), snum));
                if (!s) {
                        status = NT_STATUS_NO_MEMORY;
                        goto err_root_exit;
@@ -628,37 +583,35 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
                        goto err_root_exit;
                }
                DEBUG(3,("Connect path is '%s' for service [%s]\n",s,
-                        lp_servicename(snum)));
+                        lp_servicename(talloc_tos(), snum)));
                TALLOC_FREE(s);
        }
 
        /*
-        * New code to check if there's a share security descripter
-        * added from NT server manager. This is done after the
-        * smb.conf checks are done as we need a uid and token. JRA.
+        * Set up the share security descriptor.
+        * NOTE - we use the *INCOMING USER* session_info
+        * here, as does (indirectly) change_to_user(),
+        * which can be called on any incoming packet.
+        * This way we set up the share access based
+        * on the authenticated user, not the forced
+        * user. See bug:
         *
+        * https://bugzilla.samba.org/show_bug.cgi?id=9878
         */
 
-       create_share_access_mask(conn, snum);
-
-       if ((conn->share_access & FILE_WRITE_DATA) == 0) {
-               if ((conn->share_access & FILE_READ_DATA) == 0) {
-                       /* No access, read or write. */
-                       DEBUG(0,("make_connection: connection to %s "
-                                "denied due to security "
-                                "descriptor.\n",
-                                lp_servicename(snum)));
-                       status = NT_STATUS_ACCESS_DENIED;
-                       goto err_root_exit;
-               } else {
-                       conn->read_only = True;
-               }
+       status = check_user_share_access(conn,
+                                       vuser->session_info,
+                                       &conn->share_access,
+                                       &conn->read_only);
+       if (!NT_STATUS_IS_OK(status)) {
+               goto err_root_exit;
        }
+
        /* Initialise VFS function pointers */
 
        if (!smbd_vfs_init(conn)) {
                DEBUG(0, ("vfs_init failed for service %s\n",
-                         lp_servicename(snum)));
+                         lp_servicename(talloc_tos(), snum)));
                status = NT_STATUS_BAD_NETWORK_NAME;
                goto err_root_exit;
        }
@@ -673,31 +626,24 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
         */
 
        if ((lp_max_connections(snum) > 0)
-           && (count_current_connections(lp_servicename(SNUM(conn)), True) >=
+           && (count_current_connections(lp_servicename(talloc_tos(), SNUM(conn)), True) >=
                lp_max_connections(snum))) {
 
                DEBUG(1, ("Max connections (%d) exceeded for %s\n",
-                         lp_max_connections(snum), lp_servicename(snum)));
+                         lp_max_connections(snum),
+                         lp_servicename(talloc_tos(), snum)));
                status = NT_STATUS_INSUFFICIENT_RESOURCES;
                goto err_root_exit;
        }
 
-       /*
-        * Get us an entry in the connections db
-        */
-       if (!claim_connection(conn, lp_servicename(snum))) {
-               DEBUG(1, ("Could not store connections entry\n"));
-               status = NT_STATUS_INTERNAL_DB_ERROR;
-               goto err_root_exit;
-       }
-       claimed_connection = true;
-
        /* Invoke VFS make connection hook - this must be the first
           filesystem operation that we do. */
 
-       if (SMB_VFS_CONNECT(conn, lp_servicename(snum),
+       if (SMB_VFS_CONNECT(conn, lp_servicename(talloc_tos(), snum),
                            conn->session_info->unix_info->unix_name) < 0) {
-               DEBUG(0,("make_connection: VFS make connection failed!\n"));
+               DBG_WARNING("SMB_VFS_CONNECT for service '%s' at '%s' failed: %s\n",
+                           lp_servicename(talloc_tos(), snum), conn->connectpath,
+                           strerror(errno));
                status = NT_STATUS_UNSUCCESSFUL;
                goto err_root_exit;
        }
@@ -706,14 +652,11 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
        on_err_call_dis_hook = true;
 
        if ((!conn->printer) && (!conn->ipc) &&
-           lp_change_notify(conn->params)) {
-               if (sconn->notify_ctx == NULL) {
-                       sconn->notify_ctx = notify_init(
-                               sconn, sconn->msg_ctx, sconn->ev_ctx);
-               }
-               if (sconn->sys_notify_ctx == NULL) {
-                       sconn->sys_notify_ctx = sys_notify_context_create(
-                               sconn, sconn->ev_ctx);
+           lp_change_notify()) {
+
+               status = notify_init_sconn(sconn);
+               if (!NT_STATUS_IS_OK(status)) {
+                       goto err_root_exit;
                }
        }
 
@@ -736,19 +679,19 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
        /* Preexecs are done here as they might make the dir we are to ChDir
         * to below */
        /* execute any "root preexec = " line */
-       if (*lp_rootpreexec(snum)) {
+       if (*lp_root_preexec(talloc_tos(), snum)) {
                char *cmd = talloc_sub_advanced(talloc_tos(),
-                                       lp_servicename(SNUM(conn)),
+                                       lp_servicename(talloc_tos(), SNUM(conn)),
                                        conn->session_info->unix_info->unix_name,
                                        conn->connectpath,
                                        conn->session_info->unix_token->gid,
                                        conn->session_info->unix_info->sanitized_username,
                                        conn->session_info->info->domain_name,
-                                       lp_rootpreexec(snum));
+                                       lp_root_preexec(talloc_tos(), snum));
                DEBUG(5,("cmd=%s\n",cmd));
-               ret = smbrun(cmd,NULL);
+               ret = smbrun(cmd, NULL, NULL);
                TALLOC_FREE(cmd);
-               if (ret != 0 && lp_rootpreexec_close(snum)) {
+               if (ret != 0 && lp_root_preexec_close(snum)) {
                        DEBUG(1,("root preexec gave %d - failing "
                                 "connection\n", ret));
                        status = NT_STATUS_ACCESS_DENIED;
@@ -774,16 +717,16 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
         * to below */
 
        /* execute any "preexec = " line */
-       if (*lp_preexec(snum)) {
+       if (*lp_preexec(talloc_tos(), snum)) {
                char *cmd = talloc_sub_advanced(talloc_tos(),
-                                       lp_servicename(SNUM(conn)),
+                                       lp_servicename(talloc_tos(), SNUM(conn)),
                                        conn->session_info->unix_info->unix_name,
                                        conn->connectpath,
                                        conn->session_info->unix_token->gid,
                                        conn->session_info->unix_info->sanitized_username,
                                        conn->session_info->info->domain_name,
-                                       lp_preexec(snum));
-               ret = smbrun(cmd,NULL);
+                                       lp_preexec(talloc_tos(), snum));
+               ret = smbrun(cmd, NULL, NULL);
                TALLOC_FREE(cmd);
                if (ret != 0 && lp_preexec_close(snum)) {
                        DEBUG(1,("preexec gave %d - failing connection\n",
@@ -818,7 +761,7 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
                if (!canonicalize_connect_path(conn)) {
                        DEBUG(0, ("canonicalize_connect_path failed "
                        "for service %s, path %s\n",
-                               lp_servicename(snum),
+                               lp_servicename(talloc_tos(), snum),
                                conn->connectpath));
                        status = NT_STATUS_BAD_NETWORK_NAME;
                        goto err_root_exit;
@@ -827,15 +770,22 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
 
        /* Add veto/hide lists */
        if (!IS_IPC(conn) && !IS_PRINT(conn)) {
-               set_namearray( &conn->veto_list, lp_veto_files(snum));
-               set_namearray( &conn->hide_list, lp_hide_files(snum));
-               set_namearray( &conn->veto_oplock_list, lp_veto_oplocks(snum));
+               set_namearray( &conn->veto_list,
+                              lp_veto_files(talloc_tos(), snum));
+               set_namearray( &conn->hide_list,
+                              lp_hide_files(talloc_tos(), snum));
+               set_namearray( &conn->veto_oplock_list,
+                              lp_veto_oplock_files(talloc_tos(), snum));
                set_namearray( &conn->aio_write_behind_list,
-                               lp_aio_write_behind(snum));
+                               lp_aio_write_behind(talloc_tos(), snum));
        }
-       status = create_synthetic_smb_fname(talloc_tos(), conn->connectpath,
-                                           NULL, NULL, &smb_fname_cpath);
-       if (!NT_STATUS_IS_OK(status)) {
+       smb_fname_cpath = synthetic_smb_fname(talloc_tos(),
+                                       conn->connectpath,
+                                       NULL,
+                                       NULL,
+                                       0);
+       if (smb_fname_cpath == NULL) {
+               status = NT_STATUS_NO_MEMORY;
                goto err_root_exit;
        }
 
@@ -850,11 +800,12 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
                if (ret == 0 && !S_ISDIR(smb_fname_cpath->st.st_ex_mode)) {
                        DEBUG(0,("'%s' is not a directory, when connecting to "
                                 "[%s]\n", conn->connectpath,
-                                lp_servicename(snum)));
+                                lp_servicename(talloc_tos(), snum)));
                } else {
                        DEBUG(0,("'%s' does not exist or permission denied "
                                 "when connecting to [%s] Error was %s\n",
-                                conn->connectpath, lp_servicename(snum),
+                                conn->connectpath,
+                                lp_servicename(talloc_tos(), snum),
                                 strerror(errno) ));
                }
                status = NT_STATUS_BAD_NETWORK_NAME;
@@ -862,7 +813,8 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
        }
        conn->base_share_dev = smb_fname_cpath->st.st_ex_dev;
 
-       string_set(&conn->origpath,conn->connectpath);
+       talloc_free(conn->origpath);
+       conn->origpath = talloc_strdup(conn, conn->connectpath);
 
        /* Figure out the characteristics of the underlying filesystem. This
         * assumes that all the filesystem mounted withing a share path have
@@ -877,12 +829,13 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
         * (at least initially).
         */
 
-       if( DEBUGLVL( IS_IPC(conn) ? 3 : 1 ) ) {
+       if( DEBUGLVL( IS_IPC(conn) ? 3 : 2 ) ) {
                dbgtext( "%s (%s) ", get_remote_machine_name(),
                         tsocket_address_string(conn->sconn->remote_address,
                                                talloc_tos()) );
-               dbgtext( "%s", srv_is_signing_active(sconn) ? "signed " : "");
-               dbgtext( "connect to service %s ", lp_servicename(snum) );
+               dbgtext( "%s", srv_is_signing_active(xconn) ? "signed " : "");
+               dbgtext( "connect to service %s ",
+                        lp_servicename(talloc_tos(), snum) );
                dbgtext( "initially as user %s ",
                         conn->session_info->unix_info->unix_name );
                dbgtext( "(uid=%d, gid=%d) ", (int)effuid, (int)effgid );
@@ -902,9 +855,6 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
                /* Call VFS disconnect hook */
                SMB_VFS_DISCONNECT(conn);
        }
-       if (claimed_connection) {
-               yield_connection(conn, lp_servicename(snum));
-       }
        return status;
 }
 
@@ -912,18 +862,17 @@ static NTSTATUS make_connection_snum(struct smbd_server_connection *sconn,
  Make a connection to a service from SMB1. Internal interface.
 ****************************************************************************/
 
-static connection_struct *make_connection_smb1(struct smbd_server_connection *sconn,
+static connection_struct *make_connection_smb1(struct smb_request *req,
+                                       NTTIME now,
                                        int snum, struct user_struct *vuser,
                                        const char *pdev,
                                        NTSTATUS *pstatus)
 {
        struct smbXsrv_tcon *tcon;
        NTSTATUS status;
-       NTTIME now = 0;
        struct connection_struct *conn;
-       const char *share_name;
 
-       status = smb1srv_tcon_create(sconn->conn, now, &tcon);
+       status = smb1srv_tcon_create(req->xconn, now, &tcon);
        if (!NT_STATUS_IS_OK(status)) {
                DEBUG(0,("make_connection_smb1: Couldn't find free tcon %s.\n",
                         nt_errstr(status)));
@@ -931,7 +880,7 @@ static connection_struct *make_connection_smb1(struct smbd_server_connection *sc
                return NULL;
        }
 
-       conn = conn_new(sconn);
+       conn = conn_new(req->sconn);
        if (!conn) {
                TALLOC_FREE(tcon);
 
@@ -943,7 +892,7 @@ static connection_struct *make_connection_smb1(struct smbd_server_connection *sc
        conn->cnum = tcon->global->tcon_wire_id;
        conn->tcon = tcon;
 
-       *pstatus = make_connection_snum(sconn,
+       *pstatus = make_connection_snum(req->xconn,
                                        conn,
                                        snum,
                                        vuser,
@@ -954,14 +903,15 @@ static connection_struct *make_connection_smb1(struct smbd_server_connection *sc
                return NULL;
        }
 
-       share_name = lp_servicename(SNUM(conn));
-       tcon->global->share_name = talloc_strdup(tcon->global, share_name);
+       tcon->global->share_name = lp_servicename(tcon->global, SNUM(conn));
        if (tcon->global->share_name == NULL) {
                conn_free(conn);
                TALLOC_FREE(tcon);
                *pstatus = NT_STATUS_NO_MEMORY;
                return NULL;
        }
+       tcon->global->session_global_id =
+               vuser->session->global->session_global_id;
 
        tcon->compat = talloc_move(tcon, &conn);
        tcon->status = NT_STATUS_OK;
@@ -980,13 +930,14 @@ static connection_struct *make_connection_smb1(struct smbd_server_connection *sc
  We must set cnum before claiming connection.
 ****************************************************************************/
 
-connection_struct *make_connection_smb2(struct smbd_server_connection *sconn,
+connection_struct *make_connection_smb2(struct smbd_smb2_request *req,
                                        struct smbXsrv_tcon *tcon,
                                        int snum,
                                        struct user_struct *vuser,
                                        const char *pdev,
                                        NTSTATUS *pstatus)
 {
+       struct smbd_server_connection *sconn = req->sconn;
        connection_struct *conn = conn_new(sconn);
        if (!conn) {
                DEBUG(0,("make_connection_smb2: Couldn't find free connection.\n"));
@@ -997,7 +948,7 @@ connection_struct *make_connection_smb2(struct smbd_server_connection *sconn,
        conn->cnum = tcon->global->tcon_wire_id;
        conn->tcon = tcon;
 
-       *pstatus = make_connection_snum(sconn,
+       *pstatus = make_connection_snum(req->xconn,
                                        conn,
                                        snum,
                                        vuser,
@@ -1015,11 +966,13 @@ connection_struct *make_connection_smb2(struct smbd_server_connection *sconn,
  * @param service 
 ****************************************************************************/
 
-connection_struct *make_connection(struct smbd_server_connection *sconn,
+connection_struct *make_connection(struct smb_request *req,
+                                  NTTIME now,
                                   const char *service_in,
                                   const char *pdev, uint64_t vuid,
                                   NTSTATUS *status)
 {
+       struct smbd_server_connection *sconn = req->sconn;
        uid_t euid;
        struct user_struct *vuser = NULL;
        char *service = NULL;
@@ -1069,16 +1022,16 @@ connection_struct *make_connection(struct smbd_server_connection *sconn,
                }
                DEBUG(5, ("making a connection to [homes] service "
                          "created at session setup time\n"));
-               return make_connection_smb1(sconn,
+               return make_connection_smb1(req, now,
                                            vuser->homes_snum,
                                            vuser,
                                            dev, status);
        } else if ((vuser->homes_snum != -1)
                   && strequal(service_in,
-                              lp_servicename(vuser->homes_snum))) {
+                              lp_servicename(talloc_tos(), vuser->homes_snum))) {
                DEBUG(5, ("making a connection to 'homes' service [%s] "
                          "created at session setup time\n", service_in));
-               return make_connection_smb1(sconn,
+               return make_connection_smb1(req, now,
                                            vuser->homes_snum,
                                            vuser,
                                            dev, status);
@@ -1090,7 +1043,11 @@ connection_struct *make_connection(struct smbd_server_connection *sconn,
                return NULL;
        }
 
-       strlower_m(service);
+       if (!strlower_m(service)) {
+               DEBUG(2, ("strlower_m %s failed\n", service));
+               *status = NT_STATUS_INVALID_PARAMETER;
+               return NULL;
+       }
 
        snum = find_service(talloc_tos(), service, &service);
        if (!service) {
@@ -1116,17 +1073,17 @@ connection_struct *make_connection(struct smbd_server_connection *sconn,
        }
 
        /* Handle non-Dfs clients attempting connections to msdfs proxy */
-       if (lp_host_msdfs() && (*lp_msdfs_proxy(snum) != '\0'))  {
+       if (lp_host_msdfs() && (*lp_msdfs_proxy(talloc_tos(), snum) != '\0'))  {
                DEBUG(3, ("refusing connection to dfs proxy share '%s' "
                          "(pointing to %s)\n", 
-                       service, lp_msdfs_proxy(snum)));
+                       service, lp_msdfs_proxy(talloc_tos(), snum)));
                *status = NT_STATUS_BAD_NETWORK_NAME;
                return NULL;
        }
 
        DEBUG(5, ("making a connection to 'normal' service %s\n", service));
 
-       return make_connection_smb1(sconn, snum, vuser,
+       return make_connection_smb1(req, now, snum, vuser,
                                    dev, status);
 }
 
@@ -1136,6 +1093,9 @@ connection_struct *make_connection(struct smbd_server_connection *sconn,
 
 void close_cnum(connection_struct *conn, uint64_t vuid)
 {
+       char rootpath[2] = { '/', '\0'};
+       struct smb_filename root_fname = { .base_name = rootpath };
+
        file_close_conn(conn);
 
        if (!IS_IPC(conn)) {
@@ -1144,48 +1104,46 @@ void close_cnum(connection_struct *conn, uint64_t vuid)
 
        change_to_root_user();
 
-       DEBUG(IS_IPC(conn)?3:1, ("%s (%s) closed connection to service %s\n",
+       DEBUG(IS_IPC(conn)?3:2, ("%s (%s) closed connection to service %s\n",
                                 get_remote_machine_name(),
                                 tsocket_address_string(conn->sconn->remote_address,
                                                        talloc_tos()),
-                                lp_servicename(SNUM(conn))));
-
-       /* Call VFS disconnect hook */    
-       SMB_VFS_DISCONNECT(conn);
-
-       yield_connection(conn, lp_servicename(SNUM(conn)));
+                                lp_servicename(talloc_tos(), SNUM(conn))));
 
        /* make sure we leave the directory available for unmount */
-       vfs_ChDir(conn, "/");
+       vfs_ChDir(conn, &root_fname);
+
+       /* Call VFS disconnect hook */
+       SMB_VFS_DISCONNECT(conn);
 
        /* execute any "postexec = " line */
-       if (*lp_postexec(SNUM(conn)) && 
+       if (*lp_postexec(talloc_tos(), SNUM(conn)) &&
            change_to_user(conn, vuid))  {
                char *cmd = talloc_sub_advanced(talloc_tos(),
-                                       lp_servicename(SNUM(conn)),
+                                       lp_servicename(talloc_tos(), SNUM(conn)),
                                        conn->session_info->unix_info->unix_name,
                                        conn->connectpath,
                                        conn->session_info->unix_token->gid,
                                        conn->session_info->unix_info->sanitized_username,
                                        conn->session_info->info->domain_name,
-                                       lp_postexec(SNUM(conn)));
-               smbrun(cmd,NULL);
+                                       lp_postexec(talloc_tos(), SNUM(conn)));
+               smbrun(cmd, NULL, NULL);
                TALLOC_FREE(cmd);
                change_to_root_user();
        }
 
        change_to_root_user();
        /* execute any "root postexec = " line */
-       if (*lp_rootpostexec(SNUM(conn)))  {
+       if (*lp_root_postexec(talloc_tos(), SNUM(conn)))  {
                char *cmd = talloc_sub_advanced(talloc_tos(),
-                                       lp_servicename(SNUM(conn)),
+                                       lp_servicename(talloc_tos(), SNUM(conn)),
                                        conn->session_info->unix_info->unix_name,
                                        conn->connectpath,
                                        conn->session_info->unix_token->gid,
                                        conn->session_info->unix_info->sanitized_username,
                                        conn->session_info->info->domain_name,
-                                       lp_rootpostexec(SNUM(conn)));
-               smbrun(cmd,NULL);
+                                       lp_root_postexec(talloc_tos(), SNUM(conn)));
+               smbrun(cmd, NULL, NULL);
                TALLOC_FREE(cmd);
        }