diff options
author | Wei Yongjun <weiyongjun1@huawei.com> | 2020-04-27 10:43:48 +0000 |
---|---|---|
committer | Kalle Valo <kvalo@codeaurora.org> | 2020-05-04 11:52:07 +0300 |
commit | 2326aa011967f0afbcba7fe1a005d01f8b12900b (patch) | |
tree | ad7bad3fd2c9cb576b6ec24e42034f8b9f766643 | |
parent | 69c93f9674c97dc439cdc0527811f8ad104c2e35 (diff) |
ath10k: fix possible memory leak in ath10k_bmi_lz_data_large()
'cmd' is malloced in ath10k_bmi_lz_data_large() and should be freed
before leaving from the error handling cases, otherwise it will cause
memory leak.
Fixes: d58f466a5dee ("ath10k: add large size for BMI download data for SDIO")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200427104348.13570-1-weiyongjun1@huawei.com
-rw-r--r-- | drivers/net/wireless/ath/ath10k/bmi.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/wireless/ath/ath10k/bmi.c b/drivers/net/wireless/ath/ath10k/bmi.c index ea908107581d..5b6db6e66f65 100644 --- a/drivers/net/wireless/ath/ath10k/bmi.c +++ b/drivers/net/wireless/ath/ath10k/bmi.c @@ -380,6 +380,7 @@ static int ath10k_bmi_lz_data_large(struct ath10k *ar, const void *buffer, u32 l NULL, NULL); if (ret) { ath10k_warn(ar, "unable to write to the device\n"); + kfree(cmd); return ret; } |