Use VFS operations for file I/O.
[tprouty/samba.git] / source / smbd / nttrans.c
index c6782cd45fe4ce072c364c7b96e312ee206aaff6..923de7a197b375fbe603f9531e14a2d94ed47ee7 100644 (file)
 */
 
 #include "includes.h"
+#include "nterr.h"
 
 extern int DEBUGLEVEL;
 extern int Protocol;
-extern int chain_fnum;
-extern connection_struct Connections[];
-extern files_struct Files[];
 extern int Client;  
-extern int oplock_sock;
 extern int smb_read_error;
 extern int global_oplock_break;
+extern int chain_size;
 extern BOOL case_sensitive;
 extern BOOL case_preserve;
 extern BOOL short_case_preserve;
 
+static void remove_pending_change_notify_requests_by_mid(int mid);
+
 static char *known_nt_pipes[] = {
   "\\LANMAN",
   "\\srvsvc",
+  "\\svcctl",
   "\\samr",
   "\\wkssvc",
   "\\NETLOGON",
@@ -44,6 +45,7 @@ static char *known_nt_pipes[] = {
   "\\ntsvcs",
   "\\lsass",
   "\\lsarpc",
+  "\\winreg",
   NULL
 };
 
@@ -231,6 +233,39 @@ static int send_nt_replies(char *outbuf, int bufsize, char *params,
   return 0;
 }
 
+/****************************************************************************
+ (Hopefully) temporary call to fix bugs in NT5.0beta2. This OS sends unicode
+ strings in NT calls AND DOESN'T SET THE UNICODE BIT !!!!!!!
+****************************************************************************/
+
+static void my_wcstombs(char *dst, uint16 *src, size_t len)
+{
+  size_t i;
+
+  for(i = 0; i < len; i++)
+    dst[i] = (char)SVAL(src,i*2);
+}
+
+static void get_filename( char *fname, char *inbuf, int data_offset, int data_len, int fname_len)
+{
+  if(data_len - fname_len > 1) {
+    /*
+     * NT 5.0 Beta 2 has kindly sent us a UNICODE string
+     * without bothering to set the unicode bit. How kind.
+     *
+     * Firstly - ensure that the data offset is aligned
+     * on a 2 byte boundary - add one if not.
+     */
+    fname_len = fname_len/2;
+    if(data_offset & 1)
+      data_offset++;
+    my_wcstombs( fname, (uint16 *)(inbuf+data_offset), fname_len);
+  } else {
+    StrnCpy(fname,inbuf+data_offset,fname_len);
+  }
+  fname[fname_len] = '\0';
+}
+
 /****************************************************************************
  Save case statics.
 ****************************************************************************/
@@ -278,422 +313,579 @@ static void restore_case_semantics(uint32 file_attributes)
 
 static int map_create_disposition( uint32 create_disposition)
 {
+  int ret;
+
   switch( create_disposition ) {
   case FILE_CREATE:
     /* create if not exist, fail if exist */
-    return 0x10;
+    ret = (FILE_CREATE_IF_NOT_EXIST|FILE_EXISTS_FAIL);
+    break;
   case FILE_SUPERSEDE:
   case FILE_OVERWRITE_IF:
     /* create if not exist, trunc if exist */
-    return 0x12;
+    ret = (FILE_CREATE_IF_NOT_EXIST|FILE_EXISTS_TRUNCATE);
+    break;
   case FILE_OPEN:
     /* fail if not exist, open if exists */
-    return 0x1;
+    ret = (FILE_FAIL_IF_NOT_EXIST|FILE_EXISTS_OPEN);
+    break;
   case FILE_OPEN_IF:
     /* create if not exist, open if exists */
-    return 0x11;
+    ret = (FILE_CREATE_IF_NOT_EXIST|FILE_EXISTS_OPEN);
+    break;
   case FILE_OVERWRITE:
     /* fail if not exist, truncate if exists */
-    return 0x2;
+    ret = (FILE_FAIL_IF_NOT_EXIST|FILE_EXISTS_TRUNCATE);
+    break;
   default:
     DEBUG(0,("map_create_disposition: Incorrect value for create_disposition = %d\n",
              create_disposition ));
     return -1;
   }
+
+  DEBUG(10,("map_create_disposition: Mapped create_disposition %lx to %x\n",
+        (unsigned long)create_disposition, ret ));
+
+  return ret;
 }
 
 /****************************************************************************
  Utility function to map share modes.
 ****************************************************************************/
 
-static int map_share_mode( uint32 desired_access, uint32 share_access, uint32 file_attributes)
+static int map_share_mode( char *fname, uint32 desired_access, uint32 share_access, uint32 file_attributes)
 {
   int smb_open_mode = -1;
 
   switch( desired_access & (FILE_READ_DATA|FILE_WRITE_DATA) ) {
   case FILE_READ_DATA:
-    smb_open_mode = 0;
+    smb_open_mode = DOS_OPEN_RDONLY;
     break;
   case FILE_WRITE_DATA:
-    smb_open_mode = 1;
+    smb_open_mode = DOS_OPEN_WRONLY;
     break;
   case FILE_READ_DATA|FILE_WRITE_DATA:
-    smb_open_mode = 2;
+    smb_open_mode = DOS_OPEN_RDWR;
     break;
   }
 
+  /*
+   * NB. For DELETE_ACCESS we should really check the
+   * directory permissions, as that is what controls
+   * delete, and for WRITE_DAC_ACCESS we should really
+   * check the ownership, as that is what controls the
+   * chmod. Note that this is *NOT* a security hole (this
+   * note is for you, Andrew) as we are not *allowing*
+   * the access at this point, the actual unlink or
+   * chown or chmod call would do this. We are just helping
+   * clients out by telling them if they have a hope
+   * of any of this succeeding. POSIX acls may still
+   * deny the real call. JRA.
+   */
+
   if (smb_open_mode == -1) {
-    if(desired_access & DELETE_ACCESS)
-      smb_open_mode = 2;
-    else if( desired_access & FILE_EXECUTE)
-      smb_open_mode = 0;
+    if(desired_access & (DELETE_ACCESS|WRITE_DAC_ACCESS|WRITE_OWNER_ACCESS|
+                              FILE_EXECUTE|FILE_READ_ATTRIBUTES|
+                              FILE_WRITE_ATTRIBUTES|READ_CONTROL_ACCESS))
+      smb_open_mode = DOS_OPEN_RDONLY;
     else {
-      DEBUG(0,("map_share_mode: Incorrect value for desired_access = %x\n",
-             desired_access));
+      DEBUG(0,("map_share_mode: Incorrect value %lx for desired_access to file %s\n",
+             (unsigned long)desired_access, fname));
       return -1;
     }
   }
 
-  /* Add in the requested share mode - ignore FILE_SHARE_DELETE for now. */
+  /*
+   * Set the special bit that means allow share delete.
+   * This is held outside the normal share mode bits at 1<<15.
+   * JRA.
+   */
+
+  if(share_access & FILE_SHARE_DELETE)
+    smb_open_mode |= ALLOW_SHARE_DELETE;
+
+  /* Add in the requested share mode. */
   switch( share_access & (FILE_SHARE_READ|FILE_SHARE_WRITE)) {
   case FILE_SHARE_READ:
-    smb_open_mode |= (DENY_WRITE<<4);
+    smb_open_mode |= SET_DENY_MODE(DENY_WRITE);
     break;
   case FILE_SHARE_WRITE:
-    smb_open_mode |= (DENY_READ<<4);
+    smb_open_mode |= SET_DENY_MODE(DENY_READ);
     break;
   case (FILE_SHARE_READ|FILE_SHARE_WRITE):
-    smb_open_mode |= (DENY_NONE<<4);
+    smb_open_mode |= SET_DENY_MODE(DENY_NONE);
     break;
   case FILE_SHARE_NONE:
-    smb_open_mode |= (DENY_ALL<<4);
+    smb_open_mode |= SET_DENY_MODE(DENY_ALL);
     break;
   }
 
   /*
-   * Handle a O_SYNC request.
+   * Handle an O_SYNC request.
    */
+
   if(file_attributes & FILE_FLAG_WRITE_THROUGH)
-    smb_open_mode |= (1<<14);
+    smb_open_mode |= FILE_SYNC_OPENMODE;
 
+  DEBUG(10,("map_share_mode: Mapped desired access %lx, share access %lx, file attributes %lx \
+to open_mode %x\n", (unsigned long)desired_access, (unsigned long)share_access,
+                    (unsigned long)file_attributes, smb_open_mode ));
   return smb_open_mode;
 }
 
+/*
+ * This is a *disgusting* hack.
+ * This is *so* bad that even I'm embarrassed (and I
+ * have no shame). Here's the deal :
+ * Until we get the correct SPOOLSS code into smbd
+ * then when we're running with NT SMB support then
+ * NT makes this call with a level of zero, and then
+ * immediately follows it with an open request to
+ * the \\SRVSVC pipe. If we allow that open to
+ * succeed then NT barfs when it cannot open the
+ * \\SPOOLSS pipe immediately after and continually
+ * whines saying "Printer name is invalid" forever
+ * after. If we cause *JUST THIS NEXT OPEN* of \\SRVSVC
+ * to fail, then NT downgrades to using the downlevel code
+ * and everything works as well as before. I hate
+ * myself for adding this code.... JRA.
+ *
+ * The HACK_FAIL_TIME define allows only a 2
+ * second window for this to occur, just in
+ * case...
+ */
+
+static BOOL fail_next_srvsvc = False;
+static time_t fail_time;
+#define HACK_FAIL_TIME 2 /* In seconds. */
+
+void fail_next_srvsvc_open(void)
+{
+  fail_next_srvsvc = True;
+  fail_time = time(NULL);
+  DEBUG(10,("fail_next_srvsvc_open: setting up timeout close of \\srvsvc pipe for print fix.\n"));
+}
+
 /****************************************************************************
  Reply to an NT create and X call on a pipe.
 ****************************************************************************/
-
-static int nt_open_pipe(char *fname, char *inbuf, char *outbuf, int *ppnum)
+static int nt_open_pipe(char *fname, connection_struct *conn,
+                       char *inbuf, char *outbuf, int *ppnum)
 {
-  int cnum = SVAL(inbuf,smb_tid);
-  int pnum = -1;
-  uint16 vuid = SVAL(inbuf, smb_uid);
-  int i;
+       pipes_struct *p = NULL;
 
-  DEBUG(4,("nt_open_pipe: Opening pipe %s.\n", fname));
+       uint16 vuid = SVAL(inbuf, smb_uid);
+       int i;
+
+       DEBUG(4,("nt_open_pipe: Opening pipe %s.\n", fname));
     
-  /* See if it is one we want to handle. */
-  for( i = 0; known_nt_pipes[i]; i++ )
-    if( strequal(fname,known_nt_pipes[i]))
-      break;
+       /* See if it is one we want to handle. */
+       for( i = 0; known_nt_pipes[i]; i++ )
+               if( strequal(fname,known_nt_pipes[i]))
+                       break;
     
-  if ( known_nt_pipes[i] == NULL )
-    return(ERROR(ERRSRV,ERRaccess));
+       /*
+        * HACK alert.... see above - JRA.
+        */
+
+       if(fail_next_srvsvc && (time(NULL) > fail_time + HACK_FAIL_TIME)) {
+               fail_next_srvsvc = False;
+               fail_time = (time_t)0;
+               DEBUG(10,("nt_open_pipe: End of timeout close of \\srvsvc pipe for print fix.\n"));
+       }
+
+       if(fail_next_srvsvc && strequal(fname, "\\srvsvc")) {
+               fail_next_srvsvc = False;
+               DEBUG(10,("nt_open_pipe: Deliberately failing open of \\srvsvc pipe for print fix.\n"));
+               return(ERROR(ERRSRV,ERRaccess));
+       }
+
+       /*
+        * End hack alert.... see above - JRA.
+        */
+
+       if ( known_nt_pipes[i] == NULL )
+               return(ERROR(ERRSRV,ERRaccess));
     
-  /* Strip \\ off the name. */
-  fname++;
+       /* Strip \\ off the name. */
+       fname++;
     
-  DEBUG(3,("nt_open_pipe: Known pipe %s opening.\n", fname));
+       DEBUG(3,("nt_open_pipe: Known pipe %s opening.\n", fname));
 
-  pnum = open_rpc_pipe_hnd(fname, cnum, vuid);
-  if (pnum < 0)
-    return(ERROR(ERRSRV,ERRnofids));
+       p = open_rpc_pipe_p(fname, conn, vuid);
+       if (!p)
+               return(ERROR(ERRSRV,ERRnofids));
 
-  *ppnum = pnum + 0x800; /* Mark file handle up into high range. */
-  return 0;
+       *ppnum = p->pnum;
+
+       return 0;
 }
 
 /****************************************************************************
  Reply to an NT create and X call.
 ****************************************************************************/
 
-int reply_ntcreate_and_X(char *inbuf,char *outbuf,int length,int bufsize)
+int reply_ntcreate_and_X(connection_struct *conn,
+                        char *inbuf,char *outbuf,int length,int bufsize)
 {  
-  pstring fname;
-  int cnum = SVAL(inbuf,smb_tid);
-  int fnum = -1;
-  uint32 flags = IVAL(inbuf,smb_ntcreate_Flags);
-  uint32 desired_access = IVAL(inbuf,smb_ntcreate_DesiredAccess);
-  uint32 file_attributes = IVAL(inbuf,smb_ntcreate_FileAttributes);
-  uint32 share_access = IVAL(inbuf,smb_ntcreate_ShareAccess);
-  uint32 create_disposition = IVAL(inbuf,smb_ntcreate_CreateDisposition);
-  uint32 fname_len = MIN(((uint32)SVAL(inbuf,smb_ntcreate_NameLength)),
-                         ((uint32)sizeof(fname)-1));
-  int smb_ofun;
-  int smb_open_mode;
-  int smb_attr = (file_attributes & SAMBA_ATTRIBUTES_MASK);
-  /* Breakout the oplock request bits so we can set the
-     reply bits separately. */
-  int oplock_request = 0;
-  int unixmode;
-  int fmode=0,mtime=0,rmode=0;
-  off_t file_len = 0;
-  struct stat sbuf;
-  int smb_action = 0;
-  BOOL bad_path = False;
-  files_struct *fsp;
-  char *p = NULL;
-  
-  /* 
-   * We need to construct the open_and_X ofun value from the
-   * NT values, as that's what our code is structured to accept.
-   */    
-
-  if((smb_ofun = map_create_disposition( create_disposition )) == -1)
-    return(ERROR(ERRDOS,ERRbadaccess));
-
-  /*
-   * Now contruct the smb_open_mode value from the desired access
-   * and the share access.
-   */
-
-  if((smb_open_mode = map_share_mode( desired_access, share_access, file_attributes)) == -1)
-    return(ERROR(ERRDOS,ERRbadaccess));
-
-  /*
-   * Get the file name.
-   */
-  StrnCpy(fname,smb_buf(inbuf),fname_len);
-  fname[fname_len] = '\0';
-
-  /* If it's an IPC, use the pipe handler. */
-  if (IS_IPC(cnum)) {
-    int ret = nt_open_pipe(fname, inbuf, outbuf, &fnum);
-    if(ret != 0)
-      return ret;
-    smb_action = FILE_WAS_OPENED;
-  } else {
-
-    /*
-     * Ordinary file or directory.
-     */
-
-    /*
-     * Check if POSIX semantics are wanted.
-     */
-
-    set_posix_case_semantics(file_attributes);
-
-    unix_convert(fname,cnum,0,&bad_path);
-    
-    fnum = find_free_file();
-    if (fnum < 0) {
-      restore_case_semantics(file_attributes);
-      return(ERROR(ERRSRV,ERRnofids));
-    }
-
-    fsp = &Files[fnum];
-    
-    if (!check_name(fname,cnum)) { 
-      if((errno == ENOENT) && bad_path) {
-        unix_ERR_class = ERRDOS;
-        unix_ERR_code = ERRbadpath;
-      }
-      fsp->reserved = False;
-
-      restore_case_semantics(file_attributes);
-
-      return(UNIXERROR(ERRDOS,ERRnoaccess));
-    } 
-  
-    unixmode = unix_mode(cnum,smb_attr | aARCH);
-    
-    oplock_request = (flags & REQUEST_OPLOCK) ? EXCLUSIVE_OPLOCK : 0;
-    oplock_request |= (flags & REQUEST_BATCH_OPLOCK) ? BATCH_OPLOCK : 0;
-
-    /* 
-     * If it's a request for a directory open, deal with it separately.
-     */
-
-    if(flags & OPEN_DIRECTORY) {
-      oplock_request = 0;
-
-      open_directory(fnum, cnum, fname, smb_ofun, unixmode, &smb_action);
-
-      restore_case_semantics(file_attributes);
-
-      if(!fsp->open) {
-        fsp->reserved = False;
-        return(UNIXERROR(ERRDOS,ERRnoaccess));
-      }
-    } else {
-
+       pstring fname;
+       uint32 flags = IVAL(inbuf,smb_ntcreate_Flags);
+       uint32 desired_access = IVAL(inbuf,smb_ntcreate_DesiredAccess);
+       uint32 file_attributes = IVAL(inbuf,smb_ntcreate_FileAttributes);
+       uint32 share_access = IVAL(inbuf,smb_ntcreate_ShareAccess);
+       uint32 create_disposition = IVAL(inbuf,smb_ntcreate_CreateDisposition);
+       uint32 create_options = IVAL(inbuf,smb_ntcreate_CreateOptions);
+       uint32 fname_len = MIN(((uint32)SVAL(inbuf,smb_ntcreate_NameLength)),
+                              ((uint32)sizeof(fname)-1));
+       uint16 root_dir_fid = (uint16)IVAL(inbuf,smb_ntcreate_RootDirectoryFid);
+       int smb_ofun;
+       int smb_open_mode;
+       int smb_attr = (file_attributes & SAMBA_ATTRIBUTES_MASK);
+       /* Breakout the oplock request bits so we can set the
+          reply bits separately. */
+       int oplock_request = 0;
+    mode_t unixmode;
+       int pnum = -1;
+       int fmode=0,rmode=0;
+       SMB_OFF_T file_len = 0;
+       SMB_STRUCT_STAT sbuf;
+       int smb_action = 0;
+       BOOL bad_path = False;
+       files_struct *fsp=NULL;
+       char *p = NULL;
+
+       /* 
+        * We need to construct the open_and_X ofun value from the
+        * NT values, as that's what our code is structured to accept.
+        */    
+       
+       if((smb_ofun = map_create_disposition( create_disposition )) == -1)
+               return(ERROR(ERRDOS,ERRbadaccess));
+
+       /*
+        * Get the file name.
+        */
+
+    if(root_dir_fid != 0) {
       /*
-       * Ordinary file case.
+       * This filename is relative to a directory fid.
        */
+      files_struct *dir_fsp = file_fsp(inbuf,smb_ntcreate_RootDirectoryFid);
+      size_t dir_name_len;
+
+      if(!dir_fsp || !dir_fsp->is_directory)
+        return(ERROR(ERRDOS,ERRbadfid));
 
       /*
-       * NB. We have a potential bug here. If we cause an oplock
-       * break to ourselves, then we could end up processing filename
-       * related SMB requests whilst we await the oplock break
-       * response. As we may have changed the filename case
-       * semantics to be POSIX-like, this could mean a filename
-       * request could fail when it should succeed. This is a
-       * rare condition, but eventually we must arrange to restore
-       * the correct case semantics before issuing an oplock break
-       * request to our client. JRA.
+       * Copy in the base directory name.
        */
 
-      open_file_shared(fnum,cnum,fname,smb_open_mode,smb_ofun,unixmode,
-                       oplock_request,&rmode,&smb_action);
+      pstrcpy( fname, dir_fsp->fsp_name );
+      dir_name_len = strlen(fname);
 
-      if (!fsp->open) { 
-        /*
-         * We cheat here. The only case we care about is a directory
-         * rename, where the NT client will attempt to open the source
-         * directory for DELETE access. Note that when the NT client
-         * does this it does *not* set the directory bit in the
-         * request packet. This is translated into a read/write open
-         * request. POSIX states that any open for write request on a directory
-         * will generate an EISDIR error, so we can catch this here and open
-         * a pseudo handle that is flagged as a directory. JRA.
-         */
-
-        if(errno == EISDIR) {
-          oplock_request = 0;
-
-          open_directory(fnum, cnum, fname, smb_ofun, unixmode, &smb_action);
-
-          if(!fsp->open) {
-            fsp->reserved = False;
-            restore_case_semantics(file_attributes);
-            return(UNIXERROR(ERRDOS,ERRnoaccess));
-          }
-        } else {
-          if((errno == ENOENT) && bad_path) {
-            unix_ERR_class = ERRDOS;
-            unix_ERR_code = ERRbadpath;
-          }
-
-          fsp->reserved = False;
-
-          restore_case_semantics(file_attributes);
+      /*
+       * Ensure it ends in a '\'.
+       */
 
-          return(UNIXERROR(ERRDOS,ERRnoaccess));
-        }
-      } 
-    }
-  
-    if(fsp->is_directory) {
-      if(sys_stat(fsp->name, &sbuf) != 0) {
-        close_directory(fnum);
-        restore_case_semantics(file_attributes);
-        return(ERROR(ERRDOS,ERRnoaccess));
+      if(fname[dir_name_len-1] != '\\' && fname[dir_name_len-1] != '/') {
+        pstrcat(fname, "\\");
+        dir_name_len++;
       }
-    } else {
-      if (fstat(fsp->f_u.fd_ptr->fd,&sbuf) != 0) {
-        close_file(fnum,False);
-        restore_case_semantics(file_attributes);
-        return(ERROR(ERRDOS,ERRnoaccess));
-      } 
-    }
-  
-    restore_case_semantics(file_attributes);
-
-    file_len = sbuf.st_size;
-    fmode = dos_mode(cnum,fname,&sbuf);
-    if(fmode == 0)
-      fmode = FILE_ATTRIBUTE_NORMAL;
-    mtime = sbuf.st_mtime;
-    if (!fsp->is_directory && (fmode & aDIR)) {
-      close_file(fnum,False);
-      return(ERROR(ERRDOS,ERRnoaccess));
-    } 
-  
-    /* 
-     * If the caller set the extended oplock request bit
-     * and we granted one (by whatever means) - set the
-     * correct bit for extended oplock reply.
-     */
-    
-    if (oplock_request && lp_fake_oplocks(SNUM(cnum)))
-      smb_action |= EXTENDED_OPLOCK_GRANTED;
-  
-    if(oplock_request && fsp->granted_oplock)
-      smb_action |= EXTENDED_OPLOCK_GRANTED;
-  }
-  set_message(outbuf,34,0,True);
 
-  p = outbuf + smb_vwv2;
+      if(fname_len + dir_name_len >= sizeof(pstring))
+        return(ERROR(ERRSRV,ERRfilespecs));
 
-  /*
-   * Currently as we don't support level II oplocks we just report
-   * exclusive & batch here.
-   */
+      get_filename(&fname[dir_name_len], inbuf, smb_buf(inbuf)-inbuf, 
+                   smb_buflen(inbuf),fname_len);
+#if 0
+      StrnCpy(&fname[dir_name_len], smb_buf(inbuf),fname_len);
+      fname[dir_name_len+fname_len] = '\0';
+#endif
 
-  SCVAL(p,0, (smb_action & EXTENDED_OPLOCK_GRANTED ? 1 : 0));
-  p++;
-  SSVAL(p,0,fnum);
-  p += 2;
-  SIVAL(p,0,smb_action);
-  p += 4;
-
-  if (IS_IPC(cnum)) {
-    /*
-     * Deal with pipe return.
-     */  
-    p += 32;
-    SIVAL(p,0,FILE_ATTRIBUTE_NORMAL); /* File Attributes. */
-    p += 20;
-    /* File type. */
-    SSVAL(p,0,FILE_TYPE_MESSAGE_MODE_PIPE);
-    /* Device state. */
-    SSVAL(p,2, 0x5FF); /* ? */
-  } else {
-    /*
-     * Deal with file return.
-     */  
-    /* Create time. */  
-    put_long_date(p,get_create_time(&sbuf,lp_fake_dir_create_times(SNUM(cnum))));
-    p += 8;
-    put_long_date(p,sbuf.st_atime); /* access time */
-    p += 8;
-    put_long_date(p,sbuf.st_mtime); /* write time */
-    p += 8;
-    put_long_date(p,sbuf.st_mtime); /* change time */
-    p += 8;
-    SIVAL(p,0,fmode); /* File Attributes. */
-    p += 12;
-#if OFF_T_IS_64_BITS
-      SIVAL(p,0, file_len & 0xFFFFFFFF);
-      SIVAL(p,4, file_len >> 32);
-#else /* OFF_T_IS_64_BITS */
-      SIVAL(p,0,file_len);
-#endif /* OFF_T_IS_64_BITS */
-    p += 12;
-    SCVAL(p,0,fsp->is_directory ? 1 : 0);
-  }
+    } else {
+      
+      get_filename(fname, inbuf, smb_buf(inbuf)-inbuf, 
+                   smb_buflen(inbuf),fname_len);
 
-  chain_fnum = fnum;
+#if 0
+         StrnCpy(fname,smb_buf(inbuf),fname_len);
+      fname[fname_len] = '\0';
+#endif
+    }
+       
+       /* If it's an IPC, use the pipe handler. */
+
+       if (IS_IPC(conn) && lp_nt_pipe_support() && lp_security() != SEC_SHARE)
+       {
+               int ret = nt_open_pipe(fname, conn, inbuf, outbuf, &pnum);
+               if(ret != 0)
+                       return ret;
+
+               /*
+                * Deal with pipe return.
+                */  
+
+               set_message(outbuf,34,0,True);
+       
+               p = outbuf + smb_vwv2;
+               p++;
+               SSVAL(p,0,pnum);
+               p += 2;
+               SIVAL(p,0,FILE_WAS_OPENED);
+               p += 4;
+               p += 32;
+               SIVAL(p,0,FILE_ATTRIBUTE_NORMAL); /* File Attributes. */
+               p += 20;
+               /* File type. */
+               SSVAL(p,0,FILE_TYPE_MESSAGE_MODE_PIPE);
+               /* Device state. */
+               SSVAL(p,2, 0x5FF); /* ? */
+
+               DEBUG(5,("reply_ntcreate_and_X: open pipe = %s\n", fname));
+
+               return chain_reply(inbuf,outbuf,length,bufsize);
+       }
 
-  DEBUG(5,("reply_ntcreate_and_X: open fnum = %d, name = %s\n",
-        fnum, fsp->name ));
+       /*
+        * Now contruct the smb_open_mode value from the filename, 
+     * desired access and the share access.
+        */
+       
+       if((smb_open_mode = map_share_mode(fname, desired_access, 
+                                          share_access, 
+                                          file_attributes)) == -1) {
+               return(ERROR(ERRDOS,ERRbadaccess));
+       }
 
-  return chain_reply(inbuf,outbuf,length,bufsize);
+       /*
+        * Ordinary file or directory.
+        */
+               
+       /*
+        * Check if POSIX semantics are wanted.
+        */
+               
+       set_posix_case_semantics(file_attributes);
+               
+       unix_convert(fname,conn,0,&bad_path,NULL);
+               
+       fsp = file_new();
+       if (!fsp) {
+               restore_case_semantics(file_attributes);
+               return(ERROR(ERRSRV,ERRnofids));
+       }
+               
+       if (!check_name(fname,conn)) { 
+               if((errno == ENOENT) && bad_path) {
+                       unix_ERR_class = ERRDOS;
+                       unix_ERR_code = ERRbadpath;
+               }
+               file_free(fsp);
+               
+               restore_case_semantics(file_attributes);
+               
+               return(UNIXERROR(ERRDOS,ERRnoaccess));
+       } 
+               
+       unixmode = unix_mode(conn,smb_attr | aARCH);
+    
+       oplock_request = (flags & REQUEST_OPLOCK) ? EXCLUSIVE_OPLOCK : 0;
+       oplock_request |= (flags & REQUEST_BATCH_OPLOCK) ? BATCH_OPLOCK : 0;
+
+       /* 
+        * If it's a request for a directory open, deal with it separately.
+        */
+
+       if(create_options & FILE_DIRECTORY_FILE) {
+               oplock_request = 0;
+               
+               open_directory(fsp, conn, fname, smb_ofun, unixmode, 
+                              &smb_action);
+                       
+               restore_case_semantics(file_attributes);
+
+               if(!fsp->open) {
+                       file_free(fsp);
+                       return(UNIXERROR(ERRDOS,ERRnoaccess));
+               }
+       } else {
+               /*
+                * Ordinary file case.
+                */
+
+               /* NB. We have a potential bug here. If we
+                * cause an oplock break to ourselves, then we
+                * could end up processing filename related
+                * SMB requests whilst we await the oplock
+                * break response. As we may have changed the
+                * filename case semantics to be POSIX-like,
+                * this could mean a filename request could
+                * fail when it should succeed. This is a rare
+                * condition, but eventually we must arrange
+                * to restore the correct case semantics
+                * before issuing an oplock break request to
+                * our client. JRA.  */
+
+               open_file_shared(fsp,conn,fname,smb_open_mode,
+                                smb_ofun,unixmode,
+                                oplock_request,&rmode,&smb_action);
+
+               if (!fsp->open) { 
+                       /* We cheat here. The only case we
+                        * care about is a directory rename,
+                        * where the NT client will attempt to
+                        * open the source directory for
+                        * DELETE access. Note that when the
+                        * NT client does this it does *not*
+                        * set the directory bit in the
+                        * request packet. This is translated
+                        * into a read/write open
+                        * request. POSIX states that any open
+                        * for write request on a directory
+                        * will generate an EISDIR error, so
+                        * we can catch this here and open a
+                        * pseudo handle that is flagged as a
+                        * directory. JRA.  */
+
+                       if(errno == EISDIR) {
+                               oplock_request = 0;
+                               
+                               open_directory(fsp, conn, fname, 
+                                              smb_ofun, unixmode, 
+                                              &smb_action);
+                               
+                               if(!fsp->open) {
+                                       file_free(fsp);
+                                       restore_case_semantics(file_attributes);
+                                       return(UNIXERROR(ERRDOS,ERRnoaccess));
+                               }
+                       } else {
+                               if((errno == ENOENT) && bad_path) {
+                                       unix_ERR_class = ERRDOS;
+                                       unix_ERR_code = ERRbadpath;
+                               }
+                               
+                               file_free(fsp);
+                               
+                               restore_case_semantics(file_attributes);
+                               
+                               return(UNIXERROR(ERRDOS,ERRnoaccess));
+                       }
+               } 
+       }
+               
+       if(fsp->is_directory) {
+               if(fsp->conn->vfs_ops.stat(dos_to_unix(fsp->fsp_name, False),
+                                          &sbuf) != 0) {
+                       close_directory(fsp);
+                       restore_case_semantics(file_attributes);
+                       return(ERROR(ERRDOS,ERRnoaccess));
+               }
+       } else {
+               if (fsp->conn->vfs_ops.fstat(fsp->fd_ptr->fd,&sbuf) 
+                   != 0) {
+                       close_file(fsp,False);
+                       restore_case_semantics(file_attributes);
+                       return(ERROR(ERRDOS,ERRnoaccess));
+               } 
+       }
+               
+       restore_case_semantics(file_attributes);
+               
+       file_len = sbuf.st_size;
+       fmode = dos_mode(conn,fname,&sbuf);
+       if(fmode == 0)
+               fmode = FILE_ATTRIBUTE_NORMAL;
+       if (!fsp->is_directory && (fmode & aDIR)) {
+               close_file(fsp,False);
+               return(ERROR(ERRDOS,ERRnoaccess));
+       } 
+       
+       /* 
+        * If the caller set the extended oplock request bit
+        * and we granted one (by whatever means) - set the
+        * correct bit for extended oplock reply.
+        */
+       
+       if (oplock_request && lp_fake_oplocks(SNUM(conn)))
+               smb_action |= EXTENDED_OPLOCK_GRANTED;
+       
+       if(oplock_request && fsp->granted_oplock)
+               smb_action |= EXTENDED_OPLOCK_GRANTED;
+       
+       set_message(outbuf,34,0,True);
+       
+       p = outbuf + smb_vwv2;
+       
+       /*
+        * Currently as we don't support level II oplocks we just report
+        * exclusive & batch here.
+        */
+       
+       SCVAL(p,0, (smb_action & EXTENDED_OPLOCK_GRANTED ? 1 : 0));
+       p++;
+       SSVAL(p,0,fsp->fnum);
+       p += 2;
+       SIVAL(p,0,smb_action);
+       p += 4;
+       
+       /* Create time. */  
+       put_long_date(p,get_create_time(&sbuf,lp_fake_dir_create_times(SNUM(conn))));
+       p += 8;
+       put_long_date(p,sbuf.st_atime); /* access time */
+       p += 8;
+       put_long_date(p,sbuf.st_mtime); /* write time */
+       p += 8;
+       put_long_date(p,sbuf.st_mtime); /* change time */
+       p += 8;
+       SIVAL(p,0,fmode); /* File Attributes. */
+       p += 4;
+       SOFF_T(p, 0, file_len);
+       p += 8;
+       SOFF_T(p,0,file_len);
+       p += 12;
+       SCVAL(p,0,fsp->is_directory ? 1 : 0);
+       
+       DEBUG(5,("reply_ntcreate_and_X: fnum = %d, open name = %s\n", fsp->fnum, fsp->fsp_name));
+
+       return chain_reply(inbuf,outbuf,length,bufsize);
 }
 
 /****************************************************************************
  Reply to a NT_TRANSACT_CREATE call (needs to process SD's).
 ****************************************************************************/
-
-static int call_nt_transact_create(char *inbuf, char *outbuf, int length, 
-                                   int bufsize, int cnum,
-                                   char **ppsetup, char **ppparams, char **ppdata)
+static int call_nt_transact_create(connection_struct *conn,
+                                  char *inbuf, char *outbuf, int length, 
+                                   int bufsize, 
+                                   char **ppsetup, char **ppparams, 
+                                  char **ppdata)
 {
   pstring fname;
-  int fnum = -1;
   char *params = *ppparams;
   uint32 flags = IVAL(params,0);
   uint32 desired_access = IVAL(params,8);
   uint32 file_attributes = IVAL(params,20);
   uint32 share_access = IVAL(params,24);
   uint32 create_disposition = IVAL(params,28);
+  uint32 create_options = IVAL(params,32);
   uint32 fname_len = MIN(((uint32)IVAL(params,44)),
                          ((uint32)sizeof(fname)-1));
+  uint16 root_dir_fid = (uint16)IVAL(params,4);
   int smb_ofun;
   int smb_open_mode;
   int smb_attr = (file_attributes & SAMBA_ATTRIBUTES_MASK);
   /* Breakout the oplock request bits so we can set the
      reply bits separately. */
   int oplock_request = 0;
-  int unixmode;
-  int fmode=0,mtime=0,rmode=0;
-  off_t file_len = 0;
-  struct stat sbuf;
+  mode_t unixmode;
+  int pnum = -1;
+  int fmode=0,rmode=0;
+  SMB_OFF_T file_len = 0;
+  SMB_STRUCT_STAT sbuf;
   int smb_action = 0;
   BOOL bad_path = False;
-  files_struct *fsp;
+  files_struct *fsp = NULL;
   char *p = NULL;
 
   /* 
@@ -705,23 +897,50 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
     return(ERROR(ERRDOS,ERRbadaccess));
 
   /*
-   * Now contruct the smb_open_mode value from the desired access
-   * and the share access.
+   * Get the file name.
    */
 
-  if((smb_open_mode = map_share_mode( desired_access, share_access, file_attributes)) == -1)
-    return(ERROR(ERRDOS,ERRbadaccess));
+  if(root_dir_fid != 0) {
+    /*
+     * This filename is relative to a directory fid.
+     */
 
-  /*
-   * Get the file name.
-   */
+    files_struct *dir_fsp = file_fsp(params,4);
+    size_t dir_name_len;
 
-  StrnCpy(fname,params+53,fname_len);
-  fname[fname_len] = '\0';
+    if(!dir_fsp || !dir_fsp->is_directory)
+      return(ERROR(ERRDOS,ERRbadfid));
+
+    /*
+     * Copy in the base directory name.
+     */
+
+    pstrcpy( fname, dir_fsp->fsp_name );
+    dir_name_len = strlen(fname);
+
+    /*
+     * Ensure it ends in a '\'.
+     */
+
+    if((fname[dir_name_len-1] != '\\') && (fname[dir_name_len-1] != '/')) {
+      pstrcat(fname, "\\");
+      dir_name_len++;
+    }
+
+    if(fname_len + dir_name_len >= sizeof(pstring))
+      return(ERROR(ERRSRV,ERRfilespecs));
+
+    StrnCpy(&fname[dir_name_len], params+53, fname_len);
+    fname[dir_name_len+fname_len] = '\0';
+
+  } else {
+    StrnCpy(fname,params+53,fname_len);
+    fname[fname_len] = '\0';
+  }
 
   /* If it's an IPC, use the pipe handler. */
-  if (IS_IPC(cnum)) {
-    int ret = nt_open_pipe(fname, inbuf, outbuf, &fnum);
+  if (IS_IPC(conn)) {
+    int ret = nt_open_pipe(fname, conn, inbuf, outbuf, &pnum);
     if(ret != 0)
       return ret;
     smb_action = FILE_WAS_OPENED;
@@ -732,38 +951,44 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
 
     set_posix_case_semantics(file_attributes);
 
-    unix_convert(fname,cnum,0,&bad_path);
+    unix_convert(fname,conn,0,&bad_path,NULL);
     
-    fnum = find_free_file();
-    if (fnum < 0) {
-      restore_case_semantics(file_attributes);
-      return(ERROR(ERRSRV,ERRnofids));
+    fsp = file_new();
+    if (!fsp) {
+           restore_case_semantics(file_attributes);
+           return(ERROR(ERRSRV,ERRnofids));
     }
 
-    fsp = &Files[fnum];
-
-    if (!check_name(fname,cnum)) { 
+    if (!check_name(fname,conn)) { 
       if((errno == ENOENT) && bad_path) {
         unix_ERR_class = ERRDOS;
         unix_ERR_code = ERRbadpath;
       }
-      fsp->reserved = False;
+      file_free(fsp);
 
       restore_case_semantics(file_attributes);
 
       return(UNIXERROR(ERRDOS,ERRnoaccess));
     } 
   
-    unixmode = unix_mode(cnum,smb_attr | aARCH);
+    unixmode = unix_mode(conn,smb_attr | aARCH);
     
     oplock_request = (flags & REQUEST_OPLOCK) ? EXCLUSIVE_OPLOCK : 0;
     oplock_request |= (flags & REQUEST_BATCH_OPLOCK) ? BATCH_OPLOCK : 0;
 
+    /*
+     * Now contruct the smb_open_mode value from the desired access
+     * and the share access.
+     */
+
+    if((smb_open_mode = map_share_mode( fname, desired_access, share_access, file_attributes)) == -1)
+      return(ERROR(ERRDOS,ERRbadaccess));
+
     /*
      * If it's a request for a directory open, deal with it separately.
      */
 
-    if(flags & OPEN_DIRECTORY) {
+    if(create_options & FILE_DIRECTORY_FILE) {
 
       oplock_request = 0;
 
@@ -773,10 +998,10 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
        * CreateDirectory() call.
        */
 
-      open_directory(fnum, cnum, fname, smb_ofun, unixmode, &smb_action);
+      open_directory(fsp, conn, fname, smb_ofun, unixmode, &smb_action);
 
       if(!fsp->open) {
-        fsp->reserved = False;
+        file_free(fsp);
         return(UNIXERROR(ERRDOS,ERRnoaccess));
       }
     } else {
@@ -785,23 +1010,23 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
        * Ordinary file case.
        */
 
-      open_file_shared(fnum,cnum,fname,smb_open_mode,smb_ofun,unixmode,
-                       oplock_request,&rmode,&smb_action);
+      open_file_shared(fsp,conn,fname,smb_open_mode,smb_ofun,
+                      unixmode,oplock_request,&rmode,&smb_action);
 
       if (!fsp->open) { 
         if((errno == ENOENT) && bad_path) {
           unix_ERR_class = ERRDOS;
           unix_ERR_code = ERRbadpath;
         }
-        fsp->reserved = False;
+        file_free(fsp);
 
         restore_case_semantics(file_attributes);
 
         return(UNIXERROR(ERRDOS,ERRnoaccess));
       } 
   
-      if (fstat(fsp->f_u.fd_ptr->fd,&sbuf) != 0) {
-        close_file(fnum,False);
+      if (fsp->conn->vfs_ops.fstat(fsp->fd_ptr->fd,&sbuf) != 0) {
+        close_file(fsp,False);
 
         restore_case_semantics(file_attributes);
 
@@ -809,13 +1034,12 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
       } 
   
       file_len = sbuf.st_size;
-      fmode = dos_mode(cnum,fname,&sbuf);
+      fmode = dos_mode(conn,fname,&sbuf);
       if(fmode == 0)
         fmode = FILE_ATTRIBUTE_NORMAL;
-      mtime = sbuf.st_mtime;
 
       if (fmode & aDIR) {
-        close_file(fnum,False);
+        close_file(fsp,False);
         restore_case_semantics(file_attributes);
         return(ERROR(ERRDOS,ERRnoaccess));
       } 
@@ -826,7 +1050,7 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
        * correct bit for extended oplock reply.
        */
     
-      if (oplock_request && lp_fake_oplocks(SNUM(cnum)))
+      if (oplock_request && lp_fake_oplocks(SNUM(conn)))
         smb_action |= EXTENDED_OPLOCK_GRANTED;
   
       if(oplock_request && fsp->granted_oplock)
@@ -844,12 +1068,16 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
   p = params;
   SCVAL(p,0, (smb_action & EXTENDED_OPLOCK_GRANTED ? 1 : 0));
   p += 2;
-  SSVAL(p,0,fnum);
+  if (IS_IPC(conn)) {
+         SSVAL(p,0,pnum);
+  } else {
+         SSVAL(p,0,fsp->fnum);
+  }
   p += 2;
   SIVAL(p,0,smb_action);
   p += 8;
 
-  if (IS_IPC(cnum)) {
+  if (IS_IPC(conn)) {
     /*
      * Deal with pipe return.
      */  
@@ -865,7 +1093,7 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
      * Deal with file return.
      */
     /* Create time. */
-    put_long_date(p,get_create_time(&sbuf,lp_fake_dir_create_times(SNUM(cnum))));
+    put_long_date(p,get_create_time(&sbuf,lp_fake_dir_create_times(SNUM(conn))));
     p += 8;
     put_long_date(p,sbuf.st_atime); /* access time */
     p += 8;
@@ -874,13 +1102,10 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
     put_long_date(p,sbuf.st_mtime); /* change time */
     p += 8;
     SIVAL(p,0,fmode); /* File Attributes. */
-    p += 12;
-#if OFF_T_IS_64_BITS
-      SIVAL(p,0, file_len & 0xFFFFFFFF);
-      SIVAL(p,4, (file_len >> 32));
-#else /* OFF_T_IS_64_BITS */
-      SIVAL(p,0,file_len);
-#endif /* OFF_T_IS_64_BITS */
+    p += 4;
+    SOFF_T(p,0,file_len);
+    p += 8;
+    SOFF_T(p,0,file_len);
   }
 
   /* Send the required number of replies */
@@ -892,54 +1117,53 @@ static int call_nt_transact_create(char *inbuf, char *outbuf, int length,
 /****************************************************************************
  Reply to a NT CANCEL request.
 ****************************************************************************/
-
-int reply_ntcancel(char *inbuf,char *outbuf,int length,int bufsize)
+int reply_ntcancel(connection_struct *conn,
+                  char *inbuf,char *outbuf,int length,int bufsize)
 {
-  /*
-   * Go through and cancel any pending change notifies.
-   * TODO: When we add blocking locks we will add cancel
-   * for them here too.
-   */
-
-  int mid = SVAL(inbuf,smb_mid);
-  remove_pending_change_notify_requests_by_mid(mid);
-
-  DEBUG(3,("reply_ntcancel: cancel called on mid = %d.\n", mid));
-
-  return(-1);
+       /*
+        * Go through and cancel any pending change notifies.
+        */
+       
+       int mid = SVAL(inbuf,smb_mid);
+       remove_pending_change_notify_requests_by_mid(mid);
+       remove_pending_lock_requests_by_mid(mid);
+       
+       DEBUG(3,("reply_ntcancel: cancel called on mid = %d.\n", mid));
+
+       return(-1);
 }
 
 /****************************************************************************
  Reply to an unsolicited SMBNTtranss - just ignore it!
 ****************************************************************************/
-
-int reply_nttranss(char *inbuf,char *outbuf,int length,int bufsize)
+int reply_nttranss(connection_struct *conn,
+                  char *inbuf,char *outbuf,int length,int bufsize)
 {
-  DEBUG(4,("Ignoring nttranss of length %d\n",length));
-  return(-1);
+       DEBUG(4,("Ignoring nttranss of length %d\n",length));
+       return(-1);
 }
 
 /****************************************************************************
  Reply to an NT transact rename command.
 ****************************************************************************/
-
-static int call_nt_transact_rename(char *inbuf, char *outbuf, int length, 
-                                   int bufsize, int cnum,
+static int call_nt_transact_rename(connection_struct *conn,
+                                  char *inbuf, char *outbuf, int length, 
+                                   int bufsize,
                                    char **ppsetup, char **ppparams, char **ppdata)
 {
   char *params = *ppparams;
   pstring new_name;
-  int fnum = SVAL(params, 0);
+  files_struct *fsp = file_fsp(params, 0);
   BOOL replace_if_exists = (SVAL(params,2) & RENAME_REPLACE_IF_EXISTS) ? True : False;
   uint32 fname_len = MIN((((uint32)IVAL(inbuf,smb_nt_TotalParameterCount)-4)),
                          ((uint32)sizeof(new_name)-1));
   int outsize = 0;
 
-  CHECK_FNUM(fnum, cnum);
+  CHECK_FSP(fsp, conn);
   StrnCpy(new_name,params+4,fname_len);
   new_name[fname_len] = '\0';
 
-  outsize = rename_internals(inbuf, outbuf, Files[fnum].name,
+  outsize = rename_internals(conn, inbuf, outbuf, fsp->fsp_name,
                              new_name, replace_if_exists);
   if(outsize == 0) {
     /*
@@ -948,7 +1172,7 @@ static int call_nt_transact_rename(char *inbuf, char *outbuf, int length,
     send_nt_replies(outbuf, bufsize, NULL, 0, NULL, 0);
 
     DEBUG(3,("nt transact rename from = %s, to = %s succeeded.\n", 
-          Files[fnum].name, new_name));
+          fsp->fsp_name, new_name));
 
     outsize = -1;
   }
@@ -965,9 +1189,11 @@ static int call_nt_transact_rename(char *inbuf, char *outbuf, int length,
 
 typedef struct {
   ubi_slNode msg_next;
-  int fnum;
-  int cnum;
+  files_struct *fsp;
+  connection_struct *conn;
   time_t next_check_time;
+  time_t modify_time; /* Info from the directory we're monitoring. */ 
+  time_t status_time; /* Info from the directory we're monitoring. */
   char request_buf[smb_size];
 } change_notify_buf;
 
@@ -980,25 +1206,43 @@ static ubi_slList change_notify_queue = { NULL, (ubi_slNodePtr)&change_notify_qu
 static void change_notify_reply_packet(char *inbuf, int error_class, uint32 error_code)
 {
   extern int Client;
-  char outbuf[smb_size];
+  char outbuf[smb_size+38];
 
+  memset(outbuf, '\0', sizeof(outbuf));
   construct_reply_common(inbuf, outbuf);
 
+  /*
+   * If we're returning a 'too much in the directory changed' we need to
+   * set this is an NT error status flags. If we don't then the (probably
+   * untested) code in the NT redirector has a bug in that it doesn't re-issue
+   * the change notify.... Ah - I *love* it when I get so deeply into this I
+   * can even determine how MS failed to test stuff and why.... :-). JRA.
+   */
+
+  if(error_class == 0) /* NT Error. */
+    SSVAL(outbuf,smb_flg2, SVAL(outbuf,smb_flg2) | FLAGS2_32_BIT_ERROR_CODES);
+
   ERROR(error_class,error_code);
+
+  /*
+   * Seems NT needs a transact command with an error code
+   * in it. This is a longer packet than a simple error.
+   */
+  set_message(outbuf,18,0,False);
+
   send_smb(Client,outbuf);
 }
 
 /****************************************************************************
  Delete entries by fnum from the change notify pending queue.
 *****************************************************************************/
-
-void remove_pending_change_notify_requests_by_fid(int fnum)
+void remove_pending_change_notify_requests_by_fid(files_struct *fsp)
 {
   change_notify_buf *cnbp = (change_notify_buf *)ubi_slFirst( &change_notify_queue );
   change_notify_buf *prev = NULL;
 
   while(cnbp != NULL) {
-    if(cnbp->fnum == fnum) {
+    if(cnbp->fsp->fnum == fsp->fnum) {
       free((char *)ubi_slRemNext( &change_notify_queue, prev));
       cnbp = (change_notify_buf *)(prev ? ubi_slNext(prev) : ubi_slFirst(&change_notify_queue));
       continue;
@@ -1013,14 +1257,14 @@ void remove_pending_change_notify_requests_by_fid(int fnum)
  Delete entries by mid from the change notify pending queue. Always send reply.
 *****************************************************************************/
 
-void remove_pending_change_notify_requests_by_mid(int mid)
+static void remove_pending_change_notify_requests_by_mid(int mid)
 {
   change_notify_buf *cnbp = (change_notify_buf *)ubi_slFirst( &change_notify_queue );
   change_notify_buf *prev = NULL;
 
   while(cnbp != NULL) {
     if(SVAL(cnbp->request_buf,smb_mid) == mid) {
-      change_notify_reply_packet(cnbp->request_buf,ERRSRV,ERRaccess);
+      change_notify_reply_packet(cnbp->request_buf,0,NT_STATUS_CANCELLED);
       free((char *)ubi_slRemNext( &change_notify_queue, prev));
       cnbp = (change_notify_buf *)(prev ? ubi_slNext(prev) : ubi_slFirst(&change_notify_queue));
       continue;
@@ -1052,14 +1296,20 @@ void process_pending_change_notify_queue(time_t t)
    */
 
   while((cnbp != NULL) && (cnbp->next_check_time <= t)) {
-    struct stat st;
-    int fnum = cnbp->fnum;
-    int cnum = cnbp->cnum;
-    files_struct *fsp = &Files[fnum];
+    SMB_STRUCT_STAT st;
+    files_struct *fsp = cnbp->fsp;
+    connection_struct *conn = cnbp->conn;
     uint16 vuid = (lp_security() == SEC_SHARE) ? UID_FIELD_INVALID : 
                   SVAL(cnbp->request_buf,smb_uid);
 
-    if(!become_user(&Connections[cnum],cnum,vuid)) {
+    /*
+     * Ensure we don't have any old chain_fsp values
+     * sitting around....
+     */
+    chain_size = 0;
+    file_chain_reset();
+
+    if(!become_user(conn,vuid)) {
       DEBUG(0,("process_pending_change_notify_queue: Unable to become user vuid=%d.\n",
             vuid ));
       /*
@@ -1071,9 +1321,8 @@ void process_pending_change_notify_queue(time_t t)
       continue;
     }
 
-    if(!become_service(cnum,True)) {
-      DEBUG(0,("process_pending_change_notify_queue: Unable to become service cnum=%d. \
-Error was %s.\n", cnum, strerror(errno) ));
+    if(!become_service(conn,True)) {
+           DEBUG(0,("process_pending_change_notify_queue: Unable to become service Error was %s.\n", strerror(errno) ));
       /*
        * Remove the entry and return an error to the client.
        */
@@ -1084,9 +1333,9 @@ Error was %s.\n", cnum, strerror(errno) ));
       continue;
     }
 
-    if(sys_stat(fsp->name, &st) < 0) {
+    if(fsp->conn->vfs_ops.stat(dos_to_unix(fsp->fsp_name, False), &st) < 0) {
       DEBUG(0,("process_pending_change_notify_queue: Unable to stat directory %s. \
-Error was %s.\n", fsp->name, strerror(errno) ));
+Error was %s.\n", fsp->fsp_name, strerror(errno) ));
       /*
        * Remove the entry and return an error to the client.
        */
@@ -1097,14 +1346,14 @@ Error was %s.\n", fsp->name, strerror(errno) ));
       continue;
     }
 
-    if(fsp->f_u.dir_ptr->modify_time != st.st_mtime ||
-       fsp->f_u.dir_ptr->status_time != st.st_ctime) {
+    if(cnbp->modify_time != st.st_mtime ||
+       cnbp->status_time != st.st_ctime) {
       /*
        * Remove the entry and return a change notify to the client.
        */
-      DEBUG(5,("process_pending_change_notify_queue: directory fnum = %d, name = %s changed\n",
-            fnum, fsp->name ));
-      change_notify_reply_packet(cnbp->request_buf,ERRDOS,ERROR_NOTIFY_ENUM_DIR);
+      DEBUG(5,("process_pending_change_notify_queue: directory name = %s changed\n",
+            fsp->fsp_name ));
+      change_notify_reply_packet(cnbp->request_buf,0,NT_STATUS_NOTIFY_ENUM_DIR);
       free((char *)ubi_slRemNext( &change_notify_queue, prev));
       cnbp = (change_notify_buf *)(prev ? ubi_slNext(prev) : ubi_slFirst(&change_notify_queue));
       unbecome_user();
@@ -1125,63 +1374,27 @@ Error was %s.\n", fsp->name, strerror(errno) ));
  Reply to a notify change - queue the request and 
  don't allow a directory to be opened.
 ****************************************************************************/
-
-static int call_nt_transact_notify_change(char *inbuf, char *outbuf, int length,
-                                          int bufsize, int cnum,
-                                          char **ppsetup, char **ppparams, char **ppdata)
+static int call_nt_transact_notify_change(connection_struct *conn,
+                                         char *inbuf, char *outbuf, int length,
+                                          int bufsize, 
+                                          char **ppsetup, 
+                                         char **ppparams, char **ppdata)
 {
   char *setup = *ppsetup;
   files_struct *fsp;
-  int fnum = -1;
   change_notify_buf *cnbp;
-  struct stat st;
+  SMB_STRUCT_STAT st;
 
-  fnum = SVAL(setup,4);
+  fsp = file_fsp(setup,4);
 
-  DEBUG(3,("call_nt_transact_notify_change: fnum = %d.\n", fnum));
+  DEBUG(3,("call_nt_transact_notify_change\n"));
 
-  if(!VALID_FNUM(fnum))
+  if(!fsp)
     return(ERROR(ERRDOS,ERRbadfid));
 
-  fsp = &Files[fnum];
-
-  if((!fsp->open) || (!fsp->is_directory) || (cnum != fsp->cnum))
+  if((!fsp->open) || (!fsp->is_directory) || (conn != fsp->conn))
     return(ERROR(ERRDOS,ERRbadfid));
 
-  if(fsp->f_u.dir_ptr == NULL) {
-
-    /*
-     * No currently stored directory info - we must
-     * generate it here.
-     */
-
-    if((fsp->f_u.dir_ptr = (dir_status_struct *)malloc(sizeof(dir_status_struct))) == NULL) {
-      DEBUG(0,("call_nt_transact_notify_change: Malloc fail !\n" ));
-      return -1;
-    }
-
-    /*
-     * Setup the current directory information in the
-     * directory entry in the files_struct. We will use
-     * this to check against when the timer expires.
-     * NB. We only do this if there is no current directory
-     * information in the directory struct - as when we start
-     * monitoring file size etc. this information will start
-     * becoming increasingly expensive to maintain, so we won't
-     * want to re-generate it for every ChangeNofity call.
-     */
-
-    if(sys_stat(fsp->name, &st) < 0) {
-      DEBUG(0,("call_nt_transact_notify_change: Unable to stat fnum = %d, name = %s. \
-Error was %s\n", fnum, fsp->name, strerror(errno) ));
-      return -1;
-    }
-    fsp->f_u.dir_ptr->modify_time = st.st_mtime;
-    fsp->f_u.dir_ptr->status_time = st.st_ctime;
-
-  }
-
   /*
    * Now queue an entry on the notify change stack. We timestamp
    * the entry we are adding so that we know when to scan next.
@@ -1195,9 +1408,23 @@ Error was %s\n", fnum, fsp->name, strerror(errno) ));
     return -1;
   }
 
+  /* 
+   * Store the current timestamp on the directory we are monitoring.
+   */
+
+  if(fsp->conn->vfs_ops.stat(dos_to_unix(fsp->fsp_name, False), &st) < 0) {
+    DEBUG(0,("call_nt_transact_notify_change: Unable to stat name = %s. \
+Error was %s\n", fsp->fsp_name, strerror(errno) ));
+    free((char *)cnbp);
+    return(UNIXERROR(ERRDOS,ERRbadfid));
+  }
   memcpy(cnbp->request_buf, inbuf, smb_size);
-  cnbp->fnum = fnum;
-  cnbp->cnum = cnum;
+  cnbp->fsp = fsp;
+  cnbp->conn = conn;
+  cnbp->modify_time = st.st_mtime;
+  cnbp->status_time = st.st_ctime;
+
   cnbp->next_check_time = time(NULL) + lp_change_notify_timeout();
 
   /*
@@ -1209,7 +1436,7 @@ Error was %s\n", fnum, fsp->name, strerror(errno) ));
   ubi_slAddTail(&change_notify_queue, cnbp);
 
   DEBUG(3,("call_nt_transact_notify_change: notify change called on directory \
-fid=%d, name = %s\n", fnum, fsp->name ));
+name = %s\n", fsp->fsp_name ));
 
   return -1;
 }
@@ -1218,12 +1445,19 @@ fid=%d, name = %s\n", fnum, fsp->name ));
  Reply to query a security descriptor - currently this is not implemented (it
  is planned to be though).
 ****************************************************************************/
-
-static int call_nt_transact_query_security_desc(char *inbuf, char *outbuf, int length, 
-                                                int bufsize, int cnum,
+static int call_nt_transact_query_security_desc(connection_struct *conn,
+                                               char *inbuf, char *outbuf, 
+                                               int length, 
+                                                int bufsize, 
                                                 char **ppsetup, char **ppparams, char **ppdata)
 {
-  DEBUG(0,("call_nt_transact_query_security_desc: Currently not implemented.\n"));
+  static BOOL logged_message = False;
+
+  if(!logged_message) {
+    DEBUG(0,("call_nt_transact_query_security_desc: Currently not implemented.\n"));
+    logged_message = True; /* Only print this once... */
+  }
+
   return(ERROR(ERRSRV,ERRnosupport));
 }
    
@@ -1231,35 +1465,46 @@ static int call_nt_transact_query_security_desc(char *inbuf, char *outbuf, int l
  Reply to set a security descriptor - currently this is not implemented (it
  is planned to be though).
 ****************************************************************************/
-
-static int call_nt_transact_set_security_desc(char *inbuf, char *outbuf, int length,
-                                              int bufsize, int cnum,
-                                              char **ppsetup, char **ppparams, char **ppdata)
+static int call_nt_transact_set_security_desc(connection_struct *conn,
+                                             char *inbuf, char *outbuf, 
+                                             int length,
+                                              int bufsize, 
+                                              char **ppsetup, 
+                                             char **ppparams, char **ppdata)
 {
-  DEBUG(0,("call_nt_transact_set_security_desc: Currently not implemented.\n"));
+  static BOOL logged_message = False;
+
+  if(!logged_message) {
+    DEBUG(0,("call_nt_transact_set_security_desc: Currently not implemented.\n"));
+    logged_message = True; /* Only print this once... */
+  }
   return(ERROR(ERRSRV,ERRnosupport));
 }
    
 /****************************************************************************
  Reply to IOCTL - not implemented - no plans.
 ****************************************************************************/
-
-static int call_nt_transact_ioctl(char *inbuf, char *outbuf, int length,
-                                  int bufsize, int cnum,
+static int call_nt_transact_ioctl(connection_struct *conn,
+                                 char *inbuf, char *outbuf, int length,
+                                  int bufsize, 
                                   char **ppsetup, char **ppparams, char **ppdata)
 {
-  DEBUG(0,("call_nt_transact_ioctl: Currently not implemented.\n"));
+  static BOOL logged_message = False;
+
+  if(!logged_message) {
+    DEBUG(0,("call_nt_transact_ioctl: Currently not implemented.\n"));
+    logged_message = True; /* Only print this once... */
+  }
   return(ERROR(ERRSRV,ERRnosupport));
 }
    
 /****************************************************************************
  Reply to a SMBNTtrans.
 ****************************************************************************/
-
-int reply_nttrans(char *inbuf,char *outbuf,int length,int bufsize)
+int reply_nttrans(connection_struct *conn,
+                 char *inbuf,char *outbuf,int length,int bufsize)
 {
   int  outsize = 0;
-  int cnum = SVAL(inbuf,smb_tid);
 #if 0 /* Not used. */
   uint16 max_setup_count = CVAL(inbuf, smb_nt_MaxSetupCount);
   uint32 max_parameter_count = IVAL(inbuf, smb_nt_MaxParameterCount);
@@ -1349,8 +1594,7 @@ due to being in oplock break state.\n" ));
     while( num_data_sofar < total_data_count || num_params_sofar < total_parameter_count) {
       BOOL ret;
 
-      ret = receive_next_smb(Client,oplock_sock,inbuf,bufsize,
-                             SMB_SECONDARY_WAIT);
+      ret = receive_next_smb(inbuf,bufsize,SMB_SECONDARY_WAIT);
 
       if((ret && (CVAL(inbuf, smb_com) != SMBnttranss)) || !ret) {
         outsize = set_message(outbuf,0,0,True);
@@ -1392,39 +1636,45 @@ due to being in oplock break state.\n" ));
   /* Now we must call the relevant NT_TRANS function */
   switch(function_code) {
     case NT_TRANSACT_CREATE:
-      outsize = call_nt_transact_create(inbuf, outbuf, length, bufsize, cnum
+      outsize = call_nt_transact_create(conn, inbuf, outbuf, length, bufsize
                                         &setup, &params, &data);
       break;
     case NT_TRANSACT_IOCTL:
-      outsize = call_nt_transact_ioctl(inbuf, outbuf, length, bufsize, cnum,
+      outsize = call_nt_transact_ioctl(conn, 
+                                      inbuf, outbuf, length, bufsize, 
                                        &setup, &params, &data);
       break;
     case NT_TRANSACT_SET_SECURITY_DESC:
-      outsize = call_nt_transact_set_security_desc(inbuf, outbuf, length, bufsize, cnum,
+      outsize = call_nt_transact_set_security_desc(conn, inbuf, outbuf, 
+                                                  length, bufsize, 
                                                    &setup, &params, &data);
       break;
     case NT_TRANSACT_NOTIFY_CHANGE:
-      outsize = call_nt_transact_notify_change(inbuf, outbuf, length, bufsize, cnum,
+      outsize = call_nt_transact_notify_change(conn, inbuf, outbuf, 
+                                              length, bufsize, 
                                                &setup, &params, &data);
       break;
     case NT_TRANSACT_RENAME:
-      outsize = call_nt_transact_rename(inbuf, outbuf, length, bufsize, cnum,
+      outsize = call_nt_transact_rename(conn, inbuf, outbuf, length, 
+                                       bufsize, 
                                         &setup, &params, &data);
       break;
+
     case NT_TRANSACT_QUERY_SECURITY_DESC:
-      outsize = call_nt_transact_query_security_desc(inbuf, outbuf, length, bufsize, cnum,
+      outsize = call_nt_transact_query_security_desc(conn, inbuf, outbuf, 
+                                                    length, bufsize, 
                                                      &setup, &params, &data);
       break;
-    default:
-      /* Error in request */
-      DEBUG(0,("reply_nttrans: Unknown request %d in nttrans call\n", function_code));
-      if(setup)
-        free(setup);
-      if(params)
-       free(params);
-      if(data)
-       free(data);
-      return (ERROR(ERRSRV,ERRerror));
+  default:
+         /* Error in request */
+         DEBUG(0,("reply_nttrans: Unknown request %d in nttrans call\n", function_code));
+         if(setup)
+                 free(setup);
+         if(params)
+                 free(params);
+         if(data)
+                 free(data);
+         return (ERROR(ERRSRV,ERRerror));
   }
 
   /* As we do not know how many data packets will need to be