From 05f007c22ee94c9143d730910bd26efed7a663b8 Mon Sep 17 00:00:00 2001 From: "jam@chromium.org" Date: Fri, 6 Mar 2009 01:21:30 +0000 Subject: An assert was being hit because WaitableEvent's dtor was calling CloseHandle on a handle that we already closed. BUG=8070 Review URL: http://codereview.chromium.org/40195 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11078 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/common/child_process_host.cc | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'chrome/common') diff --git a/chrome/common/child_process_host.cc b/chrome/common/child_process_host.cc index 7839f44..a5058d2 100644 --- a/chrome/common/child_process_host.cc +++ b/chrome/common/child_process_host.cc @@ -55,6 +55,10 @@ ChildProcessHost::~ChildProcessHost() { if (handle()) { watcher_.StopWatching(); ProcessWatcher::EnsureProcessTerminated(handle()); + + // Above call took ownership, so don't want WaitableEvent to assert because + // the handle isn't valid anymore. + process_event_->Release(); } } -- cgit v1.1