Daniel Xu 6fa6d28051 lib/strncpy_from_user.c: Mask out bytes after NUL terminator. ... do_strncpy_from_user() may copy some extra bytes after the NUL terminator into the destination buffer. This usually does not matter for normal string operations. However, when BPF programs key BPF maps with strings, this matters a lot. A BPF program may read strings from user memory by calling the bpf_probe_read_user_str() helper which eventually calls do_strncpy_from_user(). The program can then key a map with the destination buffer. BPF map keys are fixed-width and string-agnostic, meaning that map keys are treated as a set of bytes. The issue is when do_strncpy_from_user() overcopies bytes after the NUL terminator, it can result in seemingly identical strings occupying multiple slots in a BPF map. This behavior is subtle and totally unexpected by the user. This commit masks out the bytes following the NUL while preserving long-sized stride in the fast path. Fixes: 6ae08ae3de ("bpf: Add probe_read_{user, kernel} and probe_read_{user, kernel}_str helpers") Signed-off-by: Daniel Xu <dxu@dxuuu.xyz> Signed-off-by: Alexei Starovoitov <ast@kernel.org> Link: https://lore.kernel.org/bpf/21efc982b3e9f2f7b0379eed642294caaa0c27a7.1605642949.git.dxu@dxuuu.xyz		2020-11-19 11:56:16 -08:00
..
blktrace.c	block: add a bdget_part helper	2020-10-05 10:38:33 -06:00
bpf_trace.c	lib/strncpy_from_user.c: Mask out bytes after NUL terminator.	2020-11-19 11:56:16 -08:00
bpf_trace.h
fgraph.c	fgraph: Convert ret_stack tasklist scanning to rcu	2020-09-21 21:06:02 -04:00
ftrace_internal.h
ftrace.c	Updates for tracing and bootconfig:	2020-10-15 15:51:28 -07:00
Kconfig
kprobe_event_gen_test.c
Makefile
power-traces.c
preemptirq_delay_test.c
ring_buffer_benchmark.c
ring_buffer.c	ring-buffer: Fix recursion protection transitions between interrupt context	2020-11-02 15:58:32 -05:00
rpm-traces.c
synth_event_gen_test.c	tracing: Add support for dynamic strings to synthetic events	2020-10-05 19:32:18 -04:00
trace_benchmark.c
trace_benchmark.h
trace_boot.c	tracing/boot: Add ftrace.instance.*.alloc_snapshot option	2020-10-15 12:01:13 -04:00
trace_branch.c
trace_clock.c
trace_dynevent.c	tracing: Delete repeated words in comments	2020-09-21 21:06:02 -04:00
trace_dynevent.h
trace_entries.h
trace_event_perf.c
trace_events_filter_test.h
trace_events_filter.c
trace_events_hist.c	tracing: Check return value of __create_val_fields() before using its result	2020-10-15 12:00:59 -04:00
trace_events_inject.c
trace_events_synth.c	tracing: Make -ENOMEM the default error for parse_synth_field()	2020-11-02 15:58:32 -05:00
trace_events_trigger.c
trace_events.c	Tracing: Fix mismatch section of adding early trace events	2020-10-16 14:56:52 -07:00
trace_export.c	treewide: Convert macro and uses of __section(foo) to __section("foo")	2020-10-25 14:51:49 -07:00
trace_functions_graph.c	tracing: make tracing_init_dentry() returns an integer instead of a d_entry pointer	2020-09-18 22:17:14 -04:00
trace_functions.c	tracing: Enable creating new instance early boot	2020-09-21 21:06:04 -04:00
trace_hwlat.c	tracing: make tracing_init_dentry() returns an integer instead of a d_entry pointer	2020-09-18 22:17:14 -04:00
trace_irqsoff.c
trace_kdb.c
trace_kprobe_selftest.c
trace_kprobe_selftest.h
trace_kprobe.c	kprobes: Use module_name() macro	2020-09-24 15:55:49 +02:00
trace_mmiotrace.c
trace_nop.c
trace_output.c	tracing: Make the space reserved for the pid wider	2020-09-18 12:42:11 -04:00
trace_output.h
trace_preemptirq.c	lockdep: fix order in trace_hardirqs_off_caller()	2020-09-14 10:08:07 +02:00
trace_printk.c	Updates for tracing and bootconfig:	2020-10-15 15:51:28 -07:00
trace_probe_tmpl.h
trace_probe.c
trace_probe.h	tracing: Move is_good_name() from trace_probe.h to trace.h	2020-10-15 12:01:13 -04:00
trace_sched_switch.c
trace_sched_wakeup.c
trace_selftest_dynamic.c
trace_selftest.c	ftrace: Handle tracing when switching between context	2020-11-02 08:52:18 -05:00
trace_seq.c
trace_stack.c	tracing: make tracing_init_dentry() returns an integer instead of a d_entry pointer	2020-09-18 22:17:14 -04:00
trace_stat.c	tracing: make tracing_init_dentry() returns an integer instead of a d_entry pointer	2020-09-18 22:17:14 -04:00
trace_stat.h
trace_synth.h	tracing: Add support for dynamic strings to synthetic events	2020-10-05 19:32:18 -04:00
trace_syscalls.c
trace_uprobe.c	tracing/uprobes: Support perf-style return probe	2020-09-21 21:06:03 -04:00
trace.c	tracing: Fix the checking of stackidx in __ftrace_trace_stack	2020-11-02 11:21:40 -05:00
trace.h	ftrace: Handle tracing when switching between context	2020-11-02 08:52:18 -05:00
tracing_map.c	tracing: Delete repeated words in comments	2020-09-21 21:06:02 -04:00
tracing_map.h