summaryrefslogtreecommitdiffstats
path: root/tools/testing/selftests/bpf/progs/test_autoload.c
diff options
context:
space:
mode:
authorSteven Rostedt <rostedt@goodmis.org>2025-04-17 18:30:03 -0400
committerSteven Rostedt (Google) <rostedt@goodmis.org>2025-04-17 22:16:56 -0400
commita8c5b0ed89a3f2c81c6ae0b041394e6eea0e7024 (patch)
tree4f95dc1b0c868ff7adc48a539da5d32191cf16be /tools/testing/selftests/bpf/progs/test_autoload.c
parent3b4e87e6a593d571183c414d81758624da01f2b9 (diff)
downloadlinux-a8c5b0ed89a3f2c81c6ae0b041394e6eea0e7024.tar.gz
linux-a8c5b0ed89a3f2c81c6ae0b041394e6eea0e7024.tar.bz2
linux-a8c5b0ed89a3f2c81c6ae0b041394e6eea0e7024.zip
tracing: Fix filter string testing
The filter string testing uses strncpy_from_kernel/user_nofault() to retrieve the string to test the filter against. The if() statement was incorrect as it considered 0 as a fault, when it is only negative that it faulted. Running the following commands: # cd /sys/kernel/tracing # echo "filename.ustring ~ \"/proc*\"" > events/syscalls/sys_enter_openat/filter # echo 1 > events/syscalls/sys_enter_openat/enable # ls /proc/$$/maps # cat trace Would produce nothing, but with the fix it will produce something like: ls-1192 [007] ..... 8169.828333: sys_openat(dfd: ffffffffffffff9c, filename: 7efc18359904, flags: 80000, mode: 0) Link: https://lore.kernel.org/all/CAEf4BzbVPQ=BjWztmEwBPRKHUwNfKBkS3kce-Rzka6zvbQeVpg@mail.gmail.com/ Cc: stable@vger.kernel.org Cc: Masami Hiramatsu <mhiramat@kernel.org> Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com> Cc: Andrew Morton <akpm@linux-foundation.org> Link: https://lore.kernel.org/20250417183003.505835fb@gandalf.local.home Fixes: 77360f9bbc7e5 ("tracing: Add test for user space strings when filtering on string pointers") Reported-by: Andrii Nakryiko <andrii.nakryiko@gmail.com> Reported-by: Mykyta Yatsenko <mykyta.yatsenko5@gmail.com> Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Diffstat (limited to 'tools/testing/selftests/bpf/progs/test_autoload.c')
0 files changed, 0 insertions, 0 deletions