mirror of
https://github.com/samba-team/samba.git
synced 2024-12-28 07:21:54 +03:00
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 commit is contained in:
parent
749fa73544
commit
8e26775134
@ -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;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user