Commit e96c674f authored Aug 12, 2014 by Namhyung Kim Committed by Arnaldo Carvalho de Melo Aug 13, 2014

perf symbols: Fix a memory leak in vmlinux_path__init()



When uname() failed, it should free vmlinux_path.

Acked-by: Jiri Olsa <jolsa@kernel.org>
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Minchan Kim <minchan@kernel.org>
Cc: Namhyung Kim <namhyung.kim@lge.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1407825645-24586-3-git-send-email-namhyung@kernel.org


Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

parent 6cc870f0

tools/perf/util/symbol.c

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -1772,7 +1772,7 @@ static int vmlinux_path__init(void)
	return 0;		return 0;

	if (uname(&uts) < 0)		if (uname(&uts) < 0)
	return -1;		goto out_fail;

	snprintf(bf, sizeof(bf), "/boot/vmlinux-%s", uts.release);		snprintf(bf, sizeof(bf), "/boot/vmlinux-%s", uts.release);
	vmlinux_path[vmlinux_path__nr_entries] = strdup(bf);		vmlinux_path[vmlinux_path__nr_entries] = strdup(bf);