The branch, v3-5-test has been updated
       via  c6e2256 Fix bug #8515 - Empty CIFS share can be blocked for other 
clients by deleting it via empty path (DELETE_PENDING until the last client)
      from  20ee081 Bug 7551: Return error of cli_push when 'put - /some/file' 
is used

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-5-test


- Log -----------------------------------------------------------------
commit c6e2256e2ef024ecdbebb9cb04d919342daeb780
Author: Volodymyr Khomenko <volodymyr_khome...@dell.com>
Date:   Wed Oct 12 09:57:57 2011 -0700

    Fix bug #8515 - Empty CIFS share can be blocked for other clients by 
deleting it via empty path (DELETE_PENDING until the last client)
    
    Disallow "." in can_set_delete_on_close().
    
    Autobuild-User: Jeremy Allison <j...@samba.org>
    Autobuild-Date: Wed Oct 12 21:07:27 CEST 2011 on sn-devel-104
    (cherry picked from commit bd260f03ab492d03c2890db47dc6fb4f1b824a1a)

-----------------------------------------------------------------------

Summary of changes:
 source3/locking/locking.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/locking/locking.c b/source3/locking/locking.c
index 095d0b1..8f192a6 100644
--- a/source3/locking/locking.c
+++ b/source3/locking/locking.c
@@ -1350,6 +1350,14 @@ NTSTATUS can_set_delete_on_close(files_struct *fsp, 
uint32 dosmode)
        /* Don't allow delete on close for non-empty directories. */
        if (fsp->is_directory) {
                SMB_ASSERT(!is_ntfs_stream_smb_fname(fsp->fsp_name));
+
+               /* Or the root of a share. */
+               if (ISDOT(fsp->fsp_name->base_name)) {
+                       DEBUG(10,("can_set_delete_on_close: can't set delete on 
"
+                                 "close for the root of a share.\n"));
+                       return NT_STATUS_ACCESS_DENIED;
+               }
+
                return can_delete_directory(fsp->conn,
                                            fsp->fsp_name->base_name);
        }


-- 
Samba Shared Repository

Reply via email to