Changeset 335 for branches/samba-3.2.x/source/smbd/trans2.c

Timestamp:

Sep 24, 2009, 9:15:13 AM (16 years ago)

Author:

Herwig Bauernfeind

Message:

Update 3.2 to 3.2.14 (final)

File:

• branches/samba-3.2.x/source/smbd/trans2.c (modified) (5 diffs)

branches/samba-3.2.x/source/smbd/trans2.c

@@ -4920 +4920 @@
         if (setting_write_time) {
                 /*
-                 * This was a setfileinfo on an open file.
+                 * This was a Windows setfileinfo on an open file.
                  * NT does this a lot. We also need to 
                  * set the time here, as it can be read by 
@@ -5997 +5997 @@
         bool delete_on_fail = False;
         enum perm_type ptype;
+        files_struct *all_fsps = NULL;
+        bool modify_mtime = true;
+        struct file_id id;
 
         if (total_data < 100) {
@@ -6143 +6146 @@
 
         /* Deal with any time changes. */
-
-        return smb_set_file_time(conn,
+        id = vfs_file_id_from_sbuf(conn, psbuf);
+        for(all_fsps = file_find_di_first(id); all_fsps;
+                        all_fsps = file_find_di_next(all_fsps)) {
+                /*
+                 * We're setting the time explicitly for UNIX.
+                 * Cancel any pending changes over all handles.
+                 */
+                all_fsps->update_write_time_on_close = false;
+                TALLOC_FREE(all_fsps->update_write_time_event);
+        }
+
+        /*
+         * Override the "setting_write_time"
+         * parameter here as it almost does what
+         * we need. Just remember if we modified
+         * mtime and send the notify ourselves.
+         */
+        if (null_timespec(ts[1])) {
+                modify_mtime = false;
+        }
+
+        status = smb_set_file_time(conn,
                                 fsp,
                                 fname,
                                 psbuf,
                                 ts,
-                                true);
+                                false);
+
+        if (modify_mtime) {
+                notify_fname(conn, NOTIFY_ACTION_MODIFIED,
+                        FILE_NOTIFY_CHANGE_LAST_WRITE, fname);
+        }
+        return status;
 }
 
@@ -6776 +6805 @@
         }
 
-        if (!CAN_WRITE(conn)) {
-                reply_doserror(req, ERRSRV, ERRaccess);
-                return;
-        }
-
         if (INFO_LEVEL_IS_UNIX(info_level) && !lp_unix_extensions()) {
                 reply_nterror(req, NT_STATUS_INVALID_LEVEL);
                 return;
+        }
+
+        if (!CAN_WRITE(conn)) {
+                /* Allow POSIX opens. The open path will deny
+                 * any non-readonly opens. */
+                if (info_level != SMB_POSIX_PATH_OPEN) {
+                        reply_doserror(req, ERRSRV, ERRaccess);
+                        return;
+                }
         }
 
@@ -7090 +7123 @@
 
         DEBUG(3,("call_trans2mkdir : name = %s\n", directory));
+
+        status = resolve_dfspath(ctx,
+                                conn,
+                                req->flags2 & FLAGS2_DFS_PATHNAMES,
+                                directory,
+                                &directory);
+        if (!NT_STATUS_IS_OK(status)) {
+                if (NT_STATUS_EQUAL(status,NT_STATUS_PATH_NOT_COVERED)) {
+                        reply_botherror(req,
+                                        NT_STATUS_PATH_NOT_COVERED,
+                                        ERRSRV, ERRbadpath);
+                }
+                reply_nterror(req, status);
+                return;
+        }
 
         status = unix_convert(ctx, conn, directory, False, &directory, NULL, &sbuf);