diff options
| author | Dmitry V. Levin <ldv@altlinux.org> | 2015-11-19 00:58:20 +0300 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2015-12-06 21:17:15 -0500 |
| commit | 6ce4bca0adfde9ee404ce659d110f2bdeff9c13b (patch) | |
| tree | 0e902fda2f9d1f4c5bf6715fd9d690cdc749ca78 | |
| parent | 5d9f3c7b620f6d1d9555223817bdfddfbd4b93a0 (diff) | |
| download | op-kernel-dev-6ce4bca0adfde9ee404ce659d110f2bdeff9c13b.zip op-kernel-dev-6ce4bca0adfde9ee404ce659d110f2bdeff9c13b.tar.gz | |
vfs: show_mountinfo: cleanup error code checks
Check err variable right after each assignment. This change makes
initialization of err redundant, so remove the initialization.
Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
| -rw-r--r-- | fs/proc_namespace.c | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/fs/proc_namespace.c b/fs/proc_namespace.c index 1d984b5..9363758 100644 --- a/fs/proc_namespace.c +++ b/fs/proc_namespace.c @@ -131,16 +131,17 @@ static int show_mountinfo(struct seq_file *m, struct vfsmount *mnt) struct mount *r = real_mount(mnt); struct super_block *sb = mnt->mnt_sb; struct path mnt_path = { .dentry = mnt->mnt_root, .mnt = mnt }; - int err = 0; + int err; seq_printf(m, "%i %i %u:%u ", r->mnt_id, r->mnt_parent->mnt_id, MAJOR(sb->s_dev), MINOR(sb->s_dev)); - if (sb->s_op->show_path) + if (sb->s_op->show_path) { err = sb->s_op->show_path(m, mnt->mnt_root); - else + if (err) + goto out; + } else { seq_dentry(m, mnt->mnt_root, " \t\n\\"); - if (err) - goto out; + } seq_putc(m, ' '); /* mountpoints outside of chroot jail will give SEQ_SKIP on this */ @@ -168,12 +169,13 @@ static int show_mountinfo(struct seq_file *m, struct vfsmount *mnt) seq_puts(m, " - "); show_type(m, sb); seq_putc(m, ' '); - if (sb->s_op->show_devname) + if (sb->s_op->show_devname) { err = sb->s_op->show_devname(m, mnt->mnt_root); - else + if (err) + goto out; + } else { mangle(m, r->mnt_devname ? r->mnt_devname : "none"); - if (err) - goto out; + } seq_puts(m, sb->s_flags & MS_RDONLY ? " ro" : " rw"); err = show_sb_opts(m, sb); if (err) |
