From cdae6d7711d621c19623ae4fbfdde9e0301260c7 Mon Sep 17 00:00:00 2001 From: Fangrui Song Date: Mon, 19 Apr 2021 19:39:10 -0700 Subject: [PATCH] [lldb] Fix one leak in reproducer Use a variable of static storage duration to reference an intentionally leaked variable. A static data area is in the GC-set of various leak checkers. This fixes 3 `check-lldb-shell` tests in a `-DLLVM_USE_SANITIZER={Leaks,Address}` build, e.g. `test/Shell/Reproducer/TestHomeDir.test` Differential Revision: https://reviews.llvm.org/D100806 --- lldb/tools/driver/Driver.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/lldb/tools/driver/Driver.cpp b/lldb/tools/driver/Driver.cpp index e4a6012..c512153 100644 --- a/lldb/tools/driver/Driver.cpp +++ b/lldb/tools/driver/Driver.cpp @@ -784,8 +784,8 @@ EXAMPLES: llvm::outs() << examples << '\n'; } -llvm::Optional InitializeReproducer(llvm::StringRef argv0, - opt::InputArgList &input_args) { +static llvm::Optional InitializeReproducer(llvm::StringRef argv0, + opt::InputArgList &input_args) { if (auto *finalize_path = input_args.getLastArg(OPT_reproducer_finalize)) { if (const char *error = SBReproducer::Finalize(finalize_path->getValue())) { WithColor::error() << "reproducer finalization failed: " << error << '\n'; @@ -853,8 +853,7 @@ llvm::Optional InitializeReproducer(llvm::StringRef argv0, // Register the reproducer signal handler. if (!input_args.hasArg(OPT_no_generate_on_signal)) { if (const char *reproducer_path = SBReproducer::GetPath()) { - // Leaking the string on purpose. - std::string *finalize_cmd = new std::string(argv0); + static std::string *finalize_cmd = new std::string(argv0); finalize_cmd->append(" --reproducer-finalize '"); finalize_cmd->append(reproducer_path); finalize_cmd->append("'"); -- 2.7.4