svn commit: samba r5497 - in branches/SAMBA_3_0/source/smbd: .
jra at samba.org
jra at samba.org
Tue Feb 22 05:11:41 GMT 2005
Author: jra
Date: 2005-02-22 05:11:37 +0000 (Tue, 22 Feb 2005)
New Revision: 5497
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=5497
Log:
Fix for DIR1 failures in build farm. It struck me that we
only care about failing with ACCESS_DENIED if we can't delete
with DELETE access requested. All other errors will be processed
as normal.
Jeremy.
Modified:
branches/SAMBA_3_0/source/smbd/nttrans.c
Changeset:
Modified: branches/SAMBA_3_0/source/smbd/nttrans.c
===================================================================
--- branches/SAMBA_3_0/source/smbd/nttrans.c 2005-02-22 05:11:30 UTC (rev 5496)
+++ branches/SAMBA_3_0/source/smbd/nttrans.c 2005-02-22 05:11:37 UTC (rev 5497)
@@ -782,7 +782,9 @@
if (desired_access & DELETE_ACCESS) {
status = can_delete(conn, fname, file_attributes, bad_path, True);
- if (!NT_STATUS_IS_OK(status) && !NT_STATUS_EQUAL(status,NT_STATUS_FILE_IS_A_DIRECTORY)) {
+ /* We're only going to fail here if it's access denied, as that's the
+ only error we care about for "can we delete this ?" questions. */
+ if (!NT_STATUS_IS_OK(status) && NT_STATUS_EQUAL(status,NT_STATUS_ACCESS_DENIED)) {
restore_case_semantics(conn, file_attributes);
END_PROFILE(SMBntcreateX);
return ERROR_NT(status);
@@ -1330,7 +1332,9 @@
if (desired_access & DELETE_ACCESS) {
status = can_delete(conn, fname, file_attributes, bad_path, True);
- if (!NT_STATUS_IS_OK(status) && !NT_STATUS_EQUAL(status,NT_STATUS_FILE_IS_A_DIRECTORY)) {
+ /* We're only going to fail here if it's access denied, as that's the
+ only error we care about for "can we delete this ?" questions. */
+ if (!NT_STATUS_IS_OK(status) && NT_STATUS_EQUAL(status,NT_STATUS_ACCESS_DENIED)) {
restore_case_semantics(conn, file_attributes);
END_PROFILE(SMBntcreateX);
return ERROR_NT(status);
More information about the samba-cvs
mailing list