]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s4/libcli/smb2: clang: Fix 'value stored to 'status' is never read'
authorNoel Power <noel.power@suse.com>
Mon, 29 Jul 2019 11:44:26 +0000 (11:44 +0000)
committerNoel Power <npower@samba.org>
Wed, 25 Sep 2019 09:47:31 +0000 (09:47 +0000)
Fixes:

source4/libcli/smb2/util.c:134:3: warning: Value stored to 'status' is never read <--[clang]
                status = smb2_util_setatr(tree, dname, FILE_ATTRIBUTE_NORMAL);
                ^        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
source4/libcli/smb2/util.c:187:5: warning: Value stored to 'status' is never read <--[clang]
                                status = smb2_util_setatr(tree, name, FILE_ATTRIBUTE_NORMAL);
                                ^        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
source4/libcli/smb2/util.c:209:3: warning: Value stored to 'status' is never read <--[clang]
                status = smb2_util_setatr(tree, dname, FILE_ATTRIBUTE_NORMAL);
                ^        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
3 warnings generated.

Signed-off-by: Noel Power <noel.power@suse.com>
Reviewed-by: Andreas Schneider <asn@samba.org>
source4/libcli/smb2/util.c

index e28c910fb236863c04895bb65c7f5825619917c6..94072bee3ea7a913002a3418daf26589cfc2ffa2 100644 (file)
@@ -131,7 +131,7 @@ int smb2_deltree(struct smb2_tree *tree, const char *dname)
 
        if (NT_STATUS_EQUAL(status, NT_STATUS_CANNOT_DELETE)) {
                /* it could be read-only */
-               status = smb2_util_setatr(tree, dname, FILE_ATTRIBUTE_NORMAL);
+               smb2_util_setatr(tree, dname, FILE_ATTRIBUTE_NORMAL);
                status = smb2_util_unlink(tree, dname);
        }
        if (NT_STATUS_IS_OK(status)) {
@@ -184,7 +184,7 @@ int smb2_deltree(struct smb2_tree *tree, const char *dname)
                        status = smb2_util_unlink(tree, name);
                        if (NT_STATUS_EQUAL(status, NT_STATUS_CANNOT_DELETE)) {
                                /* it could be read-only */
-                               status = smb2_util_setatr(tree, name, FILE_ATTRIBUTE_NORMAL);
+                               smb2_util_setatr(tree, name, FILE_ATTRIBUTE_NORMAL);
                                status = smb2_util_unlink(tree, name);
                        }
                        
@@ -206,7 +206,7 @@ int smb2_deltree(struct smb2_tree *tree, const char *dname)
        status = smb2_util_rmdir(tree, dname);
        if (NT_STATUS_EQUAL(status, NT_STATUS_CANNOT_DELETE)) {
                /* it could be read-only */
-               status = smb2_util_setatr(tree, dname, FILE_ATTRIBUTE_NORMAL);
+               smb2_util_setatr(tree, dname, FILE_ATTRIBUTE_NORMAL);
                status = smb2_util_rmdir(tree, dname);
        }