On 3/1/18 9:07 PM, Ka-Cheong Poon wrote:
Commit 0933a578cd55 ("rds: tcp: use sock_create_lite() to create the
accept socket") has a reference counting issue in TCP socket creation
when accepting a new connection. The code uses sock_create_lite() to
create a kernel socket. But it does not do __module_get() on the
socket owner. When the connection is shutdown and sock_release() is
called to free the socket, the owner's reference count is decremented
and becomes incorrect. Note that this bug only shows up when the socket
owner is configured as a kernel module.
v2: Update comments
Versioning comment typically goes below "---" and not part of
commit message.
Signed-off-by: Ka-Cheong Poon <[email protected]>
---
Patch looks fine.
Acked-by: Santosh Shilimkar <[email protected]>