Commit 9fedfb0c authored by Taeung Song's avatar Taeung Song Committed by Arnaldo Carvalho de Melo

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

When an error occur 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-2-git-send-email-treeze.taeung@gmail.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent ceb92913
...@@ -630,12 +630,13 @@ int cmd_inject(int argc, const char **argv, const char *prefix __maybe_unused) ...@@ -630,12 +630,13 @@ int cmd_inject(int argc, const char **argv, const char *prefix __maybe_unused)
if (inject.session == NULL) if (inject.session == NULL)
return -1; return -1;
if (symbol__init(&inject.session->header.env) < 0) ret = symbol__init(&inject.session->header.env);
return -1; if (ret < 0)
goto out_delete;
ret = __cmd_inject(&inject); ret = __cmd_inject(&inject);
out_delete:
perf_session__delete(inject.session); perf_session__delete(inject.session);
return ret; return ret;
} }
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