mirrors/sway
1
0
Fork 0
mirror of https://github.com/swaywm/sway.git synced 2026-05-26 21:38:40 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

seatop_move_tiling: fix stack overflow when dropping container into itself

Browse source 
This change avoids a cyclic reference in the window tree when dragging a
container and dropping it into one of its descendants.

The drop target resolution logic previously only checked if the target node
was the container itself, but missed checking if the target was a descendant
of the dragged container.

Fixes: #7949
This commit is contained in:
Scott Leggett 2026-05-22 23:03:31 +08:00 • committed by Simon Ser
parent 3cb86e4a2a
commit f1b40bc288
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
sway/input/seatop_move_tiling.c
View file

@ -256,8 +256,9 @@ static void handle_motion_postthreshold(struct sway_seat *seat) {
}
if (edge) {
e->target_node = node_get_parent(&con->node);
if (e->target_node == &e->con->node) {
e->target_node = node_get_parent(e->target_node);
if (e->target_node && (e->target_node == &e->con->node ||
node_has_ancestor(e->target_node, &e->con->node))) {
e->target_node = node_get_parent(&e->con->node);
}
e->target_edge = edge;
update_indicator(e, &box);