From 3ae7bb6b7c9bd22d1c53cf7ddc28850821827522 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Thu, 7 Oct 2004 20:01:31 +0000 Subject: Fix a nasty dangling pointer problem, due to a free'd pointer being left in a map. This caused problems if a later object happened to be allocated at the free'd object's address. llvm-svn: 16813 --- llvm/lib/Analysis/DataStructure/CompleteBottomUp.cpp | 3 +++ 1 file changed, 3 insertions(+) (limited to 'llvm/lib/Analysis/DataStructure/CompleteBottomUp.cpp') diff --git a/llvm/lib/Analysis/DataStructure/CompleteBottomUp.cpp b/llvm/lib/Analysis/DataStructure/CompleteBottomUp.cpp index ee111e9d8fe..3cd2bb0eef5 100644 --- a/llvm/lib/Analysis/DataStructure/CompleteBottomUp.cpp +++ b/llvm/lib/Analysis/DataStructure/CompleteBottomUp.cpp @@ -157,6 +157,9 @@ unsigned CompleteBUDataStructures::calculateSCCGraphs(DSGraph &FG, for (DSGraph::ReturnNodesTy::iterator I = NG->getReturnNodes().begin(); I != NG->getReturnNodes().end(); ++I) DSInfo[I->first] = &FG; + + // Remove NG from the ValMap since the pointer may get recycled. + ValMap.erase(NG); delete NG; Stack.pop_back(); -- cgit v1.2.3