Re: [PATCH 3/5] fs/namespace.c: Drop code after return

From: Richard Genoud
Date: Thu Sep 11 2008 - 09:47:49 EST


2008/9/11 Julia Lawall <julia@xxxxxxx>:
> From: Julia Lawall <julia@xxxxxxx>
>
> The extra semicolon serves no purpose.
>
> Signed-off-by: Julia Lawall <julia@xxxxxxx>
Reviewed-by: Richard Genoud <richard.genoud@xxxxxxxxx>
> ---
>
> fs/namespace.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/fs/namespace.c b/fs/namespace.c
> index 6e283c9..7e6151a 100644
> --- a/fs/namespace.c
> +++ b/fs/namespace.c
> @@ -1993,7 +1993,7 @@ static struct mnt_namespace *dup_mnt_ns(struct mnt_namespace *mnt_ns,
> if (!new_ns->root) {
> up_write(&namespace_sem);
> kfree(new_ns);
> - return ERR_PTR(-ENOMEM);;
> + return ERR_PTR(-ENOMEM);
> }
> spin_lock(&vfsmount_lock);
> list_add_tail(&new_ns->list, &new_ns->root->mnt_list);
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/