Re: [PATCH] wimax: remove dead code

From: Joe Perches
Date: Wed Nov 20 2013 - 23:18:01 EST


On Thu, 2013-11-21 at 04:43 +0100, Michael Opdenacker wrote:
> This removes a code line that is between a "return 0;" and an error label.
> This code line can never be reached.

Adding Johannes Berg who added this oversight.

> Found by Coverity (CID: 1130529)
>
> Signed-off-by: Michael Opdenacker <michael.opdenacker@xxxxxxxxxxxxxxxxxx>
> ---
> net/wimax/stack.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/net/wimax/stack.c b/net/wimax/stack.c
> index ef2191b969a7..ec8b577db135 100644
> --- a/net/wimax/stack.c
> +++ b/net/wimax/stack.c
> @@ -610,7 +610,6 @@ int __init wimax_subsys_init(void)
> d_fnend(4, NULL, "() = 0\n");
> return 0;
>
> - genl_unregister_family(&wimax_gnl_family);
> error_register_family:
> d_fnend(4, NULL, "() = %d\n", result);
> return result;



--
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/