Re: [PATCH] net: ip, diag -- Add diag interface for raw sockets

From: Cyrill Gorcunov
Date: Fri Sep 09 2016 - 16:17:43 EST


On Fri, Sep 09, 2016 at 12:55:13PM -0700, Eric Dumazet wrote:
> > +
> > + rep = nlmsg_new(sizeof(struct inet_diag_msg) +
> > + sizeof(struct inet_diag_meminfo) + 64,
> > + GFP_KERNEL);
> > + if (!rep)
>
> There is a missing sock_put(sk)
>
> > + return -ENOMEM;
> > +
> > + err = inet_sk_diag_fill(sk, NULL, rep, r,
> > + sk_user_ns(NETLINK_CB(in_skb).sk),
> > + NETLINK_CB(in_skb).portid,
> > + nlh->nlmsg_seq, 0, nlh);
>
> sock_put(sk);

Oh, missed. Thanks lot, Eric, will update!