* [PATCH 4.14] phonet: refcount leak in pep_sock_accep
@ 2022-01-07 10:53 Aayush Agarwal
2022-01-07 11:24 ` Greg KH
0 siblings, 1 reply; 4+ messages in thread
From: Aayush Agarwal @ 2022-01-07 10:53 UTC (permalink / raw)
Cc: aayush.a.agarwal, stable, Hangyu Hua, Jakub Kicinski,
Remi Denis-Courmont, David S. Miller, netdev, linux-kernel
From: Hangyu Hua <hbh25y@gmail.com>
commit bcd0f9335332 ("phonet: refcount leak in pep_sock_accep")
upstream.
sock_hold(sk) is invoked in pep_sock_accept(), but __sock_put(sk) is not
invoked in subsequent failure branches(pep_accept_conn() != 0).
Signed-off-by: Hangyu Hua <hbh25y@gmail.com>
Link: https://lore.kernel.org/r/20211209082839.33985-1-hbh25y@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Aayush Agarwal <aayush.a.agarwal@oracle.com>
---
net/phonet/pep.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/net/phonet/pep.c b/net/phonet/pep.c
index b0d958cd1823..4c4a8a42ee88 100644
--- a/net/phonet/pep.c
+++ b/net/phonet/pep.c
@@ -881,6 +881,7 @@ static struct sock *pep_sock_accept(struct sock *sk, int flags, int *errp,
err = pep_accept_conn(newsk, skb);
if (err) {
+ __sock_put(sk);
sock_put(newsk);
newsk = NULL;
goto drop;
--
2.27.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 4.14] phonet: refcount leak in pep_sock_accep
2022-01-07 10:53 [PATCH 4.14] phonet: refcount leak in pep_sock_accep Aayush Agarwal
@ 2022-01-07 11:24 ` Greg KH
2022-01-07 16:54 ` [External] : " aayush.a.agarwal
0 siblings, 1 reply; 4+ messages in thread
From: Greg KH @ 2022-01-07 11:24 UTC (permalink / raw)
To: Aayush Agarwal
Cc: stable, Hangyu Hua, Jakub Kicinski, Remi Denis-Courmont,
David S. Miller, netdev, linux-kernel
On Fri, Jan 07, 2022 at 02:53:32AM -0800, Aayush Agarwal wrote:
> From: Hangyu Hua <hbh25y@gmail.com>
>
> commit bcd0f9335332 ("phonet: refcount leak in pep_sock_accep")
> upstream.
>
> sock_hold(sk) is invoked in pep_sock_accept(), but __sock_put(sk) is not
> invoked in subsequent failure branches(pep_accept_conn() != 0).
>
> Signed-off-by: Hangyu Hua <hbh25y@gmail.com>
> Link: https://lore.kernel.org/r/20211209082839.33985-1-hbh25y@gmail.com
> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
> Signed-off-by: Aayush Agarwal <aayush.a.agarwal@oracle.com>
> ---
> net/phonet/pep.c | 1 +
> 1 file changed, 1 insertion(+)
What about releases 5.15.y, 5.10.y, 5.4.y, and 4.19.y? Is this also
relevant for those trees?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [External] : Re: [PATCH 4.14] phonet: refcount leak in pep_sock_accep
2022-01-07 11:24 ` Greg KH
@ 2022-01-07 16:54 ` aayush.a.agarwal
2022-01-09 13:00 ` Greg KH
0 siblings, 1 reply; 4+ messages in thread
From: aayush.a.agarwal @ 2022-01-07 16:54 UTC (permalink / raw)
To: Greg KH
Cc: stable, Hangyu Hua, Jakub Kicinski, Remi Denis-Courmont,
David S. Miller, netdev, linux-kernel
On 07/01/22 4:54 pm, Greg KH wrote:
> On Fri, Jan 07, 2022 at 02:53:32AM -0800, Aayush Agarwal wrote:
>> From: Hangyu Hua <hbh25y@gmail.com>
>>
>> commit bcd0f9335332 ("phonet: refcount leak in pep_sock_accep")
>> upstream.
>>
>> sock_hold(sk) is invoked in pep_sock_accept(), but __sock_put(sk) is not
>> invoked in subsequent failure branches(pep_accept_conn() != 0).
>>
>> Signed-off-by: Hangyu Hua <hbh25y@gmail.com>
>> Link: https://urldefense.com/v3/__https://lore.kernel.org/r/20211209082839.33985-1-hbh25y@gmail.com__;!!ACWV5N9M2RV99hQ!Znc0Oy9gtZZ18UDMwcZiYrfjj4GUibhEq5WJZ44m6azDWCC1hrZpkFh9AmGOqqS94cqz-A$
>> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
>> Signed-off-by: Aayush Agarwal <aayush.a.agarwal@oracle.com>
>> ---
>> net/phonet/pep.c | 1 +
>> 1 file changed, 1 insertion(+)
> What about releases 5.15.y, 5.10.y, 5.4.y, and 4.19.y? Is this also
> relevant for those trees?
>
> thanks,
>
> greg k-h
It's relevant for all currently supported stable releases: 4.4.y, 4.9.y,
4.14.y, 4.19.y, 5.4.y, 5.10.y, 5.15.y . I missed adding the tag "Cc:
stable@viger.kernel.org #4.4+". Should I send the patch again?
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [External] : Re: [PATCH 4.14] phonet: refcount leak in pep_sock_accep
2022-01-07 16:54 ` [External] : " aayush.a.agarwal
@ 2022-01-09 13:00 ` Greg KH
0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2022-01-09 13:00 UTC (permalink / raw)
To: aayush.a.agarwal
Cc: stable, Hangyu Hua, Jakub Kicinski, Remi Denis-Courmont,
David S. Miller, netdev, linux-kernel
On Fri, Jan 07, 2022 at 10:24:53PM +0530, aayush.a.agarwal@oracle.com wrote:
>
> On 07/01/22 4:54 pm, Greg KH wrote:
> > On Fri, Jan 07, 2022 at 02:53:32AM -0800, Aayush Agarwal wrote:
> > > From: Hangyu Hua <hbh25y@gmail.com>
> > >
> > > commit bcd0f9335332 ("phonet: refcount leak in pep_sock_accep")
> > > upstream.
> > >
> > > sock_hold(sk) is invoked in pep_sock_accept(), but __sock_put(sk) is not
> > > invoked in subsequent failure branches(pep_accept_conn() != 0).
> > >
> > > Signed-off-by: Hangyu Hua <hbh25y@gmail.com>
> > > Link: https://urldefense.com/v3/__https://lore.kernel.org/r/20211209082839.33985-1-hbh25y@gmail.com__;!!ACWV5N9M2RV99hQ!Znc0Oy9gtZZ18UDMwcZiYrfjj4GUibhEq5WJZ44m6azDWCC1hrZpkFh9AmGOqqS94cqz-A$
> > > Signed-off-by: Jakub Kicinski <kuba@kernel.org>
> > > Signed-off-by: Aayush Agarwal <aayush.a.agarwal@oracle.com>
> > > ---
> > > net/phonet/pep.c | 1 +
> > > 1 file changed, 1 insertion(+)
> > What about releases 5.15.y, 5.10.y, 5.4.y, and 4.19.y? Is this also
> > relevant for those trees?
> >
> > thanks,
> >
> > greg k-h
>
> It's relevant for all currently supported stable releases: 4.4.y, 4.9.y,
> 4.14.y, 4.19.y, 5.4.y, 5.10.y, 5.15.y . I missed adding the tag "Cc:
> stable@viger.kernel.org #4.4+". Should I send the patch again?
No need, I've queued it up everywhere now, thanks!
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-01-09 13:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-07 10:53 [PATCH 4.14] phonet: refcount leak in pep_sock_accep Aayush Agarwal
2022-01-07 11:24 ` Greg KH
2022-01-07 16:54 ` [External] : " aayush.a.agarwal
2022-01-09 13:00 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).