aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNam Cao <[email protected]>2022-10-25 11:12:23 +0200
committerGreg Kroah-Hartman <[email protected]>2022-10-31 09:08:30 +0100
commitc5a7eecdcdb6c9d3a907ee66db2b5cffc8bdb4e1 (patch)
treec7b33e8a99525eebf49e74b4d624ffbcd82aaba9
parentdbc97f832aa798c16453eeb4eb5fe74b5c998223 (diff)
Revert "staging: r8712u: Tracking kmemleak false positives."
This reverts commit 5d3da4a20a271e3cf5496a50cbb8118aa019374f. This commit annotated false positive for kmemleak. The reasoning is that the buffers are freed when the driver is unloaded. However, there is actually potential memory leak when probe fails. Tested-by: Philipp Hortmann <[email protected]> Signed-off-by: Nam Cao <[email protected]> Link: https://lore.kernel.org/r/26ce206b2c40c7db48c146aa6105789db9dfcc1a.1666688642.git.namcaov@gmail.com Signed-off-by: Greg Kroah-Hartman <[email protected]>
-rw-r--r--drivers/staging/rtl8712/rtl871x_recv.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/staging/rtl8712/rtl871x_recv.c b/drivers/staging/rtl8712/rtl871x_recv.c
index de9a568eaffa..4db7eed64a03 100644
--- a/drivers/staging/rtl8712/rtl871x_recv.c
+++ b/drivers/staging/rtl8712/rtl871x_recv.c
@@ -17,9 +17,7 @@
#define _RTL871X_RECV_C_
#include <linux/ip.h>
-#include <linux/slab.h>
#include <linux/if_ether.h>
-#include <linux/kmemleak.h>
#include <linux/etherdevice.h>
#include <linux/ieee80211.h>
#include <net/cfg80211.h>
@@ -61,7 +59,6 @@ void _r8712_init_recv_priv(struct recv_priv *precvpriv,
GFP_ATOMIC);
if (!precvpriv->pallocated_frame_buf)
return;
- kmemleak_not_leak(precvpriv->pallocated_frame_buf);
precvpriv->precv_frame_buf = precvpriv->pallocated_frame_buf +
RXFRAME_ALIGN_SZ -
((addr_t)(precvpriv->pallocated_frame_buf) &