Messages in this thread Patch in this message | | | Date | Sun, 8 Jan 2012 23:23:05 -0800 | From | tip-bot for Namhyung Kim <> | Subject | [tip:perf/core] perf script: Add missing closedir() calls |
| |
Commit-ID: 946ef2a24523e59e5cf931068ab7e9443c63c9df Gitweb: http://git.kernel.org/tip/946ef2a24523e59e5cf931068ab7e9443c63c9df Author: Namhyung Kim <namhyung@gmail.com> AuthorDate: Sun, 8 Jan 2012 02:25:25 +0900 Committer: Arnaldo Carvalho de Melo <acme@redhat.com> CommitDate: Sun, 8 Jan 2012 12:35:41 -0200
perf script: Add missing closedir() calls
The get_script_path() calls opendir() but misses corresponding closedir()'s. Add them.
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/1325957132-10600-1-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/builtin-script.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c index fd1909a..bb68ddf 100644 --- a/tools/perf/builtin-script.c +++ b/tools/perf/builtin-script.c @@ -1018,13 +1018,17 @@ static char *get_script_path(const char *script_root, const char *suffix) __script_root = get_script_root(&script_dirent, suffix); if (__script_root && !strcmp(script_root, __script_root)) { free(__script_root); + closedir(lang_dir); + closedir(scripts_dir); snprintf(script_path, MAXPATHLEN, "%s/%s", lang_path, script_dirent.d_name); return strdup(script_path); } free(__script_root); } + closedir(lang_dir); } + closedir(scripts_dir); return NULL; }
| |