vfs: show_vfsstat: remove redundant initialization and check of error code
authorDmitry V. Levin <ldv@altlinux.org>
Wed, 18 Nov 2015 21:58:32 +0000 (00:58 +0300)
committerAl Viro <viro@zeniv.linux.org.uk>
Mon, 7 Dec 2015 02:17:16 +0000 (21:17 -0500)
As err variable is now always checked right after each assignment, its
initialization is redundant and could be safely removed.  For the same
reason, the last check of err is also redundant and could be removed as
well.

Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/proc_namespace.c

index 93637588edf7d3b48e985d9fe90ae5dc35801fab..2256e7e23e678a11d2ab3fbaf1226a508f0ef1c8 100644 (file)
@@ -193,7 +193,7 @@ static int show_vfsstat(struct seq_file *m, struct vfsmount *mnt)
        struct mount *r = real_mount(mnt);
        struct path mnt_path = { .dentry = mnt->mnt_root, .mnt = mnt };
        struct super_block *sb = mnt_path.dentry->d_sb;
-       int err = 0;
+       int err;
 
        /* device */
        if (sb->s_op->show_devname) {
@@ -222,8 +222,7 @@ static int show_vfsstat(struct seq_file *m, struct vfsmount *mnt)
        /* optional statistics */
        if (sb->s_op->show_stats) {
                seq_putc(m, ' ');
-               if (!err)
-                       err = sb->s_op->show_stats(m, mnt_path.dentry);
+               err = sb->s_op->show_stats(m, mnt_path.dentry);
        }
 
        seq_putc(m, '\n');