perf symbols: Fix error path on symbol__init()
authorNamhyung Kim <namhyung@gmail.com>
Mon, 12 Dec 2011 15:16:52 +0000 (00:16 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 20 Dec 2011 15:40:27 +0000 (13:40 -0200)
The order of freeing comm_list and dso_list should be reversed.

Cc: Ingo Molnar <mingo@elte.hu>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1323703017-6060-4-git-send-email-namhyung@gmail.com
Signed-off-by: Namhyung Kim <namhyung@gmail.com>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/symbol.c

index e54b13d4c3570aed798352942fafaa6bce2fc3ac..215d50f2042e063ce53d9526a3aecbd0ec7ff461 100644 (file)
@@ -2604,10 +2604,10 @@ int symbol__init(void)
        symbol_conf.initialized = true;
        return 0;
 
-out_free_dso_list:
-       strlist__delete(symbol_conf.dso_list);
 out_free_comm_list:
        strlist__delete(symbol_conf.comm_list);
+out_free_dso_list:
+       strlist__delete(symbol_conf.dso_list);
        return -1;
 }