2022-05-11 13:21:09

by Wang Hai

Subject: [PATCH 0/3] Refix the socket leak in xs_setup_local()

Patch1 and patch2 roll back the wrong solution to fix socket leaks.

Patch3 adds safe teardown mechanism to re-fix socket leaks.

Wang Hai (3):
Revert "SUNRPC: Ensure gss-proxy connects on setup"
Revert "Revert "SUNRPC: attempt AF_LOCAL connect on setup""
SUNRPC: Fix local socket leak in xs_setup_local()

include/linux/sunrpc/clnt.h | 1 -
net/sunrpc/auth_gss/gss_rpc_upcall.c | 2 +-
net/sunrpc/clnt.c | 3 ---
net/sunrpc/xprtsock.c | 19 ++++++++++++++++++-
4 files changed, 19 insertions(+), 6 deletions(-)

--
2.17.1

2022-05-12 03:21:03

by Wang Hai

Subject: Re: [PATCH 0/3] Refix the socket leak in xs_setup_local()

在 2022/5/12 0:14, Trond Myklebust 写道:
> On Wed, 2022-05-11 at 21:22 +0800, Wang Hai wrote:
>> Patch1 and patch2 roll back the wrong solution to fix socket leaks.
>>
>> Patch3 adds safe teardown mechanism to re-fix socket leaks.
>>
>> Wang Hai (3):
>>   Revert "SUNRPC: Ensure gss-proxy connects on setup"
>>   Revert "Revert "SUNRPC: attempt AF_LOCAL connect on setup""
>>   SUNRPC: Fix local socket leak in xs_setup_local()
>>
>>  include/linux/sunrpc/clnt.h          |  1 -
>>  net/sunrpc/auth_gss/gss_rpc_upcall.c |  2 +-
>>  net/sunrpc/clnt.c                    |  3 ---
>>  net/sunrpc/xprtsock.c                | 19 ++++++++++++++++++-
>>  4 files changed, 19 insertions(+), 6 deletions(-)
>>
> Thanks, but there is already a fix for this queued up in the linux-next
> branch.
Thanks, sorry to bother you, I found it.

I only subscribed to the [email protected] mailing list,
not the [email protected] mailing list separately. So I
didn't notice it was fixed.

--
Wang Hai