Comment #9 on issue 8809 by [email protected]: Crash in
WaitableEventWatcher::StopWatching()
http://code.google.com/p/chromium/issues/detail?id=8809
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=12459
------------------------------------------------------------------------
r12459 | [email protected] | 2009-03-25 10:08:41 -0700 (Wed, 25 Mar 2009) |
16 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event.h?r1=12459&r2=12458
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event_posix.cc?r1=12459&r2=12458
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event_watcher.h?r1=12459&r2=12458
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event_watcher_posix.cc?r1=12459&r2=12458
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event_watcher_unittest.cc?r1=12459&r2=12458
POSIX: allow WaitableEvents to be deleted while watching them.
On Windows, one can close a HANDLE which is currently being waited on. The
MSDN
documentation says that the resulting behaviour is 'undefined', but it
doesn't
crash. Currently, on POSIX, one couldn't use WaitableEventWatcher to watch
an
event which gets deleted. This mismatch has bitten us several times now.
This patch allows WaitableEvents to be deleted while a WaitableEventWatcher
is
still watching them. It applies only to watchers, the usual Wait() and
WaitMany() calls still require that all their target be valid until the end
of
the call.
http://crbug.com/8809
Review URL: http://codereview.chromium.org/53026
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---