Fix bug #7892 - open_file_fchmod() leaves a stale lock.
[samba.git] / source3 / smbd / dosmode.c
index 3602e3f908186476b503a430d0c1894443733c02..2092712fbdecb9fd718931c614aa69b1432f7b9d 100644 (file)
@@ -2,10 +2,11 @@
    Unix SMB/CIFS implementation.
    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 2 of the License, or
+   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,
    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, write to the Free Software
-   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "includes.h"
 
+extern enum protocol_types Protocol;
+
+static uint32_t filter_mode_by_protocol(uint32_t mode)
+{
+       if (Protocol <= PROTOCOL_LANMAN2) {
+               DEBUG(10,("filter_mode_by_protocol: "
+                       "filtering result 0x%x to 0x%x\n",
+                       (unsigned int)mode,
+                       (unsigned int)(mode & 0x3f) ));
+               mode &= 0x3f;
+       }
+       return mode;
+}
+
 static int set_sparse_flag(const SMB_STRUCT_STAT * const sbuf)
 {
 #if defined (HAVE_STAT_ST_BLOCKS) && defined(STAT_ST_BLOCKSIZE)
@@ -30,10 +44,21 @@ static int set_sparse_flag(const SMB_STRUCT_STAT * const sbuf)
        return 0;
 }
 
+static int set_link_read_only_flag(const SMB_STRUCT_STAT *const sbuf)
+{
+#ifdef S_ISLNK
+#if LINKS_READ_ONLY
+       if (S_ISLNK(sbuf->st_mode) && S_ISDIR(sbuf->st_mode))
+               return aRONLY;
+#endif
+#endif
+       return 0;
+}
+
 /****************************************************************************
  Change a dos mode to a unix mode.
     Base permission for files:
-         if creating file and inheriting
+         if creating file and inheriting (i.e. parent_dir != NULL)
            apply read/write bits from parent directory.
          else   
            everybody gets read bit set
@@ -53,23 +78,26 @@ static int set_sparse_flag(const SMB_STRUCT_STAT * const sbuf)
          }
 ****************************************************************************/
 
-mode_t unix_mode(connection_struct *conn, int dosmode, const char *fname, BOOL creating_file)
+mode_t unix_mode(connection_struct *conn, int dosmode, const char *fname,
+                const char *inherit_from_dir)
 {
        mode_t result = (S_IRUSR | S_IRGRP | S_IROTH | S_IWUSR | S_IWGRP | S_IWOTH);
-       mode_t dir_mode = 0; /* Mode of the parent directory if inheriting. */
+       mode_t dir_mode = 0; /* Mode of the inherit_from directory if
+                             * inheriting. */
 
        if (!lp_store_dos_attributes(SNUM(conn)) && IS_DOS_READONLY(dosmode)) {
                result &= ~(S_IWUSR | S_IWGRP | S_IWOTH);
        }
 
-       if (fname && creating_file && lp_inherit_perms(SNUM(conn))) {
-               char *dname;
+       if (fname && (inherit_from_dir != NULL)
+           && lp_inherit_perms(SNUM(conn))) {
                SMB_STRUCT_STAT sbuf;
 
-               dname = parent_dirname(fname);
-               DEBUG(2,("unix_mode(%s) inheriting from %s\n",fname,dname));
-               if (SMB_VFS_STAT(conn,dname,&sbuf) != 0) {
-                       DEBUG(4,("unix_mode(%s) failed, [dir %s]: %s\n",fname,dname,strerror(errno)));
+               DEBUG(2, ("unix_mode(%s) inheriting from %s\n", fname,
+                         inherit_from_dir));
+               if (SMB_VFS_STAT(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! *** */
                }
 
@@ -126,17 +154,22 @@ mode_t unix_mode(connection_struct *conn, int dosmode, const char *fname, BOOL c
  Change a unix mode to a dos mode.
 ****************************************************************************/
 
-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, SMB_STRUCT_STAT *sbuf)
 {
        int result = 0;
+       enum mapreadonly_options ro_opts = (enum mapreadonly_options)lp_map_readonly(SNUM(conn));
 
-       if (lp_acl_check_permissions(SNUM(conn))) {
+       if (ro_opts == MAP_READONLY_YES) {
+               /* Original Samba method - map inverse of user "w" bit. */
+               if ((sbuf->st_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;
                }
-       } else if ((sbuf->st_mode & S_IWUSR) == 0) {
-               result |= aRONLY;
-       }
+       } /* Else never set the readonly bit. */
 
        if (MAP_ARCHIVE(conn) && ((sbuf->st_mode & S_IXUSR) != 0))
                result |= aARCH;
@@ -151,13 +184,7 @@ uint32 dos_mode_from_sbuf(connection_struct *conn, const char *path, SMB_STRUCT_
                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 "));
 
@@ -175,7 +202,7 @@ uint32 dos_mode_from_sbuf(connection_struct *conn, const char *path, SMB_STRUCT_
  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,SMB_STRUCT_STAT *sbuf, uint32 *pattr)
 {
        ssize_t sizeret;
        fstring attrstr;
@@ -185,17 +212,21 @@ static BOOL get_ea_dos_attribute(connection_struct *conn, const char *path,SMB_S
                return False;
        }
 
-       *pattr = 0;
+       /* Don't reset pattr to zero as we may already have filename-based attributes we
+          need to preserve. */
 
        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)) {
+               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. */
@@ -230,11 +261,11 @@ static BOOL get_ea_dos_attribute(connection_struct *conn, const char *path,SMB_S
  Set DOS attributes in an EA.
 ****************************************************************************/
 
-static BOOL set_ea_dos_attribute(connection_struct *conn, const char *path, SMB_STRUCT_STAT *sbuf, uint32 dosmode)
+static bool set_ea_dos_attribute(connection_struct *conn, const char *path, SMB_STRUCT_STAT *sbuf, uint32 dosmode)
 {
        fstring attrstr;
        files_struct *fsp = NULL;
-       BOOL ret = False;
+       bool ret = False;
 
        if (!lp_store_dos_attributes(SNUM(conn))) {
                return False;
@@ -249,13 +280,15 @@ 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;
                }
 
                /* We want DOS semantics, ie allow non owner with write permission to change the
-                       bits on a file. Just like file_utime below.
+                       bits on a file. Just like file_ntimes below.
                */
 
                /* Check if we have write access. */
@@ -268,15 +301,15 @@ static BOOL set_ea_dos_attribute(connection_struct *conn, const char *path, SMB_
                 * are not violating security in doing the setxattr.
                 */
 
-               fsp = open_file_fchmod(conn,path,sbuf);
-               if (!fsp)
+               if (!NT_STATUS_IS_OK(open_file_fchmod(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(NULL, fsp, NORMAL_CLOSE);
                return ret;
        }
        DEBUG(10,("set_ea_dos_attribute: set EA %s on file %s\n", attrstr, path));
@@ -284,46 +317,233 @@ static BOOL set_ea_dos_attribute(connection_struct *conn, const char *path, SMB_
 }
 
 /****************************************************************************
- Change a unix mode to a dos mode.
+ Change a unix mode to a dos mode for an ms dfs link.
 ****************************************************************************/
 
-uint32 dos_mode(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
+uint32 dos_mode_msdfs(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
 {
        uint32 result = 0;
 
-       DEBUG(8,("dos_mode: %s\n", path));
+       DEBUG(8,("dos_mode_msdfs: %s\n", path));
 
        if (!VALID_STAT(*sbuf)) {
                return 0;
        }
 
-       /* Get the DOS attributes from an EA by preference. */
-       if (get_ea_dos_attribute(conn, path, sbuf, &result)) {
-               result |= set_sparse_flag(sbuf);
-               return result;
+       /* First do any modifications that depend on the path name. */
+       /* hide files with a name starting with a . */
+       if (lp_hide_dot_files(SNUM(conn))) {
+               const char *p = strrchr_m(path,'/');
+               if (p) {
+                       p++;
+               } else {
+                       p = path;
+               }
+
+               /* 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);
+
+       /* Optimization : Only call is_hidden_path if it's not already
+          hidden. */
+       if (!(result & aHIDDEN) && IS_HIDDEN_PATH(conn,path)) {
+               result |= aHIDDEN;
+       }
+
+       if (result == 0) {
+               result = FILE_ATTRIBUTE_NORMAL;
+       }
+
+       result = filter_mode_by_protocol(result);
+
+       DEBUG(8,("dos_mode_msdfs returning "));
+
+       if (result & aHIDDEN) DEBUG(8, ("h"));
+       if (result & aRONLY ) DEBUG(8, ("r"));
+       if (result & aSYSTEM) DEBUG(8, ("s"));
+       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_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_flags & UF_DOS_ARCHIVE)
+               *dosmode |= aARCH;
+       if (sbuf->st_flags & UF_DOS_HIDDEN)
+               *dosmode |= aHIDDEN;
+       if (sbuf->st_flags & UF_DOS_RO)
+               *dosmode |= aRONLY;
+       if (sbuf->st_flags & UF_DOS_SYSTEM)
+               *dosmode |= aSYSTEM;
+       if (sbuf->st_flags & UF_DOS_NOINDEX)
+               *dosmode |= FILE_ATTRIBUTE_NONINDEXED;
+       if (S_ISDIR(sbuf->st_mode))
+               *dosmode |= aDIR;
+
+       *dosmode |= set_sparse_flag(sbuf);
+       *dosmode |= set_link_read_only_flag(sbuf);
+
+       return true;
+}
+
+/****************************************************************************
+ Sets DOS attributes, stored in st_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_flags & ~UF_DOS_FLAGS) |
+                    dos_attributes_to_stat_dos_flags(dosmode);
+
+       /* Return early if no flags changed. */
+       if (new_flags == sbuf->st_flags)
+               return true;
+
+       DEBUG(5, ("Setting stat dos attributes=0x%x, prev=0x%x\n", new_flags,
+                 sbuf->st_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;
        }
 
-       result = dos_mode_from_sbuf(conn, path, sbuf);
+       *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 result = 0;
+       bool offline, used_stat_dos_flags = false;
+
+       DEBUG(8,("dos_mode: %s\n", path));
+
+       if (!VALID_STAT(*sbuf)) {
+               return 0;
+       }
 
-       /* Now do any modifications that depend on the path name. */
+       /* First do any modifications that depend on the path name. */
        /* hide files with a name starting with a . */
        if (lp_hide_dot_files(SNUM(conn))) {
                const char *p = strrchr_m(path,'/');
-               if (p)
+               if (p) {
                        p++;
-               else
+               } 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;
+               }
        }
        
+#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);
+               }
+       }
+
+       
+       offline = SMB_VFS_IS_OFFLINE(conn, path, sbuf);
+       if (S_ISREG(sbuf->st_mode) && offline) {
+               result |= FILE_ATTRIBUTE_OFFLINE;
+       }
+
        /* Optimization : Only call is_hidden_path if it's not already
           hidden. */
        if (!(result & aHIDDEN) && IS_HIDDEN_PATH(conn,path)) {
                result |= aHIDDEN;
        }
 
+       if (result == 0) {
+               result = FILE_ATTRIBUTE_NORMAL;
+       }
+
+       result = filter_mode_by_protocol(result);
+
        DEBUG(8,("dos_mode returning "));
 
        if (result & aHIDDEN) DEBUG(8, ("h"));
@@ -331,6 +551,7 @@ uint32 dos_mode(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
        if (result & aSYSTEM) DEBUG(8, ("s"));
        if (result & aDIR   ) DEBUG(8, ("d"));
        if (result & aARCH  ) DEBUG(8, ("a"));
+       if (result & FILE_ATTRIBUTE_SPARSE ) DEBUG(8, ("[sparse]"));
        
        DEBUG(8,("\n"));
 
@@ -341,21 +562,35 @@ uint32 dos_mode(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
  chmod a file - but preserve some bits.
 ********************************************************************/
 
-int file_set_dosmode(connection_struct *conn, const char *fname, uint32 dosmode, SMB_STRUCT_STAT *st, BOOL creating_file)
+int file_set_dosmode(connection_struct *conn, const char *fname,
+                    uint32 dosmode, SMB_STRUCT_STAT *st,
+                    const char *parent_dir,
+                    bool newfile)
 {
        SMB_STRUCT_STAT st1;
        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 | FILE_ATTRIBUTE_OFFLINE);
 
        DEBUG(10,("file_set_dosmode: setting dos mode 0x%x on file %s\n", dosmode, fname));
-       if (!st || (st && !VALID_STAT(*st))) {
+
+       if (st == NULL) {
+               SET_STAT_INVALID(st1);
                st = &st1;
+       }
+
+       if (!VALID_STAT(*st)) {
                if (SMB_VFS_STAT(conn,fname,st))
                        return(-1);
        }
 
+       unixmode = st->st_mode;
+
        get_acl_group_bits(conn, fname, &st->st_mode);
 
        if (S_ISDIR(st->st_mode))
@@ -363,15 +598,56 @@ int file_set_dosmode(connection_struct *conn, const char *fname, uint32 dosmode,
        else
                dosmode &= ~aDIR;
 
-       if (dos_mode(conn,fname,st) == dosmode)
+       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_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_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;
                return 0;
        }
 
-       unixmode = unix_mode(conn,dosmode,fname, creating_file);
+       unixmode = unix_mode(conn,dosmode,fname, parent_dir);
 
        /* preserve the s bits */
        mask |= (S_ISUID | S_ISGID);
@@ -403,8 +679,15 @@ int file_set_dosmode(connection_struct *conn, const char *fname, uint32 dosmode,
                unixmode |= (st->st_mode & (S_IWUSR|S_IWGRP|S_IWOTH));
        }
 
-       if ((ret = SMB_VFS_CHMOD(conn,fname,unixmode)) == 0)
+       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);
+               }
+               st->st_mode = unixmode;
                return 0;
+       }
 
        if((errno != EPERM) && (errno != EACCES))
                return -1;
@@ -413,7 +696,7 @@ int file_set_dosmode(connection_struct *conn, const char *fname, uint32 dosmode,
                return -1;
 
        /* We want DOS semantics, ie allow non owner with write permission to change the
-               bits on a file. Just like file_utime below.
+               bits on a file. Just like file_ntimes below.
        */
 
        /* Check if we have write access. */
@@ -422,28 +705,34 @@ int file_set_dosmode(connection_struct *conn, const char *fname, uint32 dosmode,
                 * We need to open the file with write access whilst
                 * still in our current user context. This ensures we
                 * are not violating security in doing the fchmod.
-                * This file open does *not* break any oplocks we are
-                * holding. We need to review this.... may need to
-                * break batch oplocks open by others. JRA.
                 */
-               files_struct *fsp = open_file_fchmod(conn,fname,st);
-               if (!fsp)
+               files_struct *fsp;
+               if (!NT_STATUS_IS_OK(open_file_fchmod(conn, fname, st,
+                                                     &fsp)))
                        return -1;
                become_root();
-               ret = SMB_VFS_FCHMOD(fsp, fsp->fd, unixmode);
+               ret = SMB_VFS_FCHMOD(fsp, unixmode);
                unbecome_root();
-               close_file_fchmod(fsp);
+               close_file(NULL, fsp, NORMAL_CLOSE);
+               if (!newfile) {
+                       notify_fname(conn, NOTIFY_ACTION_MODIFIED,
+                               FILE_NOTIFY_CHANGE_ATTRIBUTES, fname);
+               }
+               if (ret == 0) {
+                       st->st_mode = unixmode;
+               }
        }
 
        return( ret );
 }
 
 /*******************************************************************
- Wrapper around dos_utime that possibly allows DOS semantics rather
+ Wrapper around the VFS ntimes that possibly allows DOS semantics rather
  than POSIX.
 *******************************************************************/
 
-int file_utime(connection_struct *conn, const char *fname, struct utimbuf *times)
+int file_ntimes(connection_struct *conn, const char *fname,
+               struct smb_file_time *ft)
 {
        SMB_STRUCT_STAT sbuf;
        int ret = -1;
@@ -451,14 +740,35 @@ int file_utime(connection_struct *conn, const char *fname, struct utimbuf *times
        errno = 0;
        ZERO_STRUCT(sbuf);
 
-       if(SMB_VFS_UTIME(conn,fname, times) == 0)
+       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
+          close and other paths) can end up calling this function
+          without the NEED_WRITE protection. Found by : 
+          Leo Weppelman <leo@wau.mis.ah.nl>
+       */
+
+       if (!CAN_WRITE(conn)) {
                return 0;
+       }
 
-       if((errno != EPERM) && (errno != EACCES))
+       if(SMB_VFS_NTIMES(conn, fname, ft) == 0) {
+               return 0;
+       }
+
+       if((errno != EPERM) && (errno != EACCES)) {
                return -1;
+       }
 
-       if(!lp_dos_filetimes(SNUM(conn)))
+       if(!lp_dos_filetimes(SNUM(conn))) {
                return -1;
+       }
 
        /* We have permission (given by the Samba admin) to
           break POSIX semantics and allow a user to change
@@ -470,30 +780,58 @@ int file_utime(connection_struct *conn, const char *fname, struct utimbuf *times
        if (can_write_to_file(conn, fname, &sbuf)) {
                /* We are allowed to become root and change the filetime. */
                become_root();
-               ret = SMB_VFS_UTIME(conn,fname, times);
+               ret = SMB_VFS_NTIMES(conn, fname, ft);
                unbecome_root();
        }
 
        return ret;
 }
-  
-/*******************************************************************
- Change a filetime - possibly allowing DOS semantics.
-*******************************************************************/
 
-BOOL set_filetime(connection_struct *conn, const char *fname, time_t mtime)
+/******************************************************************
+ 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_sticky_write_time_path(connection_struct *conn, const char *fname,
+                        struct file_id fileid, const struct timespec mtime)
 {
-       struct utimbuf times;
+       if (null_timespec(mtime)) {
+               return true;
+       }
 
-       if (null_mtime(mtime))
-               return(True);
+       if (!set_sticky_write_time(fileid, mtime)) {
+               return false;
+       }
 
-       times.modtime = times.actime = mtime;
+       return true;
+}
 
-       if (file_utime(conn, fname, &times)) {
-               DEBUG(4,("set_filetime(%s) failed: %s\n",fname,strerror(errno)));
-               return False;
+/******************************************************************
+ 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;
        }
-  
-       return(True);
-} 
+
+       notify_fname(fsp->conn, NOTIFY_ACTION_MODIFIED,
+                       FILE_NOTIFY_CHANGE_LAST_WRITE, fsp->fsp_name);
+
+       return true;
+}