Commit 07a716ff authored by Taeung Song's avatar Taeung Song Committed by Arnaldo Carvalho de Melo

perf report: Fill in the missing session freeing after an error occurs

When an error occurs an error value is just returned without freeing the
session. So allocating and freeing session have to be matched as a pair
even if an error occurs.
Signed-off-by: default avatarTaeung Song <treeze.taeung@gmail.com>
Acked-by: default avatarJiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/1435652124-22414-6-git-send-email-treeze.taeung@gmail.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 249ca1a8
...@@ -839,8 +839,10 @@ int cmd_report(int argc, const char **argv, const char *prefix __maybe_unused) ...@@ -839,8 +839,10 @@ int cmd_report(int argc, const char **argv, const char *prefix __maybe_unused)
if (report.header || report.header_only) { if (report.header || report.header_only) {
perf_session__fprintf_info(session, stdout, perf_session__fprintf_info(session, stdout,
report.show_full_info); report.show_full_info);
if (report.header_only) if (report.header_only) {
return 0; ret = 0;
goto error;
}
} else if (use_browser == 0) { } else if (use_browser == 0) {
fputs("# To display the perf.data header info, please use --header/--header-only options.\n#\n", fputs("# To display the perf.data header info, please use --header/--header-only options.\n#\n",
stdout); stdout);
......
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