]> nv-tegra.nvidia Code Review - linux-2.6.git/commitdiff
[PATCH] inotify delete race fix
authorJohn McCutchan <ttb@tentacle.dhs.org>
Thu, 4 Aug 2005 20:07:08 +0000 (13:07 -0700)
committerLinus Torvalds <torvalds@g5.osdl.org>
Thu, 4 Aug 2005 20:11:15 +0000 (13:11 -0700)
The included patch fixes a problem where a inotify client would receive a
delete event before the file was actually deleted.  The bug affects both
dnotify & inotify.

Signed-off-by: John McCutchan <ttb@tentacle.dhs.org>
Signed-off-by: Robert Love <rml@novell.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
fs/namei.c

index 4a27eb798118957544d52c9edbe013f30ce9cc43..83559dce4286ccaf282070363deb9a7d5c525ba0 100644 (file)
@@ -1801,8 +1801,8 @@ int vfs_rmdir(struct inode *dir, struct dentry *dentry)
        }
        up(&dentry->d_inode->i_sem);
        if (!error) {
        }
        up(&dentry->d_inode->i_sem);
        if (!error) {
-               fsnotify_rmdir(dentry, dentry->d_inode, dir);
                d_delete(dentry);
                d_delete(dentry);
+               fsnotify_rmdir(dentry, dentry->d_inode, dir);
        }
        dput(dentry);
 
        }
        dput(dentry);
 
@@ -1874,8 +1874,14 @@ int vfs_unlink(struct inode *dir, struct dentry *dentry)
 
        /* We don't d_delete() NFS sillyrenamed files--they still exist. */
        if (!error && !(dentry->d_flags & DCACHE_NFSFS_RENAMED)) {
 
        /* We don't d_delete() NFS sillyrenamed files--they still exist. */
        if (!error && !(dentry->d_flags & DCACHE_NFSFS_RENAMED)) {
+#if defined(CONFIG_INOTIFY) || defined(CONFIG_DNOTIFY)
+               dget(dentry);
+               d_delete(dentry);
                fsnotify_unlink(dentry, dir);
                fsnotify_unlink(dentry, dir);
+               dput(dentry);
+#else
                d_delete(dentry);
                d_delete(dentry);
+#endif
        }
 
        return error;
        }
 
        return error;