Commit 97fe0383 authored Oct 09, 2023 by Ian Rogers Committed by Namhyung Kim Oct 12, 2023

perf trace-event-info: Avoid passing NULL value to closedir



If opendir failed then closedir was passed NULL which is
erroneous. Caught by clang-tidy.

Signed-off-by: Ian Rogers <irogers@google.com>
Acked-by: Namhyung Kim <namhyung@kernel.org>
Cc: Ravi Bangoria <ravi.bangoria@amd.com>
Cc: Nick Desaulniers <ndesaulniers@google.com>
Cc: Yang Jihong <yangjihong1@huawei.com>
Cc: Huacai Chen <chenhuacai@kernel.org>
Cc: Nathan Chancellor <nathan@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: llvm@lists.linux.dev
Cc: Ming Wang <wangming01@loongson.cn>
Cc: Tom Rix <trix@redhat.com>
Cc: bpf@vger.kernel.org
Link: https://lore.kernel.org/r/20231009183920.200859-18-irogers@google.com


Signed-off-by: Namhyung Kim <namhyung@kernel.org>

parent c4b5140c

tools/perf/util/trace-event-info.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -313,6 +313,7 @@ static int record_event_files(struct tracepoint_path *tps)
		}
		err = 0;
		out:
		if (dir)
		closedir(dir);
		put_tracing_file(path);