r20394: This is a *VERY* early start of my work on notify.
authorVolker Lendecke <vlendec@samba.org>
Thu, 28 Dec 2006 21:50:31 +0000 (21:50 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 17:16:46 +0000 (12:16 -0500)
Checking in because Jeremy was bugging me. Potentially this becomes quite
intrusive, I'm not sure if I should open a temporary branch for this.

Jeremy, Jerry, do you think 3_0 is the right place for this?

Volker
(This used to be commit bcf5c751cbe203c00814642e26440cf88f300bce)

source3/include/messages.h
source3/include/smb.h
source3/lib/util.c
source3/smbd/files.c
source3/smbd/notify.c
source3/smbd/nttrans.c
source3/smbd/open.c
source3/smbd/reply.c

index 7b948e012ee052ce226943290b454a2ba678546f..1c01ba08c2fe83a30e649cc462fd5ee35fa8613b 100644 (file)
@@ -71,6 +71,7 @@
 #define MSG_SMB_FILE_RENAME  3011
 #define MSG_SMB_INJECT_FAULT 3012
 #define MSG_SMB_BLOCKING_LOCK_CANCEL 3013
+#define MSG_SMB_NOTIFY       3014
 
 /* winbind messages */
 #define MSG_WINBIND_FINISHED     4001
index 0e06ce1d791b7fd6a4067ee80c3ab7c1ea2e095f..c4d46600288c020a4d55e4b0568b13d09951d3b7 100644 (file)
@@ -428,6 +428,16 @@ struct vfs_fsp_data {
      */
 };
 
+struct notify_change {
+       uint32_t action;
+       char *name;
+};
+
+struct notify_changes {
+       uint32_t num_changes;
+       struct notify_change *changes;
+};
+
 typedef struct files_struct {
        struct files_struct *next, *prev;
        int fnum;
@@ -470,6 +480,8 @@ typedef struct files_struct {
 
        struct vfs_fsp_data *vfs_extension;
        FAKE_FILE_HANDLE *fake_file_handle;
+
+       struct notify_changes *notify;
 } files_struct;
 
 #include "ntquotas.h"
@@ -1343,6 +1355,20 @@ struct bitmap {
 #define FILE_NOTIFY_CHANGE_SECURITY    0x100
 #define FILE_NOTIFY_CHANGE_FILE_NAME   0x200
 
+#define FILE_NOTIFY_CHANGE_NAME \
+       (FILE_NOTIFY_CHANGE_FILE_NAME|FILE_NOTIFY_CHANGE_DIR_NAME)
+
+/* change notify action results */
+#define NOTIFY_ACTION_ADDED 1
+#define NOTIFY_ACTION_REMOVED 2
+#define NOTIFY_ACTION_MODIFIED 3
+#define NOTIFY_ACTION_OLD_NAME 4
+#define NOTIFY_ACTION_NEW_NAME 5
+#define NOTIFY_ACTION_ADDED_STREAM 6
+#define NOTIFY_ACTION_REMOVED_STREAM 7
+#define NOTIFY_ACTION_MODIFIED_STREAM 8
+
+
 /* where to find the base of the SMB packet proper */
 #define smb_base(buf) (((char *)(buf))+4)
 
index 9ac0b37612399e20a250bd93fe656518081b69d9..5e2588e5b90e74c44a69d6f4678a1079e8904040 100644 (file)
@@ -2620,6 +2620,37 @@ char *parent_dirname(const char *path)
        return dirpath;
 }
 
+BOOL parent_dirname_talloc(TALLOC_CTX *mem_ctx, const char *dir,
+                          char **parent, const char **name)
+{
+       char *p;
+       ptrdiff_t len;
+       p = strrchr_m(dir, '/'); /* Find final '/', if any */
+
+       if (p == NULL) {
+               if (!(*parent = talloc_strdup(mem_ctx, "."))) {
+                       return False;
+               }
+               if (name) {
+                       *name = "";
+               }
+               return True;
+       }
+
+       len = p-dir;
+
+       if (!(*parent = TALLOC_ARRAY(mem_ctx, char, len+1))) {
+               return False;
+       }
+       memcpy(*parent, dir, len);
+       (*parent)[len] = '\0';
+
+       if (name) {
+               *name = p+1;
+       }
+       return True;
+}
 
 /*******************************************************************
  Determine if a pattern contains any Microsoft wildcard characters.
index 7069818dee4f1f117f10317608028ded587679e6..8df7a29a65d70a75e9d82d7d601c2ff2e1066eb1 100644 (file)
@@ -100,6 +100,12 @@ NTSTATUS file_new(connection_struct *conn, files_struct **result)
 
        ZERO_STRUCTP(fsp->fh);
 
+       if (!(fsp->notify = TALLOC_ZERO_P(NULL, struct notify_changes))) {
+               SAFE_FREE(fsp->fh);
+               SAFE_FREE(fsp);
+               return NT_STATUS_NO_MEMORY;
+       }
+
        fsp->fh->ref_count = 1;
        fsp->fh->fd = -1;
 
@@ -361,6 +367,35 @@ files_struct *file_find_di_next(files_struct *start_fsp)
        return NULL;
 }
 
+/****************************************************************************
+ Find the directory fsp given a device and inode with the lowest
+ file_id. First use is for notify actions.
+****************************************************************************/
+
+files_struct *file_find_dir_lowest_id(SMB_DEV_T dev, SMB_INO_T inode)
+{
+       files_struct *fsp;
+       files_struct *min_fsp = NULL;
+
+       for (fsp = Files; fsp; fsp = fsp->next) {
+               if (!fsp->is_directory
+                   || fsp->dev != dev || fsp->inode != inode) {
+                       continue;
+               }
+
+               if (min_fsp == NULL) {
+                       min_fsp = fsp;
+                       continue;
+               }
+
+               if (fsp->fh->file_id < min_fsp->fh->file_id) {
+                       min_fsp = fsp;
+               }
+       }
+
+       return min_fsp;
+}
+
 /****************************************************************************
  Find a fsp that is open for printing.
 ****************************************************************************/
@@ -439,6 +474,8 @@ void file_free(files_struct *fsp)
                fsp->fh->ref_count--;
        }
 
+       TALLOC_FREE(fsp->notify);
+
        bitmap_clear(file_bmap, fsp->fnum - FILE_HANDLE_OFFSET);
        files_used--;
 
index 4ec53567dd4aa59133a6c49b38d6f2f121e06d49..2a5d7fc5525796ff6605b46dd4a16af7faff66e8 100644 (file)
@@ -35,22 +35,75 @@ struct change_notify {
        files_struct *fsp;
        connection_struct *conn;
        uint32 flags;
+       uint32 max_param_count;
        char request_buf[smb_size];
        void *change_data;
 };
 
 static struct change_notify *change_notify_list;
 
+static BOOL notify_marshall_changes(struct notify_changes *changes,
+                                   prs_struct *ps)
+{
+       int i;
+       UNISTR uni_name;
+
+       for (i=0; i<changes->num_changes; i++) {
+               struct notify_change *c = &changes->changes[i];
+               size_t namelen;
+               uint32 u32_tmp; /* Temp arg to prs_uint32 to avoid
+                                * signed/unsigned issues */
+
+               namelen = convert_string_allocate(
+                       NULL, CH_UNIX, CH_UTF16LE, c->name, strlen(c->name)+1,
+                       &uni_name.buffer, True);
+               if ((namelen == -1) || (uni_name.buffer == NULL)) {
+                       goto fail;
+               }
+
+               namelen -= 2;   /* Dump NULL termination */
+
+               /*
+                * Offset to next entry, only if there is one
+                */
+
+               u32_tmp = (i == changes->num_changes-1) ? 0 : namelen + 12;
+               if (!prs_uint32("offset", ps, 1, &u32_tmp)) goto fail;
+
+               u32_tmp = c->action;
+               if (!prs_uint32("action", ps, 1, &u32_tmp)) goto fail;
+
+               u32_tmp = namelen;
+               if (!prs_uint32("namelen", ps, 1, &u32_tmp)) goto fail;
+
+               if (!prs_unistr("name", ps, 1, &uni_name)) goto fail;
+
+               /*
+                * Not NULL terminated, decrease by the 2 UCS2 \0 chars
+                */
+               prs_set_offset(ps, prs_offset(ps)-2);
+
+               SAFE_FREE(uni_name.buffer);
+       }
+
+       return True;
+
+ fail:
+       SAFE_FREE(uni_name.buffer);
+       return False;
+}
+
 /****************************************************************************
  Setup the common parts of the return packet and send it.
 *****************************************************************************/
 
-static void change_notify_reply_packet(char *inbuf, NTSTATUS error_code)
+static void change_notify_reply_packet(struct change_notify *notify,
+                                      NTSTATUS error_code)
 {
        char outbuf[smb_size+38];
 
        memset(outbuf, '\0', sizeof(outbuf));
-       construct_reply_common(inbuf, outbuf);
+       construct_reply_common(notify->request_buf, outbuf);
 
        ERROR_NT(error_code);
 
@@ -65,6 +118,47 @@ static void change_notify_reply_packet(char *inbuf, NTSTATUS error_code)
                exit_server_cleanly("change_notify_reply_packet: send_smb failed.");
 }
 
+static void change_notify_reply(struct change_notify *notify)
+{
+       char *outbuf = NULL;
+       prs_struct ps;
+       size_t buflen = smb_size+38+notify->max_param_count;
+
+       if (!prs_init(&ps, 0, NULL, False)
+           || !notify_marshall_changes(notify->fsp->notify, &ps)) {
+               change_notify_reply_packet(notify, NT_STATUS_NO_MEMORY);
+               goto done;
+       }
+
+       if (prs_offset(&ps) > notify->max_param_count) {
+               /*
+                * We exceed what the client is willing to accept. Send
+                * nothing.
+                */
+               change_notify_reply_packet(notify, NT_STATUS_OK);
+               goto done;
+       }
+
+       if (!(outbuf = SMB_MALLOC_ARRAY(char, buflen))) {
+               change_notify_reply_packet(notify, NT_STATUS_NO_MEMORY);
+               goto done;
+       }
+
+       memset(outbuf, '\0', sizeof(outbuf));
+       construct_reply_common(notify->request_buf, outbuf);
+
+       if (send_nt_replies(outbuf, buflen, NT_STATUS_OK, prs_data_p(&ps),
+                           prs_offset(&ps), NULL, 0) == -1) {
+               exit_server("change_notify_reply_packet: send_smb failed.");
+       }
+
+ done:
+       SAFE_FREE(outbuf);
+       prs_mem_free(&ps);
+       notify->fsp->notify->num_changes = 0;
+       TALLOC_FREE(notify->fsp->notify->changes);
+}
+
 /****************************************************************************
  Remove an entry from the list and free it, also closing any
  directory handle if necessary.
@@ -89,7 +183,7 @@ void remove_pending_change_notify_requests_by_fid(files_struct *fsp, NTSTATUS st
        for (cnbp=change_notify_list; cnbp; cnbp=next) {
                next=cnbp->next;
                if (cnbp->fsp->fnum == fsp->fnum) {
-                       change_notify_reply_packet(cnbp->request_buf,status);
+                       change_notify_reply_packet(cnbpstatus);
                        change_notify_remove(cnbp);
                }
        }
@@ -106,7 +200,7 @@ void remove_pending_change_notify_requests_by_mid(int mid)
        for (cnbp=change_notify_list; cnbp; cnbp=next) {
                next=cnbp->next;
                if(SVAL(cnbp->request_buf,smb_mid) == mid) {
-                       change_notify_reply_packet(cnbp->request_buf,NT_STATUS_CANCELLED);
+                       change_notify_reply_packet(cnbpNT_STATUS_CANCELLED);
                        change_notify_remove(cnbp);
                }
        }
@@ -128,7 +222,7 @@ void remove_pending_change_notify_requests_by_filename(files_struct *fsp, NTSTAT
                 * the filename are identical.
                 */
                if((cnbp->fsp->conn == fsp->conn) && strequal(cnbp->fsp->fsp_name,fsp->fsp_name)) {
-                       change_notify_reply_packet(cnbp->request_buf,status);
+                       change_notify_reply_packet(cnbpstatus);
                        change_notify_remove(cnbp);
                }
        }
@@ -171,9 +265,13 @@ BOOL process_pending_change_notify_queue(time_t t)
 
                vuid = (lp_security() == SEC_SHARE) ? UID_FIELD_INVALID : SVAL(cnbp->request_buf,smb_uid);
 
-               if (cnotify->check_notify(cnbp->conn, vuid, cnbp->fsp->fsp_name, cnbp->flags, cnbp->change_data, t)) {
-                       DEBUG(10,("process_pending_change_notify_queue: dir %s changed !\n", cnbp->fsp->fsp_name ));
-                       change_notify_reply_packet(cnbp->request_buf,STATUS_NOTIFY_ENUM_DIR);
+               if ((cnbp->fsp->notify->num_changes != 0)
+                   || cnotify->check_notify(cnbp->conn, vuid,
+                                            cnbp->fsp->fsp_name, cnbp->flags,
+                                            cnbp->change_data, t)) {
+                       DEBUG(10,("process_pending_change_notify_queue: dir "
+                                 "%s changed !\n", cnbp->fsp->fsp_name ));
+                       change_notify_reply(cnbp);
                        change_notify_remove(cnbp);
                }
        }
@@ -188,13 +286,14 @@ BOOL process_pending_change_notify_queue(time_t t)
  error.
 ****************************************************************************/
 
-BOOL change_notify_set(char *inbuf, files_struct *fsp, connection_struct *conn, uint32 flags)
+BOOL change_notify_set(char *inbuf, files_struct *fsp, connection_struct *conn,
+                      uint32 flags, uint32 max_param_count)
 {
        struct change_notify *cnbp;
 
        if((cnbp = SMB_MALLOC_P(struct change_notify)) == NULL) {
                DEBUG(0,("change_notify_set: malloc fail !\n" ));
-               return -1;
+               return False;
        }
 
        ZERO_STRUCTP(cnbp);
@@ -203,7 +302,9 @@ BOOL change_notify_set(char *inbuf, files_struct *fsp, connection_struct *conn,
        cnbp->fsp = fsp;
        cnbp->conn = conn;
        cnbp->flags = flags;
-       cnbp->change_data = cnotify->register_notify(conn, fsp->fsp_name, flags);
+       cnbp->max_param_count = max_param_count;
+       cnbp->change_data = cnotify->register_notify(conn, fsp->fsp_name,
+                                                    flags);
        
        if (!cnbp->change_data) {
                SAFE_FREE(cnbp);
@@ -227,6 +328,174 @@ int change_notify_fd(void)
        return -1;
 }
 
+/* notify message definition
+
+Offset  Data                   length.
+0      SMB_DEV_T dev           8
+8      SMB_INO_T inode         8
+16     uint32 action           4
+20..   name
+*/
+
+#define MSG_NOTIFY_MESSAGE_SIZE 21 /* Includes at least the '\0' terminator */
+
+struct notify_message {
+       SMB_DEV_T dev;
+       SMB_INO_T inode;
+       uint32_t action;
+       char *name;
+};
+
+static DATA_BLOB notify_message_to_buf(const struct notify_message *msg)
+{
+       DATA_BLOB result;
+       size_t len;
+
+       len = strlen(msg->name);
+
+       result = data_blob(NULL, MSG_NOTIFY_MESSAGE_SIZE + len);
+       if (!result.data) {
+               return result;
+       }
+
+       SDEV_T_VAL(result.data, 0, msg->dev);
+       SINO_T_VAL(result.data, 8, msg->inode);
+       SIVAL(result.data, 16, msg->action);
+       memcpy(result.data+20, msg->name, len+1);
+
+       return result;
+}
+
+static BOOL buf_to_notify_message(void *buf, size_t len,
+                                 struct notify_message *msg)
+{
+       if (len < MSG_NOTIFY_MESSAGE_SIZE) {
+               DEBUG(0, ("Got invalid notify message of len %d\n", len));
+               return False;
+       }
+
+       msg->dev     = DEV_T_VAL(buf, 0);
+       msg->inode   = INO_T_VAL(buf, 8);
+       msg->action  = IVAL(buf, 16);
+       msg->name    = ((char *)buf)+20;
+       return True;
+}
+
+void notify_action(connection_struct *conn, const char *parent,
+                  const char *name, uint32_t action)
+{
+       struct share_mode_lock *lck;
+       SMB_STRUCT_STAT sbuf;
+       int i;
+       struct notify_message msg;
+       DATA_BLOB blob;
+
+       struct process_id *pids;
+       int num_pids;
+
+       if (SMB_VFS_STAT(conn, parent, &sbuf) != 0) {
+               /*
+                * Not 100% critical, ignore failure
+                */
+               return;
+       }
+
+       if (!(lck = get_share_mode_lock(NULL, sbuf.st_dev, sbuf.st_ino,
+                                       NULL, NULL))) {
+               return;
+       }
+
+       msg.dev = sbuf.st_dev;
+       msg.inode = sbuf.st_ino;
+       msg.action = action;
+       msg.name = CONST_DISCARD(char *, name);
+
+       blob = notify_message_to_buf(&msg);
+       if (blob.data == NULL) {
+               DEBUG(0, ("notify_message_to_buf failed\n"));
+               return;
+       }
+
+       pids = NULL;
+       num_pids = 0;
+
+       become_root_uid_only();
+
+       for (i=0; i<lck->num_share_modes; i++) {
+               struct share_mode_entry *e = &lck->share_modes[i];
+               int j;
+               struct process_id *tmp;
+
+               for (j=0; j<num_pids; j++) {
+                       if (procid_equal(&e->pid, &pids[j])) {
+                               break;
+                       }
+               }
+
+               if (j < num_pids) {
+                       /*
+                        * Already sent to that process, skip it
+                        */
+                       continue;
+               }
+
+               message_send_pid(lck->share_modes[i].pid, MSG_SMB_NOTIFY,
+                                blob.data, blob.length, True);
+
+               if (!(tmp = TALLOC_REALLOC_ARRAY(lck, pids, struct process_id,
+                                                num_pids+1))) {
+                       DEBUG(0, ("realloc failed\n"));
+                       break;
+               }
+               pids = tmp;
+               pids[num_pids] = e->pid;
+               num_pids += 1;
+       }
+
+       unbecome_root_uid_only();
+
+       data_blob_free(&blob);
+       TALLOC_FREE(lck);
+}
+
+static void notify_message(int msgtype, struct process_id pid,
+                          void *buf, size_t len)
+{
+       struct notify_message msg;
+       files_struct *fsp;
+       struct notify_change *changes, *change;
+
+       if (!buf_to_notify_message(buf, len, &msg)) {
+               return;
+       }
+
+       DEBUG(10, ("Received notify_message for 0x%x/%.0f: %d\n",
+                  (unsigned)msg.dev, (double)msg.inode, msg.action));
+
+       if (!(fsp = file_find_dir_lowest_id(msg.dev, msg.inode))) {
+               DEBUG(10, ("notify_message: did not find fsp\n"));
+               return;
+       }
+
+       if (!(changes = TALLOC_REALLOC_ARRAY(
+                     fsp->notify, fsp->notify->changes,
+                     struct notify_change, fsp->notify->num_changes+1))) {
+               DEBUG(0, ("talloc_realloc failed\n"));
+               return;
+       }
+
+       fsp->notify->changes = changes;
+
+       change = &(fsp->notify->changes[fsp->notify->num_changes]);
+
+       if (!(change->name = talloc_strdup(changes, msg.name))) {
+               DEBUG(0, ("talloc_strdup failed\n"));
+               return;
+       }
+       change->action = msg.action;
+       fsp->notify->num_changes += 1;
+}
+
 /****************************************************************************
  Initialise the change notify subsystem.
 ****************************************************************************/
@@ -250,5 +519,7 @@ BOOL init_change_notify(void)
                return False;
        }
 
+       message_register(MSG_SMB_NOTIFY, notify_message);
+
        return True;
 }
index 6359f931c019a009aaf188bf566a84c8c17e0be1..976ecf45241435d574a50ad34149dda7889d81ab 100644 (file)
@@ -68,9 +68,8 @@ static char *nttrans_realloc(char **ptr, size_t size)
  HACK ! Always assumes smb_setup field is zero.
 ****************************************************************************/
 
-static int send_nt_replies(char *outbuf, int bufsize, NTSTATUS nt_error,
-                          char *params, int paramsize, char *pdata,
-                          int datasize)
+int send_nt_replies(char *outbuf, int bufsize, NTSTATUS nt_error,
+                   char *params, int paramsize, char *pdata, int datasize)
 {
        int data_to_send = datasize;
        int params_to_send = paramsize;
@@ -1848,7 +1847,7 @@ static int call_nt_transact_notify_change(connection_struct *conn, char *inbuf,
                return ERROR_DOS(ERRDOS,ERRbadfid);
        }
 
-       if (!change_notify_set(inbuf, fsp, conn, flags)) {
+       if (!change_notify_set(inbuf, fsp, conn, flags, max_param_count)) {
                return(UNIXERROR(ERRDOS,ERRbadfid));
        }
 
index 28170e9cd8531b4fb0ce31bdd75bb532a38b4c10..040b0543aaff8909753e3ec107c570de17e2e38a 100644 (file)
@@ -1127,7 +1127,8 @@ NTSTATUS open_file_ntcreate(connection_struct *conn,
        uint32 open_access_mask = access_mask;
        NTSTATUS status;
        int ret_flock;
-       const char *parent_dir;
+       char *parent_dir;
+       const char *newname;
 
        if (conn->printer) {
                /* 
@@ -1144,8 +1145,8 @@ NTSTATUS open_file_ntcreate(connection_struct *conn,
                return print_fsp_open(conn, fname, result);
        }
 
-       if (!(parent_dir = talloc_strdup(tmp_talloc_ctx(),
-                                        parent_dirname(fname)))) {
+       if (!parent_dirname_talloc(tmp_talloc_ctx(), fname, &parent_dir,
+                                  &newname)) {
                return NT_STATUS_NO_MEMORY;
        }
 
@@ -1878,7 +1879,8 @@ static NTSTATUS mkdir_internal(connection_struct *conn, const char *name,
 {
        int ret= -1;
        mode_t mode;
-       const char *parent_dir;
+       char *parent_dir;
+       const char *dirname;
 
        if(!CAN_WRITE(conn)) {
                DEBUG(5,("mkdir_internal: failing create on read-only share "
@@ -1890,8 +1892,8 @@ static NTSTATUS mkdir_internal(connection_struct *conn, const char *name,
                return map_nt_error_from_unix(errno);
        }
 
-       if (!(parent_dir = talloc_strdup(tmp_talloc_ctx(),
-                                        parent_dirname(name)))) {
+       if (!parent_dirname_talloc(tmp_talloc_ctx(), name, &parent_dir,
+                                  &dirname)) {
                return NT_STATUS_NO_MEMORY;
        }
 
@@ -1936,6 +1938,8 @@ static NTSTATUS mkdir_internal(connection_struct *conn, const char *name,
                change_dir_owner_to_parent(conn, parent_dir, name, psbuf);
        }
 
+       notify_action(conn, parent_dir, dirname, NOTIFY_ACTION_ADDED);
+
        return NT_STATUS_OK;
 }
 
index f6d52113108407701680354a29c0c1f175c5dc9b..f10b649bf191891fac5dfa78ca55e114bef5aaaa 100644 (file)
@@ -3860,7 +3860,7 @@ static BOOL recursive_rmdir(connection_struct *conn, char *directory)
  The internals of the rmdir code - called elsewhere.
 ****************************************************************************/
 
-BOOL rmdir_internals(connection_struct *conn, char *directory)
+BOOL rmdir_internals(connection_struct *conn, const char *directory)
 {
        BOOL ok;
        SMB_STRUCT_STAT st;
@@ -3933,10 +3933,25 @@ BOOL rmdir_internals(connection_struct *conn, char *directory)
                }
        }
 
-       if (!ok)
-               DEBUG(3,("rmdir_internals: couldn't remove directory %s : %s\n", directory,strerror(errno)));
+       if (!ok) {
+               DEBUG(3,("rmdir_internals: couldn't remove directory %s : "
+                        "%s\n", directory,strerror(errno)));
+               return False;
+       }
+
+       {
+               char *parent_dir;
+               const char *dirname;
+
+               if (parent_dirname_talloc(tmp_talloc_ctx(), directory,
+                                         &parent_dir, &dirname)) {
+                       notify_action(conn, parent_dir, dirname,
+                                     NOTIFY_ACTION_REMOVED);
+                       TALLOC_FREE(parent_dir); /* Not strictly necessary */
+               }
+       }
 
-       return ok;
+       return True;
 }
 
 /****************************************************************************