linux-kernel - [RESEND PATCH 1/3] vfs: show_vfsmnt: remove redundant initialization of error code

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-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160111153646.GB30070@altlinux.org>
Date:	Mon, 11 Jan 2016 18:36:46 +0300
From:	"Dmitry V. Levin" <ldv@...linux.org>
To:	Alexander Viro <viro@...iv.linux.org.uk>
Cc:	Mateusz Guzik <mguzik@...hat.com>, linux-fsdevel@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: [RESEND PATCH 1/3] vfs: show_vfsmnt: remove redundant initialization of error code

Date: Wed Nov 18 21:07:10 2015 +0000

As err variable is now always checked right after the first assignment,
its initialization is redundant and could be safely removed.

Suggested-by: Mateusz Guzik <mguzik@...hat.com>
Signed-off-by: Dmitry V. Levin <ldv@...linux.org>
---
 fs/proc_namespace.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/proc_namespace.c b/fs/proc_namespace.c
index 876459559..cbc9c27 100644
--- a/fs/proc_namespace.c
+++ b/fs/proc_namespace.c
@@ -95,9 +95,9 @@ static int show_vfsmnt(struct seq_file *m, struct vfsmount *mnt)
 {
 	struct proc_mounts *p = m->private;
 	struct mount *r = real_mount(mnt);
-	int err = 0;
 	struct path mnt_path = { .dentry = mnt->mnt_root, .mnt = mnt };
 	struct super_block *sb = mnt_path.dentry->d_sb;
+	int err;
 
 	if (sb->s_op->show_devname) {
 		err = sb->s_op->show_devname(m, mnt_path.dentry);

-- 
ldv