wifi: ath12k: fix skb_ext_desc leak in ath12k_dp_tx() error path
authorNicolas Escande <nico.escande@gmail.com>
Wed, 22 Jan 2025 16:01:12 +0000 (17:01 +0100)
committerJeff Johnson <jeff.johnson@oss.qualcomm.com>
Thu, 23 Jan 2025 16:02:58 +0000 (08:02 -0800)
When vlan support was added, we missed that when
ath12k_dp_prepare_htt_metadata() returns an error we also need to free
the skb holding the metadata before going on with the cleanup process.

Compile tested only.

Fixes: 26dd8ccdba4d ("wifi: ath12k: dynamic VLAN support")
Signed-off-by: Nicolas Escande <nico.escande@gmail.com>
Reviewed-by: Aditya Kumar Singh <aditya.kumar.singh@oss.qualcomm.com>
Link: https://patch.msgid.link/20250122160112.3234558-1-nico.escande@gmail.com
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
drivers/net/wireless/ath/ath12k/dp_tx.c

index a8d341a6df01ea786bf4005fa419a568031611d6..e0b85f959cd4ae407fa2186e3b94f937b7988f60 100644 (file)
@@ -398,6 +398,7 @@ map:
                        if (ret < 0) {
                                ath12k_dbg(ab, ATH12K_DBG_DP_TX,
                                           "Failed to add HTT meta data, dropping packet\n");
+                               kfree_skb(skb_ext_desc);
                                goto fail_unmap_dma;
                        }
                }