Fix memory leak by storing returned pointer in std::unique_ptr
authorRichard Trieu <rtrieu@google.com>
Tue, 30 Aug 2016 21:12:48 +0000 (21:12 +0000)
committerRichard Trieu <rtrieu@google.com>
Tue, 30 Aug 2016 21:12:48 +0000 (21:12 +0000)
llvm-svn: 280157

clang/lib/Tooling/Tooling.cpp

index 162a491..529c47e 100644 (file)
@@ -244,10 +244,9 @@ bool ToolInvocation::run() {
   const char *const BinaryName = Argv[0];
   IntrusiveRefCntPtr<DiagnosticOptions> DiagOpts = new DiagnosticOptions();
   unsigned MissingArgIndex, MissingArgCount;
-  llvm::opt::InputArgList ParsedArgs =
-      driver::createDriverOptTable()->ParseArgs(
-          ArrayRef<const char *>(Argv).slice(1), MissingArgIndex,
-          MissingArgCount);
+  std::unique_ptr<llvm::opt::OptTable> Opts(driver::createDriverOptTable());
+  llvm::opt::InputArgList ParsedArgs = Opts->ParseArgs(
+      ArrayRef<const char *>(Argv).slice(1), MissingArgIndex, MissingArgCount);
   ParseDiagnosticArgs(*DiagOpts, ParsedArgs);
   TextDiagnosticPrinter DiagnosticPrinter(
       llvm::errs(), &*DiagOpts);