summaryrefslogtreecommitdiff
path: root/source4
diff options
context:
space:
mode:
authorVolker Lendecke <vlendec@samba.org>2004-10-10 17:03:37 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 12:59:45 -0500
commit3b6a240284763726ae7b696c4e3e7a462451b424 (patch)
treed5241cc7ce74545dfd9167d6bb96c6bbb2d5dd83 /source4
parent18dc1cd00f09d051497cf89e348f8ba794c63230 (diff)
downloadsamba-3b6a240284763726ae7b696c4e3e7a462451b424.tar.gz
samba-3b6a240284763726ae7b696c4e3e7a462451b424.tar.bz2
samba-3b6a240284763726ae7b696c4e3e7a462451b424.zip
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)
Diffstat (limited to 'source4')
-rw-r--r--source4/torture/raw/write.c103
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;
}