Eric Sunshine <[email protected]> writes: > On Tue, Apr 5, 2016 at 6:05 AM, Elia Pinto <[email protected]> wrote: >> The correct api is trace_printf_key >> >> Signed-off-by: Elia Pinto <[email protected]> >> --- >> diff --git a/Documentation/technical/api-trace.txt >> b/Documentation/technical/api-trace.txt >> @@ -28,7 +28,7 @@ static struct trace_key trace_foo = TRACE_KEY_INIT(FOO); >> static void trace_print_foo(const char *message) >> { >> - trace_print_key(&trace_foo, message); >> + trace_printf_key(&trace_foo, message); >> } > > Since you're touching this already, I wonder if it would make sense to > rewrite this example to avoid the dangerous sending of an arbitrary > string (which might contain %) to a printf-like function. Like this, > for example: > > trace_printf_key(&trace_foo, "%s", message);
Thanks, will squash in. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
