[SCM] Samba Shared Repository - branch master updated

David Disseldorp ddiss at samba.org
Fri Jul 21 17:11:02 UTC 2017


The branch, master has been updated
       via  1dcacff vfs_ceph: fix cephwrap_chdir()
      from  4e04f02 selftest: Add test for password change when NTLM is disabled

https://git.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 1dcacff083019810e207a3d123a81fe32d9dde1a
Author: David Disseldorp <ddiss at samba.org>
Date:   Fri Jul 14 23:55:29 2017 +0200

    vfs_ceph: fix cephwrap_chdir()
    
    When provided a '/' path (i.e. CephFS root), vfs_ceph does a *local*
    chdir() to the share path. This breaks smb client directory listings.
    
    Bug: https://bugzilla.samba.org/show_bug.cgi?id=12911
    
    Signed-off-by: David Disseldorp <ddiss at samba.org>
    Reviewed-by: Jeremy Allison <jra at samba.org>
    
    Autobuild-User(master): David Disseldorp <ddiss at samba.org>
    Autobuild-Date(master): Fri Jul 21 19:10:46 CEST 2017 on sn-devel-144

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

Summary of changes:
 source3/modules/vfs_ceph.c | 8 --------
 1 file changed, 8 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/modules/vfs_ceph.c b/source3/modules/vfs_ceph.c
index 73106c4..9abd321 100644
--- a/source3/modules/vfs_ceph.c
+++ b/source3/modules/vfs_ceph.c
@@ -936,14 +936,6 @@ static int cephwrap_chdir(struct vfs_handle_struct *handle,
 {
 	int result = -1;
 	DBG_DEBUG("[CEPH] chdir(%p, %s)\n", handle, smb_fname->base_name);
-	/*
-	 * If the path is just / use chdir because Ceph is below / and
-	 * cannot deal with changing directory above its mount point
-	 */
-	if (!strcmp(smb_fname->base_name, "/")) {
-		return chdir(smb_fname->base_name);
-	}
-
 	result = ceph_chdir(handle->data, smb_fname->base_name);
 	DBG_DEBUG("[CEPH] chdir(...) = %d\n", result);
 	WRAP_RETURN(result);


-- 
Samba Shared Repository



More information about the samba-cvs mailing list