]> bbs.cooldavid.org Git - net-next-2.6.git/blobdiff - fs/notify/fanotify/fanotify.c
fsnotify: fsnotify_add_notify_event should return an event
[net-next-2.6.git] / fs / notify / fanotify / fanotify.c
index 4feed8601e2916b981dff9edda63eb43c1d4f492..f3c40c0e2b867c0ee7f552a034a506c741f3c015 100644 (file)
@@ -2,9 +2,12 @@
 #include <linux/fdtable.h>
 #include <linux/fsnotify_backend.h>
 #include <linux/init.h>
+#include <linux/jiffies.h>
 #include <linux/kernel.h> /* UINT_MAX */
 #include <linux/mount.h>
+#include <linux/sched.h>
 #include <linux/types.h>
+#include <linux/wait.h>
 
 static bool should_merge(struct fsnotify_event *old, struct fsnotify_event *new)
 {
@@ -27,71 +30,94 @@ static bool should_merge(struct fsnotify_event *old, struct fsnotify_event *new)
        return false;
 }
 
-/* Note, if we return an event in *arg that a reference is being held... */
-static int fanotify_merge(struct list_head *list,
-                         struct fsnotify_event *event,
-                         void **arg)
+/* and the list better be locked by something too! */
+static struct fsnotify_event *fanotify_merge(struct list_head *list,
+                                            struct fsnotify_event *event)
 {
        struct fsnotify_event_holder *test_holder;
-       struct fsnotify_event *test_event;
+       struct fsnotify_event *test_event = NULL;
        struct fsnotify_event *new_event;
-       struct fsnotify_event **return_event = (struct fsnotify_event **)arg;
-       int ret = 0;
 
        pr_debug("%s: list=%p event=%p\n", __func__, list, event);
 
-       *return_event = NULL;
-
-       /* and the list better be locked by something too! */
 
        list_for_each_entry_reverse(test_holder, list, event_list) {
-               test_event = test_holder->event;
-               if (should_merge(test_event, event)) {
-                       fsnotify_get_event(test_event);
-                       *return_event = test_event;
-
-                       ret = -EEXIST;
-                       /* if they are exactly the same we are done */
-                       if (test_event->mask == event->mask)
-                               goto out;
-
-                       /*
-                        * if the refcnt == 1 this is the only queue
-                        * for this event and so we can update the mask
-                        * in place.
-                        */
-                       if (atomic_read(&test_event->refcnt) == 1) {
-                               test_event->mask |= event->mask;
-                               goto out;
-                       }
-
-                       /* can't allocate memory, merge was no possible */
-                       new_event = fsnotify_clone_event(test_event);
-                       if (unlikely(!new_event)) {
-                               ret = 0;
-                               goto out;
-                       }
-
-                       /* we didn't return the test_event, so drop that ref */
-                       fsnotify_put_event(test_event);
-                       /* the reference we return on new_event is from clone */
-                       *return_event = new_event;
-
-                       /* build new event and replace it on the list */
-                       new_event->mask = (test_event->mask | event->mask);
-                       fsnotify_replace_event(test_holder, new_event);
-
+               if (should_merge(test_holder->event, event)) {
+                       test_event = test_holder->event;
                        break;
                }
        }
-out:
+
+       if (!test_event)
+               return NULL;
+
+       fsnotify_get_event(test_event);
+
+       /* if they are exactly the same we are done */
+       if (test_event->mask == event->mask)
+               return test_event;
+
+       /*
+        * if the refcnt == 2 this is the only queue
+        * for this event and so we can update the mask
+        * in place.
+        */
+       if (atomic_read(&test_event->refcnt) == 2) {
+               test_event->mask |= event->mask;
+               return test_event;
+       }
+
+       new_event = fsnotify_clone_event(test_event);
+
+       /* done with test_event */
+       fsnotify_put_event(test_event);
+
+       /* couldn't allocate memory, merge was not possible */
+       if (unlikely(!new_event))
+               return ERR_PTR(-ENOMEM);
+
+       /* build new event and replace it on the list */
+       new_event->mask = (test_event->mask | event->mask);
+       fsnotify_replace_event(test_holder, new_event);
+
+       /* we hold a reference on new_event from clone_event */
+       return new_event;
+}
+
+#ifdef CONFIG_FANOTIFY_ACCESS_PERMISSIONS
+static int fanotify_get_response_from_access(struct fsnotify_group *group,
+                                            struct fsnotify_event *event)
+{
+       int ret;
+
+       pr_debug("%s: group=%p event=%p\n", __func__, group, event);
+
+       wait_event(group->fanotify_data.access_waitq, event->response);
+
+       /* userspace responded, convert to something usable */
+       spin_lock(&event->lock);
+       switch (event->response) {
+       case FAN_ALLOW:
+               ret = 0;
+               break;
+       case FAN_DENY:
+       default:
+               ret = -EPERM;
+       }
+       event->response = 0;
+       spin_unlock(&event->lock);
+
+       pr_debug("%s: group=%p event=%p about to return ret=%d\n", __func__,
+                group, event, ret);
+       
        return ret;
 }
+#endif
 
 static int fanotify_handle_event(struct fsnotify_group *group, struct fsnotify_event *event)
 {
-       int ret;
-       struct fsnotify_event *used_event;
+       int ret = 0;
+       struct fsnotify_event *notify_event = NULL;
 
        BUILD_BUG_ON(FAN_ACCESS != FS_ACCESS);
        BUILD_BUG_ON(FAN_MODIFY != FS_MODIFY);
@@ -100,15 +126,27 @@ static int fanotify_handle_event(struct fsnotify_group *group, struct fsnotify_e
        BUILD_BUG_ON(FAN_OPEN != FS_OPEN);
        BUILD_BUG_ON(FAN_EVENT_ON_CHILD != FS_EVENT_ON_CHILD);
        BUILD_BUG_ON(FAN_Q_OVERFLOW != FS_Q_OVERFLOW);
+       BUILD_BUG_ON(FAN_OPEN_PERM != FS_OPEN_PERM);
+       BUILD_BUG_ON(FAN_ACCESS_PERM != FS_ACCESS_PERM);
 
        pr_debug("%s: group=%p event=%p\n", __func__, group, event);
 
-       ret = fsnotify_add_notify_event(group, event, NULL, fanotify_merge, (void **)&used_event);
-       /* -EEXIST means this event was merged with another, not that it was an error */
-       if (ret == -EEXIST)
-               ret = 0;
-       if (used_event)
-               fsnotify_put_event(used_event);
+       notify_event = fsnotify_add_notify_event(group, event, NULL, fanotify_merge);
+       if (IS_ERR(notify_event))
+               return PTR_ERR(notify_event);
+
+#ifdef CONFIG_FANOTIFY_ACCESS_PERMISSIONS
+       if (event->mask & FAN_ALL_PERM_EVENTS) {
+               /* if we merged we need to wait on the new event */
+               if (notify_event)
+                       event = notify_event;
+               ret = fanotify_get_response_from_access(group, event);
+       }
+#endif
+
+       if (notify_event)
+               fsnotify_put_event(notify_event);
+
        return ret;
 }