lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 25 Aug 2017 23:14:41 -0600 From: Andreas Dilger <adilger@...ger.ca> To: tytso@....edu Cc: linux-ext4@...r.kernel.org, Andreas Dilger <adilger@...ger.ca> Subject: [PATCH] misc: quiet llvm build warnings Quiet some relatively harmless build warnings: mkquota.c:527:15: warning: format specifies type 'long' but the argument has type 'qsize_t' (aka 'long long') [-Wformat] dq->dq_id, dq->dq_dqb.dqb_curspace, ^~~~~~~~~~~~~~~~~~~~~~~ tune2fs.c:928:18: warning: '&&' within '||' [-Wlogical-op-parentheses] if (pass == 1 && (inode->i_flags & EXT4_EA_INODE_FL) || ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~ tune2fs.c:928:18: place parentheses around '&&' to silence warning Signed-off-by: Andreas Dilger <adilger@...ger.ca> --- lib/support/mkquota.c | 2 +- misc/tune2fs.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/support/mkquota.c b/lib/support/mkquota.c index 11a878e..4251e46 100644 --- a/lib/support/mkquota.c +++ b/lib/support/mkquota.c @@ -523,7 +523,7 @@ static int scan_dquots_callback(struct dquot *dquot, void *cb_data) dq->dq_dqb.dqb_curinodes != dquot->dq_dqb.dqb_curinodes) { scan_data->usage_is_inconsistent = 1; fprintf(stderr, "[QUOTA WARNING] Usage inconsistent for ID %u:" - "actual (%ld, %ld) != expected (%ld, %ld)\n", + "actual (%lld, %lld) != expected (%lld, %lld)\n", dq->dq_id, dq->dq_dqb.dqb_curspace, dq->dq_dqb.dqb_curinodes, dquot->dq_dqb.dqb_curspace, diff --git a/misc/tune2fs.c b/misc/tune2fs.c index 9b53dc2..c7c1d6d 100644 --- a/misc/tune2fs.c +++ b/misc/tune2fs.c @@ -925,8 +925,8 @@ static void rewrite_inodes(ext2_filsys fs) if (!ino) break; - if (pass == 1 && (inode->i_flags & EXT4_EA_INODE_FL) || - pass == 2 && !(inode->i_flags & EXT4_EA_INODE_FL)) + if ((pass == 1 && (inode->i_flags & EXT4_EA_INODE_FL))|| + (pass == 2 && !(inode->i_flags & EXT4_EA_INODE_FL))) rewrite_one_inode(&ctx, ino, inode); } while (ino); -- 1.8.0
Powered by blists - more mailing lists