diff options
| author |   Wei Yongjun <weiyongjun1@huawei.com> | 2019-09-04 14:16:01 +0000 |
|---|---|---|
| committer |   Kalle Valo <kvalo@codeaurora.org> | 2019-09-06 17:10:33 +0300 |
| commit | eb9affaeff701ce90fcf476ff5332624f9d0ddd2 (patch) | |
| tree | bc3eb7880fc10041d738d9954a70060eb2d7e00a /drivers/net/wireless/realtek/rtw88/debug.c | |
| parent | hostap: remove set but not used variable 'copied' in prism2_io_debug_proc_read (diff) | |
| download | linux-dev-eb9affaeff701ce90fcf476ff5332624f9d0ddd2.tar.xz linux-dev-eb9affaeff701ce90fcf476ff5332624f9d0ddd2.zip | |
rtw88: fix seq_file memory leak
When using single_open(), single_release() should be used instead
of seq_release(), otherwise there is a memory leak.
This is detected by Coccinelle semantic patch.
Fixes: e3037485c68e ("rtw88: new Realtek 802.11ac driver")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/wireless/realtek/rtw88/debug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/realtek/rtw88/debug.c b/drivers/net/wireless/realtek/rtw88/debug.c index 5d235968d475..6ad985e98e42 100644 --- a/drivers/net/wireless/realtek/rtw88/debug.c +++ b/drivers/net/wireless/realtek/rtw88/debug.c @@ -77,7 +77,7 @@ static const struct file_operations file_ops_single_r = { .open = rtw_debugfs_single_open_rw, .read = seq_read, .llseek = seq_lseek, - .release = seq_release, + .release = single_release, }; static const struct file_operations file_ops_single_rw = { |