mbox series

[mptcp-next,v2,0/3] mptcp: fix bug in userspace PM APIs

Message ID 20220420042524.2268380-1-kishen.maloor@intel.com (mailing list archive)
Headers show
Series mptcp: fix bug in userspace PM APIs | expand

Message

Kishen Maloor April 20, 2022, 4:25 a.m. UTC
This series fixes issue #267:
https://github.com/multipath-tcp/mptcp_net-next/issues/267

Specifically, calls to mptcp_token_get_sock() from userspace PM APIs
hold a reference to the msk which needs to be freed before leaving those
functions. This was causing a memory leak which was caught in
the CI debug build.

v2:
-Fixed code style.

Kishen Maloor (3):
  Squash-to: mptcp: netlink: Add MPTCP_PM_CMD_ANNOUNCE
  Squash-to: mptcp: netlink: Add MPTCP_PM_CMD_REMOVE
  Squash-to: mptcp: netlink: allow userspace-driven subflow
    establishment

 net/mptcp/pm_userspace.c | 93 +++++++++++++++++++++++-----------------
 1 file changed, 53 insertions(+), 40 deletions(-)


base-commit: e561667a821b0ecccba582520829f38da50567a3
--
2.31.1

Comments

Matthieu Baerts April 20, 2022, 3:22 p.m. UTC | #1
Hi Kishen, Geliang, Mat,

On 20/04/2022 06:25, Kishen Maloor wrote:
> This series fixes issue #267: 
> https://github.com/multipath-tcp/mptcp_net-next/issues/267
> 
> Specifically, calls to mptcp_token_get_sock() from userspace PM APIs 
> hold a reference to the msk which needs to be freed before leaving
> those functions. This was causing a memory leak which was caught in 
> the CI debug build.

Thank you for the patches and the reviews!

Please next time add more details in the different commit messages,
mainly the reason and ideally a "Closes:" tag.


Now in our tree:

New patches for t/upstream:
- b5427dca0016: "squashed" patch 1/3 in "mptcp: netlink: Add
MPTCP_PM_CMD_ANNOUNCE"
- 93a0837b24aa: "squashed" patch 2/3 in "mptcp: netlink: Add
MPTCP_PM_CMD_REMOVE"
- 1cfb03d21837: "squashed" patch 3/3 in "mptcp: netlink: allow
userspace-driven subflow establishment"
- Results: 8001d7b9c400..1e2cdcf7b0cc (export)


Builds and tests are now in progress:

https://cirrus-ci.com/github/multipath-tcp/mptcp_net-next/export/20220420T152103

https://github.com/multipath-tcp/mptcp_net-next/actions/workflows/build-validation.yml?query=branch:export


Cheers,
Matt