Commit 4cc97614 authored by Markus Elfring's avatar Markus Elfring Committed by Arnaldo Carvalho de Melo

perf header: Delete an unnecessary check before the calling free_event_desc()

The free_event_desc() function tests whether its argument is NULL and
then returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.
Signed-off-by: default avatarMarkus Elfring <elfring@users.sourceforge.net>
Cc: Julia Lawall <julia.lawall@lip6.fr>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: kernel-janitors@vger.kernel.org
Link: http://lkml.kernel.org/r/558C2ABA.3000603@users.sourceforge.netSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 060664f3
...@@ -1063,8 +1063,7 @@ read_event_desc(struct perf_header *ph, int fd) ...@@ -1063,8 +1063,7 @@ read_event_desc(struct perf_header *ph, int fd)
free(buf); free(buf);
return events; return events;
error: error:
if (events) free_event_desc(events);
free_event_desc(events);
events = NULL; events = NULL;
goto out; goto out;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment