Commit 506fde11 authored Feb 01, 2017 by Taeung Song Committed by Arnaldo Carvalho de Melo Feb 08, 2017

perf tools: Use zfree() instead of ad hoc equivalent



We have zfree(&ptr) for this very common pattern:

   free(ptr);
   ptr = NULL;

So use it in a few more places.

Signed-off-by: Taeung Song <treeze.taeung@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/r/1485952447-7013-4-git-send-email-treeze.taeung@gmail.com


[ rewrote commit log ]
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

parent 5aa365f2

tools/perf/util/parse-events.c

+2 −4

Original line number	Diff line number	Diff line
		@@ -254,8 +254,7 @@ struct tracepoint_path tracepoint_name_to_path(const char name)
		if (path->system == NULL \|\| path->name == NULL) {
		zfree(&path->system);
		zfree(&path->name);
		free(path);
		path = NULL;
		zfree(&path);
		}

		return path;
		@@ -1482,8 +1481,7 @@ static void perf_pmu__parse_cleanup(void)
		p = perf_pmu_events_list + i;
		free(p->symbol);
		}
		free(perf_pmu_events_list);
		perf_pmu_events_list = NULL;
		zfree(&perf_pmu_events_list);
		perf_pmu_events_list_num = 0;
		}
		}

Admin message