Re: [PATCH] tracing: remove redundant assignment to pointer 'event'

[Steven Rostedt <rostedt@goodmis.org>]

On Mon, 17 Jun 2019 13:37:22 +0100
Colin King <colin.king@canonical.com> wrote:

> From: Colin Ian King <colin.king@canonical.com>
> 
> The value assigned to pointer 'event' is never read and hence it
> is redundant and can be removed.
> 
> Addresses-Coverity: ("Unused value")
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  kernel/trace/trace_events_hist.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/kernel/trace/trace_events_hist.c b/kernel/trace/trace_events_hist.c
> index ca6b0dff60c5..0013b43d8b4d 100644
> --- a/kernel/trace/trace_events_hist.c
> +++ b/kernel/trace/trace_events_hist.c
> @@ -1294,7 +1294,6 @@ static int __create_synth_event(int argc, const char *name, const char **argv)
>  	event = alloc_synth_event(name, n_fields, fields);
>  	if (IS_ERR(event)) {
>  		ret = PTR_ERR(event);
> -		event = NULL;

This is one of those cases where I rather not touch it.

Yeah, it may not be read, but assigning event to NULL isn't dangerous
here. And if we change the code to expect event to be NULL or something
real, it is better to keep this.

-- Steve



>  		goto err;
>  	}
>  	ret = register_synth_event(event);