summaryrefslogtreecommitdiff
path: root/kernel/module/tracking.c
diff options
context:
space:
mode:
authorDavid Carlier <devnexen@gmail.com>2026-05-08 20:57:47 +0100
committerSteven Rostedt <rostedt@goodmis.org>2026-05-20 16:10:56 -0400
commit576ec047d20b368b43c4d5db98c4f2e0f3c101ec (patch)
tree4541ee6fc2225395470a8ea00eb824362f412eec /kernel/module/tracking.c
parent5200f5f493f79f14bbdc349e402a40dfb32f23c8 (diff)
tracing: Avoid NULL return from hist_field_name() on truncation
hist_field_name() returns "" everywhere except the fully-qualified VAR_REF/EXPR case, where snprintf() truncation returns NULL early and bypasses the bottom NULL->"" guard. Callers don't expect NULL: strcat(expr, hist_field_name(field, 0)) at trace_events_hist.c:1758 and the strcmp() in the sort-key match loop at :4804 both deref it. system and event_name are bounded by MAX_EVENT_NAME_LEN, but the field name on a VAR_REF is kstrdup'd from a histogram variable name parsed out of the trigger string and has no length cap, so a long enough var name in a fully qualified reference can reach the truncation path. Keep the length check but leave field_name as "" on overflow. Link: https://patch.msgid.link/20260508195747.25492-1-devnexen@gmail.com Fixes: 5ec1d1e97de1 ("tracing: Rebuild full_name on each hist_field_name() call") Signed-off-by: David Carlier <devnexen@gmail.com> Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/module/tracking.c')
0 files changed, 0 insertions, 0 deletions