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: Mon, 28 Jan 2013 17:12:18 -0800 From: "Darrick J. Wong" <darrick.wong@...cle.com> To: Guo Chao <yan@...ux.vnet.ibm.com> Cc: tytso@....edu, linux-ext4@...r.kernel.org Subject: Re: [PATCH 2/4] ext4: remove unused variable in add_dirent_to_buf() On Fri, Jan 18, 2013 at 04:01:12PM +0800, Guo Chao wrote: > After commit 978fef9 (create __ext4_insert_dentry for dir entry > insertion), 'reclen' is not used anymore. > > Signed-off-by: Guo Chao <yan@...ux.vnet.ibm.com> This one looks ok too. Reviewed-by: Darrick J. Wong <darrick.wong@...cle.com> > --- > fs/ext4/namei.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c > index a445247..b3717a3 100644 > --- a/fs/ext4/namei.c > +++ b/fs/ext4/namei.c > @@ -1703,7 +1703,6 @@ static int add_dirent_to_buf(handle_t *handle, struct dentry *dentry, > const char *name = dentry->d_name.name; > int namelen = dentry->d_name.len; > unsigned int blocksize = dir->i_sb->s_blocksize; > - unsigned short reclen; > int csum_size = 0; > int err; > > @@ -1711,7 +1710,6 @@ static int add_dirent_to_buf(handle_t *handle, struct dentry *dentry, > EXT4_FEATURE_RO_COMPAT_METADATA_CSUM)) > csum_size = sizeof(struct ext4_dir_entry_tail); > > - reclen = EXT4_DIR_REC_LEN(namelen); > if (!de) { > err = ext4_find_dest_de(dir, inode, > bh, bh->b_data, blocksize - csum_size, > -- > 1.7.9.5 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@...r.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists