mirror of
https://github.com/torvalds/linux.git
synced 2026-05-25 23:52:08 +02:00
nvme-tcp: fix use-after-free of netns by kernel TCP socket.
Commit1be52169c3("nvme-tcp: fix selinux denied when calling sock_sendmsg") converted sock_create() in nvme_tcp_alloc_queue() to sock_create_kern(). sock_create_kern() creates a kernel socket, which does not hold a reference to netns. If the code does not manage the netns lifetime properly, use-after-free could happen. Also, TCP kernel socket with sk_net_refcnt 0 has a socket leak problem: it remains FIN_WAIT_1 if it misses FIN after close() because tcp_close() stops all timers. To fix such problems, let's hold netns ref by sk_net_refcnt_upgrade(). We had the same issue in CIFS, SMC, etc, and applied the same solution, see commitef7134c7fc("smb: client: Fix use-after-free of network namespace.") and commit9744d2bf19("smc: Fix use-after-free in tcp_write_timer_handler()."). Fixes:1be52169c3("nvme-tcp: fix selinux denied when calling sock_sendmsg") Signed-off-by: Kuniyuki Iwashima <kuniyu@amazon.com> Signed-off-by: Christoph Hellwig <hch@lst.de>
This commit is contained in:
parent
e3105f54a5
commit
b013b817f3
|
|
@ -1803,6 +1803,8 @@ static int nvme_tcp_alloc_queue(struct nvme_ctrl *nctrl, int qid,
|
|||
ret = PTR_ERR(sock_file);
|
||||
goto err_destroy_mutex;
|
||||
}
|
||||
|
||||
sk_net_refcnt_upgrade(queue->sock->sk);
|
||||
nvme_tcp_reclassify_socket(queue->sock);
|
||||
|
||||
/* Single syn retry */
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user