From: Alexey Samsonov Date: Fri, 28 Jun 2013 15:08:29 +0000 (+0000) Subject: llvm-symbolizer: don't leave dangling pointers after flushing LLVMSymbolizer. Add... X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=0ed872cdd1e5d7d60983854ea49aff13aa3280c1;p=oota-llvm.git llvm-symbolizer: don't leave dangling pointers after flushing LLVMSymbolizer. Add a destructor. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@185154 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/tools/llvm-symbolizer/LLVMSymbolize.cpp b/tools/llvm-symbolizer/LLVMSymbolize.cpp index 59564161b4e..57f34c24729 100644 --- a/tools/llvm-symbolizer/LLVMSymbolize.cpp +++ b/tools/llvm-symbolizer/LLVMSymbolize.cpp @@ -202,6 +202,8 @@ std::string LLVMSymbolizer::symbolizeData(const std::string &ModuleName, void LLVMSymbolizer::flush() { DeleteContainerSeconds(Modules); DeleteContainerPointers(ParsedBinariesAndObjects); + BinaryForPath.clear(); + ObjectFileForArch.clear(); } static std::string getDarwinDWARFResourceForPath(const std::string &Path) { diff --git a/tools/llvm-symbolizer/LLVMSymbolize.h b/tools/llvm-symbolizer/LLVMSymbolize.h index d1f2285aa8a..03c765cc9c3 100644 --- a/tools/llvm-symbolizer/LLVMSymbolize.h +++ b/tools/llvm-symbolizer/LLVMSymbolize.h @@ -48,6 +48,9 @@ public: }; LLVMSymbolizer(const Options &Opts = Options()) : Opts(Opts) {} + ~LLVMSymbolizer() { + flush(); + } // Returns the result of symbolization for module name/offset as // a string (possibly containing newlines).