s3/smbd: update some more DEBUG macros in smbd_smb2_create_send
[nivanova/samba-autobuild/.git] / source3 / smbd / notify_inotify.c
index efb659f7c07addd20dd2762cecb54ba463cafe25..74855a9f6f9eedf8642b42dbe34e95b67e29f086 100644 (file)
@@ -24,8 +24,7 @@
 #include "includes.h"
 #include "../librpc/gen_ndr/notify.h"
 #include "smbd/smbd.h"
-
-#ifdef HAVE_INOTIFY
+#include "lib/util/sys_rw_data.h"
 
 #include <sys/inotify.h>
 
@@ -49,7 +48,8 @@ struct inotify_watch_context {
        int wd;
        void (*callback)(struct sys_notify_context *ctx, 
                         void *private_data,
-                        struct notify_event *ev);
+                        struct notify_event *ev,
+                        uint32_t filter);
        void *private_data;
        uint32_t mask; /* the inotify mask */
        uint32_t filter; /* the windows completion filter */
@@ -57,6 +57,60 @@ struct inotify_watch_context {
 };
 
 
+/*
+  map from a change notify mask to a inotify mask. Remove any bits
+  which we can handle
+*/
+static const struct {
+       uint32_t notify_mask;
+       uint32_t inotify_mask;
+} inotify_mapping[] = {
+       {FILE_NOTIFY_CHANGE_FILE_NAME,   IN_CREATE|IN_DELETE|IN_MOVED_FROM|IN_MOVED_TO},
+       {FILE_NOTIFY_CHANGE_DIR_NAME,    IN_CREATE|IN_DELETE|IN_MOVED_FROM|IN_MOVED_TO},
+       {FILE_NOTIFY_CHANGE_ATTRIBUTES,  IN_ATTRIB|IN_MOVED_TO|IN_MOVED_FROM|IN_MODIFY},
+       {FILE_NOTIFY_CHANGE_LAST_WRITE,  IN_ATTRIB},
+       {FILE_NOTIFY_CHANGE_LAST_ACCESS, IN_ATTRIB},
+       {FILE_NOTIFY_CHANGE_EA,          IN_ATTRIB},
+       {FILE_NOTIFY_CHANGE_SECURITY,    IN_ATTRIB}
+};
+
+static uint32_t inotify_map(uint32_t *filter)
+{
+       int i;
+       uint32_t out=0;
+       for (i=0;i<ARRAY_SIZE(inotify_mapping);i++) {
+               if (inotify_mapping[i].notify_mask & *filter) {
+                       out |= inotify_mapping[i].inotify_mask;
+                       *filter &= ~inotify_mapping[i].notify_mask;
+               }
+       }
+       return out;
+}
+
+/*
+ * Map inotify mask back to filter. This returns all filters that
+ * could have created the inotify watch.
+ */
+static uint32_t inotify_map_mask_to_filter(uint32_t mask)
+{
+       int i;
+       uint32_t filter = 0;
+
+       for (i = 0; i < ARRAY_SIZE(inotify_mapping); i++) {
+               if (inotify_mapping[i].inotify_mask & mask) {
+                       filter |= inotify_mapping[i].notify_mask;
+               }
+       }
+
+       if (mask & IN_ISDIR) {
+               filter &= ~FILE_NOTIFY_CHANGE_FILE_NAME;
+       } else {
+               filter &= ~FILE_NOTIFY_CHANGE_DIR_NAME;
+       }
+
+       return filter;
+}
+
 /*
   destroy the inotify private context
 */
@@ -74,6 +128,8 @@ static int inotify_destructor(struct inotify_private *in)
 static bool filter_match(struct inotify_watch_context *w,
                         struct inotify_event *e)
 {
+       bool ok;
+
        DEBUG(10, ("filter_match: e->mask=%x, w->mask=%x, w->filter=%x\n",
                   e->mask, w->mask, w->filter));
 
@@ -85,28 +141,25 @@ static bool filter_match(struct inotify_watch_context *w,
 
        /* SMB separates the filters for files and directories */
        if (e->mask & IN_ISDIR) {
-               if ((w->filter & FILE_NOTIFY_CHANGE_DIR_NAME) == 0) {
-                       return False;
-               }
-       } else {
-               if ((e->mask & IN_ATTRIB) &&
-                   (w->filter & (FILE_NOTIFY_CHANGE_ATTRIBUTES|
-                                 FILE_NOTIFY_CHANGE_LAST_WRITE|
-                                 FILE_NOTIFY_CHANGE_LAST_ACCESS|
-                                 FILE_NOTIFY_CHANGE_EA|
-                                 FILE_NOTIFY_CHANGE_SECURITY))) {
-                       return True;
-               }
-               if ((e->mask & IN_MODIFY) && 
-                   (w->filter & FILE_NOTIFY_CHANGE_ATTRIBUTES)) {
-                       return True;
-               }
-               if ((w->filter & FILE_NOTIFY_CHANGE_FILE_NAME) == 0) {
-                       return False;
-               }
+               ok = ((w->filter & FILE_NOTIFY_CHANGE_DIR_NAME) != 0);
+               return ok;
+       }
+
+       if ((e->mask & IN_ATTRIB) &&
+           (w->filter & (FILE_NOTIFY_CHANGE_ATTRIBUTES|
+                         FILE_NOTIFY_CHANGE_LAST_WRITE|
+                         FILE_NOTIFY_CHANGE_LAST_ACCESS|
+                         FILE_NOTIFY_CHANGE_EA|
+                         FILE_NOTIFY_CHANGE_SECURITY))) {
+               return True;
+       }
+       if ((e->mask & IN_MODIFY) &&
+           (w->filter & FILE_NOTIFY_CHANGE_ATTRIBUTES)) {
+               return True;
        }
 
-       return True;
+       ok = ((w->filter & FILE_NOTIFY_CHANGE_FILE_NAME) != 0);
+       return ok;
 }
 
 
@@ -118,11 +171,13 @@ static bool filter_match(struct inotify_watch_context *w,
 */
 static void inotify_dispatch(struct inotify_private *in, 
                             struct inotify_event *e, 
+                            int prev_wd,
                             uint32_t prev_cookie,
                             struct inotify_event *e2)
 {
        struct inotify_watch_context *w, *next;
        struct notify_event ne;
+       uint32_t filter;
 
        DEBUG(10, ("inotify_dispatch called with mask=%x, name=[%s]\n",
                   e->mask, e->len ? e->name : ""));
@@ -140,13 +195,14 @@ static void inotify_dispatch(struct inotify_private *in,
        } else if (e->mask & IN_DELETE) {
                ne.action = NOTIFY_ACTION_REMOVED;
        } else if (e->mask & IN_MOVED_FROM) {
-               if (e2 != NULL && e2->cookie == e->cookie) {
+               if (e2 != NULL && e2->cookie == e->cookie &&
+                   e2->wd == e->wd) {
                        ne.action = NOTIFY_ACTION_OLD_NAME;
                } else {
                        ne.action = NOTIFY_ACTION_REMOVED;
                }
        } else if (e->mask & IN_MOVED_TO) {
-               if (e->cookie == prev_cookie) {
+               if ((e->cookie == prev_cookie) && (e->wd == prev_wd)) {
                        ne.action = NOTIFY_ACTION_NEW_NAME;
                } else {
                        ne.action = NOTIFY_ACTION_ADDED;
@@ -156,33 +212,40 @@ static void inotify_dispatch(struct inotify_private *in,
        }
        ne.path = e->name;
 
-       DEBUG(10, ("inotify_dispatch: ne.action = %d, ne.path = %s\n",
-                  ne.action, ne.path));
+       filter = inotify_map_mask_to_filter(e->mask);
+
+       DBG_DEBUG("ne.action = %d, ne.path = %s, filter = %d\n",
+                 ne.action, ne.path, filter);
 
        /* find any watches that have this watch descriptor */
        for (w=in->watches;w;w=next) {
                next = w->next;
                if (w->wd == e->wd && filter_match(w, e)) {
-                       w->callback(in->ctx, w->private_data, &ne);
+                       ne.dir = w->path;
+                       w->callback(in->ctx, w->private_data, &ne, filter);
                }
        }
 
-       /* SMB expects a file rename to generate three events, two for
-          the rename and the other for a modify of the
-          destination. Strange! */
-       if (ne.action != NOTIFY_ACTION_NEW_NAME ||
-           (e->mask & IN_ISDIR) != 0) {
-               return;
-       }
+       if ((ne.action == NOTIFY_ACTION_NEW_NAME) &&
+           ((e->mask & IN_ISDIR) == 0)) {
 
-       ne.action = NOTIFY_ACTION_MODIFIED;
-       e->mask = IN_ATTRIB;
+               /*
+                * SMB expects a file rename to generate three events, two for
+                * the rename and the other for a modify of the
+                * destination. Strange!
+                */
 
-       for (w=in->watches;w;w=next) {
-               next = w->next;
-               if (w->wd == e->wd && filter_match(w, e) &&
-                   !(w->filter & FILE_NOTIFY_CHANGE_CREATION)) {
-                       w->callback(in->ctx, w->private_data, &ne);
+               ne.action = NOTIFY_ACTION_MODIFIED;
+               e->mask = IN_ATTRIB;
+
+               for (w=in->watches;w;w=next) {
+                       next = w->next;
+                       if (w->wd == e->wd && filter_match(w, e) &&
+                           !(w->filter & FILE_NOTIFY_CHANGE_CREATION)) {
+                               ne.dir = w->path;
+                               w->callback(in->ctx, w->private_data, &ne,
+                                           filter);
+                       }
                }
        }
 }
@@ -198,7 +261,8 @@ static void inotify_handler(struct tevent_context *ev, struct tevent_fd *fde,
        int bufsize = 0;
        struct inotify_event *e0, *e;
        uint32_t prev_cookie=0;
-       NTSTATUS status;
+       int prev_wd = -1;
+       ssize_t ret;
 
        /*
          we must use FIONREAD as we cannot predict the length of the
@@ -216,10 +280,10 @@ static void inotify_handler(struct tevent_context *ev, struct tevent_fd *fde,
        if (e == NULL) return;
        ((uint8_t *)e)[bufsize] = '\0';
 
-       status = read_data(in->fd, (char *)e0, bufsize);
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(0,("Failed to read all inotify data - %s\n",
-                       nt_errstr(status)));
+       ret = read_data(in->fd, e0, bufsize);
+       if (ret != bufsize) {
+               DEBUG(0, ("Failed to read all inotify data - %s\n",
+                         strerror(errno)));
                talloc_free(e0);
                /* the inotify fd will now be out of sync,
                 * can't keep reading data off it */
@@ -234,7 +298,8 @@ static void inotify_handler(struct tevent_context *ev, struct tevent_fd *fde,
                if (bufsize >= sizeof(*e)) {
                        e2 = (struct inotify_event *)(e->len + sizeof(*e) + (char *)e);
                }
-               inotify_dispatch(in, e, prev_cookie, e2);
+               inotify_dispatch(in, e, prev_wd, prev_cookie, e2);
+               prev_wd = e->wd;
                prev_cookie = e->cookie;
                e = e2;
        }
@@ -246,21 +311,22 @@ static void inotify_handler(struct tevent_context *ev, struct tevent_fd *fde,
   setup the inotify handle - called the first time a watch is added on
   this context
 */
-static NTSTATUS inotify_setup(struct sys_notify_context *ctx)
+static int inotify_setup(struct sys_notify_context *ctx)
 {
        struct inotify_private *in;
+       struct tevent_fd *fde;
 
-       if (!lp_parm_bool(-1, "notify", "inotify", True)) {
-               return NT_STATUS_INVALID_SYSTEM_SERVICE;
+       in = talloc(ctx, struct inotify_private);
+       if (in == NULL) {
+               return ENOMEM;
        }
 
-       in = talloc(ctx, struct inotify_private);
-       NT_STATUS_HAVE_NO_MEMORY(in);
        in->fd = inotify_init();
        if (in->fd == -1) {
-               DEBUG(0,("Failed to init inotify - %s\n", strerror(errno)));
+               int ret = errno;
+               DEBUG(0, ("Failed to init inotify - %s\n", strerror(ret)));
                talloc_free(in);
-               return map_nt_error_from_unix(errno);
+               return ret;
        }
        in->ctx = ctx;
        in->watches = NULL;
@@ -269,40 +335,14 @@ static NTSTATUS inotify_setup(struct sys_notify_context *ctx)
        talloc_set_destructor(in, inotify_destructor);
 
        /* add a event waiting for the inotify fd to be readable */
-       tevent_add_fd(ctx->ev, in, in->fd, TEVENT_FD_READ, inotify_handler, in);
-
-       return NT_STATUS_OK;
-}
-
-
-/*
-  map from a change notify mask to a inotify mask. Remove any bits
-  which we can handle
-*/
-static const struct {
-       uint32_t notify_mask;
-       uint32_t inotify_mask;
-} inotify_mapping[] = {
-       {FILE_NOTIFY_CHANGE_FILE_NAME,   IN_CREATE|IN_DELETE|IN_MOVED_FROM|IN_MOVED_TO},
-       {FILE_NOTIFY_CHANGE_DIR_NAME,    IN_CREATE|IN_DELETE|IN_MOVED_FROM|IN_MOVED_TO},
-       {FILE_NOTIFY_CHANGE_ATTRIBUTES,  IN_ATTRIB|IN_MOVED_TO|IN_MOVED_FROM|IN_MODIFY},
-       {FILE_NOTIFY_CHANGE_LAST_WRITE,  IN_ATTRIB},
-       {FILE_NOTIFY_CHANGE_LAST_ACCESS, IN_ATTRIB},
-       {FILE_NOTIFY_CHANGE_EA,          IN_ATTRIB},
-       {FILE_NOTIFY_CHANGE_SECURITY,    IN_ATTRIB}
-};
-
-static uint32_t inotify_map(uint32_t *filter)
-{
-       int i;
-       uint32_t out=0;
-       for (i=0;i<ARRAY_SIZE(inotify_mapping);i++) {
-               if (inotify_mapping[i].notify_mask & *filter) {
-                       out |= inotify_mapping[i].inotify_mask;
-                       *filter &= ~inotify_mapping[i].notify_mask;
-               }
+       fde = tevent_add_fd(ctx->ev, in, in->fd, TEVENT_FD_READ,
+                           inotify_handler, in);
+       if (fde == NULL) {
+               ctx->private_data = NULL;
+               TALLOC_FREE(in);
+               return ENOMEM;
        }
-       return out;
+       return 0;
 }
 
 /*
@@ -314,17 +354,19 @@ static int watch_destructor(struct inotify_watch_context *w)
        int wd = w->wd;
        DLIST_REMOVE(w->in->watches, w);
 
-       /* only rm the watch if its the last one with this wd */
        for (w=in->watches;w;w=w->next) {
-               if (w->wd == wd) break;
-       }
-       if (w == NULL) {
-               DEBUG(10, ("Deleting inotify watch %d\n", wd));
-               if (inotify_rm_watch(in->fd, wd) == -1) {
-                       DEBUG(1, ("inotify_rm_watch returned %s\n",
-                                 strerror(errno)));
+               if (w->wd == wd) {
+                       /*
+                        * Another inotify_watch_context listens on this path,
+                        * leave the kernel level watch in place
+                        */
+                       return 0;
                }
+       }
 
+       DEBUG(10, ("Deleting inotify watch %d\n", wd));
+       if (inotify_rm_watch(in->fd, wd) == -1) {
+               DEBUG(1, ("inotify_rm_watch returned %s\n", strerror(errno)));
        }
        return 0;
 }
@@ -334,18 +376,19 @@ static int watch_destructor(struct inotify_watch_context *w)
   add a watch. The watch is removed when the caller calls
   talloc_free() on *handle
 */
-NTSTATUS inotify_watch(struct sys_notify_context *ctx,
-                      const char *path,
-                      uint32_t *filter,
-                      uint32_t *subdir_filter,
-                      void (*callback)(struct sys_notify_context *ctx, 
-                                       void *private_data,
-                                       struct notify_event *ev),
-                      void *private_data, 
-                      void *handle_p)
+int inotify_watch(TALLOC_CTX *mem_ctx,
+                 struct sys_notify_context *ctx,
+                 const char *path,
+                 uint32_t *filter,
+                 uint32_t *subdir_filter,
+                 void (*callback)(struct sys_notify_context *ctx,
+                                  void *private_data,
+                                  struct notify_event *ev,
+                                  uint32_t filter),
+                 void *private_data,
+                 void *handle_p)
 {
        struct inotify_private *in;
-       int wd;
        uint32_t mask;
        struct inotify_watch_context *w;
        uint32_t orig_filter = *filter;
@@ -353,9 +396,11 @@ NTSTATUS inotify_watch(struct sys_notify_context *ctx,
 
        /* maybe setup the inotify fd */
        if (ctx->private_data == NULL) {
-               NTSTATUS status;
-               status = inotify_setup(ctx);
-               NT_STATUS_NOT_OK_RETURN(status);
+               int ret;
+               ret = inotify_setup(ctx);
+               if (ret != 0) {
+                       return ret;
+               }
        }
 
        in = talloc_get_type(ctx->private_data, struct inotify_private);
@@ -363,45 +408,44 @@ NTSTATUS inotify_watch(struct sys_notify_context *ctx,
        mask = inotify_map(filter);
        if (mask == 0) {
                /* this filter can't be handled by inotify */
-               return NT_STATUS_INVALID_PARAMETER;
+               return EINVAL;
        }
 
        /* using IN_MASK_ADD allows us to cope with inotify() returning the same
           watch descriptor for multiple watches on the same path */
        mask |= (IN_MASK_ADD | IN_ONLYDIR);
 
-       /* get a new watch descriptor for this path */
-       wd = inotify_add_watch(in->fd, path, mask);
-       if (wd == -1) {
-               *filter = orig_filter;
-               DEBUG(1, ("inotify_add_watch returned %s\n", strerror(errno)));
-               return map_nt_error_from_unix(errno);
-       }
-
-       DEBUG(10, ("inotify_add_watch for %s mask %x returned wd %d\n",
-                  path, mask, wd));
-
-       w = talloc(in, struct inotify_watch_context);
+       w = talloc(mem_ctx, struct inotify_watch_context);
        if (w == NULL) {
-               inotify_rm_watch(in->fd, wd);
                *filter = orig_filter;
-               return NT_STATUS_NO_MEMORY;
+               return ENOMEM;
        }
 
        w->in = in;
-       w->wd = wd;
        w->callback = callback;
        w->private_data = private_data;
        w->mask = mask;
        w->filter = orig_filter;
        w->path = talloc_strdup(w, path);
        if (w->path == NULL) {
-               inotify_rm_watch(in->fd, wd);
                *filter = orig_filter;
                TALLOC_FREE(w);
-               return NT_STATUS_NO_MEMORY;
+               return ENOMEM;
+       }
+
+       /* get a new watch descriptor for this path */
+       w->wd = inotify_add_watch(in->fd, path, mask);
+       if (w->wd == -1) {
+               int err = errno;
+               *filter = orig_filter;
+               TALLOC_FREE(w);
+               DEBUG(1, ("inotify_add_watch returned %s\n", strerror(err)));
+               return err;
        }
 
+       DEBUG(10, ("inotify_add_watch for %s mask %x returned wd %d\n",
+                  path, mask, w->wd));
+
        (*handle) = w;
 
        DLIST_ADD(in->watches, w);
@@ -409,7 +453,5 @@ NTSTATUS inotify_watch(struct sys_notify_context *ctx,
        /* the caller frees the handle to stop watching */
        talloc_set_destructor(w, watch_destructor);
 
-       return NT_STATUS_OK;
+       return 0;
 }
-
-#endif