Re: [PATCH] ath10k: fix memory leak on caldata on error exit path
flat view
From: Valo, Kalle <hidden>
Date: 2016-09-02 15:45:17
Also in:
lkml, netdev
Colin King [off-list ref] writes:
quoted hunk
From: Colin Ian King <redacted> caldata is not being free'd on the error exit path, causing a memory leak. kfree it to fix the leak. Signed-off-by: Colin Ian King <redacted> --- drivers/net/wireless/ath/ath10k/pci.c | 1 + 1 file changed, 1 insertion(+)diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless=
/ath/ath10k/pci.c
quoted hunk
index 9a22c47..886337c 100644--- a/drivers/net/wireless/ath/ath10k/pci.c +++ b/drivers/net/wireless/ath/ath10k/pci.c@@ -2725,6 +2725,7 @@ static int ath10k_pci_hif_fetch_cal_eeprom(struct a=
th10k *ar, void **data,
return 0; =20 err_free: + kfree(caldata); kfree(data); =20 return -EINVAL;
I don't think we should free data at all:
static int ath10k_download_cal_eeprom(struct ath10k *ar)
{
size_t data_len;
void *data =3D NULL;
int ret;
ret =3D ath10k_hif_fetch_cal_eeprom(ar, &data, &data_len);
Instead we should free only caldata, right?
--=20
Kalle Valo=