linux-ext4 - [PATCH] ext4: replace ternary operator with min()

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 25 Oct 2022 22:51:35 +0800 (GMT+08:00)
From:   wangkailong@...i.cn
To:     tytso@....edu, adilger.kernel@...ger.ca, linkinjeon@...nel.org,
        sfrench@...ba.org, senozhatsky@...omium.org, tom@...pey.com
Cc:     roman.gushchin@...ux.dev, akpm@...ux-foundation.org,
        willy@...radead.org, linux-ext4@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-cifs@...r.kernel.org
Subject:  [PATCH] ext4: replace ternary operator with min()

Fix the following coccicheck warning:

fs/ext4/inline.c:183: WARNING opportunity for min()
fs/isofs/inode.c:1153: WARNING opportunity for min()
fs/ksmbd/oplock.c:882: WARNING opportunity for min()
fs/ksmbd/oplock.c:916: WARNING opportunity for min()
fs/hpfs/name.c:79: WARNING opportunity for min()

Signed-off-by: KaiLong Wang <wangkailong@...i.cn>
---
 fs/ext4/inline.c  | 3 +--
 fs/hpfs/name.c    | 2 +-
 fs/isofs/inode.c  | 2 +-
 fs/ksmbd/oplock.c | 4 ++--
 4 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/fs/ext4/inline.c b/fs/ext4/inline.c
index a4fbe825694b..19481f32ddfc 100644
--- a/fs/ext4/inline.c
+++ b/fs/ext4/inline.c
@@ -180,8 +180,7 @@ static int ext4_read_inline_data(struct inode *inode, void *buffer,
 
 	BUG_ON(len > EXT4_I(inode)->i_inline_size);
 
-	cp_len = len < EXT4_MIN_INLINE_DATA_SIZE ?
-			len : EXT4_MIN_INLINE_DATA_SIZE;
+	cp_len = min(len, EXT4_MIN_INLINE_DATA_SIZE);
 
 	raw_inode = ext4_raw_inode(iloc);
 	memcpy(buffer, (void *)(raw_inode->i_block), cp_len);
diff --git a/fs/hpfs/name.c b/fs/hpfs/name.c
index ef7ba77f36b8..ca6160a6d6f9 100644
--- a/fs/hpfs/name.c
+++ b/fs/hpfs/name.c
@@ -76,7 +76,7 @@ int hpfs_compare_names(struct super_block *s,
 		       const unsigned char *n1, unsigned l1,
 		       const unsigned char *n2, unsigned l2, int last)
 {
-	unsigned l = l1 < l2 ? l1 : l2;
+	unsigned l = min(l1, l2);
 	unsigned i;
 	if (last) return -1;
 	for (i = 0; i < l; i++) {
diff --git a/fs/isofs/inode.c b/fs/isofs/inode.c
index df9d70588b60..e8c1cad04bd2 100644
--- a/fs/isofs/inode.c
+++ b/fs/isofs/inode.c
@@ -1150,7 +1150,7 @@ static int isofs_get_block(struct inode *inode, sector_t iblock,
 	}
 
 	ret = isofs_get_blocks(inode, iblock, &bh_result, 1);
-	return ret < 0 ? ret : 0;
+	return min(ret, 0);
 }
 
 static int isofs_bmap(struct inode *inode, sector_t block)
diff --git a/fs/ksmbd/oplock.c b/fs/ksmbd/oplock.c
index d7d47b82451d..94ec8d02d877 100644
--- a/fs/ksmbd/oplock.c
+++ b/fs/ksmbd/oplock.c
@@ -879,7 +879,7 @@ static int oplock_break(struct oplock_info *brk_opinfo, int req_op_level)
 
 		err = oplock_break_pending(brk_opinfo, req_op_level);
 		if (err)
-			return err < 0 ? err : 0;
+			return min(err, 0);
 
 		if (brk_opinfo->open_trunc) {
 			/*
@@ -913,7 +913,7 @@ static int oplock_break(struct oplock_info *brk_opinfo, int req_op_level)
 	} else {
 		err = oplock_break_pending(brk_opinfo, req_op_level);
 		if (err)
-			return err < 0 ? err : 0;
+			return min(err, 0);
 
 		if (brk_opinfo->level == SMB2_OPLOCK_LEVEL_BATCH ||
 		    brk_opinfo->level == SMB2_OPLOCK_LEVEL_EXCLUSIVE)
-- 
2.25.1