Re: [PATCH] net: sock: correctly handle failed prog retrieval from fd

From: Alexei Starovoitov
Date: Fri Dec 12 2014 - 01:34:55 EST


On Thu, Dec 11, 2014 at 9:33 PM, Sasha Levin <sasha.levin@xxxxxxxxxx> wrote:
> diff --git a/net/core/filter.c b/net/core/filter.c
> index 8cc3c03..ec9baea 100644
> --- a/net/core/filter.c
> +++ b/net/core/filter.c
> @@ -1103,8 +1103,8 @@ int sk_attach_bpf(u32 ufd, struct sock *sk)
> return -EPERM;
>
> prog = bpf_prog_get(ufd);
> - if (!prog)
> - return -EINVAL;
> + if (IS_ERR(prog))
> + return PTR_ERR(prog);

thank you for the fix, but the same fix is already applied
to net-next and pushed all the way to:
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=198bf1b046e370a7d3987b195cff5f1efebec3ac
sorry for the headache that this bug caused.
--
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/