All of lore.kernel.org
 help / color / mirror / Atom feed
* [MPTCP] [PATCH] mptcp: Add sock_put() call on error path
@ 2019-08-12 23:57 Peter Krystad
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Krystad @ 2019-08-12 23:57 UTC (permalink / raw)
  To: mptcp 

[-- Attachment #1: Type: text/plain, Size: 555 bytes --]

Per final review comment

squashto: Add handling of outgoing MP_JOIN requests

Signed-off-by: Peter Krystad <peter.krystad(a)linux.intel.com>
---
 net/mptcp/subflow.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/mptcp/subflow.c b/net/mptcp/subflow.c
index 448c9e7f567a..27afc64d377f 100644
--- a/net/mptcp/subflow.c
+++ b/net/mptcp/subflow.c
@@ -257,6 +257,7 @@ int subflow_connect(struct sock *sk, struct sockaddr_in *local,
 	return err;
 
 failed:
+	sock_put(sf->sk);
 	sock_release(sf);
 	return err;
 }
-- 
2.17.2


^ permalink raw reply related	[flat|nested] 2+ messages in thread
* Re: [MPTCP] [PATCH] mptcp: Add sock_put() call on error path
@ 2019-08-13 15:16 Matthieu Baerts
  0 siblings, 0 replies; 2+ messages in thread
From: Matthieu Baerts @ 2019-08-13 15:16 UTC (permalink / raw)
  To: mptcp 

[-- Attachment #1: Type: text/plain, Size: 541 bytes --]

Hi Mat, Peter,

On 13/08/2019 01:57, Peter Krystad wrote:
> Per final review comment
> 
> squashto: Add handling of outgoing MP_JOIN requests

Thank you for finding the issue and fixing it!

- 527d1a3ee6a8: "squashed" in "mptcp: Add handling of outgoing MP_JOIN
requests"
- 6e195bff312b..a873cedf4afa: result

Tests are still OK.

Cheers,
Matt
-- 
Matthieu Baerts | R&D Engineer
matthieu.baerts(a)tessares.net
Tessares SA | Hybrid Access Solutions
www.tessares.net
1 Avenue Jean Monnet, 1348 Louvain-la-Neuve, Belgium

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-08-13 15:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-08-12 23:57 [MPTCP] [PATCH] mptcp: Add sock_put() call on error path Peter Krystad
  -- strict thread matches above, loose matches on Subject: below --
2019-08-13 15:16 Matthieu Baerts

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.