The branch, master has been updated
       via  fb35a22 test: Fix quoting
       via  5deb685 s3-vfs: Fix developer build of vfs_ceph module.
      from  a59fb32 ctdb: improve helpfulness of debug message when taking 
reclock fails

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


- Log -----------------------------------------------------------------
commit fb35a22b7aae07bce233d5539c5048b308271010
Author: Volker Lendecke <[email protected]>
Date:   Mon Dec 8 17:12:24 2014 +0100

    test: Fix quoting
    
    Signed-off-by: Volker Lendecke <[email protected]>
    Reviewed-by: Guenther Deschner <[email protected]>
    
    Autobuild-User(master): Günther Deschner <[email protected]>
    Autobuild-Date(master): Fri Jan  9 17:33:31 CET 2015 on sn-devel-104

commit 5deb685ecd6f611bcf357f7040dc62582a50d2aa
Author: Günther Deschner <[email protected]>
Date:   Fri Jan 9 12:54:51 2015 +0100

    s3-vfs: Fix developer build of vfs_ceph module.
    
    Guenther
    
    Signed-off-by: Günther Deschner <[email protected]>
    Reviewed-by: Michael Adam <[email protected]>

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

Summary of changes:
 source3/modules/vfs_ceph.c                | 30 ++++++++++++++++++++----------
 source3/script/tests/test_smbclient_s3.sh |  2 +-
 2 files changed, 21 insertions(+), 11 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/modules/vfs_ceph.c b/source3/modules/vfs_ceph.c
index b0a0024..ec72312 100644
--- a/source3/modules/vfs_ceph.c
+++ b/source3/modules/vfs_ceph.c
@@ -750,9 +750,10 @@ static int cephwrap_ntimes(struct vfs_handle_struct 
*handle,
                         struct smb_file_time *ft)
 {
        struct utimbuf buf;
+       int result;
        buf.actime = ft->atime.tv_sec;
        buf.modtime = ft->mtime.tv_sec;
-       int result = ceph_utime(handle->data, smb_fname->base_name, &buf);
+       result = ceph_utime(handle->data, smb_fname->base_name, &buf);
        DEBUG(10, ("[CEPH] ntimes(%p, %s, {%ld, %ld, %ld, %ld}) = %d\n", 
handle, smb_fname_str_dbg(smb_fname),
                                ft->mtime.tv_sec, ft->atime.tv_sec, 
ft->ctime.tv_sec,
                                ft->create_time.tv_sec, result));
@@ -1050,8 +1051,9 @@ static const char *cephwrap_connectpath(struct 
vfs_handle_struct *handle,
 
 static ssize_t cephwrap_getxattr(struct vfs_handle_struct *handle,const char 
*path, const char *name, void *value, size_t size)
 {
+       int ret;
        DEBUG(10, ("[CEPH] getxattr(%p, %s, %s, %p, %llu)\n", handle, path, 
name, value, llu(size)));
-       int ret = ceph_getxattr(handle->data, path, name, value, size);
+       ret = ceph_getxattr(handle->data, path, name, value, size);
        DEBUG(10, ("[CEPH] getxattr(...) = %d\n", ret));
        if (ret < 0) {
                WRAP_RETURN(ret);
@@ -1062,8 +1064,9 @@ static ssize_t cephwrap_getxattr(struct vfs_handle_struct 
*handle,const char *pa
 
 static ssize_t cephwrap_fgetxattr(struct vfs_handle_struct *handle, struct 
files_struct *fsp, const char *name, void *value, size_t size)
 {
+       int ret;
        DEBUG(10, ("[CEPH] fgetxattr(%p, %p, %s, %p, %llu)\n", handle, fsp, 
name, value, llu(size)));
-       int ret = ceph_getxattr(handle->data, fsp->fsp_name->base_name, name, 
value, size);
+       ret = ceph_getxattr(handle->data, fsp->fsp_name->base_name, name, 
value, size);
        DEBUG(10, ("[CEPH] fgetxattr(...) = %d\n", ret));
        if (ret < 0) {
                WRAP_RETURN(ret);
@@ -1074,8 +1077,9 @@ static ssize_t cephwrap_fgetxattr(struct 
vfs_handle_struct *handle, struct files
 
 static ssize_t cephwrap_listxattr(struct vfs_handle_struct *handle, const char 
*path, char *list, size_t size)
 {
+       int ret;
        DEBUG(10, ("[CEPH] listxattr(%p, %s, %p, %llu)\n", handle, path, list, 
llu(size)));
-       int ret = ceph_listxattr(handle->data, path, list, size);
+       ret = ceph_listxattr(handle->data, path, list, size);
        DEBUG(10, ("[CEPH] listxattr(...) = %d\n", ret));
        if (ret < 0) {
                WRAP_RETURN(ret);
@@ -1087,8 +1091,9 @@ static ssize_t cephwrap_listxattr(struct 
vfs_handle_struct *handle, const char *
 #if 0
 static ssize_t cephwrap_llistxattr(struct vfs_handle_struct *handle, const 
char *path, char *list, size_t size)
 {
+       int ret;
        DEBUG(10, ("[CEPH] llistxattr(%p, %s, %p, %llu)\n", handle, path, list, 
llu(size)));
-       int ret = ceph_llistxattr(handle->data, path, list, size);
+       ret = ceph_llistxattr(handle->data, path, list, size);
        DEBUG(10, ("[CEPH] listxattr(...) = %d\n", ret));
        if (ret < 0) {
                WRAP_RETURN(ret);
@@ -1100,8 +1105,9 @@ static ssize_t cephwrap_llistxattr(struct 
vfs_handle_struct *handle, const char
 
 static ssize_t cephwrap_flistxattr(struct vfs_handle_struct *handle, struct 
files_struct *fsp, char *list, size_t size)
 {
+       int ret;
        DEBUG(10, ("[CEPH] flistxattr(%p, %p, %s, %llu)\n", handle, fsp, list, 
llu(size)));
-       int ret = ceph_listxattr(handle->data, fsp->fsp_name->base_name, list, 
size);
+       ret = ceph_listxattr(handle->data, fsp->fsp_name->base_name, list, 
size);
        DEBUG(10, ("[CEPH] flistxattr(...) = %d\n", ret));
        if (ret < 0) {
                WRAP_RETURN(ret);
@@ -1112,32 +1118,36 @@ static ssize_t cephwrap_flistxattr(struct 
vfs_handle_struct *handle, struct file
 
 static int cephwrap_removexattr(struct vfs_handle_struct *handle, const char 
*path, const char *name)
 {
+       int ret;
        DEBUG(10, ("[CEPH] removexattr(%p, %s, %s)\n", handle, path, name));
-       int ret = ceph_removexattr(handle->data, path, name);
+       ret = ceph_removexattr(handle->data, path, name);
        DEBUG(10, ("[CEPH] removexattr(...) = %d\n", ret));
        WRAP_RETURN(ret);
 }
 
 static int cephwrap_fremovexattr(struct vfs_handle_struct *handle, struct 
files_struct *fsp, const char *name)
 {
+       int ret;
        DEBUG(10, ("[CEPH] fremovexattr(%p, %p, %s)\n", handle, fsp, name));
-       int ret = ceph_removexattr(handle->data, fsp->fsp_name->base_name, 
name);
+       ret = ceph_removexattr(handle->data, fsp->fsp_name->base_name, name);
        DEBUG(10, ("[CEPH] fremovexattr(...) = %d\n", ret));
        WRAP_RETURN(ret);
 }
 
 static int cephwrap_setxattr(struct vfs_handle_struct *handle, const char 
*path, const char *name, const void *value, size_t size, int flags)
 {
+       int ret;
        DEBUG(10, ("[CEPH] setxattr(%p, %s, %s, %p, %llu, %d)\n", handle, path, 
name, value, llu(size), flags));
-       int ret = ceph_setxattr(handle->data, path, name, value, size, flags);
+       ret = ceph_setxattr(handle->data, path, name, value, size, flags);
        DEBUG(10, ("[CEPH] setxattr(...) = %d\n", ret));
        WRAP_RETURN(ret);
 }
 
 static int cephwrap_fsetxattr(struct vfs_handle_struct *handle, struct 
files_struct *fsp, const char *name, const void *value, size_t size, int flags)
 {
+       int ret;
        DEBUG(10, ("[CEPH] fsetxattr(%p, %p, %s, %p, %llu, %d)\n", handle, fsp, 
name, value, llu(size), flags));
-       int ret = ceph_setxattr(handle->data, fsp->fsp_name->base_name, name, 
value, size, flags);
+       ret = ceph_setxattr(handle->data, fsp->fsp_name->base_name, name, 
value, size, flags);
        DEBUG(10, ("[CEPH] fsetxattr(...) = %d\n", ret));
        WRAP_RETURN(ret);
 }
diff --git a/source3/script/tests/test_smbclient_s3.sh 
b/source3/script/tests/test_smbclient_s3.sh
index f6f3705..e786c9f 100755
--- a/source3/script/tests/test_smbclient_s3.sh
+++ b/source3/script/tests/test_smbclient_s3.sh
@@ -212,7 +212,7 @@ mkdir a_test_dir
 quit
 EOF
 
-    cmd='CLI_FORCE_INTERACTIVE=yes $SMBCLIENT -U% //$SERVER/$1" -I $SERVER_IP 
$ADDARGS < $tmpfile 2>&1'
+    cmd='CLI_FORCE_INTERACTIVE=yes $SMBCLIENT -U% "//$SERVER/$1" -I $SERVER_IP 
$ADDARGS < $tmpfile 2>&1'
     eval echo "$cmd"
     out=`eval $cmd`
     ret=$?


-- 
Samba Shared Repository

Reply via email to