s3:smbd: remove dirptr and dirpath from connection_struct
[samba.git] / source3 / smbd / dir.c
index ca6f8bfd8dc22a2690afaf177f6a6c559fda2aaa..1c84decbde1d88f1cbe6694318d836a5e0301bcd 100644 (file)
 */
 
 #include "includes.h"
+#include "smbd/globals.h"
 
 /*
    This module implements directory related functions for Samba.
 */
 
-extern struct current_user current_user;
-
 /* "Special" directory offsets. */
 #define END_OF_DIRECTORY_OFFSET ((long)-1)
 #define START_OF_DIRECTORY_OFFSET ((long)0)
@@ -63,9 +62,6 @@ struct dptr_struct {
        bool did_stat; /* Optimisation for non-wcard searches. */
 };
 
-static struct bitmap *dptr_bmap;
-static struct dptr_struct *dirptrs;
-static int dirhandles_open = 0;
 
 #define INVALID_DPTR_KEY (-3)
 
@@ -121,17 +117,13 @@ bool make_dir_struct(TALLOC_CTX *ctx,
 
 void init_dptrs(void)
 {
-       static bool dptrs_init=False;
-
-       if (dptrs_init)
+       if (dptr_bmap)
                return;
 
        dptr_bmap = bitmap_allocate(MAX_DIRECTORY_HANDLES);
 
        if (!dptr_bmap)
                exit_server("out of memory in init_dptrs");
-
-       dptrs_init = True;
 }
 
 /****************************************************************************
@@ -415,8 +407,6 @@ NTSTATUS dptr_create(connection_struct *conn, const char *path, bool old_handle,
                return map_nt_error_from_unix(errno);
        }
 
-       string_set(&conn->dirpath,path);
-
        if (dirhandles_open >= MAX_OPEN_DIRECTORIES) {
                dptr_idleoldest();
        }
@@ -560,11 +550,12 @@ int dptr_dnum(struct dptr_struct *dptr)
  Return the next visible file name, skipping veto'd and invisible files.
 ****************************************************************************/
 
-static const char *dptr_normal_ReadDirName(struct dptr_struct *dptr, long *poffset, SMB_STRUCT_STAT *pst)
+static const char *dptr_normal_ReadDirName(struct dptr_struct *dptr,
+                                          long *poffset, SMB_STRUCT_STAT *pst)
 {
        /* Normal search for the next file. */
        const char *name;
-       while ((name = ReadDirName(dptr->dir_hnd, poffset)) != NULL) {
+       while ((name = ReadDirName(dptr->dir_hnd, poffset, pst)) != NULL) {
                if (is_visible_file(dptr->conn, dptr->path, name, pst, True)) {
                        return name;
                }
@@ -576,90 +567,123 @@ static const char *dptr_normal_ReadDirName(struct dptr_struct *dptr, long *poffs
  Return the next visible file name, skipping veto'd and invisible files.
 ****************************************************************************/
 
-const char *dptr_ReadDirName(TALLOC_CTX *ctx,
+char *dptr_ReadDirName(TALLOC_CTX *ctx,
                        struct dptr_struct *dptr,
                        long *poffset,
                        SMB_STRUCT_STAT *pst)
 {
+       struct smb_filename *smb_fname_base = NULL;
+       char *name = NULL;
+       char *pathreal = NULL;
+       char *found_name = NULL;
+       int ret;
+       const char *name_temp = NULL;
+       NTSTATUS status;
+
        SET_STAT_INVALID(*pst);
 
-       if (dptr->has_wild) {
-               return dptr_normal_ReadDirName(dptr, poffset, pst);
+       if (dptr->has_wild || dptr->did_stat) {
+               name_temp = dptr_normal_ReadDirName(dptr, poffset, pst);
+               name = talloc_strdup(ctx, name_temp);
+               return name;
        }
 
-       /* If poffset is -1 then we know we returned this name before and we have
-          no wildcards. We're at the end of the directory. */
+       /* If poffset is -1 then we know we returned this name before and we
+        * have no wildcards. We're at the end of the directory. */
        if (*poffset == END_OF_DIRECTORY_OFFSET) {
                return NULL;
        }
 
-       if (!dptr->did_stat) {
-               char *pathreal = NULL;
+       /* We know the stored wcard contains no wildcard characters.
+        * See if we can match with a stat call. If we can't, then set
+        * did_stat to true to ensure we only do this once and keep
+        * searching. */
+
+       dptr->did_stat = true;
+
+       /* First check if it should be visible. */
+       if (!is_visible_file(dptr->conn, dptr->path, dptr->wcard,
+           pst, true))
+       {
+               /* This only returns false if the file was found, but
+                  is explicitly not visible. Set us to end of
+                  directory, but return NULL as we know we can't ever
+                  find it. */
+               goto ret;
+       }
 
-               /* We know the stored wcard contains no wildcard characters. See if we can match
-                  with a stat call. If we can't, then set did_stat to true to
-                  ensure we only do this once and keep searching. */
+       if (VALID_STAT(*pst)) {
+               name = talloc_strdup(ctx, dptr->wcard);
+               goto ret;
+       }
 
-               dptr->did_stat = True;
+       pathreal = talloc_asprintf(ctx,
+                               "%s/%s",
+                               dptr->path,
+                               dptr->wcard);
+       if (!pathreal)
+               return NULL;
 
-               /* First check if it should be visible. */
-               if (!is_visible_file(dptr->conn, dptr->path, dptr->wcard, pst, True)) {
-                       /* This only returns False if the file was found, but
-                          is explicitly not visible. Set us to end of directory,
-                          but return NULL as we know we can't ever find it. */
-                       dptr->dir_hnd->offset = *poffset = END_OF_DIRECTORY_OFFSET;
-                       return NULL;
-               }
+       /* Create an smb_filename with stream_name == NULL. */
+       status = create_synthetic_smb_fname(ctx, pathreal, NULL, NULL,
+                                           &smb_fname_base);
+       if (!NT_STATUS_IS_OK(status)) {
+               return NULL;
+       }
 
-               if (VALID_STAT(*pst)) {
-                       /* We need to set the underlying dir_hnd offset to -1 also as
-                          this function is usually called with the output from TellDir. */
-                       dptr->dir_hnd->offset = *poffset = END_OF_DIRECTORY_OFFSET;
-                       return dptr->wcard;
+       if (SMB_VFS_STAT(dptr->conn, smb_fname_base) == 0) {
+               *pst = smb_fname_base->st;
+               TALLOC_FREE(smb_fname_base);
+               name = talloc_strdup(ctx, dptr->wcard);
+               goto clean;
+       } else {
+               TALLOC_FREE(smb_fname_base);
+               /* If we get any other error than ENOENT or ENOTDIR
+                  then the file exists we just can't stat it. */
+               if (errno != ENOENT && errno != ENOTDIR) {
+                       name = talloc_strdup(ctx, dptr->wcard);
+                       goto clean;
                }
+       }
 
-               pathreal = talloc_asprintf(ctx,
-                                       "%s/%s",
-                                       dptr->path,
-                                       dptr->wcard);
-               if (!pathreal) {
-                       return NULL;
-               }
+       /* Stat failed. We know this is authoratiative if we are
+        * providing case sensitive semantics or the underlying
+        * filesystem is case sensitive.
+        */
+       if (dptr->conn->case_sensitive ||
+           !(dptr->conn->fs_capabilities & FILE_CASE_SENSITIVE_SEARCH))
+       {
+               goto clean;
+       }
 
-               if (SMB_VFS_STAT(dptr->conn,pathreal,pst) == 0) {
-                       /* We need to set the underlying dir_hnd offset to -1 also as
-                          this function is usually called with the output from TellDir. */
-                       dptr->dir_hnd->offset = *poffset = END_OF_DIRECTORY_OFFSET;
-                       TALLOC_FREE(pathreal);
-                       return dptr->wcard;
-               } else {
-                       /* If we get any other error than ENOENT or ENOTDIR
-                          then the file exists we just can't stat it. */
-                       if (errno != ENOENT && errno != ENOTDIR) {
-                               /* We need to set the underlying dir_hdn offset to -1 also as
-                                  this function is usually called with the output from TellDir. */
-                               dptr->dir_hnd->offset = *poffset = END_OF_DIRECTORY_OFFSET;
-                               TALLOC_FREE(pathreal);
-                               return dptr->wcard;
-                       }
-               }
+       /*
+        * Try case-insensitive stat if the fs has the ability. This avoids
+        * scanning the whole directory.
+        */
+       ret = SMB_VFS_GET_REAL_FILENAME(dptr->conn, dptr->path, dptr->wcard,
+                                       ctx, &found_name);
+       if (ret == 0) {
+               name = found_name;
+               goto clean;
+       } else if (errno == ENOENT) {
+               /* The case-insensitive lookup was authoritative. */
+               goto clean;
+       }
 
-               TALLOC_FREE(pathreal);
+       TALLOC_FREE(pathreal);
 
-               /* Stat failed. We know this is authoratiative if we are
-                * providing case sensitive semantics or the underlying
-                * filesystem is case sensitive.
-                */
+       name_temp = dptr_normal_ReadDirName(dptr, poffset, pst);
+       name = talloc_strdup(ctx, name_temp);
+       return name;
 
-               if (dptr->conn->case_sensitive ||
-                   !(dptr->conn->fs_capabilities & FILE_CASE_SENSITIVE_SEARCH)) {
-                       /* We need to set the underlying dir_hnd offset to -1 also as
-                          this function is usually called with the output from TellDir. */
-                       dptr->dir_hnd->offset = *poffset = END_OF_DIRECTORY_OFFSET;
-                       return NULL;
-               }
-       }
-       return dptr_normal_ReadDirName(dptr, poffset, pst);
+clean:
+       TALLOC_FREE(pathreal);
+ret:
+       /* We need to set the underlying dir_hnd offset to -1
+        * also as this function is usually called with the
+        * output from TellDir. */
+       dptr->dir_hnd->offset = *poffset = END_OF_DIRECTORY_OFFSET;
+       return name;
 }
 
 /****************************************************************************
@@ -688,6 +712,14 @@ void dptr_DirCacheAdd(struct dptr_struct *dptr, const char *name, long offset)
        DirCacheAdd(dptr->dir_hnd, name, offset);
 }
 
+/****************************************************************************
+ Initialize variables & state data at the beginning of all search SMB requests.
+****************************************************************************/
+void dptr_init_search_op(struct dptr_struct *dptr)
+{
+       SMB_VFS_INIT_SEARCH_OP(dptr->conn, dptr->dir_hnd->dir);
+}
+
 /****************************************************************************
  Fill the 5 byte server reserved dptr field.
 ****************************************************************************/
@@ -791,118 +823,251 @@ static bool mangle_mask_match(connection_struct *conn,
        return mask_match_search(mname,mask,False);
 }
 
-/****************************************************************************
- Get an 8.3 directory entry.
-****************************************************************************/
-
-bool get_dir_entry(TALLOC_CTX *ctx,
-               connection_struct *conn,
-               const char *mask,
-               uint32 dirtype,
-               char **pp_fname_out,
-               SMB_OFF_T *size,
-               uint32 *mode,
-               time_t *date,
-               bool check_descend)
+bool smbd_dirptr_get_entry(TALLOC_CTX *ctx,
+                          struct dptr_struct *dirptr,
+                          const char *mask,
+                          uint32_t dirtype,
+                          bool dont_descend,
+                          bool ask_sharemode,
+                          bool (*match_fn)(TALLOC_CTX *ctx,
+                                           void *private_data,
+                                           const char *dname,
+                                           const char *mask,
+                                           char **_fname),
+                          bool (*mode_fn)(TALLOC_CTX *ctx,
+                                          void *private_data,
+                                          struct smb_filename *smb_fname,
+                                          uint32_t *_mode),
+                          void *private_data,
+                          char **_fname,
+                          struct smb_filename **_smb_fname,
+                          uint32_t *_mode,
+                          long *_prev_offset)
 {
-       const char *dname = NULL;
-       bool found = False;
-       SMB_STRUCT_STAT sbuf;
-       char *pathreal = NULL;
-       const char *filename = NULL;
+       connection_struct *conn = dirptr->conn;
        bool needslash;
 
-       *pp_fname_out = NULL;
+       *_smb_fname = NULL;
+       *_mode = 0;
 
-       needslash = ( conn->dirpath[strlen(conn->dirpath) -1] != '/');
+       needslash = ( dirptr->path[strlen(dirptr->path) -1] != '/');
 
-       if (!conn->dirptr) {
-               return(False);
-       }
+       while (true) {
+               long cur_offset;
+               long prev_offset;
+               SMB_STRUCT_STAT sbuf;
+               char *dname = NULL;
+               bool isdots;
+               char *fname = NULL;
+               char *pathreal = NULL;
+               struct smb_filename *smb_fname = NULL;
+               uint32_t mode = 0;
+               bool ok;
+               NTSTATUS status;
 
-       while (!found) {
-               long curoff = dptr_TellDir(conn->dirptr);
-               dname = dptr_ReadDirName(ctx, conn->dirptr, &curoff, &sbuf);
+               cur_offset = dptr_TellDir(dirptr);
+               prev_offset = cur_offset;
+               dname = dptr_ReadDirName(ctx, dirptr, &cur_offset, &sbuf);
 
-               DEBUG(6,("readdir on dirptr 0x%lx now at offset %ld\n",
-                       (long)conn->dirptr,TellDir(conn->dirptr->dir_hnd)));
+               DEBUG(6,("smbd_dirptr_get_entry: dirptr 0x%lx now at offset %ld\n",
+                       (long)dirptr, cur_offset));
 
                if (dname == NULL) {
-                       return(False);
+                       return false;
                }
 
-               filename = dname;
-
-               /* notice the special *.* handling. This appears to be the only difference
-                       between the wildcard handling in this routine and in the trans2 routines.
-                       see masktest for a demo
-               */
-               if ((strcmp(mask,"*.*") == 0) ||
-                   mask_match_search(filename,mask,False) ||
-                   mangle_mask_match(conn,filename,mask)) {
-                       char mname[13];
-
-                       if (!mangle_is_8_3(filename, False, conn->params)) {
-                               if (!name_to_8_3(filename,mname,False,
-                                          conn->params)) {
-                                       continue;
-                               }
-                               filename = mname;
-                       }
+               isdots = (ISDOT(dname) || ISDOTDOT(dname));
+               if (dont_descend && !isdots) {
+                       TALLOC_FREE(dname);
+                       continue;
+               }
 
-                       if (needslash) {
-                               pathreal = talloc_asprintf(ctx,
-                                               "%s/%s",
-                                               conn->dirpath,
-                                               dname);
-                       } else {
-                               pathreal = talloc_asprintf(ctx,
-                                               "%s%s",
-                                               conn->dirpath,
-                                               dname);
-                       }
-                       if (!pathreal) {
-                               return False;
-                       }
+               /*
+                * fname may get mangled, dname is never mangled.
+                * Whenever we're accessing the filesystem we use
+                * pathreal which is composed from dname.
+                */
 
-                       if (!VALID_STAT(sbuf) && (SMB_VFS_STAT(conn, pathreal, &sbuf)) != 0) {
-                               DEBUG(5,("Couldn't stat 1 [%s]. Error = %s\n",
-                                       pathreal, strerror(errno) ));
-                               TALLOC_FREE(pathreal);
-                               continue;
-                       }
+               ok = match_fn(ctx, private_data, dname, mask, &fname);
+               if (!ok) {
+                       TALLOC_FREE(dname);
+                       continue;
+               }
 
-                       *mode = dos_mode(conn,pathreal,&sbuf);
+               pathreal = talloc_asprintf(ctx, "%s%s%s",
+                                          dirptr->path,
+                                          needslash?"/":"",
+                                          dname);
+               if (!pathreal) {
+                       TALLOC_FREE(dname);
+                       TALLOC_FREE(fname);
+                       return false;
+               }
 
-                       if (!dir_check_ftype(conn,*mode,dirtype)) {
-                               DEBUG(5,("[%s] attribs 0x%x didn't match 0x%x\n",filename,(unsigned int)*mode,(unsigned int)dirtype));
-                               TALLOC_FREE(pathreal);
-                               continue;
+               /* Create smb_fname with NULL stream_name. */
+               status = create_synthetic_smb_fname(ctx, pathreal,
+                                                   NULL, &sbuf,
+                                                   &smb_fname);
+               TALLOC_FREE(pathreal);
+               if (!NT_STATUS_IS_OK(status)) {
+                       TALLOC_FREE(dname);
+                       TALLOC_FREE(fname);
+                       return false;
+               }
+
+               ok = mode_fn(ctx, private_data, smb_fname, &mode);
+               if (!ok) {
+                       TALLOC_FREE(dname);
+                       TALLOC_FREE(fname);
+                       TALLOC_FREE(smb_fname);
+                       continue;
+               }
+
+               if (!dir_check_ftype(conn, mode, dirtype)) {
+                       DEBUG(5,("[%s] attribs 0x%x didn't match 0x%x\n",
+                               fname, (unsigned int)mode, (unsigned int)dirtype));
+                       TALLOC_FREE(dname);
+                       TALLOC_FREE(fname);
+                       TALLOC_FREE(smb_fname);
+                       continue;
+               }
+
+               if (ask_sharemode) {
+                       struct timespec write_time_ts;
+                       struct file_id fileid;
+
+                       fileid = vfs_file_id_from_sbuf(conn,
+                                                      &smb_fname->st);
+                       get_file_infos(fileid, NULL, &write_time_ts);
+                       if (!null_timespec(write_time_ts)) {
+                               update_stat_ex_mtime(&smb_fname->st,
+                                                    write_time_ts);
                        }
+               }
+
+               DEBUG(3,("smbd_dirptr_get_entry mask=[%s] found %s "
+                       "fname=%s (%s)\n",
+                       mask, smb_fname_str_dbg(smb_fname),
+                       dname, fname));
+
+               DirCacheAdd(dirptr->dir_hnd, dname, cur_offset);
 
-                       *size = sbuf.st_size;
-                       *date = sbuf.st_mtime;
+               TALLOC_FREE(dname);
 
-                       DEBUG(3,("get_dir_entry mask=[%s] found %s "
-                               "fname=%s (%s)\n",
-                               mask,
-                               pathreal,
-                               dname,
-                               filename));
+               *_fname = fname;
+               *_smb_fname = smb_fname;
+               *_mode = mode;
+               *_prev_offset = prev_offset;
+
+               return true;
+       }
 
-                       found = True;
+       return false;
+}
+
+/****************************************************************************
+ Get an 8.3 directory entry.
+****************************************************************************/
 
-                       *pp_fname_out = talloc_strdup(ctx, filename);
-                       if (!*pp_fname_out) {
-                               return False;
+static bool smbd_dirptr_8_3_match_fn(TALLOC_CTX *ctx,
+                                    void *private_data,
+                                    const char *dname,
+                                    const char *mask,
+                                    char **_fname)
+{
+       connection_struct *conn = (connection_struct *)private_data;
+
+       if ((strcmp(mask,"*.*") == 0) ||
+           mask_match_search(dname, mask, false) ||
+           mangle_mask_match(conn, dname, mask)) {
+               char mname[13];
+               const char *fname;
+
+               if (!mangle_is_8_3(dname, false, conn->params)) {
+                       bool ok = name_to_8_3(dname, mname, false,
+                                             conn->params);
+                       if (!ok) {
+                               return false;
                        }
+                       fname = mname;
+               } else {
+                       fname = dname;
+               }
+
+               *_fname = talloc_strdup(ctx, fname);
+               if (*_fname == NULL) {
+                       return false;
+               }
+
+               return true;
+       }
+
+       return false;
+}
 
-                       DirCacheAdd(conn->dirptr->dir_hnd, dname, curoff);
-                       TALLOC_FREE(pathreal);
+static bool smbd_dirptr_8_3_mode_fn(TALLOC_CTX *ctx,
+                                   void *private_data,
+                                   struct smb_filename *smb_fname,
+                                   uint32_t *_mode)
+{
+       connection_struct *conn = (connection_struct *)private_data;
+
+       if (!VALID_STAT(smb_fname->st)) {
+               if ((SMB_VFS_STAT(conn, smb_fname)) != 0) {
+                       DEBUG(5,("smbd_dirptr_8_3_mode_fn: "
+                                "Couldn't stat [%s]. Error "
+                                "= %s\n",
+                                smb_fname_str_dbg(smb_fname),
+                                strerror(errno)));
+                       return false;
                }
        }
 
-       return(found);
+       *_mode = dos_mode(conn, smb_fname);
+       return true;
+}
+
+bool get_dir_entry(TALLOC_CTX *ctx,
+               struct dptr_struct *dirptr,
+               const char *mask,
+               uint32_t dirtype,
+               char **_fname,
+               SMB_OFF_T *_size,
+               uint32_t *_mode,
+               struct timespec *_date,
+               bool check_descend,
+               bool ask_sharemode)
+{
+       connection_struct *conn = dirptr->conn;
+       char *fname = NULL;
+       struct smb_filename *smb_fname = NULL;
+       uint32_t mode = 0;
+       long prev_offset;
+       bool ok;
+
+       ok = smbd_dirptr_get_entry(ctx,
+                                  dirptr,
+                                  mask,
+                                  dirtype,
+                                  check_descend,
+                                  ask_sharemode,
+                                  smbd_dirptr_8_3_match_fn,
+                                  smbd_dirptr_8_3_mode_fn,
+                                  conn,
+                                  &fname,
+                                  &smb_fname,
+                                  &mode,
+                                  &prev_offset);
+       if (!ok) {
+               return false;
+       }
+
+       *_fname = talloc_move(ctx, &fname);
+       *_size = smb_fname->st.st_ex_size;
+       *_mode = mode;
+       *_date = smb_fname->st.st_ex_mtime;
+       TALLOC_FREE(smb_fname);
+       return true;
 }
 
 /*******************************************************************
@@ -911,13 +1076,9 @@ bool get_dir_entry(TALLOC_CTX *ctx,
  use it for anything security sensitive.
 ********************************************************************/
 
-static bool user_can_read_file(connection_struct *conn, char *name, SMB_STRUCT_STAT *pst)
+static bool user_can_read_file(connection_struct *conn,
+                              struct smb_filename *smb_fname)
 {
-       SEC_DESC *psd = NULL;
-       files_struct *fsp;
-       NTSTATUS status;
-       uint32 access_granted;
-
        /*
         * If user is a member of the Admin group
         * we never hide files from them.
@@ -927,38 +1088,7 @@ static bool user_can_read_file(connection_struct *conn, char *name, SMB_STRUCT_S
                return True;
        }
 
-       SMB_ASSERT(VALID_STAT(*pst));
-
-       /* Pseudo-open the file (note - no fd's created). */
-
-       if(S_ISDIR(pst->st_mode)) {
-                status = open_directory(conn, NULL, name, pst,
-                       READ_CONTROL_ACCESS,
-                       FILE_SHARE_READ|FILE_SHARE_WRITE,
-                       FILE_OPEN,
-                       0, /* no create options. */
-                       FILE_ATTRIBUTE_DIRECTORY,
-                       NULL, &fsp);
-       } else {
-               status = open_file_stat(conn, NULL, name, pst, &fsp);
-       }
-
-       if (!NT_STATUS_IS_OK(status)) {
-               return False;
-       }
-
-       /* Get NT ACL -allocated in main loop talloc context. No free needed here. */
-       status = SMB_VFS_FGET_NT_ACL(fsp,
-                       (OWNER_SECURITY_INFORMATION|GROUP_SECURITY_INFORMATION|DACL_SECURITY_INFORMATION), &psd);
-       close_file(fsp, NORMAL_CLOSE);
-
-       /* No access if SD get failed. */
-       if (!NT_STATUS_IS_OK(status)) {
-               return False;
-       }
-
-       return se_access_check(psd, current_user.nt_user_token, FILE_READ_DATA,
-                                 &access_granted, &status);
+       return can_access_file_acl(conn, smb_fname, FILE_READ_DATA);
 }
 
 /*******************************************************************
@@ -968,14 +1098,9 @@ static bool user_can_read_file(connection_struct *conn, char *name, SMB_STRUCT_S
  use it for anything security sensitive.
 ********************************************************************/
 
-static bool user_can_write_file(connection_struct *conn, char *name, SMB_STRUCT_STAT *pst)
+static bool user_can_write_file(connection_struct *conn,
+                               const struct smb_filename *smb_fname)
 {
-       SEC_DESC *psd = NULL;
-       files_struct *fsp;
-       int info;
-       NTSTATUS status;
-       uint32 access_granted;
-
        /*
         * If user is a member of the Admin group
         * we never hide files from them.
@@ -985,46 +1110,23 @@ static bool user_can_write_file(connection_struct *conn, char *name, SMB_STRUCT_
                return True;
        }
 
-       SMB_ASSERT(VALID_STAT(*pst));
+       SMB_ASSERT(VALID_STAT(smb_fname->st));
 
        /* Pseudo-open the file */
 
-       if(S_ISDIR(pst->st_mode)) {
+       if(S_ISDIR(smb_fname->st.st_ex_mode)) {
                return True;
-       } else {
-               status = open_file_ntcreate(conn, NULL, name, pst,
-                       FILE_WRITE_ATTRIBUTES,
-                       FILE_SHARE_READ|FILE_SHARE_WRITE,
-                       FILE_OPEN,
-                       0,
-                       FILE_ATTRIBUTE_NORMAL,
-                       INTERNAL_OPEN_ONLY,
-                       &info, &fsp);
-       }
-
-       if (!NT_STATUS_IS_OK(status)) {
-               return False;
        }
 
-       /* Get NT ACL -allocated in main loop talloc context. No free needed here. */
-       status = SMB_VFS_FGET_NT_ACL(fsp,
-                       (OWNER_SECURITY_INFORMATION|GROUP_SECURITY_INFORMATION|DACL_SECURITY_INFORMATION), &psd);
-       close_file(fsp, NORMAL_CLOSE);
-
-       /* No access if SD get failed. */
-       if (!NT_STATUS_IS_OK(status)) {
-               return False;
-       }
-
-       return se_access_check(psd, current_user.nt_user_token, FILE_WRITE_DATA,
-                                 &access_granted, &status);
+       return can_write_to_file(conn, smb_fname);
 }
 
 /*******************************************************************
   Is a file a "special" type ?
 ********************************************************************/
 
-static bool file_is_special(connection_struct *conn, char *name, SMB_STRUCT_STAT *pst)
+static bool file_is_special(connection_struct *conn,
+                           const struct smb_filename *smb_fname)
 {
        /*
         * If user is a member of the Admin group
@@ -1034,27 +1136,31 @@ static bool file_is_special(connection_struct *conn, char *name, SMB_STRUCT_STAT
        if (conn->admin_user)
                return False;
 
-       SMB_ASSERT(VALID_STAT(*pst));
+       SMB_ASSERT(VALID_STAT(smb_fname->st));
 
-       if (S_ISREG(pst->st_mode) || S_ISDIR(pst->st_mode) || S_ISLNK(pst->st_mode))
+       if (S_ISREG(smb_fname->st.st_ex_mode) ||
+           S_ISDIR(smb_fname->st.st_ex_mode) ||
+           S_ISLNK(smb_fname->st.st_ex_mode))
                return False;
 
        return True;
 }
 
 /*******************************************************************
- Should the file be seen by the client ? NOTE: A successful return
- is no guarantee of the file's existence ... you also have to check
- whether pst is valid.
+ Should the file be seen by the client?
+ NOTE: A successful return is no guarantee of the file's existence.
 ********************************************************************/
 
-bool is_visible_file(connection_struct *conn, const char *dir_path, const char *name, SMB_STRUCT_STAT *pst, bool use_veto)
+bool is_visible_file(connection_struct *conn, const char *dir_path,
+                    const char *name, SMB_STRUCT_STAT *pst, bool use_veto)
 {
        bool hide_unreadable = lp_hideunreadable(SNUM(conn));
        bool hide_unwriteable = lp_hideunwriteable_files(SNUM(conn));
        bool hide_special = lp_hide_special_files(SNUM(conn));
-
-       SET_STAT_INVALID(*pst);
+       char *entry = NULL;
+       struct smb_filename *smb_fname_base = NULL;
+       NTSTATUS status;
+       bool ret = false;
 
        if ((strcmp(".",name) == 0) || (strcmp("..",name) == 0)) {
                return True; /* . and .. are always visible. */
@@ -1067,50 +1173,71 @@ bool is_visible_file(connection_struct *conn, const char *dir_path, const char *
        }
 
        if (hide_unreadable || hide_unwriteable || hide_special) {
-               char *entry = NULL;
-
-               if (asprintf(&entry, "%s/%s", dir_path, name) == -1) {
-                       return False;
+               entry = talloc_asprintf(talloc_tos(), "%s/%s", dir_path, name);
+               if (!entry) {
+                       ret = false;
+                       goto out;
                }
 
                /* If it's a dfs symlink, ignore _hide xxxx_ options */
                if (lp_host_msdfs() &&
                                lp_msdfs_root(SNUM(conn)) &&
                                is_msdfs_link(conn, entry, NULL)) {
-                       SAFE_FREE(entry);
-                       return True;
+                       ret = true;
+                       goto out;
+               }
+
+               /* Create an smb_filename with stream_name == NULL. */
+               status = create_synthetic_smb_fname(talloc_tos(), entry, NULL,
+                                                   pst, &smb_fname_base);
+               if (!NT_STATUS_IS_OK(status)) {
+                       ret = false;
+                       goto out;
                }
 
                /* If the file name does not exist, there's no point checking
                 * the configuration options. We succeed, on the basis that the
                 * checks *might* have passed if the file was present.
                 */
-               if (SMB_VFS_STAT(conn, entry, pst) != 0) {
-                       SAFE_FREE(entry);
-                       return True;
+               if (!VALID_STAT(*pst)) {
+                       if (SMB_VFS_STAT(conn, smb_fname_base) != 0) {
+                               ret = true;
+                               goto out;
+                       } else {
+                               *pst = smb_fname_base->st;
+                       }
                }
 
                /* Honour _hide unreadable_ option */
-               if (hide_unreadable && !user_can_read_file(conn, entry, pst)) {
-                       DEBUG(10,("is_visible_file: file %s is unreadable.\n", entry ));
-                       SAFE_FREE(entry);
-                       return False;
+               if (hide_unreadable &&
+                   !user_can_read_file(conn, smb_fname_base)) {
+                       DEBUG(10,("is_visible_file: file %s is unreadable.\n",
+                                entry ));
+                       ret = false;
+                       goto out;
                }
                /* Honour _hide unwriteable_ option */
-               if (hide_unwriteable && !user_can_write_file(conn, entry, pst)) {
-                       DEBUG(10,("is_visible_file: file %s is unwritable.\n", entry ));
-                       SAFE_FREE(entry);
-                       return False;
+               if (hide_unwriteable && !user_can_write_file(conn,
+                                                            smb_fname_base)) {
+                       DEBUG(10,("is_visible_file: file %s is unwritable.\n",
+                                entry ));
+                       ret = false;
+                       goto out;
                }
                /* Honour _hide_special_ option */
-               if (hide_special && file_is_special(conn, entry, pst)) {
-                       DEBUG(10,("is_visible_file: file %s is special.\n", entry ));
-                       SAFE_FREE(entry);
-                       return False;
+               if (hide_special && file_is_special(conn, smb_fname_base)) {
+                       DEBUG(10,("is_visible_file: file %s is special.\n",
+                                entry ));
+                       ret = false;
+                       goto out;
                }
-               SAFE_FREE(entry);
        }
-       return True;
+
+       ret = true;
+ out:
+       TALLOC_FREE(smb_fname_base);
+       TALLOC_FREE(entry);
+       return ret;
 }
 
 static int smb_Dir_destructor(struct smb_Dir *dirp)
@@ -1140,6 +1267,7 @@ struct smb_Dir *OpenDir(TALLOC_CTX *mem_ctx, connection_struct *conn,
 
        dirp->dir_path = talloc_strdup(dirp, name);
        if (!dirp->dir_path) {
+               errno = ENOMEM;
                goto fail;
        }
 
@@ -1161,17 +1289,21 @@ struct smb_Dir *OpenDir(TALLOC_CTX *mem_ctx, connection_struct *conn,
 }
 
 /*******************************************************************
- Read from a directory. Also return current offset.
+ Read from a directory.
+ Return directory entry, current offset, and optional stat information.
  Don't check for veto or invisible files.
 ********************************************************************/
 
-const char *ReadDirName(struct smb_Dir *dirp, long *poffset)
+const char *ReadDirName(struct smb_Dir *dirp, long *poffset,
+                       SMB_STRUCT_STAT *sbuf)
 {
        const char *n;
        connection_struct *conn = dirp->conn;
 
        /* Cheat to allow . and .. to be the first entries returned. */
-       if (((*poffset == START_OF_DIRECTORY_OFFSET) || (*poffset == DOT_DOT_DIRECTORY_OFFSET)) && (dirp->file_number < 2)) {
+       if (((*poffset == START_OF_DIRECTORY_OFFSET) ||
+            (*poffset == DOT_DOT_DIRECTORY_OFFSET)) && (dirp->file_number < 2))
+       {
                if (dirp->file_number == 0) {
                        n = ".";
                        *poffset = dirp->offset = START_OF_DIRECTORY_OFFSET;
@@ -1189,7 +1321,7 @@ const char *ReadDirName(struct smb_Dir *dirp, long *poffset)
                SeekDir(dirp, *poffset);
        }
 
-       while ((n = vfs_readdirname(conn, dirp->dir))) {
+       while ((n = vfs_readdirname(conn, dirp->dir, sbuf))) {
                /* Ignore . and .. - we've already returned them. */
                if (*n == '.') {
                        if ((n[1] == '\0') || (n[1] == '.' && n[2] == '\0')) {
@@ -1322,7 +1454,7 @@ bool SearchDir(struct smb_Dir *dirp, const char *name, long *poffset)
        SMB_VFS_REWINDDIR(conn, dirp->dir);
        dirp->file_number = 0;
        *poffset = START_OF_DIRECTORY_OFFSET;
-       while ((entry = ReadDirName(dirp, poffset))) {
+       while ((entry = ReadDirName(dirp, poffset, NULL))) {
                if (conn->case_sensitive ? (strcmp(entry, name) == 0) : strequal(entry, name)) {
                        return True;
                }
@@ -1340,6 +1472,7 @@ NTSTATUS can_delete_directory(struct connection_struct *conn,
        NTSTATUS status = NT_STATUS_OK;
        long dirpos = 0;
        const char *dname;
+       SMB_STRUCT_STAT st;
        struct smb_Dir *dir_hnd = OpenDir(talloc_tos(), conn, dirname,
                                          NULL, 0);
 
@@ -1347,9 +1480,7 @@ NTSTATUS can_delete_directory(struct connection_struct *conn,
                return map_nt_error_from_unix(errno);
        }
 
-       while ((dname = ReadDirName(dir_hnd,&dirpos))) {
-               SMB_STRUCT_STAT st;
-
+       while ((dname = ReadDirName(dir_hnd, &dirpos, &st))) {
                /* Quick check for "." and ".." */
                if (dname[0] == '.') {
                        if (!dname[1] || (dname[1] == '.' && !dname[2])) {
@@ -1361,7 +1492,8 @@ NTSTATUS can_delete_directory(struct connection_struct *conn,
                        continue;
                }
 
-               DEBUG(10,("can_delete_directory: got name %s - can't delete\n", dname ));
+               DEBUG(10,("can_delete_directory: got name %s - can't delete\n",
+                        dname ));
                status = NT_STATUS_DIRECTORY_NOT_EMPTY;
                break;
        }