diff options
| author |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-01-06 16:28:28 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-01-17 17:23:31 +0100 |
| commit | 30374434edab20e25776f8ecb4bc9d1e54309487 (patch) | |
| tree | 80246c4a5d50d6ba9dac386bb25128172899e014 /drivers/usb/core/usb.c | |
| parent | MAINTAINERS: add the Renesas RZ/N1 USBF controller entry (diff) | |
| download | wireguard-linux-30374434edab20e25776f8ecb4bc9d1e54309487.tar.xz wireguard-linux-30374434edab20e25776f8ecb4bc9d1e54309487.zip | |
USB: fix memory leak with using debugfs_lookup()
When calling debugfs_lookup() the result must have dput() called on it,
otherwise the memory will leak over time. To make things simpler, just
call debugfs_lookup_and_remove() instead which handles all of the logic at
once.
Cc: Alan Stern <stern@rowland.harvard.edu>
Cc: Jilin Yuan <yuanjilin@cdjrlc.com>
Link: https://lore.kernel.org/r/20230106152828.3790902-1-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/core/usb.c')
| -rw-r--r-- | drivers/usb/core/usb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/core/usb.c b/drivers/usb/core/usb.c index 11b15d7b357a..a415206cab04 100644 --- a/drivers/usb/core/usb.c +++ b/drivers/usb/core/usb.c @@ -998,7 +998,7 @@ static void usb_debugfs_init(void) static void usb_debugfs_cleanup(void) { - debugfs_remove(debugfs_lookup("devices", usb_debug_root)); + debugfs_lookup_and_remove("devices", usb_debug_root); } /* |