diff options
| author | Alexey V. Vissarionov <gremlin@altlinux.org> | 2023-02-15 20:31:37 +0200 |
|---|---|---|
| committer | Kalle Valo <quic_kvalo@quicinc.com> | 2023-02-17 18:03:04 +0200 |
| commit | 778f83f889e7fca37780d9640fcbd0229ae38eaa (patch) | |
| tree | bc43bbd20a27484f69797bae32b575716022f01e /drivers/net/wireless/ath/ath6kl/bmi.c | |
| parent | cf8f3d4deb02a8fdc806c46d4112b69868544697 (diff) | |
wifi: ath6kl: minor fix for allocation size
Although the "param" pointer occupies more or equal space compared
to "*param", the allocation size should use the size of variable
itself.
Found by Linux Verification Center (linuxtesting.org) with SVACE.
Fixes: bdcd81707973cf8a ("Add ath6kl cleaned up driver")
Signed-off-by: Alexey V. Vissarionov <gremlin@altlinux.org>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20230117110414.GC12547@altlinux.org
Diffstat (limited to 'drivers/net/wireless/ath/ath6kl/bmi.c')
| -rw-r--r-- | drivers/net/wireless/ath/ath6kl/bmi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/ath/ath6kl/bmi.c b/drivers/net/wireless/ath/ath6kl/bmi.c index bde5a10d470c..af98e871199d 100644 --- a/drivers/net/wireless/ath/ath6kl/bmi.c +++ b/drivers/net/wireless/ath/ath6kl/bmi.c @@ -246,7 +246,7 @@ int ath6kl_bmi_execute(struct ath6kl *ar, u32 addr, u32 *param) return -EACCES; } - size = sizeof(cid) + sizeof(addr) + sizeof(param); + size = sizeof(cid) + sizeof(addr) + sizeof(*param); if (size > ar->bmi.max_cmd_size) { WARN_ON(1); return -EINVAL; |