diff options
| author |   Jeff Layton <jlayton@kernel.org> | 2025-08-08 07:45:23 -0400 |
|---|---|---|
| committer |   Jakub Kicinski <kuba@kernel.org> | 2025-08-11 20:07:44 -0700 |
| commit | 52966bf71de98fef4ca7b3be1349adc7459d6d53 (patch) | |
| tree | 70994f9d79667f524a9098b58d76a693c5bbeaec | |
| parent | net: usb: qmi_wwan: add Telit Cinterion FN990A w/audio composition (diff) | |
| download | wireguard-linux-52966bf71de98fef4ca7b3be1349adc7459d6d53.tar.xz wireguard-linux-52966bf71de98fef4ca7b3be1349adc7459d6d53.zip | |
ref_tracker: use %p instead of %px in debugfs dentry name
As Kees points out, this is a kernel address leak, and debugging is
not a sufficiently good reason to expose the real kernel address.
Fixes: 65b584f53611 ("ref_tracker: automatically register a file in debugfs for a ref_tracker_dir")
Reported-by: Kees Cook <kees@kernel.org>
Closes: https://lore.kernel.org/netdev/202507301603.62E553F93@keescook/
Signed-off-by: Jeff Layton <jlayton@kernel.org>
Reviewed-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
| -rw-r--r-- | lib/ref_tracker.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index a9e6ffcff04b..cce12287708e 100644 --- a/lib/ref_tracker.c +++ b/lib/ref_tracker.c @@ -434,7 +434,7 @@ void ref_tracker_dir_debugfs(struct ref_tracker_dir *dir) if (dentry && !xa_is_err(dentry)) return; - ret = snprintf(name, sizeof(name), "%s@%px", dir->class, dir); + ret = snprintf(name, sizeof(name), "%s@%p", dir->class, dir); name[sizeof(name) - 1] = '\0'; if (ret < sizeof(name)) { |