From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 630CA3C4178; Tue, 16 Jun 2026 17:43:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781631802; cv=none; b=uDkF8YH/j+rFbD8COUo5I5WxgawE7cELRWgA5NL7okn0TnDSEY4DAIX6Ql7nFbgPq2ufTXT/ffkSKIaW8O4kAWb9Gzz7W6KZRuLBGKohZ8Wj+04VGvcq8cbJ8UipKyWxwWZDRjNFE++OGvw2a3kNLHGw0lBwpGETFO8YfLlEe2o= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781631802; c=relaxed/simple; bh=t5BVxks0zRdWF1FzqbzpIFXLx/jTfI27GL+bRgueaNE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=BkbmtI7P6M6Y/Mgczr8cEKQh/9IVmXoC4HLY3/oC8frDsPwBLQIGBEi1qzT5aURLvy8y501LtuqBxz/GLVldMetZ/qgEg0GfESRXqs8ue+fHF1TQgmlOzZLOrB0zi4jPyMFGMcmQrQIBvJJy5AWuo50AaXciIREoQqLl/yBW/7Y= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=r7thrx3T; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="r7thrx3T" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 326AB1F000E9; Tue, 16 Jun 2026 17:43:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=korg; t=1781631800; bh=ePlb/aRBztxw1xcs+ECLVTtwK9LYuDlOzlBCxH7egnI=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=r7thrx3TuXWDAhi+mdb6CS18DYJXcVnzMLjZgnEJ9d+YJNU+z1Nv6l/a5ylClJJXd 79X+KsqUGYTj2alCW2Yr+tKhg2KleaKeNsKgR+jZq9zm15tcn7rmrcSl+crYAfYy7X 3yhoOwLU/jYN2b7QFjIv8QHZpIqvma1crTCUITgQ= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Raf Dickson , Stefano Garzarella , Paolo Abeni Subject: [PATCH 6.1 290/522] vsock/vmci: fix sk_ack_backlog leak on failed handshake Date: Tue, 16 Jun 2026 20:27:17 +0530 Message-ID: <20260616145139.498361367@linuxfoundation.org> X-Mailer: git-send-email 2.54.0 In-Reply-To: <20260616145125.307082728@linuxfoundation.org> References: <20260616145125.307082728@linuxfoundation.org> User-Agent: quilt/0.69 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 6.1-stable review patch. If anyone has any objections, please let me know. ------------------ From: Raf Dickson commit c05fa14db43ebef3bd862ca9d073981c0358b3f0 upstream. When vmci_transport_recv_connecting_server() returns an error, vmci_transport_recv_listen() calls vsock_remove_pending() but never calls sk_acceptq_removed(). This leaves sk_ack_backlog incremented permanently. Repeated handshake failures (malformed packets, queue pair alloc failure, event subscribe failure) cause sk_ack_backlog to climb toward sk_max_ack_backlog. Once it reaches the limit the listener permanently refuses all new connections with -ECONNREFUSED, a silent denial of service requiring a process restart to recover. The two existing sk_acceptq_removed() calls in af_vsock.c do not cover this path: line 764 checks vsock_is_pending() which returns false after vsock_remove_pending(), and line 1889 is only reached on successful accept(). Fix by balancing sk_acceptq_added() with sk_acceptq_removed() on the error path. Fixes: d021c344051a ("VSOCK: Introduce VM Sockets") Cc: stable@vger.kernel.org Signed-off-by: Raf Dickson Acked-by: Stefano Garzarella Link: https://patch.msgid.link/20260526104356.469928-1-rafdog35@gmail.com Signed-off-by: Paolo Abeni Signed-off-by: Greg Kroah-Hartman --- net/vmw_vsock/vmci_transport.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) --- a/net/vmw_vsock/vmci_transport.c +++ b/net/vmw_vsock/vmci_transport.c @@ -972,8 +972,10 @@ static int vmci_transport_recv_listen(st err = -EINVAL; } - if (err < 0) + if (err < 0) { vsock_remove_pending(sk, pending); + sk_acceptq_removed(sk); + } release_sock(pending); vmci_transport_release_pending(pending);