summaryrefslogtreecommitdiff
path: root/source3/smbd
diff options
context:
space:
mode:
authorTim Prouty <tprouty@samba.org>2008-12-06 16:08:35 -0800
committerTim Prouty <tprouty@samba.org>2008-12-09 18:06:49 -0800
commit7b9f6dda131f471ae61c12e7eb06d67b8f02b1cf (patch)
tree7e948a71bc8d910f439d3d429eb9d31ccf334ade /source3/smbd
parent4a9b092eb43603bae6190b8a5fdee20c9ebae26c (diff)
downloadsamba-7b9f6dda131f471ae61c12e7eb06d67b8f02b1cf.tar.gz
samba-7b9f6dda131f471ae61c12e7eb06d67b8f02b1cf.tar.bz2
samba-7b9f6dda131f471ae61c12e7eb06d67b8f02b1cf.zip
s3: [3/3]: Fix a delete on close divergence from windows and the associated torture test
This third patch cleans up by removing all of the code that is made obsolete by the first patch. It should cause no functional changes.
Diffstat (limited to 'source3/smbd')
-rw-r--r--source3/smbd/open.c5
-rw-r--r--source3/smbd/reply.c2
2 files changed, 2 insertions, 5 deletions
diff --git a/source3/smbd/open.c b/source3/smbd/open.c
index 8882e5438b..77ad1664ef 100644
--- a/source3/smbd/open.c
+++ b/source3/smbd/open.c
@@ -1981,7 +1981,7 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
}
set_share_mode(lck, fsp, conn->server_info->utok.uid, 0,
- fsp->oplock_type, new_file_created);
+ fsp->oplock_type);
/* Handle strange delete on close create semantics. */
if (create_options & FILE_DELETE_ON_CLOSE) {
@@ -2419,8 +2419,7 @@ static NTSTATUS open_directory(connection_struct *conn,
return status;
}
- set_share_mode(lck, fsp, conn->server_info->utok.uid, 0, NO_OPLOCK,
- True);
+ set_share_mode(lck, fsp, conn->server_info->utok.uid, 0, NO_OPLOCK);
/* For directories the delete on close bit at open time seems
always to be honored on close... See test 19 in Samba4 BASE-DELETE. */
diff --git a/source3/smbd/reply.c b/source3/smbd/reply.c
index 00c744ce1a..9f7a1896b8 100644
--- a/source3/smbd/reply.c
+++ b/source3/smbd/reply.c
@@ -5583,8 +5583,6 @@ NTSTATUS rename_internals_fsp(connection_struct *conn,
* depends on these semantics. JRA.
*/
- set_allow_initial_delete_on_close(lck, fsp, True);
-
if (create_options & FILE_DELETE_ON_CLOSE) {
status = can_set_delete_on_close(fsp, True, 0);