On Wed, 23 Jul 2025 10:30:56 +0900
"Masami Hiramatsu (Google)" <mhira...@kernel.org> wrote:

> diff --git a/kernel/trace/trace_fprobe.c b/kernel/trace/trace_fprobe.c
> index add08ffb04d7..610f8d53be8a 100644
> --- a/kernel/trace/trace_fprobe.c
> +++ b/kernel/trace/trace_fprobe.c
> @@ -1384,14 +1384,17 @@ static int trace_fprobe_create_internal(int argc, 
> const char *argv[],
>  
>  static int trace_fprobe_create_cb(int argc, const char *argv[])
>  {
> -     struct traceprobe_parse_context ctx = {
> -             .flags = TPARG_FL_KERNEL | TPARG_FL_FPROBE,
> -     };
> +     struct traceprobe_parse_context *ctx __free(traceprobe_parse_context) = 
> NULL;
>       int ret;
>  
> +     ctx = kzalloc(sizeof(*ctx), GFP_KERNEL);
> +     if (!ctx)
> +             return -ENOMEM;
> +
> +     ctx->flags = TPARG_FL_KERNEL | TPARG_FL_FPROBE,

Oops, this last should be ';' instead of ','!

-- 
Masami Hiramatsu (Google) <mhira...@kernel.org>

Reply via email to