[PATCH ath-next v3 1/4] wifi: ath12k: Handle error cases during extended skb allocation

[Vasanthakumar Thiagarajan]

On 4/11/2025 11:31 AM, P Praneesh wrote:
> Currently, in the case of extended skb allocation, the buffer is freed
> before the DMA unmap operation. This premature deletion can result in
> skb->data corruption, as the memory region could be re-allocated for other
> purposes. Fix this issue by reordering the failure cases by calling
> dma_unmap_single() first, then followed by the corresponding kfree_skb().
> This helps avoid data corruption in case of failures in dp_tx().
> 
> Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.4.1-00199-QCAHKSWPL_SILICONZ-1
> Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0.c5-00481-QCAHMTSWPL_V1.0_V2.0_SILICONZ-3
> 
> Fixes: d889913205cf ("wifi: ath12k: driver for Qualcomm Wi-Fi 7 devices")
> Signed-off-by: P Praneesh <praneesh.p at oss.qualcomm.com>

Reviewed-by: Vasanthakumar Thiagarajan <vasanthakumar.thiagarajan at oss.qualcomm.com>