[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-changelog] [xen master] libfsimage: fix clang 10 build



commit e54c433adf01a242bf6e9fe9378a2c83d3f8b419
Author:     Roger Pau Monne <roger.pau@xxxxxxxxxx>
AuthorDate: Fri Mar 13 09:45:57 2020 +0100
Commit:     Wei Liu <wl@xxxxxxx>
CommitDate: Fri Mar 13 10:20:21 2020 +0000

    libfsimage: fix clang 10 build
    
    clang complains with:
    
    fsys_zfs.c:826:2: error: converting the enum constant to a boolean 
[-Werror,-Wint-in-bool-context]
            VERIFY_DN_TYPE(dn, DMU_OT_PLAIN_FILE_CONTENTS);
            ^
    
/wrkdirs/usr/ports/sysutils/xen-tools/work/xen-4.13.0/tools/libfsimage/zfs/../../../tools/libfsimage/zfs/fsys_zfs.h:74:11:
 note: expanded from macro 'VERIFY_DN_TYPE'
            if (type && (dnp)->dn_type != type) { \
                     ^
    1 error generated.
    
    Fix this by not forcing an implicit conversion of the enum into a
    boolean and instead comparing with the 0 enumerator.
    
    Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
    Acked-by: Wei Liu <wl@xxxxxxx>
---
 tools/libfsimage/zfs/fsys_zfs.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/libfsimage/zfs/fsys_zfs.h b/tools/libfsimage/zfs/fsys_zfs.h
index 5cd627dbac..721972a05a 100644
--- a/tools/libfsimage/zfs/fsys_zfs.h
+++ b/tools/libfsimage/zfs/fsys_zfs.h
@@ -71,7 +71,7 @@ typedef       unsigned int size_t;
  * Can only be used in functions returning non-0 for failure.
  */
 #define        VERIFY_DN_TYPE(dnp, type) \
-       if (type && (dnp)->dn_type != type) { \
+       if (type != DMU_OT_NONE && (dnp)->dn_type != type) { \
                return (ERR_FSYS_CORRUPT); \
        }
 
@@ -80,7 +80,7 @@ typedef       unsigned int size_t;
  * Can only be used in functions returning 0 for failure.
  */
 #define        VERIFY_OS_TYPE(osp, type) \
-       if (type && (osp)->os_type != type) { \
+       if (type != DMU_OST_NONE && (osp)->os_type != type) { \
                errnum = ERR_FSYS_CORRUPT; \
                return (0); \
        }
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.