diff options
| author | Dmitry Antipov <dmantipov@yandex.ru> | 2023-08-02 18:07:15 +0200 |
|---|---|---|
| committer | Kalle Valo <kvalo@kernel.org> | 2023-08-21 17:56:26 +0200 |
| commit | 9c8fd72a5c2a031cbc680a2990107ecd958ffcdb (patch) | |
| tree | 4fe86611dadc1fa61f4549c9d05aa74a0b81728c /drivers/net/wireless/marvell/mwifiex/debugfs.c | |
| parent | pds_core: Fix some kernel-doc comments (diff) | |
| download | linux-9c8fd72a5c2a031cbc680a2990107ecd958ffcdb.tar.xz linux-9c8fd72a5c2a031cbc680a2990107ecd958ffcdb.zip | |
wifi: mwifiex: fix memory leak in mwifiex_histogram_read()
Always free the zeroed page on return from 'mwifiex_histogram_read()'.
Fixes: cbf6e05527a7 ("mwifiex: add rx histogram statistics support")
Acked-by: Brian Norris <briannorris@chromium.org>
Signed-off-by: Dmitry Antipov <dmantipov@yandex.ru>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20230802160726.85545-1-dmantipov@yandex.ru
Diffstat (limited to 'drivers/net/wireless/marvell/mwifiex/debugfs.c')
| -rw-r--r-- | drivers/net/wireless/marvell/mwifiex/debugfs.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/net/wireless/marvell/mwifiex/debugfs.c b/drivers/net/wireless/marvell/mwifiex/debugfs.c index 52b18f4a774b..0cdd6c50c1c0 100644 --- a/drivers/net/wireless/marvell/mwifiex/debugfs.c +++ b/drivers/net/wireless/marvell/mwifiex/debugfs.c @@ -253,8 +253,11 @@ mwifiex_histogram_read(struct file *file, char __user *ubuf, if (!p) return -ENOMEM; - if (!priv || !priv->hist_data) - return -EFAULT; + if (!priv || !priv->hist_data) { + ret = -EFAULT; + goto free_and_exit; + } + phist_data = priv->hist_data; p += sprintf(p, "\n" @@ -309,6 +312,8 @@ mwifiex_histogram_read(struct file *file, char __user *ubuf, ret = simple_read_from_buffer(ubuf, count, ppos, (char *)page, (unsigned long)p - page); +free_and_exit: + free_page(page); return ret; } |