Closed Bug 230103 Opened 21 years ago Closed 21 years ago

Looper must be locked in NS_METHOD nsWindow::CaptureMouse() method

Categories

(Core Graveyard :: GFX: BeOS, defect)

Other
BeOS
defect
Not set
major

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sergei_d, Assigned: sergei_d)

Details

(Keywords: crash)

Attachments

(1 file)

Currently it isn't.
It maylead to crashes like:
The looper must be locked

_debugger:
_debugger:
+0007 ec084853: * c3 retn
mozilla-bin:sc
frame retaddr
fd00147c ec169e20 BLooper::check_lock(void) + 00000044
fd001494 ec21dd27 BView::SetEventMask(unsigned long, unsigned long) + 0000004b
fd0014ac ed650e04 nsWindow::CaptureMouse(int) + 0000002c
fd0014c4 ecfc4c59 nsResizerFrame::HandleEvent(nsIPresContext *, nsGUIEvent *,
nsEventStatus *) + 000000d9

Preparing simple patch
Assignee: arougthopher → sergei_d
Attached patch Patch (diff -up)Splinter Review
Adding Lock/UnlockLooper() to CaptureMouse method
Comment on attachment 138414 [details] [diff] [review]
Patch (diff -up)

review request
Attachment #138414 - Flags: review?(cbiesinger)
Attachment #138414 - Flags: review?(cbiesinger) → review+
Checking in nsWindow.cpp;
/cvsroot/mozilla/widget/src/beos/nsWindow.cpp,v  <--  nsWindow.cpp
new revision: 1.71; previous revision: 1.70
done
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: