diff options
| author | Namhyung Kim <[email protected]> | 2017-06-08 16:31:02 +0900 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <[email protected]> | 2017-06-08 15:38:47 -0300 |
| commit | 44ad6b8852529eb39066edbedc027a6901da6803 (patch) | |
| tree | b21fa536818dd709a241f83820a5c8b566d692ca | |
| parent | 3619ef76b37d4803bc9daee9d03d82c8526db378 (diff) | |
perf tools: Fix a memory leak in __open_dso()
The 'name' variable should be freed on the error path.
Signed-off-by: Namhyung Kim <[email protected]>
Acked-by: Jiri Olsa <[email protected]>
Cc: Adrian Hunter <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Wang Nan <[email protected]>
Cc: [email protected]
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
| -rw-r--r-- | tools/perf/util/dso.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/perf/util/dso.c b/tools/perf/util/dso.c index b27d127cdf68..1f29e4fe7af0 100644 --- a/tools/perf/util/dso.c +++ b/tools/perf/util/dso.c @@ -412,8 +412,10 @@ static int __open_dso(struct dso *dso, struct machine *machine) return -EINVAL; } - if (!is_regular_file(name)) + if (!is_regular_file(name)) { + free(name); return -EINVAL; + } fd = do_open(name); free(name); |