inotify: clean up the inotify_add_watch out path
authorEric Paris <eparis@redhat.com>
Tue, 11 May 2010 21:16:23 +0000 (17:16 -0400)
committerEric Paris <eparis@redhat.com>
Fri, 14 May 2010 15:51:07 +0000 (11:51 -0400)
inotify_add_watch explictly frees the unused inode mark, but it can just
use the generic code.  Just do that.

Signed-off-by: Eric Paris <eparis@redhat.com>
fs/notify/inotify/inotify_user.c

index 472cdf29ef82598586a52f32c405bc43a8ab6b26..40da732eb73cf1fbf83643becbce8f2f30063b38 100644 (file)
@@ -578,16 +578,13 @@ retry:
        /* return the watch descriptor for this new entry */
        ret = tmp_ientry->wd;
 
-       /* match the ref from fsnotify_init_markentry() */
-       fsnotify_put_mark(&tmp_ientry->fsn_entry);
-
        /* if this mark added a new event update the group mask */
        if (mask & ~group->mask)
                fsnotify_recalc_group_mask(group);
 
 out_err:
-       if (ret < 0)
-               kmem_cache_free(inotify_inode_mark_cachep, tmp_ientry);
+       /* match the ref from fsnotify_init_markentry() */
+       fsnotify_put_mark(&tmp_ientry->fsn_entry);
 
        return ret;
 }