On 2018/6/1 19:16, YueHaibing wrote: > sizeof(struct ce_desc) should be a copy-paste mistake > just use sizeof(struct ce_desc_64) to avoid mem leak > > Fixes: b7ba83f7c414 ("ath10k: add support for shadow register for WNC3990") > Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx> > --- > drivers/net/wireless/ath/ath10k/ce.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/ath/ath10k/ce.c b/drivers/net/wireless/ath/ath10k/ce.c > index 3b96a43..35dec2a 100644 > --- a/drivers/net/wireless/ath/ath10k/ce.c > +++ b/drivers/net/wireless/ath/ath10k/ce.c > @@ -1512,7 +1512,7 @@ ath10k_ce_alloc_src_ring_64(struct ath10k *ar, unsigned int ce_id, > ret = ath10k_ce_alloc_shadow_base(ar, src_ring, nentries); > if (ret) { > dma_free_coherent(ar->dev, > - (nentries * sizeof(struct ce_desc) + > + (nentries * sizeof(struct ce_desc64) + sorry ,I post the wrong patch,will send V2 > CE_DESC_RING_ALIGN), > src_ring->base_addr_owner_space_unaligned, > base_addr); >