Fix bug #7892 - open_file_fchmod() leaves a stale lock.
[samba.git] / source3 / smbd / dosmode.c
index 1172fe3e6b55fe0747f632413e3a41d2483e8d5d..2092712fbdecb9fd718931c614aa69b1432f7b9d 100644 (file)
@@ -6,7 +6,7 @@
    
    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)
@@ -31,29 +44,15 @@ static int set_sparse_flag(const SMB_STRUCT_STAT * const sbuf)
        return 0;
 }
 
-/****************************************************************************
- Work out whether this file is offline
-****************************************************************************/
-
-#ifndef ISDOT
-#define ISDOT(p) (*(p) == '.' && *((p) + 1) == '\0')
-#endif /* ISDOT */
-
-#ifndef ISDOTDOT
-#define ISDOTDOT(p) (*(p) == '.' && *((p) + 1) == '.' && *((p) + 2) == '\0')
-#endif /* ISDOTDOT */
-
-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;
 }
 
 /****************************************************************************
@@ -185,13 +184,7 @@ static uint32 dos_mode_from_sbuf(connection_struct *conn, const char *path, SMB_
                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 "));
 
@@ -209,7 +202,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,SMB_STRUCT_STAT *sbuf, uint32 *pattr)
 {
        ssize_t sizeret;
        fstring attrstr;
@@ -224,13 +217,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. */
@@ -265,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;
@@ -284,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. */
@@ -303,14 +301,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(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));
@@ -340,8 +339,10 @@ 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;
                }
        }
@@ -354,6 +355,12 @@ uint32 dos_mode_msdfs(connection_struct *conn, const char *path,SMB_STRUCT_STAT
                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"));
@@ -368,6 +375,113 @@ uint32 dos_mode_msdfs(connection_struct *conn, const char *path,SMB_STRUCT_STAT
        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;
+       }
+
+       *attributes_changed = true;
+       return true;
+}
+#endif /* HAVE_STAT_DOS_FLAGS */
+
 /****************************************************************************
  Change a unix mode to a dos mode.
 ****************************************************************************/
@@ -375,6 +489,7 @@ uint32 dos_mode_msdfs(connection_struct *conn, const char *path,SMB_STRUCT_STAT
 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));
 
@@ -391,21 +506,30 @@ 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_mode) && offline) {
+               result |= FILE_ATTRIBUTE_OFFLINE;
        }
 
        /* Optimization : Only call is_hidden_path if it's not already
@@ -414,6 +538,12 @@ uint32 dos_mode(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
                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"));
@@ -434,24 +564,33 @@ uint32 dos_mode(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
 
 int file_set_dosmode(connection_struct *conn, const char *fname,
                     uint32 dosmode, SMB_STRUCT_STAT *st,
-                    const char *parent_dir)
+                    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;
+       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))
@@ -459,11 +598,52 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
        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;
        }
 
@@ -499,8 +679,15 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
                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;
@@ -509,7 +696,7 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
                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. */
@@ -518,28 +705,34 @@ int file_set_dosmode(connection_struct *conn, const char *fname,
                 * 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;
-               if (!NT_STATUS_IS_OK(open_file_fchmod(conn,fname,st,&fsp)))
+               if (!NT_STATUS_IS_OK(open_file_fchmod(conn, fname, st,
+                                                     &fsp)))
                        return -1;
                become_root();
-               ret = SMB_VFS_FCHMOD(fsp, fsp->fh->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;
@@ -547,6 +740,13 @@ int file_utime(connection_struct *conn, const char *fname, struct utimbuf *times
        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
           close and other paths) can end up calling this function
@@ -558,14 +758,17 @@ int file_utime(connection_struct *conn, const char *fname, struct utimbuf *times
                return 0;
        }
 
-       if(SMB_VFS_UTIME(conn,fname, times) == 0)
+       if(SMB_VFS_NTIMES(conn, fname, ft) == 0) {
                return 0;
+       }
 
-       if((errno != EPERM) && (errno != EACCES))
+       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
@@ -577,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 (!set_sticky_write_time(fileid, mtime)) {
+               return false;
+       }
 
-       if (null_mtime(mtime))
-               return(True);
+       return true;
+}
 
-       times.modtime = times.actime = mtime;
+/******************************************************************
+ Force a "sticky" write time on an fsp. This will always be
+ returned on all future write time queries and set on close.
+******************************************************************/
 
-       if (file_utime(conn, fname, &times)) {
-               DEBUG(4,("set_filetime(%s) failed: %s\n",fname,strerror(errno)));
-               return False;
+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;
+}