Skip to content

Commit

Permalink
Properly initialize all fields in CrashReporterCleanupContext. This c…
Browse files Browse the repository at this point in the history
…aused the buildbot failure earlier.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@128071 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
tkremenek committed Mar 22, 2011
1 parent c943a9a commit 3311d95
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions include/llvm/Support/CrashRecoveryContext.h
Original file line number Diff line number Diff line change
Expand Up @@ -101,11 +101,10 @@ class CrashRecoveryContextCleanup {
protected:
CrashRecoveryContext *context;
CrashRecoveryContextCleanup(CrashRecoveryContext *context)
: context(context) {}
: context(context), cleanupFired(false) {}
public:
bool cleanupFired;

CrashRecoveryContextCleanup() : cleanupFired(false) {}
virtual ~CrashRecoveryContextCleanup();
virtual void recoverResources() = 0;

Expand Down
2 changes: 1 addition & 1 deletion lib/Support/CrashRecoveryContext.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ CrashRecoveryContext::~CrashRecoveryContext() {
CrashRecoveryContextCleanup *tmp = i;
i = tmp->next;
tmp->cleanupFired = true;
//tmp->recoverResources();
tmp->recoverResources();
delete tmp;
}
tlIsRecoveringFromCrash.erase();
Expand Down

0 comments on commit 3311d95

Please sign in to comment.