mirrors/labwc
1
0
Fork 0
mirror of https://github.com/labwc/labwc.git synced 2026-04-12 08:21:13 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

xwayland: fix segv bug when starting game

Browse source 
(threadid=<optimized out>, signo=signo@entry=6, no_tid=no_tid@entry=0)
    at pthread_kill.c:44
    at pthread_kill.c:78
    (fmt=0x7739d9f9bb68 "%s%s%s:%u: %s%sAssertion `%s' failed.\n%n", assertion=assertion@entry=0x5a76573f9222 "root", file=file@entry=0x5a76573f9606 "../labwc/src/view.c", line=line@entry=2013, function=function@entry=0x5a7657400320 <__PRETTY_FUNCTION__.14> "view_move_to_front") at assert.c:94
    (assertion=assertion@entry=0x5a76573f9222 "root", file=file@entry=0x5a76573f9606 "../labwc/src/view.c", line=line@entry=2013, function=function@entry=0x5a7657400320 <__PRETTY_FUNCTION__.14> "view_move_to_front") at assert.c:103
    at ../labwc/src/view.c:2013
    at ../labwc/src/view-impl-common.c:30
    at ../labwc/src/xwayland.c:677
    at ../wayland-1.22.0/src/wayland-server.c:2241
    (signal=signal@entry=0x5a7659025160, data=data@entry=0x5a7659024e90)
    at ../wayland-1.22.0/src/wayland-server.c:2241
    at ../subprojects/wlroots/types/wlr_compositor.c:493
    (cif=cif@entry=0x7ffc74d32530, fn=<optimized out>, rvalue=<optimized out>, avalue=<optimized out>, closure=closure@entry=0x0) at ../src/x86/ffi64.c:673
    (cif=cif@entry=0x7ffc74d32530, fn=<optimized out>, rvalue=rvalue@entry=0x0, avalue=avalue@entry=0x7ffc74d32600) at ../src/x86/ffi64.c:710
    (closure=closure@entry=0x5a7658f5adc0, target=<optimized out>,
    target@entry=0x5a7659025240, opcode=opcode@entry=6, data=<optimized out>,
    data@entry=0x5a7658609820, flags=2) at ../wayland-1.22.0/src/connection.c:1025
    (fd=<optimized out>, mask=<optimized out>, data=<optimized out>)
    at ../wayland-1.22.0/src/wayland-server.c:438
    (loop=0x5a7657816e60, timeout=timeout@entry=-1)
    at ../wayland-1.22.0/src/event-loop.c:1027
    at ../wayland-1.22.0/src/wayland-server.c:1493
    at ../labwc/src/main.c:179

Reported-by: @kode54
This commit is contained in:
Johan Malm 2024-03-17 20:51:29 +00:00
parent 7e419f7584
commit 1c51ae3de1
1 changed files with 17 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

19
src/view.c
View file

@ -2010,7 +2010,18 @@ view_move_to_front(struct view *view)
} }
struct view *root = view_get_root(view); struct view *root = view_get_root(view);
assert(root);
/*
* The case of root == NULL is unlikely, but has been reported when
* starting games on XWayland. It's believed to be caused by setting
* override-redirect on the root wlr_xwayland_surface making it not be
* associated with a view anymore.
*
* I this case we just raise the original view instead.
*/
if (!root) {
root = view;
}
move_to_front(root); move_to_front(root);
for_each_subview(root, move_to_front); for_each_subview(root, move_to_front);
@ -2027,7 +2038,11 @@ view_move_to_back(struct view *view)
{ {
assert(view); assert(view);
struct view *root = view_get_root(view); struct view *root = view_get_root(view);
assert(root);
/* See comment in view_move_to_front() */
if (!root) {
root = view;
}
for_each_subview(root, move_to_back); for_each_subview(root, move_to_back);
move_to_back(root); move_to_back(root);