i2p/haiku-beta6
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Tracker: Make sure destWindow is locked before you "Move to" it

Browse Source 
... fixes another crash reported in #20072, this time concerning
Looper must be locked.

Change-Id: Ia4161609066bd2a5094f51410058dce75a630dd9
This commit is contained in:
John Scipione
2026-05-07 23:50:20 -04:00
parent 9de308df97
commit 37246ea447
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/kits/tracker/PoseView.cpp
+5 -2
View File
@@ -5202,10 +5202,11 @@ BPoseView::MoveSelectionInto(Model* destFolder, BContainerWindow* srcWindow,
if (okToMove) {
PoseList* selectionList = srcWindow->PoseView()->SelectionList();
BList* pointList = NULL;
if (destWindow != NULL) {
if (destWindow != NULL && destWindow->LockLooper()) {
// destination window is available e.g. drag and drop
pointList = destWindow->PoseView()->GetDropPointList(dragStart, dropPoint,
selectionList, srcWindow->PoseView(), pinToGrid);
destWindow->UnlockLooper();
} else {
// destination window is closed e.g. Move to/Copy to/Link to
// pose positions will be offset by their position in the source window
@@ -5242,8 +5243,10 @@ BPoseView::MoveSelectionInto(Model* destFolder, BContainerWindow* srcWindow,
moveMode = kMoveSelectionTo;
// move will replace current selection
if (moveMode == kMoveSelectionTo && destWindow != NULL)
if (moveMode == kMoveSelectionTo && destWindow != NULL && destWindow->LockLooper()) {
destWindow->PoseView()->ClearSelection();
destWindow->UnlockLooper();
}
FSMoveToFolder(srcList, destEntry, moveMode, pointList);
return;