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
| ||
|
Message-Id: <20200708111512.97DF5A405C@b06wcsmtp001.portsmouth.uk.ibm.com> Date: Wed, 8 Jul 2020 16:45:11 +0530 From: Ritesh Harjani <riteshh@...ux.ibm.com> To: Markus Elfring <Markus.Elfring@....de>, linux-ext4@...r.kernel.org, Andreas Dilger <adilger.kernel@...ger.ca>, "Theodore Ts'o" <tytso@....edu> Cc: kernel-janitors@...r.kernel.org, LKML <linux-kernel@...r.kernel.org> Subject: Re: [PATCH] ext4: Delete unnecessary checks before brelse() On 6/13/20 11:37 PM, Markus Elfring wrote: > From: Markus Elfring <elfring@...rs.sourceforge.net> > Date: Sat, 13 Jun 2020 19:12:24 +0200 > > The brelse() function tests whether its argument is NULL > and then returns immediately. > Thus remove the tests which are not needed around the shown calls. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring <elfring@...rs.sourceforge.net> Sure, LGTM. Feel free to add Reviewed-by: Ritesh Harjani <riteshh@...ux.ibm.com> > --- > fs/ext4/extents.c | 6 ++---- > fs/ext4/xattr.c | 3 +-- > 2 files changed, 3 insertions(+), 6 deletions(-) > > diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c > index 221f240eae60..315276d50aa8 100644 > --- a/fs/ext4/extents.c > +++ b/fs/ext4/extents.c > @@ -693,10 +693,8 @@ void ext4_ext_drop_refs(struct ext4_ext_path *path) > return; > depth = path->p_depth; > for (i = 0; i <= depth; i++, path++) { > - if (path->p_bh) { > - brelse(path->p_bh); > - path->p_bh = NULL; > - } > + brelse(path->p_bh); > + path->p_bh = NULL; > } > } > > diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c > index 9b29a40738ac..eb997ce21be3 100644 > --- a/fs/ext4/xattr.c > +++ b/fs/ext4/xattr.c > @@ -1354,8 +1354,7 @@ static int ext4_xattr_inode_write(handle_t *handle, struct inode *ea_inode, > > block = 0; > while (wsize < bufsize) { > - if (bh != NULL) > - brelse(bh); > + brelse(bh); > csize = (bufsize - wsize) > blocksize ? blocksize : > bufsize - wsize; > bh = ext4_getblk(handle, ea_inode, block, 0); > -- > 2.27.0 >
Powered by blists - more mailing lists