r2895: Another nut to crack, not sure how hard: The statinfo on a file does obviously

not change during a write. The nasty thing: Excel 2003 obviosly does depend on
this.

Volker
(This used to be commit 8e26775134671114425ce1ecf7a22bad4e763d1f)

1 files changed, 103 insertions, 0 deletions

diff --git a/source4/torture/raw/write.c b/source4/torture/raw/write.c
index 77c73a42b4..7aff22129b 100644
--- a/source4/torture/raw/write.c
+++ b/source4/torture/raw/write.c
@@ -670,6 +670,105 @@ done:
 	return ret;
 }
 
+/* Windows does obviously not update the stat info during a write call. I
+ * *think* this is the problem causing a spurious Excel 2003 on XP error
+ * message when saving a file. Excel does a setfileinfo, writes, and then does
+ * a getpath(!)info. Or so... For Samba sometimes it displays an error message
+ * that the file might have been changed in between. What i've been able to
+ * trace down is that this happens if the getpathinfo after the write shows a
+ * different last write time than the setfileinfo showed. This is really
+ * nasty....
+ */
+
+static BOOL test_finfo_after_write(struct smbcli_state *cli, TALLOC_CTX *mem_ctx)
+{
+	union smb_open io1;
+	union smb_fileinfo finfo1, finfo2;
+	union smb_write io3;
+	const char *fname = BASEDIR "\\torture_file.txt";
+	NTSTATUS status;
+	int fnum = -1;
+	BOOL ret = True;
+
+	io1.generic.level = RAW_OPEN_NTCREATEX;
+	io1.ntcreatex.in.flags = NTCREATEX_FLAGS_EXTENDED;
+	io1.ntcreatex.in.root_fid = 0;
+	io1.ntcreatex.in.access_mask = GENERIC_RIGHTS_FILE_WRITE|GENERIC_RIGHTS_FILE_READ;
+	io1.ntcreatex.in.alloc_size = 1024*1024;
+	io1.ntcreatex.in.file_attr = FILE_ATTRIBUTE_NORMAL;
+	io1.ntcreatex.in.share_access = NTCREATEX_SHARE_ACCESS_NONE;
+	io1.ntcreatex.in.open_disposition = NTCREATEX_DISP_CREATE;
+	io1.ntcreatex.in.create_options = 0;
+	io1.ntcreatex.in.impersonation = NTCREATEX_IMPERSONATION_ANONYMOUS;
+	io1.ntcreatex.in.security_flags = 0;
+	io1.ntcreatex.in.fname = fname;
+
+	status = smb_raw_open(cli->tree, mem_ctx, &io1);
+
+	if (!NT_STATUS_IS_OK(status))
+		return False;
+
+	fnum = io1.ntcreatex.out.fnum;
+
+	finfo1.basic_info.level = RAW_FILEINFO_BASIC_INFO;
+	finfo1.basic_info.in.fnum = fnum;
+
+	status = smb_raw_fileinfo(cli->tree, mem_ctx, &finfo1);
+
+	if (!NT_STATUS_IS_OK(status)) {
+		DEBUG(0, ("fileinfo failed: %s\n", nt_errstr(status)));
+		return False;
+	}
+
+	io3.generic.level = RAW_WRITE_WRITE;
+	io3.write.in.fnum = fnum;
+	io3.write.in.count = 1;
+	io3.write.in.offset = 0;
+	io3.write.in.remaining = 0;
+	io3.write.in.data = "x";
+
+	msleep(1000);
+
+	status = smb_raw_write(cli->tree, &io3);
+
+	if (!NT_STATUS_IS_OK(status))
+		return False;
+
+	finfo2.basic_info.level = RAW_FILEINFO_BASIC_INFO;
+	finfo2.basic_info.in.fname = fname;
+
+	status = smb_raw_pathinfo(cli->tree, mem_ctx, &finfo2);
+
+	if (!NT_STATUS_IS_OK(status)) {
+		DEBUG(0, ("fileinfo failed: %s\n", nt_errstr(status)));
+		return False;
+	}
+
+	if (finfo1.basic_info.out.create_time !=
+	    finfo2.basic_info.out.create_time) {
+		ret = False;
+	}
+		
+	if (finfo1.basic_info.out.access_time !=
+	    finfo2.basic_info.out.access_time) {
+		ret = False;
+	}
+		
+	if (finfo1.basic_info.out.write_time !=
+	    finfo2.basic_info.out.write_time) {
+		ret = False;
+	}
+		
+	if (finfo1.basic_info.out.change_time !=
+	    finfo2.basic_info.out.change_time) {
+		ret = False;
+	}
+	
+	smbcli_close(cli->tree, fnum);
+	smbcli_unlink(cli->tree, fname);
+
+	return ret;
+}
 
 /* 
    basic testing of write calls
@@ -686,6 +785,10 @@ BOOL torture_raw_write(int dummy)
 
 	mem_ctx = talloc_init("torture_raw_write");
 
+	if (!test_finfo_after_write(cli, mem_ctx)) {
+		ret = False;
+	}
+
 	if (!test_write(cli, mem_ctx)) {
 		ret = False;
 	}