Added the O_NOFOLLOW flag if follow symlinks is set off.
[ira/wip.git] / source / smbd / open.c
index 92bba76619af28e48a555043995109ec1da1f3ff..b42c1bacc34dba2c96fae3a4284eea4494a7cfa7 100644 (file)
@@ -3,6 +3,7 @@
    Version 1.9.
    file opening and share modes
    Copyright (C) Andrew Tridgell 1992-1998
+   Copyright (C) Jeremy Allison 2001
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
 
 #include "includes.h"
 
-extern int DEBUGLEVEL;
-
-extern pstring sesssetup_user;
+extern userdom_struct current_user_info;
 extern uint16 global_oplock_port;
 extern BOOL global_client_failed_oplock_break;
-extern int global_smbpid;
 
 /****************************************************************************
  fd support routines - attempt to do a dos_open.
@@ -35,17 +33,27 @@ extern int global_smbpid;
 static int fd_open(struct connection_struct *conn, char *fname, 
                   int flags, mode_t mode)
 {
-       int fd = conn->vfs_ops.open(dos_to_unix(fname,False),flags,mode);
+       int fd;
+#ifdef O_NONBLOCK
+       flags |= O_NONBLOCK;
+#endif
+
+#ifdef O_NOFOLLOW
+       if (!lp_symlinks(SNUM(conn)))
+               flags |= O_NOFOLLOW;
+#endif
+
+       fd = conn->vfs_ops.open(conn,fname,flags,mode);
 
        /* Fix for files ending in '.' */
        if((fd == -1) && (errno == ENOENT) &&
-          (strchr(fname,'.')==NULL)) {
+          (strchr_m(fname,'.')==NULL)) {
                pstrcat(fname,".");
-               fd = conn->vfs_ops.open(dos_to_unix(fname,False),flags,mode);
+               fd = conn->vfs_ops.open(conn,fname,flags,mode);
        }
 
-       DEBUG(10,("fd_open: name %s, mode = %d, fd = %d. %s\n", fname, (int)mode, fd,
-               (fd == -1) ? strerror(errno) : "" ));
+       DEBUG(10,("fd_open: name %s, flags = 0%o mode = 0%o, fd = %d. %s\n", fname,
+               flags, (int)mode, fd, (fd == -1) ? strerror(errno) : "" ));
 
        return fd;
 }
@@ -56,21 +64,9 @@ static int fd_open(struct connection_struct *conn, char *fname,
 
 int fd_close(struct connection_struct *conn, files_struct *fsp)
 {
-       int ret = 0;
-
-       /*
-        * Deal with transferring any pending fd's if there
-        * are POSIX locks outstanding.
-        */
-
-       if(!fd_close_posix_locks(conn,fsp))
+       if (fsp->fd == -1)
                return -1;
-
-       if(fsp->fd != -1)
-               ret = conn->vfs_ops.close(fsp->fd);
-
-       fsp->fd = -1;
-       return ret;
+       return fd_close_posix(conn, fsp);
 }
 
 
@@ -96,12 +92,12 @@ static void check_for_pipe(char *fname)
 ****************************************************************************/
 
 static BOOL open_file(files_struct *fsp,connection_struct *conn,
-                     char *fname1,int flags,mode_t mode)
+                     char *fname1,SMB_STRUCT_STAT *psbuf,int flags,mode_t mode)
 {
        extern struct current_user current_user;
        pstring fname;
        int accmode = (flags & O_ACCMODE);
-       SMB_STRUCT_STAT sbuf;
+       int local_flags = flags;
 
        fsp->fd = -1;
        fsp->oplock_type = NO_OPLOCK;
@@ -136,17 +132,45 @@ static BOOL open_file(files_struct *fsp,connection_struct *conn,
                }
        }
 
+       /*
+        * This little piece of insanity is inspired by the
+        * fact that an NT client can open a file for O_RDONLY,
+        * but set the create disposition to FILE_EXISTS_TRUNCATE.
+        * If the client *can* write to the file, then it expects to
+        * truncate the file, even though it is opening for readonly.
+        * Quicken uses this stupid trick in backup file creation...
+        * Thanks *greatly* to "David W. Chapman Jr." <dwcjr@inethouston.net>
+        * for helping track this one down. It didn't bite us in 2.0.x
+        * as we always opened files read-write in that release. JRA.
+        */
+
+       if ((accmode == O_RDONLY) && ((flags & O_TRUNC) == O_TRUNC))
+               local_flags = (flags & ~O_ACCMODE)|O_RDWR;
+
+       /*
+        * We can't actually truncate here as the file may be locked.
+        * open_file_shared will take care of the truncate later. JRA.
+        */
+
+       local_flags &= ~O_TRUNC;
+
        /* actually do the open */
-       fsp->fd = fd_open(conn, fname, flags, mode);
+       fsp->fd = fd_open(conn, fname, local_flags, mode);
 
        if (fsp->fd == -1)  {
-               DEBUG(3,("Error opening file %s (%s) (flags=%d)\n",
-                        fname,strerror(errno),flags));
+               DEBUG(3,("Error opening file %s (%s) (local_flags=%d) (flags=%d)\n",
+                        fname,strerror(errno),local_flags,flags));
                check_for_pipe(fname);
                return False;
        }
 
-       conn->vfs_ops.fstat(fsp->fd, &sbuf);
+       if (!VALID_STAT(*psbuf)) {
+               if (vfs_fstat(fsp,fsp->fd,psbuf) == -1) {
+                       DEBUG(0,("Error doing fstat on open file %s (%s)\n", fname,strerror(errno) ));
+                       fd_close(conn, fsp);
+                       return False;
+               }
+       }
 
        /*
         * POSIX allows read-only opens of directories. We don't
@@ -154,19 +178,17 @@ static BOOL open_file(files_struct *fsp,connection_struct *conn,
         * so catch a directory open and return an EISDIR. JRA.
         */
 
-       if(S_ISDIR(sbuf.st_mode)) {
+       if(S_ISDIR(psbuf->st_mode)) {
                fd_close(conn, fsp);
                errno = EISDIR;
                return False;
        }
 
-       fsp->mode = sbuf.st_mode;
-       fsp->inode = sbuf.st_ino;
-       fsp->dev = sbuf.st_dev;
-       GetTimeOfDay(&fsp->open_time);
+       fsp->mode = psbuf->st_mode;
+       fsp->inode = psbuf->st_ino;
+       fsp->dev = psbuf->st_dev;
        fsp->vuid = current_user.vuid;
-       fsp->smbpid = global_smbpid;
-       fsp->size = 0;
+       fsp->size = psbuf->st_size;
        fsp->pos = -1;
        fsp->can_lock = True;
        fsp->can_read = ((flags & O_WRONLY)==0);
@@ -180,19 +202,11 @@ static BOOL open_file(files_struct *fsp,connection_struct *conn,
        fsp->stat_open = False;
        fsp->directory_delete_on_close = False;
        fsp->conn = conn;
-       /*
-        * Note that the file name here is the *untranslated* name
-        * ie. it is still in the DOS codepage sent from the client.
-        * All use of this filename will pass though the sys_xxxx
-        * functions which will do the dos_to_unix translation before
-        * mapping into a UNIX filename. JRA.
-        */
        string_set(&fsp->fsp_name,fname);
-       fsp->wbmpx_ptr = NULL;      
        fsp->wcp = NULL; /* Write cache pointer. */
 
        DEBUG(2,("%s opened file %s read=%s write=%s (numopen=%d)\n",
-                *sesssetup_user ? sesssetup_user : conn->user,fsp->fsp_name,
+                *current_user_info.smb_name ? current_user_info.smb_name : conn->user,fsp->fsp_name,
                 BOOLSTR(fsp->can_read), BOOLSTR(fsp->can_write),
                 conn->num_files_open + 1));
 
@@ -209,16 +223,13 @@ static int truncate_unless_locked(struct connection_struct *conn, files_struct *
 {
        SMB_BIG_UINT mask = (SMB_BIG_UINT)-1;
 
-       if (!fsp->can_write)
-               return -1;
-
-       if (is_locked(fsp,fsp->conn,mask,0,WRITE_LOCK)){
+       if (is_locked(fsp,fsp->conn,mask,0,WRITE_LOCK,True)){
                errno = EACCES;
                unix_ERR_class = ERRDOS;
                unix_ERR_code = ERRlock;
                return -1;
        } else {
-               return conn->vfs_ops.ftruncate(fsp->fd,0); 
+               return conn->vfs_ops.ftruncate(fsp,fsp->fd,0); 
        }
 }
 
@@ -227,7 +238,7 @@ return True if the filename is one of the special executable types
 ********************************************************************/
 static BOOL is_executable(const char *fname)
 {
-       if ((fname = strrchr(fname,'.'))) {
+       if ((fname = strrchr_m(fname,'.'))) {
                if (strequal(fname,".com") ||
                    strequal(fname,".dll") ||
                    strequal(fname,".exe") ||
@@ -337,18 +348,27 @@ static int access_table(int new_deny,int old_deny,int old_mode,
 check if we can open a file with a share mode
 ****************************************************************************/
 
-static int check_share_mode( share_mode_entry *share, int deny_mode, 
+static BOOL check_share_mode(connection_struct *conn, share_mode_entry *share, int share_mode, 
                             const char *fname, BOOL fcbopen, int *flags)
 {
+       int deny_mode = GET_DENY_MODE(share_mode);
        int old_open_mode = GET_OPEN_MODE(share->share_mode);
        int old_deny_mode = GET_DENY_MODE(share->share_mode);
 
        /*
-        * Don't allow any open once the delete on close flag has been
+        * share modes = false means don't bother to check for
+        * DENY mode conflict. This is a *really* bad idea :-). JRA.
+        */
+
+       if(!lp_share_modes(SNUM(conn)))
+               return True;
+
+       /*
+        * Don't allow any opens once the delete on close flag has been
         * set.
         */
 
-       if(GET_DELETE_ON_CLOSE_FLAG(share->share_mode)) {
+       if (GET_DELETE_ON_CLOSE_FLAG(share->share_mode)) {
                DEBUG(5,("check_share_mode: Failing open on file %s as delete on close flag is set.\n",
                        fname ));
                unix_ERR_class = ERRDOS;
@@ -356,9 +376,38 @@ static int check_share_mode( share_mode_entry *share, int deny_mode,
                return False;
        }
 
+       /*
+        * If delete access was requested and the existing share mode doesn't have
+        * ALLOW_SHARE_DELETE then deny.
+        */
+
+       if (GET_DELETE_ACCESS_REQUESTED(share_mode) && !GET_ALLOW_SHARE_DELETE(share->share_mode)) {
+               DEBUG(5,("check_share_mode: Failing open on file %s as delete access requested and allow share delete not set.\n",
+                       fname ));
+               unix_ERR_class = ERRDOS;
+               unix_ERR_code = ERRbadshare;
+
+               return False;
+       }
+
+       /*
+        * The inverse of the above.
+        * If delete access was granted and the new share mode doesn't have
+        * ALLOW_SHARE_DELETE then deny.
+        */
+
+       if (GET_DELETE_ACCESS_REQUESTED(share->share_mode) && !GET_ALLOW_SHARE_DELETE(share_mode)) {
+               DEBUG(5,("check_share_mode: Failing open on file %s as delete access granted and allow share delete not requested.\n",
+                       fname ));
+               unix_ERR_class = ERRDOS;
+               unix_ERR_code = ERRbadshare;
+
+               return False;
+       }
+
        {
                int access_allowed = access_table(deny_mode,old_deny_mode,old_open_mode,
-                                                                               (share->pid == getpid()),is_executable(fname));
+                                                                               (share->pid == sys_getpid()),is_executable(fname));
 
                if ((access_allowed == AFAIL) ||
                        (!fcbopen && (access_allowed == AREAD && *flags == O_RDWR)) ||
@@ -396,124 +445,182 @@ static int open_mode_check(connection_struct *conn, const char *fname, SMB_DEV_T
                                                        SMB_INO_T inode, int share_mode, int *p_flags, int *p_oplock_request,
                                                        BOOL *p_all_current_opens_are_level_II)
 {
-  int i;
-  int num_share_modes;
-  int oplock_contention_count = 0;
-  share_mode_entry *old_shares = 0;
-  BOOL fcbopen = False;
-  int deny_mode = GET_DENY_MODE(share_mode);
-  BOOL broke_oplock;   
-
-  if(GET_OPEN_MODE(share_mode) == DOS_OPEN_FCB)
-    fcbopen = True;
-
-  num_share_modes = get_share_modes(conn, dev, inode, &old_shares);
-
-  if(num_share_modes == 0)
-    return 0;
-
-  /*
-   * Check if the share modes will give us access.
-   */
-
-  do {
-
-    broke_oplock = False;
-    *p_all_current_opens_are_level_II = True;
-
-    for(i = 0; i < num_share_modes; i++) {
-      share_mode_entry *share_entry = &old_shares[i];
-
-      /* 
-       * By observation of NetBench, oplocks are broken *before* share
-       * modes are checked. This allows a file to be closed by the client
-       * if the share mode would deny access and the client has an oplock. 
-       * Check if someone has an oplock on this file. If so we must break 
-       * it before continuing. 
-       */
-
-      if((*p_oplock_request && EXCLUSIVE_OPLOCK_TYPE(share_entry->op_type)) ||
-         (!*p_oplock_request && (share_entry->op_type != NO_OPLOCK))) {
-
-        BOOL opb_ret;
-
-        DEBUG(5,("open_mode_check: breaking oplock (%x) on file %s, \
-dev = %x, inode = %.0f\n", share_entry->op_type, fname, (unsigned int)dev, (double)inode));
-
-        /* Oplock break - unlock to request it. */
-        unlock_share_entry(conn, dev, inode);
-
-        opb_ret = request_oplock_break(share_entry, dev, inode);
-
-        /* Now relock. */
-        lock_share_entry(conn, dev, inode);
-
-        if(opb_ret == False) {
-          free((char *)old_shares);
-          DEBUG(0,("open_mode_check: FAILED when breaking oplock (%x) on file %s, \
+       int i;
+       int num_share_modes;
+       int oplock_contention_count = 0;
+       share_mode_entry *old_shares = 0;
+       BOOL fcbopen = False;
+       BOOL broke_oplock;      
+       
+       if(GET_OPEN_MODE(share_mode) == DOS_OPEN_FCB)
+               fcbopen = True;
+       
+       num_share_modes = get_share_modes(conn, dev, inode, &old_shares);
+       
+       if(num_share_modes == 0)
+               return 0;
+       
+       /*
+        * Check if the share modes will give us access.
+        */
+       
+       do {
+               share_mode_entry broken_entry;
+               
+               broke_oplock = False;
+               *p_all_current_opens_are_level_II = True;
+               
+               for(i = 0; i < num_share_modes; i++) {
+                       share_mode_entry *share_entry = &old_shares[i];
+                       
+                       /* 
+                        * By observation of NetBench, oplocks are broken *before* share
+                        * modes are checked. This allows a file to be closed by the client
+                        * if the share mode would deny access and the client has an oplock. 
+                        * Check if someone has an oplock on this file. If so we must break 
+                        * it before continuing. 
+                        */
+                       
+                       if((*p_oplock_request && EXCLUSIVE_OPLOCK_TYPE(share_entry->op_type)) ||
+                          (!*p_oplock_request && (share_entry->op_type != NO_OPLOCK))) {
+                               
+                               BOOL opb_ret;
+
+                               DEBUG(5,("open_mode_check: oplock_request = %d, breaking oplock (%x) on file %s, \
+dev = %x, inode = %.0f\n", *p_oplock_request, share_entry->op_type, fname, (unsigned int)dev, (double)inode));
+                               
+                               /* Oplock break - unlock to request it. */
+                               unlock_share_entry(conn, dev, inode);
+                               
+                               opb_ret = request_oplock_break(share_entry);
+                               
+                               /* Now relock. */
+                               lock_share_entry(conn, dev, inode);
+                               
+                               if(opb_ret == False) {
+                                       DEBUG(0,("open_mode_check: FAILED when breaking oplock (%x) on file %s, \
 dev = %x, inode = %.0f\n", old_shares[i].op_type, fname, (unsigned int)dev, (double)inode));
-          errno = EACCES;
-          unix_ERR_class = ERRDOS;
-          unix_ERR_code = ERRbadshare;
-          return -1;
-        }
-
-        broke_oplock = True;
-        *p_all_current_opens_are_level_II = False;
-        break;
-
-      } else if (!LEVEL_II_OPLOCK_TYPE(share_entry->op_type)) {
-        *p_all_current_opens_are_level_II = False;
-      }
-
-      /* someone else has a share lock on it, check to see 
-         if we can too */
-
-      if(check_share_mode(share_entry, deny_mode, fname, fcbopen, p_flags) == False) {
-        free((char *)old_shares);
-        errno = EACCES;
-        return -1;
-      }
-
-    } /* end for */
-
-    if(broke_oplock) {
-      free((char *)old_shares);
-      num_share_modes = get_share_modes(conn, dev, inode, &old_shares);
-      oplock_contention_count++;
-    }
-  } while(broke_oplock);
-
-  if(old_shares != 0)
-    free((char *)old_shares);
-
-  /*
-   * Refuse to grant an oplock in case the contention limit is
-   * reached when going through the lock list multiple times.
-   */
-
-  if(oplock_contention_count >= lp_oplock_contention_limit(SNUM(conn))) {
-    *p_oplock_request = 0;
-    DEBUG(4,("open_mode_check: oplock contention = %d. Not granting oplock.\n",
-          oplock_contention_count ));
-  }
-
-  return num_share_modes;
+                                       SAFE_FREE(old_shares);
+                                       errno = EACCES;
+                                       unix_ERR_class = ERRDOS;
+                                       unix_ERR_code = ERRbadshare;
+                                       return -1;
+                               }
+                               
+                               broke_oplock = True;
+                               broken_entry = *share_entry;
+                               break;
+                               
+                       } else if (!LEVEL_II_OPLOCK_TYPE(share_entry->op_type)) {
+                               *p_all_current_opens_are_level_II = False;
+                       }
+                       
+                       /* someone else has a share lock on it, check to see 
+                          if we can too */
+                       
+                       if(check_share_mode(conn, share_entry, share_mode, fname, fcbopen, p_flags) == False) {
+                               SAFE_FREE(old_shares);
+                               errno = EACCES;
+                               return -1;
+                       }
+                       
+               } /* end for */
+               
+               if(broke_oplock) {
+                       SAFE_FREE(old_shares);
+                       num_share_modes = get_share_modes(conn, dev, inode, &old_shares);
+                       oplock_contention_count++;
+                       
+                       /* Paranoia check that this is no longer an exlusive entry. */
+                       for(i = 0; i < num_share_modes; i++) {
+                               share_mode_entry *share_entry = &old_shares[i];
+                               
+                               if (share_modes_identical(&broken_entry, share_entry) && 
+                                   EXCLUSIVE_OPLOCK_TYPE(share_entry->op_type) ) {
+                                       
+                                       /*
+                                        * This should not happen. The target left this oplock
+                                        * as exlusive.... The process *must* be dead.... 
+                                        */
+                                       
+                                       DEBUG(0,("open_mode_check: exlusive oplock left by process %d after break ! For file %s, \
+dev = %x, inode = %.0f. Deleting it to continue...\n", (int)broken_entry.pid, fname, (unsigned int)dev, (double)inode));
+                                       
+                                       if (process_exists(broken_entry.pid)) {
+                                               DEBUG(0,("open_mode_check: Existent process %d left active oplock.\n",
+                                                        broken_entry.pid ));
+                                       }
+                                       
+                                       if (del_share_entry(dev, inode, &broken_entry, NULL) == -1) {
+                                               errno = EACCES;
+                                               unix_ERR_class = ERRDOS;
+                                               unix_ERR_code = ERRbadshare;
+                                               return -1;
+                                       }
+                                       
+                                       /*
+                                        * We must reload the share modes after deleting the 
+                                        * other process's entry.
+                                        */
+                                       
+                                       SAFE_FREE(old_shares);
+                                       num_share_modes = get_share_modes(conn, dev, inode, &old_shares);
+                                       break;
+                               }
+                       } /* end for paranoia... */
+               } /* end if broke_oplock */
+               
+       } while(broke_oplock);
+       
+       if(old_shares != 0)
+               SAFE_FREE(old_shares);
+       
+       /*
+        * Refuse to grant an oplock in case the contention limit is
+        * reached when going through the lock list multiple times.
+        */
+       
+       if(oplock_contention_count >= lp_oplock_contention_limit(SNUM(conn))) {
+               *p_oplock_request = 0;
+               DEBUG(4,("open_mode_check: oplock contention = %d. Not granting oplock.\n",
+                        oplock_contention_count ));
+       }
+       
+       return num_share_modes;
 }
 
 /****************************************************************************
- Open a file with a share mode.
+set a kernel flock on a file for NFS interoperability
+this requires a patch to Linux
 ****************************************************************************/
+static void kernel_flock(files_struct *fsp, int deny_mode)
+{
+#if HAVE_KERNEL_SHARE_MODES
+       int kernel_mode = 0;
+       if (deny_mode == DENY_READ) kernel_mode = LOCK_MAND|LOCK_WRITE;
+       else if (deny_mode == DENY_WRITE) kernel_mode = LOCK_MAND|LOCK_READ;
+       else if (deny_mode == DENY_ALL) kernel_mode = LOCK_MAND;
+       if (kernel_mode) flock(fsp->fd, kernel_mode);
+#endif
+       ;;
+}
 
-files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mode,int ofun,
-                     mode_t mode,int oplock_request, int *Access,int *action)
+
+/****************************************************************************
+ Open a file with a share mode. On output from this open we are guarenteeing
+ that 
+****************************************************************************/
+files_struct *open_file_shared(connection_struct *conn,char *fname, SMB_STRUCT_STAT *psbuf, 
+                              int share_mode,int ofun, mode_t mode,int oplock_request, 
+                              int *Access,int *action)
 {
        int flags=0;
        int flags2=0;
        int deny_mode = GET_DENY_MODE(share_mode);
        BOOL allow_share_delete = GET_ALLOW_SHARE_DELETE(share_mode);
-       SMB_STRUCT_STAT sbuf;
-       BOOL file_existed = vfs_file_exist(conn, fname, &sbuf);
+       BOOL delete_access_requested = GET_DELETE_ACCESS_REQUESTED(share_mode);
+       BOOL delete_on_close = GET_DELETE_ON_CLOSE_FLAG(share_mode);
+       BOOL file_existed = VALID_STAT(*psbuf);
        BOOL fcbopen = False;
        SMB_DEV_T dev = 0;
        SMB_INO_T inode = 0;
@@ -529,15 +636,13 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
                        ignored */
                *Access = DOS_OPEN_WRONLY;
                *action = FILE_WAS_CREATED;
-               return print_fsp_open(conn, fname);
+               return print_fsp_open(conn);
        }
 
-       fsp = file_new();
+       fsp = file_new(conn);
        if(!fsp)
                return NULL;
 
-       fsp->fd = -1;
-
        DEBUG(10,("open_file_shared: fname = %s, share_mode = %x, ofun = %x, mode = %o, oplock request = %d\n",
                fname, share_mode, ofun, (int)mode,  oplock_request ));
 
@@ -574,10 +679,10 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
                return NULL;
        }
       
-       if (GET_FILE_CREATE_DISPOSITION(ofun) == FILE_CREATE_IF_NOT_EXIST)
+       if (CAN_WRITE(conn) && (GET_FILE_CREATE_DISPOSITION(ofun) == FILE_CREATE_IF_NOT_EXIST))
                flags2 |= O_CREAT;
 
-       if (GET_FILE_OPEN_DISPOSITION(ofun) == FILE_EXISTS_TRUNCATE)
+       if (CAN_WRITE(conn) && (GET_FILE_OPEN_DISPOSITION(ofun) == FILE_EXISTS_TRUNCATE))
                flags2 |= O_TRUNC;
 
        if (GET_FILE_OPEN_DISPOSITION(ofun) == FILE_EXISTS_FAIL)
@@ -609,7 +714,7 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
 #endif /* O_SYNC */
   
        if (flags != O_RDONLY && file_existed && 
-                       (!CAN_WRITE(conn) || IS_DOS_READONLY(dos_mode(conn,fname,&sbuf)))) {
+                       (!CAN_WRITE(conn) || IS_DOS_READONLY(dos_mode(conn,fname,psbuf)))) {
                if (!fcbopen) {
                        DEBUG(5,("open_file_shared: read/write access requested for file %s on read only %s\n",
                                fname, !CAN_WRITE(conn) ? "share" : "file" ));
@@ -628,15 +733,36 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
        }
 
        if (file_existed) {
-               dev = sbuf.st_dev;
-               inode = sbuf.st_ino;
+
+               dev = psbuf->st_dev;
+               inode = psbuf->st_ino;
 
                lock_share_entry(conn, dev, inode);
 
                num_share_modes = open_mode_check(conn, fname, dev, inode, share_mode,
                                                                &flags, &oplock_request, &all_current_opens_are_level_II);
                if(num_share_modes == -1) {
+
+                       /*
+                        * This next line is a subtlety we need for MS-Access. If a file open will
+                        * fail due to share permissions and also for security (access)
+                        * reasons, we need to return the access failed error, not the
+                        * share error. This means we must attempt to open the file anyway
+                        * in order to get the UNIX access error - even if we're going to
+                        * fail the open for share reasons. This is bad, as we're burning
+                        * another fd if there are existing locks but there's nothing else
+                        * we can do. We also ensure we're not going to create or tuncate
+                        * the file as we only want an access decision at this stage. JRA.
+                        */
+                       fsp_open = open_file(fsp,conn,fname,psbuf,flags|(flags2&~(O_TRUNC|O_CREAT)),mode);
+
+                       DEBUG(4,("open_file_shared : share_mode deny - calling open_file with \
+flags=0x%X flags2=0x%X mode=0%o returned %d\n",
+                               flags,(flags2&~(O_TRUNC|O_CREAT)),(int)mode,(int)fsp_open ));
+
                        unlock_share_entry(conn, dev, inode);
+                       if (fsp_open)
+                               fd_close(conn, fsp);
                        file_free(fsp);
                        return NULL;
                }
@@ -649,10 +775,14 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
        DEBUG(4,("calling open_file with flags=0x%X flags2=0x%X mode=0%o\n",
                        flags,flags2,(int)mode));
 
-       fsp_open = open_file(fsp,conn,fname,flags|(flags2&~(O_TRUNC)),mode);
+       /*
+        * open_file strips any O_TRUNC flags itself.
+        */
+
+       fsp_open = open_file(fsp,conn,fname,psbuf,flags|flags2,mode);
 
        if (!fsp_open && (flags == O_RDWR) && (errno != ENOENT) && fcbopen) {
-               if((fsp_open = open_file(fsp,conn,fname,O_RDONLY,mode)) == True)
+               if((fsp_open = open_file(fsp,conn,fname,psbuf,O_RDONLY,mode)) == True)
                        flags = O_RDONLY;
        }
 
@@ -684,11 +814,27 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
                        return NULL;
                }
 
+               /*
+                * If there are any share modes set then the file *did*
+                * exist. Ensure we return the correct value for action.
+                */
+
+               if (num_share_modes > 0)
+                       file_existed = True;
+
                /*
                 * We exit this block with the share entry *locked*.....
                 */
        }
 
+       /* note that we ignore failure for the following. It is
+           basically a hack for NFS, and NFS will never set one of
+           these only read them. Nobody but Samba can ever set a deny
+           mode and we have already checked our more authoritative
+           locking database for permission to set this deny mode. If
+           the kernel refuses the operations then the kernel is wrong */
+       kernel_flock(fsp, deny_mode);
+
        /*
         * At this point onwards, we can guarentee that the share entry
         * is locked, whether we created the file or not, and that the
@@ -699,11 +845,16 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
         * If requested, truncate the file.
         */
 
-       if ((flags2&O_TRUNC) && (truncate_unless_locked(conn,fsp) == -1)) {
-               unlock_share_entry_fsp(fsp);
-               fd_close(conn,fsp);
-               file_free(fsp);
-               return NULL;
+       if (flags2&O_TRUNC) {
+               /*
+                * We are modifing the file after open - update the stat struct..
+                */
+               if ((truncate_unless_locked(conn,fsp) == -1) || (vfs_fstat(fsp,fsp->fd,psbuf)==-1)) {
+                       unlock_share_entry_fsp(fsp);
+                       fd_close(conn,fsp);
+                       file_free(fsp);
+                       return NULL;
+               }
        }
 
        switch (flags) {
@@ -720,7 +871,10 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
 
        fsp->share_mode = SET_DENY_MODE(deny_mode) | 
                                                SET_OPEN_MODE(open_mode) | 
-                                               SET_ALLOW_SHARE_DELETE(allow_share_delete);
+                                               SET_ALLOW_SHARE_DELETE(allow_share_delete) |
+                                               SET_DELETE_ACCESS_REQUESTED(delete_access_requested);
+
+       DEBUG(10,("open_file_shared : share_mode = %x\n", fsp->share_mode ));
 
        if (Access)
                (*Access) = open_mode;
@@ -753,6 +907,27 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
 
        set_share_mode(fsp, port, oplock_request);
 
+       if (delete_on_close) {
+               NTSTATUS result = set_delete_on_close_internal(fsp, delete_on_close);
+
+               if (NT_STATUS_V(result) !=  NT_STATUS_V(NT_STATUS_OK)) {
+                       unlock_share_entry_fsp(fsp);
+                       fd_close(conn,fsp);
+                       file_free(fsp);
+                       return NULL;
+               }
+       }
+       
+       /*
+        * Take care of inherited ACLs on created files. JRA.
+        */
+
+       if (!file_existed && (conn->vfs_ops.fchmod_acl != NULL)) {
+               int saved_errno = errno; /* We might get ENOSYS in the next call.. */
+               if (conn->vfs_ops.fchmod_acl(fsp, fsp->fd, mode) == -1 && errno == ENOSYS)
+                       errno = saved_errno; /* Ignore ENOSYS */
+       }
+               
        unlock_share_entry_fsp(fsp);
 
        conn->num_files_open++;
@@ -765,28 +940,26 @@ files_struct *open_file_shared(connection_struct *conn,char *fname,int share_mod
  with the 'stat_open' flag set 
 ****************************************************************************/
 
-files_struct *open_file_stat(connection_struct *conn,
-                  char *fname, int smb_ofun, SMB_STRUCT_STAT *pst, int *action)
+files_struct *open_file_stat(connection_struct *conn, char *fname,
+                                                       SMB_STRUCT_STAT *psbuf, int smb_ofun, int *action)
 {
        extern struct current_user current_user;
-       files_struct *fsp = file_new();
-
-       if(!fsp)
-               return NULL;
+       files_struct *fsp = NULL;
 
-       if(conn->vfs_ops.stat(dos_to_unix(fname, False), pst) < 0) {
-               DEBUG(0,("open_file_stat: unable to stat name = %s. Error was %s\n",
-                        fname, strerror(errno) ));
-               file_free(fsp);
+       if (!VALID_STAT(*psbuf)) {
+               DEBUG(0,("open_file_stat: unable to stat name = %s. Error was %s\n", fname, strerror(errno) ));
                return NULL;
        }
 
-       if(S_ISDIR(pst->st_mode)) {
+       if(S_ISDIR(psbuf->st_mode)) {
                DEBUG(0,("open_file_stat: %s is a directory !\n", fname ));
-               file_free(fsp);
                return NULL;
        }
 
+       fsp = file_new(conn);
+       if(!fsp)
+               return NULL;
+
        *action = FILE_WAS_OPENED;
        
        DEBUG(5,("open_file_stat: opening file %s as a stat entry\n", fname));
@@ -795,12 +968,11 @@ files_struct *open_file_stat(connection_struct *conn,
         * Setup the files_struct for it.
         */
        
-       fsp->fd = -1;
-       fsp->mode = 0;
-       GetTimeOfDay(&fsp->open_time);
+       fsp->mode = psbuf->st_mode;
+       fsp->inode = psbuf->st_ino;
+       fsp->dev = psbuf->st_dev;
+       fsp->size = psbuf->st_size;
        fsp->vuid = current_user.vuid;
-       fsp->smbpid = global_smbpid;
-       fsp->size = 0;
        fsp->pos = -1;
        fsp->can_lock = False;
        fsp->can_read = False;
@@ -814,40 +986,76 @@ files_struct *open_file_stat(connection_struct *conn,
        fsp->stat_open = True;
        fsp->directory_delete_on_close = False;
        fsp->conn = conn;
-       /*
-        * Note that the file name here is the *untranslated* name
-        * ie. it is still in the DOS codepage sent from the client.
-        * All use of this filename will pass though the sys_xxxx
-        * functions which will do the dos_to_unix translation before
-        * mapping into a UNIX filename. JRA.
-        */
        string_set(&fsp->fsp_name,fname);
-       fsp->wbmpx_ptr = NULL;
-    fsp->wcp = NULL; /* Write cache pointer. */
+       fsp->wcp = NULL; /* Write cache pointer. */
 
        conn->num_files_open++;
 
        return fsp;
 }
 
+/****************************************************************************
+ Open a file for for write to ensure that we can fchmod it.
+****************************************************************************/
+
+files_struct *open_file_fchmod(connection_struct *conn, char *fname, SMB_STRUCT_STAT *psbuf)
+{
+       files_struct *fsp = NULL;
+       BOOL fsp_open;
+
+       if (!VALID_STAT(*psbuf))
+               return NULL;
+
+       fsp = file_new(conn);
+       if(!fsp)
+               return NULL;
+
+       fsp_open = open_file(fsp,conn,fname,psbuf,O_WRONLY,0);
+
+       /* 
+        * This is not a user visible file open.
+        * Don't set a share mode and don't increment
+        * the conn->num_files_open.
+        */
+
+       if (!fsp_open) {
+               file_free(fsp);
+               return NULL;
+       }
+
+       return fsp;
+}
+
+/****************************************************************************
+ Close the fchmod file fd - ensure no locks are lost.
+****************************************************************************/
+
+int close_file_fchmod(files_struct *fsp)
+{
+       int ret = fd_close(fsp->conn, fsp);
+       file_free(fsp);
+       return ret;
+}
+
 /****************************************************************************
  Open a directory from an NT SMB call.
 ****************************************************************************/
 
-files_struct *open_directory(connection_struct *conn,
-                  char *fname, int smb_ofun, mode_t unixmode, int *action)
+files_struct *open_directory(connection_struct *conn, char *fname,
+                                                       SMB_STRUCT_STAT *psbuf, int share_mode, int smb_ofun, mode_t unixmode, int *action)
 {
        extern struct current_user current_user;
-       SMB_STRUCT_STAT st;
        BOOL got_stat = False;
-       files_struct *fsp = file_new();
+       files_struct *fsp = file_new(conn);
+       BOOL delete_on_close = GET_DELETE_ON_CLOSE_FLAG(share_mode);
 
        if(!fsp)
                return NULL;
 
-       if(conn->vfs_ops.stat(dos_to_unix(fname, False), &st) == 0) {
+       fsp->conn = conn; /* The vfs_fXXX() macros need this. */
+
+       if (VALID_STAT(*psbuf))
                got_stat = True;
-       }
 
        if (got_stat && (GET_FILE_OPEN_DISPOSITION(smb_ofun) == FILE_EXISTS_FAIL)) {
                file_free(fsp);
@@ -859,7 +1067,7 @@ files_struct *open_directory(connection_struct *conn,
 
                if (got_stat) {
 
-                       if(!S_ISDIR(st.st_mode)) {
+                       if(!S_ISDIR(psbuf->st_mode)) {
                                DEBUG(0,("open_directory: %s is not a directory !\n", fname ));
                                file_free(fsp);
                                errno = EACCES;
@@ -880,13 +1088,18 @@ files_struct *open_directory(connection_struct *conn,
                                return NULL;
                        }
 
-                       if(conn->vfs_ops.mkdir(dos_to_unix(fname, False), 
-                                               unix_mode(conn,aDIR, fname)) < 0) {
-                               DEBUG(0,("open_directory: unable to create %s. Error was %s\n",
+                       if(vfs_mkdir(conn,fname, unix_mode(conn,aDIR, fname)) < 0) {
+                               DEBUG(2,("open_directory: unable to create %s. Error was %s\n",
                                         fname, strerror(errno) ));
                                file_free(fsp);
                                return NULL;
                        }
+
+                       if(vfs_stat(conn,fname, psbuf) != 0) {
+                               file_free(fsp);
+                               return NULL;
+                       }
+
                        *action = FILE_WAS_CREATED;
 
                }
@@ -903,7 +1116,7 @@ files_struct *open_directory(connection_struct *conn,
                        return NULL;
                }
 
-               if(!S_ISDIR(st.st_mode)) {
+               if(!S_ISDIR(psbuf->st_mode)) {
                        DEBUG(0,("open_directory: %s is not a directory !\n", fname ));
                        file_free(fsp);
                        return NULL;
@@ -912,24 +1125,22 @@ files_struct *open_directory(connection_struct *conn,
                *action = FILE_WAS_OPENED;
        }
        
-       DEBUG(5,("open_directory: opening directory %s\n",
-                fname));
+       DEBUG(5,("open_directory: opening directory %s\n", fname));
 
        /*
         * Setup the files_struct for it.
         */
        
-       fsp->fd = -1;
-       fsp->mode = 0;
-       GetTimeOfDay(&fsp->open_time);
+       fsp->mode = psbuf->st_mode;
+       fsp->inode = psbuf->st_ino;
+       fsp->dev = psbuf->st_dev;
+       fsp->size = psbuf->st_size;
        fsp->vuid = current_user.vuid;
-       fsp->smbpid = global_smbpid;
-       fsp->size = 0;
        fsp->pos = -1;
        fsp->can_lock = True;
        fsp->can_read = False;
        fsp->can_write = False;
-       fsp->share_mode = 0;
+       fsp->share_mode = share_mode;
        fsp->print_file = False;
        fsp->modified = False;
        fsp->oplock_type = NO_OPLOCK;
@@ -937,16 +1148,16 @@ files_struct *open_directory(connection_struct *conn,
        fsp->is_directory = True;
        fsp->directory_delete_on_close = False;
        fsp->conn = conn;
-       /*
-        * Note that the file name here is the *untranslated* name
-        * ie. it is still in the DOS codepage sent from the client.
-        * All use of this filename will pass though the sys_xxxx
-        * functions which will do the dos_to_unix translation before
-        * mapping into a UNIX filename. JRA.
-        */
        string_set(&fsp->fsp_name,fname);
-       fsp->wbmpx_ptr = NULL;
 
+       if (delete_on_close) {
+               NTSTATUS result = set_delete_on_close_internal(fsp, delete_on_close);
+
+               if (NT_STATUS_V(result) !=  NT_STATUS_V(NT_STATUS_OK)) {
+                       file_free(fsp);
+                       return NULL;
+               }
+       }
        conn->num_files_open++;
 
        return fsp;
@@ -964,11 +1175,11 @@ BOOL check_file_sharing(connection_struct *conn,char *fname, BOOL rename_op)
   share_mode_entry *old_shares = 0;
   int num_share_modes;
   SMB_STRUCT_STAT sbuf;
-  pid_t pid = getpid();
+  pid_t pid = sys_getpid();
   SMB_DEV_T dev;
   SMB_INO_T inode;
 
-  if (conn->vfs_ops.stat(dos_to_unix(fname,False),&sbuf) == -1)
+  if (vfs_stat(conn,fname,&sbuf) == -1)
     return(True);
 
   dev = sbuf.st_dev;
@@ -1046,13 +1257,12 @@ dev = %x, inode = %.0f\n", share_entry->op_type, fname, (unsigned int)dev, (doub
 
             /* Oplock break.... */
             unlock_share_entry(conn, dev, inode);
-            if(request_oplock_break(share_entry, dev, inode) == False)
+            if(request_oplock_break(share_entry) == False)
             {
-              free((char *)old_shares);
-
               DEBUG(0,("check_file_sharing: FAILED when breaking oplock (%x) on file %s, \
 dev = %x, inode = %.0f\n", old_shares[i].op_type, fname, (unsigned int)dev, (double)inode));
 
+              SAFE_FREE(old_shares);
               return False;
             }
             lock_share_entry(conn, dev, inode);
@@ -1082,7 +1292,7 @@ dev = %x, inode = %.0f\n", old_shares[i].op_type, fname, (unsigned int)dev, (dou
 
       if(broke_oplock)
       {
-        free((char *)old_shares);
+        SAFE_FREE(old_shares);
         num_share_modes = get_share_modes(conn, dev, inode, &old_shares);
       }
     } while(broke_oplock);
@@ -1103,7 +1313,6 @@ dev = %x, inode = %.0f\n", old_shares[i].op_type, fname, (unsigned int)dev, (dou
 free_and_exit:
 
   unlock_share_entry(conn, dev, inode);
-  if(old_shares != NULL)
-    free((char *)old_shares);
+  SAFE_FREE(old_shares);
   return(ret);
 }