tracing: Have trace event string test handle zero length strings
authorSteven Rostedt (Google) <rostedt@goodmis.org>
Wed, 23 Mar 2022 14:32:51 +0000 (10:32 -0400)
committerSteven Rostedt (Google) <rostedt@goodmis.org>
Wed, 23 Mar 2022 14:51:58 +0000 (10:51 -0400)
commiteca344a7362e0f34f179298fd8366bcd556eede1
tree2df9b0079f1248d7e010f4b99acbe5ac39a4c96b
parent78cbc6513217b00be6a9904415ef7ff3619eb035
tracing: Have trace event string test handle zero length strings

If a trace event has in its TP_printk():

 "%*.s", len, len ? __get_str(string) : NULL

It is perfectly valid if len is zero and passing in the NULL.
Unfortunately, the runtime string check at time of reading the trace sees
the NULL and flags it as a bad string and produces a WARN_ON().

Handle this case by passing into the test function if the format has an
asterisk (star) and if so, if the length is zero, then mark it as safe.

Link: https://lore.kernel.org/all/YjsWzuw5FbWPrdqq@bfoster/
Cc: stable@vger.kernel.org
Reported-by: Brian Foster <bfoster@redhat.com>
Tested-by: Brian Foster <bfoster@redhat.com>
Fixes: 9a6944fee68e2 ("tracing: Add a verifier to check string pointers for trace events")
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
kernel/trace/trace.c