xfs_dir2_free_hdr_t entries are unsigned, so they can't be negative. remove these unnessary checks in process_leaf_node_dir_v2_free. Reported by coverity. Signed-off-by: Vivek Trivedi <t.vivek@xxxxxxxxxxx> --- db/check.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/db/check.c b/db/check.c index 9c1541d..9d986b9 100644 --- a/db/check.c +++ b/db/check.c @@ -3073,9 +3073,7 @@ process_leaf_node_dir_v3_free( return; } if (be32_to_cpu(free->hdr.nvalid) > maxent || - be32_to_cpu(free->hdr.nvalid) < 0 || be32_to_cpu(free->hdr.nused) > maxent || - be32_to_cpu(free->hdr.nused) < 0 || be32_to_cpu(free->hdr.nused) > be32_to_cpu(free->hdr.nvalid)) { if (!sflag || v) -- 1.7.9.5 _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs