Re: [PATCH v2] ftrace: Fix use-after-free for dynamic ftrace_ops

From: Li Huafei
Date: Wed Nov 02 2022 - 23:38:43 EST




On 2022/11/3 11:24, Steven Rostedt wrote:
> On Wed, 2 Nov 2022 23:23:34 -0400
> Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
>
>> I think you misunderstood me. What I was suggesting was to get rid of
>> the ftrace_enabled check. The DYNAMIC part is most definitely needed.
>>
>> if (!command) {
>> if (ops->flags & FTRACE_OPS_FL_DYNAMIC)
>> goto out;
>> return 0;
>> }
>
>
> Nevermind, I forgot that the out is before the DYNAMIC check.
>
> ;-)
>

Yes. There we have the DYNAMIC check.

> -- Steve
>
> .
>