From: James Lamanna Signed-off-by: James Lamanna Signed-off-by: Domen Puncer Signed-off-by: Alexey Dobriyan Signed-off-by: Jan Kara Signed-off-by: Andrew Morton --- fs/reiserfs/super.c | 25 +++++++++---------------- fs/reiserfs/xattr_acl.c | 3 +-- 2 files changed, 10 insertions(+), 18 deletions(-) diff -puN fs/reiserfs/super.c~reiserfs-free-checking-cleanup fs/reiserfs/super.c --- 25/fs/reiserfs/super.c~reiserfs-free-checking-cleanup Tue Sep 20 14:11:20 2005 +++ 25-akpm/fs/reiserfs/super.c Tue Sep 20 14:11:20 2005 @@ -1024,12 +1024,8 @@ static int reiserfs_parse_options(struct strcpy(REISERFS_SB(s)->s_qf_names[qtype], arg); *mount_options |= 1 << REISERFS_QUOTA; } else { - if (REISERFS_SB(s)->s_qf_names[qtype]) { - kfree(REISERFS_SB(s)-> - s_qf_names[qtype]); - REISERFS_SB(s)->s_qf_names[qtype] = - NULL; - } + kfree(REISERFS_SB(s)->s_qf_names[qtype]); + REISERFS_SB(s)->s_qf_names[qtype] = NULL; } } if (c == 'f') { @@ -1158,11 +1154,10 @@ static int reiserfs_remount(struct super if (!reiserfs_parse_options (s, arg, &mount_options, &blocks, NULL, &commit_max_age)) { #ifdef CONFIG_QUOTA - for (i = 0; i < MAXQUOTAS; i++) - if (REISERFS_SB(s)->s_qf_names[i]) { - kfree(REISERFS_SB(s)->s_qf_names[i]); - REISERFS_SB(s)->s_qf_names[i] = NULL; - } + for (i = 0; i < MAXQUOTAS; i++) { + kfree(REISERFS_SB(s)->s_qf_names[i]); + REISERFS_SB(s)->s_qf_names[i] = NULL; + } #endif return -EINVAL; } @@ -1940,13 +1935,11 @@ static int reiserfs_fill_super(struct su brelse(SB_BUFFER_WITH_SB(s)); #ifdef CONFIG_QUOTA for (j = 0; j < MAXQUOTAS; j++) { - if (sbi->s_qf_names[j]) - kfree(sbi->s_qf_names[j]); + kfree(sbi->s_qf_names[j]); + sbi->s_qf_names[j] = NULL; } #endif - if (sbi != NULL) { - kfree(sbi); - } + kfree(sbi); s->s_fs_info = NULL; return errval; diff -puN fs/reiserfs/xattr_acl.c~reiserfs-free-checking-cleanup fs/reiserfs/xattr_acl.c --- 25/fs/reiserfs/xattr_acl.c~reiserfs-free-checking-cleanup Tue Sep 20 14:11:20 2005 +++ 25-akpm/fs/reiserfs/xattr_acl.c Tue Sep 20 14:11:20 2005 @@ -296,8 +296,7 @@ reiserfs_set_acl(struct inode *inode, in } } - if (value) - kfree(value); + kfree(value); if (!error) { /* Release the old one */ _