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 AB7B022B594; Fri, 7 Aug 2026 14:46:51 +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=1786114013; cv=none; b=WXAg7XzBLWWC+3cQVS+m7L8XVXzIp4co1R+TFlgZXM5KX3w0MIPRdYOGVSDJ6v9a0Fm1FvCE8XxG6+sqX9srhgp77UGHXRpun5zYBW6m9hkMazZ9OuzWe+fezo8yE1qsIiZgN5eiar4n1eqANKYGWmokT61bY53aunSJZduCt7w= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786114013; c=relaxed/simple; bh=NV3MH+feR/3NPZ3F/8M2WDcQnkreQ3uceIzMXFvgNRM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=eZ4gqrI3tnnWc3Ugoxc2BsTDxfMExXV5zXcZPet5ntTbIwfl7FLgaCQ6PfjJpT0KVHJr8mJqXWa+/l9r9dLQ/r4/R4hsTQ7JYU7yX+GcabIf3j6AAn9FS+K6NT5ebUYbvR7wQ1TpJaaIVFapbEOiWlEddhSHbDVPRb73nLuO9EY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=K+ao16rt; 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="K+ao16rt" Received: by smtp.kernel.org (Postfix) with ESMTPSA id C025E1F000E9; Fri, 7 Aug 2026 14:46:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=korg; t=1786114011; bh=xiutd1QlgJ4oEJQKUruu8LwojRlsnAZfKkA3SL9902k=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=K+ao16rtRB8DYBwKsTLdM2J28KeCYRFZwDPsomhxdR46+nS3aj04syz/80KQ55UA6 kuT7A0D08I7WVqXdXgyGZyeK329dU0eo9fW+9/ZDkoL4nzu8f3dtGupod5dWzgn5Lk u/dgWuHQlTsUuXcr+Ual5/AOowWSv8QeQg59BIMg= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Chenguang Zhao , Vadim Fedorenko , "David S. Miller" , Sasha Levin Subject: [PATCH 6.12 112/337] net: sxgbe: free TX rings on RX allocation failure Date: Fri, 7 Aug 2026 16:35:15 +0200 Message-ID: <20260807143420.960379992@linuxfoundation.org> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260807143418.516897842@linuxfoundation.org> References: <20260807143418.516897842@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.12-stable review patch. If anyone has any objections, please let me know. ------------------ From: Chenguang Zhao [ Upstream commit c870f7e2890b9f78ac84515a9809cc5c183c975e ] When RX descriptor ring allocation fails, init_dma_desc_rings() only frees the partially allocated RX rings and returns. The TX rings that were allocated earlier in the same function are leaked. Rearrange error labels to clean up TX rings upon RX failures. Fixes: 1edb9ca69e8a ("net: sxgbe: add basic framework for Samsung 10Gb ethernet driver") Signed-off-by: Chenguang Zhao Reviewed-by: Vadim Fedorenko Signed-off-by: David S. Miller Signed-off-by: Sasha Levin --- drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c b/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c index d2ba283f6123f..a4f457fa570d9 100644 --- a/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c +++ b/drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c @@ -598,14 +598,13 @@ static int init_dma_desc_rings(struct net_device *netd) return 0; -txalloc_err: - while (queue_num--) - free_tx_ring(priv->device, priv->txq[queue_num], tx_rsize); - return ret; - rxalloc_err: while (queue_num--) free_rx_ring(priv->device, priv->rxq[queue_num], rx_rsize); + queue_num = SXGBE_TX_QUEUES; +txalloc_err: + while (queue_num--) + free_tx_ring(priv->device, priv->txq[queue_num], tx_rsize); return ret; } -- 2.53.0