s3 file_access: Convert some more functions over to use smb_filneame
[ira/wip.git] / source3 / smbd / dosmode.c
index d3813f9b41b0af24c85733dd7c2f1713f666b114..26d3320a4c5f4dcc2d62f15be9d370f4ddd66ad9 100644 (file)
@@ -3,17 +3,17 @@
    dos mode handling functions
    Copyright (C) Andrew Tridgell 1992-1998
    Copyright (C) James Peach 2006
-   
+
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
-   
+
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU General Public License for more details.
-   
+
    You should have received a copy of the GNU General Public License
    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 static int set_sparse_flag(const SMB_STRUCT_STAT * const sbuf)
 {
 #if defined (HAVE_STAT_ST_BLOCKS) && defined(STAT_ST_BLOCKSIZE)
-       if (sbuf->st_size > sbuf->st_blocks * (SMB_OFF_T)STAT_ST_BLOCKSIZE) {
+       if (sbuf->st_ex_size > sbuf->st_ex_blocks * (SMB_OFF_T)STAT_ST_BLOCKSIZE) {
                return FILE_ATTRIBUTE_SPARSE;
        }
 #endif
        return 0;
 }
 
-/****************************************************************************
- Work out whether this file is offline
-****************************************************************************/
-
-static uint32 set_offline_flag(connection_struct *conn, const char *const path)
+static int set_link_read_only_flag(const SMB_STRUCT_STAT *const sbuf)
 {
-       if (ISDOT(path) || ISDOTDOT(path)) {
-               return 0;
-       }
-
-       if (!lp_dmapi_support(SNUM(conn)) || !dmapi_have_session()) {
-               return 0;
-       }
-
-       return dmapi_file_flags(path);
+#ifdef S_ISLNK
+#if LINKS_READ_ONLY
+       if (S_ISLNK(sbuf->st_mode) && S_ISDIR(sbuf->st_mode))
+               return aRONLY;
+#endif
+#endif
+       return 0;
 }
 
 /****************************************************************************
@@ -87,14 +81,14 @@ mode_t unix_mode(connection_struct *conn, int dosmode, const char *fname,
 
                DEBUG(2, ("unix_mode(%s) inheriting from %s\n", fname,
                          inherit_from_dir));
-               if (SMB_VFS_STAT(conn, inherit_from_dir, &sbuf) != 0) {
+               if (vfs_stat_smb_fname(conn, inherit_from_dir, &sbuf) != 0) {
                        DEBUG(4,("unix_mode(%s) failed, [dir %s]: %s\n", fname,
                                 inherit_from_dir, strerror(errno)));
                        return(0);      /* *** shouldn't happen! *** */
                }
 
                /* Save for later - but explicitly remove setuid bit for safety. */
-               dir_mode = sbuf.st_mode & ~S_ISUID;
+               dir_mode = sbuf.st_ex_mode & ~S_ISUID;
                DEBUG(2,("unix_mode(%s) inherit mode %o\n",fname,(int)dir_mode));
                /* Clear "result" */
                result = 0;
@@ -123,7 +117,7 @@ mode_t unix_mode(connection_struct *conn, int dosmode, const char *fname,
 
                if (lp_map_system(SNUM(conn)) && IS_DOS_SYSTEM(dosmode))
                        result |= S_IXGRP;
+
                if (lp_map_hidden(SNUM(conn)) && IS_DOS_HIDDEN(dosmode))
                        result |= S_IXOTH;  
 
@@ -146,43 +140,45 @@ mode_t unix_mode(connection_struct *conn, int dosmode, const char *fname,
  Change a unix mode to a dos mode.
 ****************************************************************************/
 
-static uint32 dos_mode_from_sbuf(connection_struct *conn, const char *path, SMB_STRUCT_STAT *sbuf)
+static uint32 dos_mode_from_sbuf(connection_struct *conn, const char *path, const SMB_STRUCT_STAT *sbuf)
 {
        int result = 0;
        enum mapreadonly_options ro_opts = (enum mapreadonly_options)lp_map_readonly(SNUM(conn));
 
        if (ro_opts == MAP_READONLY_YES) {
                /* Original Samba method - map inverse of user "w" bit. */
-               if ((sbuf->st_mode & S_IWUSR) == 0) {
+               if ((sbuf->st_ex_mode & S_IWUSR) == 0) {
                        result |= aRONLY;
                }
        } else if (ro_opts == MAP_READONLY_PERMISSIONS) {
-               /* Check actual permissions for read-only. */
-               if (!can_write_to_file(conn, path, sbuf)) {
-                       result |= aRONLY;
+               struct smb_filename *smb_fname = NULL;
+               NTSTATUS status;
+
+               status = create_synthetic_smb_fname_split(talloc_tos(), path,
+                                                         sbuf, &smb_fname);
+               if (NT_STATUS_IS_OK(status)) {
+                       /* Check actual permissions for read-only. */
+                       if (!can_write_to_file(conn, smb_fname)) {
+                               result |= aRONLY;
+                       }
                }
+               TALLOC_FREE(smb_fname);
        } /* Else never set the readonly bit. */
 
-       if (MAP_ARCHIVE(conn) && ((sbuf->st_mode & S_IXUSR) != 0))
+       if (MAP_ARCHIVE(conn) && ((sbuf->st_ex_mode & S_IXUSR) != 0))
                result |= aARCH;
 
-       if (MAP_SYSTEM(conn) && ((sbuf->st_mode & S_IXGRP) != 0))
+       if (MAP_SYSTEM(conn) && ((sbuf->st_ex_mode & S_IXGRP) != 0))
                result |= aSYSTEM;
-       
-       if (MAP_HIDDEN(conn) && ((sbuf->st_mode & S_IXOTH) != 0))
+
+       if (MAP_HIDDEN(conn) && ((sbuf->st_ex_mode & S_IXOTH) != 0))
                result |= aHIDDEN;   
-  
-       if (S_ISDIR(sbuf->st_mode))
+
+       if (S_ISDIR(sbuf->st_ex_mode))
                result = aDIR | (result & aRONLY);
 
        result |= set_sparse_flag(sbuf);
-#ifdef S_ISLNK
-#if LINKS_READ_ONLY
-       if (S_ISLNK(sbuf->st_mode) && S_ISDIR(sbuf->st_mode))
-               result |= aRONLY;
-#endif
-#endif
+       result |= set_link_read_only_flag(sbuf);
 
        DEBUG(8,("dos_mode_from_sbuf returning "));
 
@@ -191,7 +187,7 @@ static uint32 dos_mode_from_sbuf(connection_struct *conn, const char *path, SMB_
        if (result & aSYSTEM) DEBUG(8, ("s"));
        if (result & aDIR   ) DEBUG(8, ("d"));
        if (result & aARCH  ) DEBUG(8, ("a"));
-       
+
        DEBUG(8,("\n"));
        return result;
 }
@@ -200,7 +196,7 @@ static uint32 dos_mode_from_sbuf(connection_struct *conn, const char *path, SMB_
  Get DOS attributes from an EA.
 ****************************************************************************/
 
-static bool get_ea_dos_attribute(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf, uint32 *pattr)
+static bool get_ea_dos_attribute(connection_struct *conn, const char *path, const SMB_STRUCT_STAT *sbuf, uint32 *pattr)
 {
        ssize_t sizeret;
        fstring attrstr;
@@ -215,13 +211,16 @@ static bool get_ea_dos_attribute(connection_struct *conn, const char *path,SMB_S
 
        sizeret = SMB_VFS_GETXATTR(conn, path, SAMBA_XATTR_DOS_ATTRIB, attrstr, sizeof(attrstr));
        if (sizeret == -1) {
-#if defined(ENOTSUP) && defined(ENOATTR)
-               if ((errno != ENOTSUP) && (errno != ENOATTR) && (errno != EACCES) && (errno != EPERM)) {
+               if (errno == ENOSYS
+#if defined(ENOTSUP)
+                       || errno == ENOTSUP) {
+#else
+                               ) {
+#endif
                        DEBUG(1,("get_ea_dos_attributes: Cannot get attribute from EA on file %s: Error = %s\n",
                                path, strerror(errno) ));
                        set_store_dos_attributes(SNUM(conn), False);
                }
-#endif
                return False;
        }
        /* Null terminate string. */
@@ -234,7 +233,7 @@ static bool get_ea_dos_attribute(connection_struct *conn, const char *path,SMB_S
                 return False;
         }
 
-       if (S_ISDIR(sbuf->st_mode)) {
+       if (S_ISDIR(sbuf->st_ex_mode)) {
                dosattr |= aDIR;
        }
        *pattr = (uint32)(dosattr & SAMBA_ATTRIBUTES_MASK);
@@ -246,7 +245,7 @@ static bool get_ea_dos_attribute(connection_struct *conn, const char *path,SMB_S
        if (dosattr & aSYSTEM) DEBUG(8, ("s"));
        if (dosattr & aDIR   ) DEBUG(8, ("d"));
        if (dosattr & aARCH  ) DEBUG(8, ("a"));
-       
+
        DEBUG(8,("\n"));
 
        return True;
@@ -275,6 +274,8 @@ static bool set_ea_dos_attribute(connection_struct *conn, const char *path, SMB_
 #else
                                ) {
 #endif
+                               DEBUG(1,("set_ea_dos_attributes: Cannot set attribute EA on file %s: Error = %s\n",
+                                       path, strerror(errno) ));
                                set_store_dos_attributes(SNUM(conn), False);
                        }
                        return False;
@@ -294,14 +295,15 @@ static bool set_ea_dos_attribute(connection_struct *conn, const char *path, SMB_
                 * are not violating security in doing the setxattr.
                 */
 
-               if (!NT_STATUS_IS_OK(open_file_fchmod(conn,path,sbuf,&fsp)))
+               if (!NT_STATUS_IS_OK(open_file_fchmod(NULL, conn, path, sbuf,
+                                                     &fsp)))
                        return ret;
                become_root();
                if (SMB_VFS_SETXATTR(conn, path, SAMBA_XATTR_DOS_ATTRIB, attrstr, strlen(attrstr), 0) == 0) {
                        ret = True;
                }
                unbecome_root();
-               close_file_fchmod(fsp);
+               close_file_fchmod(NULL, fsp);
                return ret;
        }
        DEBUG(10,("set_ea_dos_attribute: set EA %s on file %s\n", attrstr, path));
@@ -312,13 +314,14 @@ static bool set_ea_dos_attribute(connection_struct *conn, const char *path, SMB_
  Change a unix mode to a dos mode for an ms dfs link.
 ****************************************************************************/
 
-uint32 dos_mode_msdfs(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
+uint32 dos_mode_msdfs(connection_struct *conn, const char *path, const SMB_STRUCT_STAT *psbuf)
 {
+       SMB_STRUCT_STAT sbuf = *psbuf;
        uint32 result = 0;
 
        DEBUG(8,("dos_mode_msdfs: %s\n", path));
 
-       if (!VALID_STAT(*sbuf)) {
+       if (!VALID_STAT(sbuf)) {
                return 0;
        }
 
@@ -331,13 +334,15 @@ uint32 dos_mode_msdfs(connection_struct *conn, const char *path,SMB_STRUCT_STAT
                } else {
                        p = path;
                }
-               
-               if (p[0] == '.' && p[1] != '.' && p[1] != 0) {
+
+               /* Only . and .. are not hidden. */
+               if (p[0] == '.' && !((p[1] == '\0') ||
+                               (p[1] == '.' && p[2] == '\0'))) {
                        result |= aHIDDEN;
                }
        }
-       
-       result |= dos_mode_from_sbuf(conn, path, sbuf);
+
+       result |= dos_mode_from_sbuf(conn, path, &sbuf);
 
        /* Optimization : Only call is_hidden_path if it's not already
           hidden. */
@@ -353,23 +358,132 @@ uint32 dos_mode_msdfs(connection_struct *conn, const char *path,SMB_STRUCT_STAT
        if (result & aDIR   ) DEBUG(8, ("d"));
        if (result & aARCH  ) DEBUG(8, ("a"));
        if (result & FILE_ATTRIBUTE_SPARSE ) DEBUG(8, ("[sparse]"));
-       
+
        DEBUG(8,("\n"));
 
        return(result);
 }
 
+#ifdef HAVE_STAT_DOS_FLAGS
+/****************************************************************************
+ Convert dos attributes (FILE_ATTRIBUTE_*) to dos stat flags (UF_*)
+****************************************************************************/
+
+int dos_attributes_to_stat_dos_flags(uint32_t dosmode)
+{
+       uint32_t dos_stat_flags = 0;
+
+       if (dosmode & aARCH)
+               dos_stat_flags |= UF_DOS_ARCHIVE;
+       if (dosmode & aHIDDEN)
+               dos_stat_flags |= UF_DOS_HIDDEN;
+       if (dosmode & aRONLY)
+               dos_stat_flags |= UF_DOS_RO;
+       if (dosmode & aSYSTEM)
+               dos_stat_flags |= UF_DOS_SYSTEM;
+       if (dosmode & FILE_ATTRIBUTE_NONINDEXED)
+               dos_stat_flags |= UF_DOS_NOINDEX;
+
+       return dos_stat_flags;
+}
+
+/****************************************************************************
+ Gets DOS attributes, accessed via st_ex_flags in the stat struct.
+****************************************************************************/
+
+static bool get_stat_dos_flags(connection_struct *conn,
+                              const char *fname,
+                              const SMB_STRUCT_STAT *sbuf,
+                              uint32_t *dosmode)
+{
+       SMB_ASSERT(sbuf && VALID_STAT(*sbuf));
+       SMB_ASSERT(dosmode);
+
+       if (!lp_store_dos_attributes(SNUM(conn))) {
+               return false;
+       }
+
+       DEBUG(5, ("Getting stat dos attributes for %s.\n", fname));
+
+       if (sbuf->st_ex_flags & UF_DOS_ARCHIVE)
+               *dosmode |= aARCH;
+       if (sbuf->st_ex_flags & UF_DOS_HIDDEN)
+               *dosmode |= aHIDDEN;
+       if (sbuf->st_ex_flags & UF_DOS_RO)
+               *dosmode |= aRONLY;
+       if (sbuf->st_ex_flags & UF_DOS_SYSTEM)
+               *dosmode |= aSYSTEM;
+       if (sbuf->st_ex_flags & UF_DOS_NOINDEX)
+               *dosmode |= FILE_ATTRIBUTE_NONINDEXED;
+       if (S_ISDIR(sbuf->st_ex_mode))
+               *dosmode |= aDIR;
+
+       *dosmode |= set_sparse_flag(sbuf);
+       *dosmode |= set_link_read_only_flag(sbuf);
+
+       return true;
+}
+
+/****************************************************************************
+ Sets DOS attributes, stored in st_ex_flags of the inode.
+****************************************************************************/
+
+static bool set_stat_dos_flags(connection_struct *conn,
+                               const char *fname,
+                               SMB_STRUCT_STAT *sbuf,
+                               uint32_t dosmode,
+                               bool *attributes_changed)
+{
+       uint32_t new_flags = 0;
+       int error = 0;
+
+       SMB_ASSERT(sbuf && VALID_STAT(*sbuf));
+       SMB_ASSERT(attributes_changed);
+
+       *attributes_changed = false;
+
+       if (!lp_store_dos_attributes(SNUM(conn))) {
+               return false;
+       }
+
+       DEBUG(5, ("Setting stat dos attributes for %s.\n", fname));
+
+       new_flags = (sbuf->st_ex_flags & ~UF_DOS_FLAGS) |
+                    dos_attributes_to_stat_dos_flags(dosmode);
+
+       /* Return early if no flags changed. */
+       if (new_flags == sbuf->st_ex_flags)
+               return true;
+
+       DEBUG(5, ("Setting stat dos attributes=0x%x, prev=0x%x\n", new_flags,
+                 sbuf->st_ex_flags));
+
+       /* Set new flags with chflags. */
+       error = SMB_VFS_CHFLAGS(conn, fname, new_flags);
+       if (error) {
+               DEBUG(0, ("Failed setting new stat dos attributes (0x%x) on "
+                         "file %s! errno=%d\n", new_flags, fname, errno));
+               return false;
+       }
+
+       *attributes_changed = true;
+       return true;
+}
+#endif /* HAVE_STAT_DOS_FLAGS */
+
 /****************************************************************************
  Change a unix mode to a dos mode.
 ****************************************************************************/
 
-uint32 dos_mode(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
+uint32 dos_mode(connection_struct *conn, const char *path, const SMB_STRUCT_STAT *psbuf)
 {
+       SMB_STRUCT_STAT sbuf = *psbuf;
        uint32 result = 0;
+       bool offline, used_stat_dos_flags = false;
 
        DEBUG(8,("dos_mode: %s\n", path));
 
-       if (!VALID_STAT(*sbuf)) {
+       if (!VALID_STAT(sbuf)) {
                return 0;
        }
 
@@ -382,21 +496,29 @@ uint32 dos_mode(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
                } else {
                        p = path;
                }
-               
-               if (p[0] == '.' && p[1] != '.' && p[1] != 0) {
+
+               /* Only . and .. are not hidden. */
+               if (p[0] == '.' && !((p[1] == '\0') ||
+                               (p[1] == '.' && p[2] == '\0'))) {
                        result |= aHIDDEN;
                }
        }
-       
-       /* Get the DOS attributes from an EA by preference. */
-       if (get_ea_dos_attribute(conn, path, sbuf, &result)) {
-               result |= set_sparse_flag(sbuf);
-       } else {
-               result |= dos_mode_from_sbuf(conn, path, sbuf);
+
+#ifdef HAVE_STAT_DOS_FLAGS
+       used_stat_dos_flags = get_stat_dos_flags(conn, path, &sbuf, &result);
+#endif
+       if (!used_stat_dos_flags) {
+               /* Get the DOS attributes from an EA by preference. */
+               if (get_ea_dos_attribute(conn, path, &sbuf, &result)) {
+                       result |= set_sparse_flag(&sbuf);
+               } else {
+                       result |= dos_mode_from_sbuf(conn, path, &sbuf);
+               }
        }
 
-       if (S_ISREG(sbuf->st_mode)) {
-               result |= set_offline_flag(conn, path);
+       offline = SMB_VFS_IS_OFFLINE(conn, path, &sbuf);
+       if (S_ISREG(sbuf.st_ex_mode) && offline) {
+               result |= FILE_ATTRIBUTE_OFFLINE;
        }
 
        /* Optimization : Only call is_hidden_path if it's not already
@@ -413,7 +535,7 @@ uint32 dos_mode(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
        if (result & aDIR   ) DEBUG(8, ("d"));
        if (result & aARCH  ) DEBUG(8, ("a"));
        if (result & FILE_ATTRIBUTE_SPARSE ) DEBUG(8, ("[sparse]"));
-       
+
        DEBUG(8,("\n"));
 
        return(result);
@@ -432,10 +554,11 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
        int mask=0;
        mode_t tmp;
        mode_t unixmode;
-       int ret = -1;
+       int ret = -1, lret = -1;
+       uint32_t old_mode;
 
        /* We only allow READONLY|HIDDEN|SYSTEM|DIRECTORY|ARCHIVE here. */
-       dosmode &= SAMBA_ATTRIBUTES_MASK;
+       dosmode &= (SAMBA_ATTRIBUTES_MASK | FILE_ATTRIBUTE_OFFLINE);
 
        DEBUG(10,("file_set_dosmode: setting dos mode 0x%x on file %s\n", dosmode, fname));
 
@@ -445,31 +568,65 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
        }
 
        if (!VALID_STAT(*st)) {
-               if (SMB_VFS_STAT(conn,fname,st))
+               if (vfs_stat_smb_fname(conn,fname,st))
                        return(-1);
        }
 
-       unixmode = st->st_mode;
+       unixmode = st->st_ex_mode;
 
-       get_acl_group_bits(conn, fname, &st->st_mode);
+       get_acl_group_bits(conn, fname, &st->st_ex_mode);
 
-       if (S_ISDIR(st->st_mode))
+       if (S_ISDIR(st->st_ex_mode))
                dosmode |= aDIR;
        else
                dosmode &= ~aDIR;
 
-       if (dos_mode(conn,fname,st) == dosmode) {
-               st->st_mode = unixmode;
+       old_mode = dos_mode(conn,fname,st);
+
+       if (dosmode & FILE_ATTRIBUTE_OFFLINE) {
+               if (!(old_mode & FILE_ATTRIBUTE_OFFLINE)) {
+                       lret = SMB_VFS_SET_OFFLINE(conn, fname);
+                       if (lret == -1) {
+                               DEBUG(0, ("set_dos_mode: client has asked to set "
+                                         "FILE_ATTRIBUTE_OFFLINE to %s/%s but there was "
+                                         "an error while setting it or it is not supported.\n",
+                                         parent_dir, fname));
+                       }
+               }
+       }
+
+       dosmode  &= ~FILE_ATTRIBUTE_OFFLINE;
+       old_mode &= ~FILE_ATTRIBUTE_OFFLINE;
+
+       if (old_mode == dosmode) {
+               st->st_ex_mode = unixmode;
                return(0);
        }
 
+#ifdef HAVE_STAT_DOS_FLAGS
+       {
+               bool attributes_changed;
+
+               if (set_stat_dos_flags(conn, fname, st, dosmode,
+                                      &attributes_changed))
+               {
+                       if (!newfile && attributes_changed) {
+                               notify_fname(conn, NOTIFY_ACTION_MODIFIED,
+                                   FILE_NOTIFY_CHANGE_ATTRIBUTES, fname);
+                       }
+                       st->st_ex_mode = unixmode;
+                       return 0;
+               }
+       }
+#endif
+
        /* Store the DOS attributes in an EA by preference. */
        if (set_ea_dos_attribute(conn, fname, st, dosmode)) {
                if (!newfile) {
                        notify_fname(conn, NOTIFY_ACTION_MODIFIED,
                                FILE_NOTIFY_CHANGE_ATTRIBUTES, fname);
                }
-               st->st_mode = unixmode;
+               st->st_ex_mode = unixmode;
                return 0;
        }
 
@@ -491,10 +648,10 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
        if (!MAP_HIDDEN(conn))
                mask |= S_IXOTH;
 
-       unixmode |= (st->st_mode & mask);
+       unixmode |= (st->st_ex_mode & mask);
 
        /* if we previously had any r bits set then leave them alone */
-       if ((tmp = st->st_mode & (S_IRUSR|S_IRGRP|S_IROTH))) {
+       if ((tmp = st->st_ex_mode & (S_IRUSR|S_IRGRP|S_IROTH))) {
                unixmode &= ~(S_IRUSR|S_IRGRP|S_IROTH);
                unixmode |= tmp;
        }
@@ -502,15 +659,16 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
        /* if we previously had any w bits set then leave them alone 
                whilst adding in the new w bits, if the new mode is not rdonly */
        if (!IS_DOS_READONLY(dosmode)) {
-               unixmode |= (st->st_mode & (S_IWUSR|S_IWGRP|S_IWOTH));
+               unixmode |= (st->st_ex_mode & (S_IWUSR|S_IWGRP|S_IWOTH));
        }
 
-       if ((ret = SMB_VFS_CHMOD(conn,fname,unixmode)) == 0) {
-               if (!newfile) {
+       ret = SMB_VFS_CHMOD(conn, fname, unixmode);
+       if (ret == 0) {
+               if(!newfile || (lret != -1)) {
                        notify_fname(conn, NOTIFY_ACTION_MODIFIED,
-                               FILE_NOTIFY_CHANGE_ATTRIBUTES, fname);
+                                    FILE_NOTIFY_CHANGE_ATTRIBUTES, fname);
                }
-               st->st_mode = unixmode;
+               st->st_ex_mode = unixmode;
                return 0;
        }
 
@@ -535,18 +693,19 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
                 * break batch oplocks open by others. JRA.
                 */
                files_struct *fsp;
-               if (!NT_STATUS_IS_OK(open_file_fchmod(conn,fname,st,&fsp)))
+               if (!NT_STATUS_IS_OK(open_file_fchmod(NULL, conn, fname, st,
+                                                     &fsp)))
                        return -1;
                become_root();
                ret = SMB_VFS_FCHMOD(fsp, unixmode);
                unbecome_root();
-               close_file_fchmod(fsp);
+               close_file_fchmod(NULL, fsp);
                if (!newfile) {
                        notify_fname(conn, NOTIFY_ACTION_MODIFIED,
                                FILE_NOTIFY_CHANGE_ATTRIBUTES, fname);
                }
                if (ret == 0) {
-                       st->st_mode = unixmode;
+                       st->st_ex_mode = unixmode;
                }
        }
 
@@ -558,13 +717,21 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
  than POSIX.
 *******************************************************************/
 
-int file_ntimes(connection_struct *conn, const char *fname, const struct timespec ts[2])
+int file_ntimes(connection_struct *conn, const char *fname,
+               struct smb_file_time *ft)
 {
-       SMB_STRUCT_STAT sbuf;
+       struct smb_filename *smb_fname = NULL;
+       NTSTATUS status;
        int ret = -1;
 
        errno = 0;
-       ZERO_STRUCT(sbuf);
+
+       DEBUG(6, ("file_ntime: actime: %s",
+                 time_to_asc(convert_timespec_to_time_t(ft->atime))));
+       DEBUG(6, ("file_ntime: modtime: %s",
+                 time_to_asc(convert_timespec_to_time_t(ft->mtime))));
+       DEBUG(6, ("file_ntime: createtime: %s",
+                 time_to_asc(convert_timespec_to_time_t(ft->create_time))));
 
        /* Don't update the time on read-only shares */
        /* We need this as set_filetime (which can be called on
@@ -577,7 +744,7 @@ int file_ntimes(connection_struct *conn, const char *fname, const struct timespe
                return 0;
        }
 
-       if(SMB_VFS_NTIMES(conn, fname, ts) == 0) {
+       if(SMB_VFS_NTIMES(conn, fname, ft) == 0) {
                return 0;
        }
 
@@ -595,41 +762,70 @@ int file_ntimes(connection_struct *conn, const char *fname, const struct timespe
           (as DOS does).
         */
 
+       status = create_synthetic_smb_fname_split(talloc_tos(), fname, NULL,
+                                                 &smb_fname);
+
+       if (!NT_STATUS_IS_OK(status)) {
+               return -1;
+       }
+
        /* Check if we have write access. */
-       if (can_write_to_file(conn, fname, &sbuf)) {
+       if (can_write_to_file(conn, smb_fname)) {
                /* We are allowed to become root and change the filetime. */
                become_root();
-               ret = SMB_VFS_NTIMES(conn, fname, ts);
+               ret = SMB_VFS_NTIMES(conn, fname, ft);
                unbecome_root();
        }
+       TALLOC_FREE(smb_fname);
 
        return ret;
 }
 
-/*******************************************************************
- Change a filetime - possibly allowing DOS semantics.
-*******************************************************************/
+/******************************************************************
+ Force a "sticky" write time on a pathname. This will always be
+ returned on all future write time queries and set on close.
+******************************************************************/
 
-bool set_filetime(connection_struct *conn, const char *fname,
-               const struct timespec mtime)
+bool set_sticky_write_time_path(connection_struct *conn, const char *fname,
+                        struct file_id fileid, const struct timespec mtime)
 {
-       struct timespec ts[2];
-
        if (null_timespec(mtime)) {
-               return(True);
+               return true;
        }
 
-       ts[1] = mtime; /* mtime. */
-       ts[0] = ts[1]; /* atime. */
+       if (!set_sticky_write_time(fileid, mtime)) {
+               return false;
+       }
 
-       if (file_ntimes(conn, fname, ts)) {
-               DEBUG(4,("set_filetime(%s) failed: %s\n",
-                                       fname,strerror(errno)));
-               return False;
+       return true;
+}
+
+/******************************************************************
+ Force a "sticky" write time on an fsp. This will always be
+ returned on all future write time queries and set on close.
+******************************************************************/
+
+bool set_sticky_write_time_fsp(struct files_struct *fsp, const struct timespec mtime)
+{
+       fsp->write_time_forced = true;
+       TALLOC_FREE(fsp->update_write_time_event);
+
+       return set_sticky_write_time_path(fsp->conn, fsp->fsp_name,
+                       fsp->file_id, mtime);
+}
+
+/******************************************************************
+ Update a write time immediately, without the 2 second delay.
+******************************************************************/
+
+bool update_write_time(struct files_struct *fsp)
+{
+       if (!set_write_time(fsp->file_id, timespec_current())) {
+               return false;
        }
 
-       notify_fname(conn, NOTIFY_ACTION_MODIFIED,
-               FILE_NOTIFY_CHANGE_LAST_WRITE, fname);
+       notify_fname(fsp->conn, NOTIFY_ACTION_MODIFIED,
+                       FILE_NOTIFY_CHANGE_LAST_WRITE, fsp->fsp_name);
 
        return true;
 }