r8219: Merge the new open code from HEAD to 3.0. Haven't yet run the torture
[samba.git] / source3 / smbd / dosmode.c
index 1336e27281ba5d849d540b3fc39fb7f4b97db1b1..a2bc424b8e751734ed075146526973218ed3875d 100644 (file)
@@ -1,6 +1,5 @@
 /* 
-   Unix SMB/Netbios implementation.
-   Version 1.9.
+   Unix SMB/CIFS implementation.
    dos mode handling functions
    Copyright (C) Andrew Tridgell 1992-1998
    
 
 #include "includes.h"
 
-extern int DEBUGLEVEL;
+static int set_sparse_flag(const SMB_STRUCT_STAT * const sbuf)
+{
+#if defined (HAVE_STAT_ST_BLOCKS) && defined(STAT_ST_BLOCKSIZE)
+       if (sbuf->st_size > sbuf->st_blocks * (SMB_OFF_T)STAT_ST_BLOCKSIZE) {
+               return FILE_ATTRIBUTE_SPARSE;
+       }
+#endif
+       return 0;
+}
 
 /****************************************************************************
-  change a dos mode to a unix mode
-    base permission for files:
-         everybody gets read bit set
+ Change a dos mode to a unix mode.
+    Base permission for files:
+         if creating file and inheriting
+           apply read/write bits from parent directory.
+         else   
+           everybody gets read bit set
          dos readonly is represented in unix by removing everyone's write bit
          dos archive is represented in unix by the user's execute bit
          dos system is represented in unix by the group's execute bit
          dos hidden is represented in unix by the other's execute bit
-         Then apply create mask,
-         then add force bits.
-    base permission for directories:
+         if !inheriting {
+           Then apply create mask,
+           then add force bits.
+         }
+    Base permission for directories:
          dos directory is represented in unix by unix's dir bit and the exec bit
-         Then apply create mask,
-         then add force bits.
+         if !inheriting {
+           Then apply create mask,
+           then add force bits.
+         }
 ****************************************************************************/
-mode_t unix_mode(connection_struct *conn,int dosmode)
-{
-  mode_t result = (S_IRUSR | S_IRGRP | S_IROTH);
 
-  if ( !IS_DOS_READONLY(dosmode) )
-    result |= (S_IWUSR | S_IWGRP | S_IWOTH);
-  if (IS_DOS_DIR(dosmode)) {
-    /* We never make directories read only for the owner as under DOS a user
-       can always create a file in a read-only directory. */
-    result |= (S_IFDIR | S_IXUSR | S_IXGRP | S_IXOTH | S_IWUSR);
-    /* Apply directory mask */
-    result &= lp_dir_mode(SNUM(conn));
-    /* Add in force bits */
-    result |= lp_force_dir_mode(SNUM(conn));
-  } else { 
-    if (lp_map_archive(SNUM(conn)) && IS_DOS_ARCHIVE(dosmode))
-      result |= S_IXUSR;
-
-    if (lp_map_system(SNUM(conn)) && IS_DOS_SYSTEM(dosmode))
-      result |= S_IXGRP;
-    if (lp_map_hidden(SNUM(conn)) && IS_DOS_HIDDEN(dosmode))
-      result |= S_IXOTH;  
+mode_t unix_mode(connection_struct *conn, int dosmode, const char *fname, BOOL creating_file)
+{
+       mode_t result = (S_IRUSR | S_IRGRP | S_IROTH | S_IWUSR | S_IWGRP | S_IWOTH);
+       mode_t dir_mode = 0; /* Mode of the parent directory if inheriting. */
+
+       if (!lp_store_dos_attributes(SNUM(conn)) && IS_DOS_READONLY(dosmode)) {
+               result &= ~(S_IWUSR | S_IWGRP | S_IWOTH);
+       }
+
+       if (fname && creating_file && lp_inherit_perms(SNUM(conn))) {
+               char *dname;
+               SMB_STRUCT_STAT sbuf;
+
+               dname = parent_dirname(fname);
+               DEBUG(2,("unix_mode(%s) inheriting from %s\n",fname,dname));
+               if (SMB_VFS_STAT(conn,dname,&sbuf) != 0) {
+                       DEBUG(4,("unix_mode(%s) failed, [dir %s]: %s\n",fname,dname,strerror(errno)));
+                       return(0);      /* *** shouldn't happen! *** */
+               }
+
+               /* Save for later - but explicitly remove setuid bit for safety. */
+               dir_mode = sbuf.st_mode & ~S_ISUID;
+               DEBUG(2,("unix_mode(%s) inherit mode %o\n",fname,(int)dir_mode));
+               /* Clear "result" */
+               result = 0;
+       } 
+
+       if (IS_DOS_DIR(dosmode)) {
+               /* We never make directories read only for the owner as under DOS a user
+               can always create a file in a read-only directory. */
+               result |= (S_IFDIR | S_IWUSR);
+
+               if (dir_mode) {
+                       /* Inherit mode of parent directory. */
+                       result |= dir_mode;
+               } else {
+                       /* Provisionally add all 'x' bits */
+                       result |= (S_IXUSR | S_IXGRP | S_IXOTH);                 
+
+                       /* Apply directory mask */
+                       result &= lp_dir_mask(SNUM(conn));
+                       /* Add in force bits */
+                       result |= lp_force_dir_mode(SNUM(conn));
+               }
+       } else { 
+               if (lp_map_archive(SNUM(conn)) && IS_DOS_ARCHIVE(dosmode))
+                       result |= S_IXUSR;
+
+               if (lp_map_system(SNUM(conn)) && IS_DOS_SYSTEM(dosmode))
+                       result |= S_IXGRP;
  
-    /* Apply mode mask */
-    result &= lp_create_mode(SNUM(conn));
-    /* Add in force bits */
-    result |= lp_force_create_mode(SNUM(conn));
-  }
-  return(result);
+               if (lp_map_hidden(SNUM(conn)) && IS_DOS_HIDDEN(dosmode))
+                       result |= S_IXOTH;  
+
+               if (dir_mode) {
+                       /* Inherit 666 component of parent directory mode */
+                       result |= dir_mode & (S_IRUSR | S_IRGRP | S_IROTH | S_IWUSR | S_IWGRP | S_IWOTH);
+               } else {
+                       /* Apply mode mask */
+                       result &= lp_create_mask(SNUM(conn));
+                       /* Add in force bits */
+                       result |= lp_force_create_mode(SNUM(conn));
+               }
+       }
+
+       DEBUG(3,("unix_mode(%s) returning 0%o\n",fname,(int)result ));
+       return(result);
 }
 
-
 /****************************************************************************
-  change a unix mode to a dos mode
+ Change a unix mode to a dos mode.
 ****************************************************************************/
-int dos_mode(connection_struct *conn,char *path,SMB_STRUCT_STAT *sbuf)
-{
-  int result = 0;
 
-  DEBUG(8,("dos_mode: %s\n", path));
+uint32 dos_mode_from_sbuf(connection_struct *conn, const char *path, SMB_STRUCT_STAT *sbuf)
+{
+       int result = 0;
+
+       if (lp_acl_check_permissions(SNUM(conn))) {
+               if (!can_write_to_file(conn, path, sbuf)) {
+                       result |= aRONLY;
+               }
+       } else if ((sbuf->st_mode & S_IWUSR) == 0) {
+               result |= aRONLY;
+       }
+
+       if (MAP_ARCHIVE(conn) && ((sbuf->st_mode & S_IXUSR) != 0))
+               result |= aARCH;
+
+       if (MAP_SYSTEM(conn) && ((sbuf->st_mode & S_IXGRP) != 0))
+               result |= aSYSTEM;
+       
+       if (MAP_HIDDEN(conn) && ((sbuf->st_mode & S_IXOTH) != 0))
+               result |= aHIDDEN;   
+  
+       if (S_ISDIR(sbuf->st_mode))
+               result = aDIR | (result & aRONLY);
 
-  if ((sbuf->st_mode & S_IWUSR) == 0)
-      result |= aRONLY;
+       result |= set_sparse_flag(sbuf);
+#ifdef S_ISLNK
+#if LINKS_READ_ONLY
+       if (S_ISLNK(sbuf->st_mode) && S_ISDIR(sbuf->st_mode))
+               result |= aRONLY;
+#endif
+#endif
 
-  if (MAP_ARCHIVE(conn) && ((sbuf->st_mode & S_IXUSR) != 0))
-    result |= aARCH;
+       DEBUG(8,("dos_mode_from_sbuf returning "));
 
-  if (MAP_SYSTEM(conn) && ((sbuf->st_mode & S_IXGRP) != 0))
-    result |= aSYSTEM;
+       if (result & aHIDDEN) DEBUG(8, ("h"));
+       if (result & aRONLY ) DEBUG(8, ("r"));
+       if (result & aSYSTEM) DEBUG(8, ("s"));
+       if (result & aDIR   ) DEBUG(8, ("d"));
+       if (result & aARCH  ) DEBUG(8, ("a"));
+       
+       DEBUG(8,("\n"));
+       return result;
+}
 
-  if (MAP_HIDDEN(conn) && ((sbuf->st_mode & S_IXOTH) != 0))
-    result |= aHIDDEN;   
-  
-  if (S_ISDIR(sbuf->st_mode))
-    result = aDIR | (result & aRONLY);
+/****************************************************************************
+ Get DOS attributes from an EA.
+****************************************************************************/
 
-#ifdef S_ISLNK
-#if LINKS_READ_ONLY
-  if (S_ISLNK(sbuf->st_mode) && S_ISDIR(sbuf->st_mode))
-    result |= aRONLY;
+static BOOL get_ea_dos_attribute(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf, uint32 *pattr)
+{
+       ssize_t sizeret;
+       fstring attrstr;
+       unsigned int dosattr;
+
+       if (!lp_store_dos_attributes(SNUM(conn))) {
+               return False;
+       }
+
+       *pattr = 0;
+
+       sizeret = SMB_VFS_GETXATTR(conn, path, SAMBA_XATTR_DOS_ATTRIB, attrstr, sizeof(attrstr));
+       if (sizeret == -1) {
+#if defined(ENOTSUP) && defined(ENOATTR)
+               if ((errno != ENOTSUP) && (errno != ENOATTR) && (errno != EACCES)) {
+                       DEBUG(1,("get_ea_dos_attributes: Cannot get attribute from EA on file %s: Error = %s\n",
+                               path, strerror(errno) ));
+                       set_store_dos_attributes(SNUM(conn), False);
+               }
 #endif
+               return False;
+       }
+       /* Null terminate string. */
+       attrstr[sizeret] = 0;
+       DEBUG(10,("get_ea_dos_attribute: %s attrstr = %s\n", path, attrstr));
+
+       if (sizeret < 2 || attrstr[0] != '0' || attrstr[1] != 'x' ||
+                       sscanf(attrstr, "%x", &dosattr) != 1) {
+               DEBUG(1,("get_ea_dos_attributes: Badly formed DOSATTRIB on file %s - %s\n", path, attrstr));
+                return False;
+        }
+
+       if (S_ISDIR(sbuf->st_mode)) {
+               dosattr |= aDIR;
+       }
+       *pattr = (uint32)(dosattr & SAMBA_ATTRIBUTES_MASK);
+
+       DEBUG(8,("get_ea_dos_attribute returning (0x%x)", dosattr));
+
+       if (dosattr & aHIDDEN) DEBUG(8, ("h"));
+       if (dosattr & aRONLY ) DEBUG(8, ("r"));
+       if (dosattr & aSYSTEM) DEBUG(8, ("s"));
+       if (dosattr & aDIR   ) DEBUG(8, ("d"));
+       if (dosattr & aARCH  ) DEBUG(8, ("a"));
+       
+       DEBUG(8,("\n"));
+
+       return True;
+}
+
+/****************************************************************************
+ Set DOS attributes in an EA.
+****************************************************************************/
+
+static BOOL set_ea_dos_attribute(connection_struct *conn, const char *path, SMB_STRUCT_STAT *sbuf, uint32 dosmode)
+{
+       fstring attrstr;
+       files_struct *fsp = NULL;
+       BOOL ret = False;
+
+       if (!lp_store_dos_attributes(SNUM(conn))) {
+               return False;
+       }
+
+       snprintf(attrstr, sizeof(attrstr)-1, "0x%x", dosmode & SAMBA_ATTRIBUTES_MASK);
+       if (SMB_VFS_SETXATTR(conn, path, SAMBA_XATTR_DOS_ATTRIB, attrstr, strlen(attrstr), 0) == -1) {
+               if((errno != EPERM) && (errno != EACCES)) {
+                       if (errno == ENOSYS
+#if defined(ENOTSUP)
+                               || errno == ENOTSUP) {
+#else
+                               ) {
 #endif
+                               set_store_dos_attributes(SNUM(conn), False);
+                       }
+                       return False;
+               }
+
+               /* We want DOS semantics, ie allow non owner with write permission to change the
+                       bits on a file. Just like file_utime below.
+               */
+
+               /* Check if we have write access. */
+               if(!CAN_WRITE(conn) || !lp_dos_filemode(SNUM(conn)))
+                       return False;
+
+               /*
+                * We need to open the file with write access whilst
+                * still in our current user context. This ensures we
+                * are not violating security in doing the setxattr.
+                */
+
+               fsp = open_file_fchmod(conn,path,sbuf);
+               if (!fsp)
+                       return ret;
+               become_root();
+               if (SMB_VFS_SETXATTR(conn, path, SAMBA_XATTR_DOS_ATTRIB, attrstr, strlen(attrstr), 0) == 0) {
+                       ret = True;
+               }
+               unbecome_root();
+               close_file_fchmod(fsp);
+               return ret;
+       }
+       DEBUG(10,("set_ea_dos_attribute: set EA %s on file %s\n", attrstr, path));
+       return True;
+}
+
+/****************************************************************************
+ Change a unix mode to a dos mode.
+****************************************************************************/
 
-  /* hide files with a name starting with a . */
-  if (lp_hide_dot_files(SNUM(conn)))
-    {
-      char *p = strrchr(path,'/');
-      if (p)
-       p++;
-      else
-       p = path;
-      
-      if (p[0] == '.' && p[1] != '.' && p[1] != 0)
-       result |= aHIDDEN;
-    }
-
-  /* Optimization : Only call is_hidden_path if it's not already
-     hidden. */
-  if (!(result & aHIDDEN) && IS_HIDDEN_PATH(conn,path))
-  {
-    result |= aHIDDEN;
-  }
-
-  DEBUG(8,("dos_mode returning "));
-
-  if (result & aHIDDEN) DEBUG(8, ("h"));
-  if (result & aRONLY ) DEBUG(8, ("r"));
-  if (result & aSYSTEM) DEBUG(8, ("s"));
-  if (result & aDIR   ) DEBUG(8, ("d"));
-  if (result & aARCH  ) DEBUG(8, ("a"));
-
-  DEBUG(8,("\n"));
-
-  return(result);
+uint32 dos_mode(connection_struct *conn, const char *path,SMB_STRUCT_STAT *sbuf)
+{
+       uint32 result = 0;
+
+       DEBUG(8,("dos_mode: %s\n", path));
+
+       if (!VALID_STAT(*sbuf)) {
+               return 0;
+       }
+
+       /* Get the DOS attributes from an EA by preference. */
+       if (get_ea_dos_attribute(conn, path, sbuf, &result)) {
+               result |= set_sparse_flag(sbuf);
+               return result;
+       }
+
+       result = dos_mode_from_sbuf(conn, path, sbuf);
+
+       /* Now do any modifications that depend on the path name. */
+       /* hide files with a name starting with a . */
+       if (lp_hide_dot_files(SNUM(conn))) {
+               const char *p = strrchr_m(path,'/');
+               if (p)
+                       p++;
+               else
+                       p = path;
+               
+               if (p[0] == '.' && p[1] != '.' && p[1] != 0)
+                       result |= aHIDDEN;
+       }
+       
+       /* Optimization : Only call is_hidden_path if it's not already
+          hidden. */
+       if (!(result & aHIDDEN) && IS_HIDDEN_PATH(conn,path)) {
+               result |= aHIDDEN;
+       }
+
+       DEBUG(8,("dos_mode returning "));
+
+       if (result & aHIDDEN) DEBUG(8, ("h"));
+       if (result & aRONLY ) DEBUG(8, ("r"));
+       if (result & aSYSTEM) DEBUG(8, ("s"));
+       if (result & aDIR   ) DEBUG(8, ("d"));
+       if (result & aARCH  ) DEBUG(8, ("a"));
+       
+       DEBUG(8,("\n"));
+
+       return(result);
 }
 
 /*******************************************************************
-chmod a file - but preserve some bits
+ chmod a file - but preserve some bits.
 ********************************************************************/
-int file_chmod(connection_struct *conn,char *fname,int dosmode,SMB_STRUCT_STAT *st)
+
+int file_set_dosmode(connection_struct *conn, const char *fname, uint32 dosmode, SMB_STRUCT_STAT *st, BOOL creating_file)
 {
-  SMB_STRUCT_STAT st1;
-  int mask=0;
-  int tmp;
-  int unixmode;
+       SMB_STRUCT_STAT st1;
+       int mask=0;
+       mode_t tmp;
+       mode_t unixmode;
+       int ret = -1;
+
+       DEBUG(10,("file_set_dosmode: setting dos mode 0x%x on file %s\n", dosmode, fname));
+       if (!st || (st && !VALID_STAT(*st))) {
+               st = &st1;
+               if (SMB_VFS_STAT(conn,fname,st))
+                       return(-1);
+       }
 
-  if (!st) {
-    st = &st1;
-    if (dos_stat(fname,st)) return(-1);
-  }
+       get_acl_group_bits(conn, fname, &st->st_mode);
 
-  if (S_ISDIR(st->st_mode)) dosmode |= aDIR;
+       if (S_ISDIR(st->st_mode))
+               dosmode |= aDIR;
+       else
+               dosmode &= ~aDIR;
 
-  if (dos_mode(conn,fname,st) == dosmode) return(0);
+       if (dos_mode(conn,fname,st) == dosmode)
+               return(0);
 
-  unixmode = unix_mode(conn,dosmode);
+       /* Store the DOS attributes in an EA by preference. */
+       if (set_ea_dos_attribute(conn, fname, st, dosmode)) {
+               return 0;
+       }
 
-  /* preserve the s bits */
-  mask |= (S_ISUID | S_ISGID);
+       unixmode = unix_mode(conn,dosmode,fname, creating_file);
 
-  /* preserve the t bit */
+       /* preserve the s bits */
+       mask |= (S_ISUID | S_ISGID);
+
+       /* preserve the t bit */
 #ifdef S_ISVTX
-  mask |= S_ISVTX;
+       mask |= S_ISVTX;
 #endif
 
-  /* possibly preserve the x bits */
-  if (!MAP_ARCHIVE(conn)) mask |= S_IXUSR;
-  if (!MAP_SYSTEM(conn)) mask |= S_IXGRP;
-  if (!MAP_HIDDEN(conn)) mask |= S_IXOTH;
+       /* possibly preserve the x bits */
+       if (!MAP_ARCHIVE(conn))
+               mask |= S_IXUSR;
+       if (!MAP_SYSTEM(conn))
+               mask |= S_IXGRP;
+       if (!MAP_HIDDEN(conn))
+               mask |= S_IXOTH;
+
+       unixmode |= (st->st_mode & mask);
+
+       /* if we previously had any r bits set then leave them alone */
+       if ((tmp = st->st_mode & (S_IRUSR|S_IRGRP|S_IROTH))) {
+               unixmode &= ~(S_IRUSR|S_IRGRP|S_IROTH);
+               unixmode |= tmp;
+       }
+
+       /* if we previously had any w bits set then leave them alone 
+               whilst adding in the new w bits, if the new mode is not rdonly */
+       if (!IS_DOS_READONLY(dosmode)) {
+               unixmode |= (st->st_mode & (S_IWUSR|S_IWGRP|S_IWOTH));
+       }
+
+       if ((ret = SMB_VFS_CHMOD(conn,fname,unixmode)) == 0)
+               return 0;
+
+       if((errno != EPERM) && (errno != EACCES))
+               return -1;
+
+       if(!lp_dos_filemode(SNUM(conn)))
+               return -1;
+
+       /* We want DOS semantics, ie allow non owner with write permission to change the
+               bits on a file. Just like file_utime below.
+       */
+
+       /* Check if we have write access. */
+       if (CAN_WRITE(conn)) {
+               /*
+                * We need to open the file with write access whilst
+                * still in our current user context. This ensures we
+                * are not violating security in doing the fchmod.
+                * This file open does *not* break any oplocks we are
+                * holding. We need to review this.... may need to
+                * break batch oplocks open by others. JRA.
+                */
+               files_struct *fsp = open_file_fchmod(conn,fname,st);
+               if (!fsp)
+                       return -1;
+               become_root();
+               ret = SMB_VFS_FCHMOD(fsp, fsp->fh->fd, unixmode);
+               unbecome_root();
+               close_file_fchmod(fsp);
+       }
+
+       return( ret );
+}
 
-  unixmode |= (st->st_mode & mask);
+/*******************************************************************
+ Wrapper around dos_utime that possibly allows DOS semantics rather
+ than POSIX.
+*******************************************************************/
 
-  /* if we previously had any r bits set then leave them alone */
-  if ((tmp = st->st_mode & (S_IRUSR|S_IRGRP|S_IROTH))) {
-    unixmode &= ~(S_IRUSR|S_IRGRP|S_IROTH);
-    unixmode |= tmp;
-  }
+int file_utime(connection_struct *conn, const char *fname, struct utimbuf *times)
+{
+       SMB_STRUCT_STAT sbuf;
+       int ret = -1;
 
-  /* if we previously had any w bits set then leave them alone 
-   if the new mode is not rdonly */
-  if (!IS_DOS_READONLY(dosmode) &&
-      (tmp = st->st_mode & (S_IWUSR|S_IWGRP|S_IWOTH))) {
-    unixmode &= ~(S_IWUSR|S_IWGRP|S_IWOTH);
-    unixmode |= tmp;
-  }
+       errno = 0;
+       ZERO_STRUCT(sbuf);
 
-  return(dos_chmod(fname,unixmode));
-}
+       if(SMB_VFS_UTIME(conn,fname, times) == 0)
+               return 0;
 
+       if((errno != EPERM) && (errno != EACCES))
+               return -1;
 
-/*******************************************************************
-Wrapper around dos_utime that possibly allows DOS semantics rather
-than POSIX.
-*******************************************************************/
-int file_utime(connection_struct *conn, char *fname, struct utimbuf *times)
-{
-  extern struct current_user current_user;
-  SMB_STRUCT_STAT sb;
-  int ret = -1;
-
-  errno = 0;
-
-  if(dos_utime(fname, times) == 0)
-    return 0;
-
-  if((errno != EPERM) && (errno != EACCES))
-    return -1;
-
-  if(!lp_dos_filetimes(SNUM(conn)))
-    return -1;
-
-  /* We have permission (given by the Samba admin) to
-     break POSIX semantics and allow a user to change
-     the time on a file they don't own but can write to
-     (as DOS does).
-   */
-
-  if(dos_stat(fname,&sb) != 0)
-    return -1;
-
-  /* Check if we have write access. */
-  if (CAN_WRITE(conn)) {
-         if (((sb.st_mode & S_IWOTH) ||
-              conn->admin_user ||
-              ((sb.st_mode & S_IWUSR) && current_user.uid==sb.st_uid) ||
-              ((sb.st_mode & S_IWGRP) &&
-               in_group(sb.st_gid,current_user.gid,
-                        current_user.ngroups,current_user.groups)))) {
-                 /* We are allowed to become root and change the filetime. */
-                 become_root(False);
-                 ret = dos_utime(fname, times);
-                 unbecome_root(False);
-         }
-  }
-
-  return ret;
+       if(!lp_dos_filetimes(SNUM(conn)))
+               return -1;
+
+       /* We have permission (given by the Samba admin) to
+          break POSIX semantics and allow a user to change
+          the time on a file they don't own but can write to
+          (as DOS does).
+        */
+
+       /* Check if we have write access. */
+       if (can_write_to_file(conn, fname, &sbuf)) {
+               /* We are allowed to become root and change the filetime. */
+               become_root();
+               ret = SMB_VFS_UTIME(conn,fname, times);
+               unbecome_root();
+       }
+
+       return ret;
 }
   
 /*******************************************************************
-Change a filetime - possibly allowing DOS semantics.
+ Change a filetime - possibly allowing DOS semantics.
 *******************************************************************/
-BOOL set_filetime(connection_struct *conn, char *fname, time_t mtime)
+
+BOOL set_filetime(connection_struct *conn, const char *fname, time_t mtime)
 {
-  struct utimbuf times;
+       struct utimbuf times;
 
-  if (null_mtime(mtime)) return(True);
+       if (null_mtime(mtime))
+               return(True);
 
-  times.modtime = times.actime = mtime;
+       times.modtime = times.actime = mtime;
 
-  if (file_utime(conn, fname, &times)) {
-    DEBUG(4,("set_filetime(%s) failed: %s\n",fname,strerror(errno)));
-  }
+       if (file_utime(conn, fname, &times)) {
+               DEBUG(4,("set_filetime(%s) failed: %s\n",fname,strerror(errno)));
+               return False;
+       }
   
-  return(True);
+       return(True);
 } 
-
-