Re: [PATCH] tracing: initialize variable in create_dyn_event()

From: Steven Rostedt
Date: Thu Mar 14 2019 - 18:19:12 EST


On Thu, 14 Mar 2019 14:05:02 -0700
Frank Rowand <frowand.list@xxxxxxxxx> wrote:

> Hi Steve,
>
> Ping.

Thanks, this got buried.

> > static int create_dyn_event(int argc, char **argv)
> > {
> > struct dyn_event_operations *ops;
> > - int ret;
> > + int ret = 0;

Probably should be ret = -ENODEV;

As if ret doesn't get set, then no dynamic code has been registered,
and something is truly wrong.

-- Steve


> >
> > if (argv[0][0] == '-' || argv[0][0] == '!')
> > return dyn_event_release(argc, argv, NULL);
> >