[rds-devel] [PATCH v2 net] rds: Incorrect reference counting in TCP socket creation

santosh.shilimkar at oracle.com santosh.shilimkar at oracle.com
Thu Mar 1 22:22:07 PST 2018



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 <ka-cheong.poon at oracle.com>
> ---
Patch looks fine.
Acked-by: Santosh Shilimkar <santosh.shilimkar at oracle.com>



More information about the rds-devel mailing list