Re: [PATCH mptcp-net v2] mptcp: fix extra_subflows leak on failed passive join

Matthieu Baerts <[email protected]> Tue, 21 Jul 2026 19:22:05 +0200
Newsgroups dev.linux.lists.mptcp
Organization NGI0 Core
Message-ID <[email protected]>
Hi Chenguang,

On 01/07/2026 05:20, Chenguang Zhao wrote:
> From: Chenguang Zhao <[email protected]>
> 
> mptcp_pm_allow_new_subflow() increments extra_subflows
> before __mptcp_finish_join() on the passive MP_JOIN path.
> On synchronous join failure the subflow is dropped without
> calling mptcp_close_ssk(), so the counter is not rolled back.
> 
> Call mptcp_pm_close_subflow() when the join completion fails.
> 
> Fixes: 3e5014909b56 ("mptcp: cleanup MPJ subflow list handling")
> Signed-off-by: Chenguang Zhao <[email protected]>
> Reviewed-by: Matthieu Baerts (NGI0) <[email protected]>

Thank you for the patch. Now in our tree (with small adaptations in the
commit message):

New patches for t/upstream-net and t/upstream:
- 5cc023a79ad5: mptcp: decrement subflows counter on failed passive join
- Results: 6e75226ac108..4c2362a7db02 (export-net)
- Results: 4c2752580a3e..4285a4fe811b (export)

Tests are now in progress:

- export-net:
https://github.com/multipath-tcp/mptcp_net-next/commit/bc033e934bd44ead5a065f26cb78af76769e90de/checks
- export:
https://github.com/multipath-tcp/mptcp_net-next/commit/254350c480d5cc9124500ba3588f2d24a1905608/checks

Cheers,
Matt
-- 
Sponsored by the NGI0 Core fund.