Bug 382843

Summary:	Reliably crashes on wayland when clicking logout button
Product:	[Frameworks and Libraries] libplasma	Reporter:	Fabian Vogt <fabian>
Component:	tooltips	Assignee:	David Edmundson <kde>
Status:	RESOLVED DUPLICATE
Severity:	crash	CC:	kde
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Fabian Vogt 2017-07-28 08:58:13 UTC

I get this backtrace 100% reproducible:

#0  0x00007f87f15baa90 in raise () from /lib64/libc.so.6
#1  0x00007f87f15bc0f6 in abort () from /lib64/libc.so.6
#2  0x00007f87f05af41f in ?? ()
#3  0x00007f87f05ab4dd in wl_proxy_marshal_array_constructor_versioned () from /usr/lib64/libwayland-client.so.0
#4  0x00007f87f05ab5ec in wl_proxy_marshal () from /usr/lib64/libwayland-client.so.0
#5  0x00007f87e4714188 in QtWaylandClient::QWaylandWlShellSurface::updateTransientParent(QWindow*) () from /usr/lib64/libQt5WaylandClient.so.5
#6  0x00007f87e47124fb in QtWaylandClient::QWaylandWindow::initWindow() () from /usr/lib64/libQt5WaylandClient.so.5
#7  0x00007f87e4712b0d in QtWaylandClient::QWaylandWindow::setVisible(bool) () from /usr/lib64/libQt5WaylandClient.so.5
#8  0x00007f874fdf9cf4 in QtWaylandClient::QWaylandEglWindow::setVisible(bool) () from /usr/lib64/qt5/plugins/wayland-graphics-integration-client/libwayland-egl.so
#9  0x00007f87f2790c9d in QWindow::setVisible(bool) () from /usr/lib64/libQt5Gui.so.5
#10 0x00007f87f7a5c0cd in PlasmaQuick::Dialog::setVisible (this=0x556ed82ce3c0, visible=<optimized out>) at /usr/src/debug/plasma-framework-5.36.0git.20170727T045342~8125fbcbf/src/plasmaquick/dialog.cpp:1369
#11 0x00007f87450f4d9f in ToolTip::showToolTip (this=0x556edc1c0ae0) at /usr/src/debug/plasma-framework-5.36.0git.20170727T045342~8125fbcbf/src/declarativeimports/core/tooltip.cpp:160
#12 0x00007f87f1f6035c in QMetaObject::activate(QObject*, int, int, void**) () from /usr/lib64/libQt5Core.so.5
#13 0x00007f87f1f6c6a7 in QTimer::timeout(QTimer::QPrivateSignal) () from /usr/lib64/libQt5Core.so.5
#14 0x00007f87f1f6c9d8 in QTimer::timerEvent(QTimerEvent*) () from /usr/lib64/libQt5Core.so.5
#15 0x00007f87f1f60bfb in QObject::event(QEvent*) () from /usr/lib64/libQt5Core.so.5
#16 0x00007f87f2f4193c in QApplicationPrivate::notify_helper(QObject*, QEvent*) () from /usr/lib64/libQt5Widgets.so.5
#17 0x00007f87f2f48cb4 in QApplication::notify(QObject*, QEvent*) () from /usr/lib64/libQt5Widgets.so.5
#18 0x00007f87f1f32d48 in QCoreApplication::notifyInternal2(QObject*, QEvent*) () from /usr/lib64/libQt5Core.so.5
#19 0x00007f87f1f87c3e in QTimerInfoList::activateTimers() () from /usr/lib64/libQt5Core.so.5
#20 0x00007f87f1f88389 in ?? () from /usr/lib64/libQt5Core.so.5
#21 0x00007f87ec69eb67 in g_main_context_dispatch () from /usr/lib64/libglib-2.0.so.0
#22 0x00007f87ec69ed98 in ?? () from /usr/lib64/libglib-2.0.so.0
#23 0x00007f87ec69ee2c in g_main_context_iteration () from /usr/lib64/libglib-2.0.so.0
#24 0x00007f87f1f8868f in QEventDispatcherGlib::processEvents(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/lib64/libQt5Core.so.5
#25 0x00007f87f1f3135a in QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/lib64/libQt5Core.so.5
#26 0x00007f87f1f39be4 in QCoreApplication::exec() () from /usr/lib64/libQt5Core.so.5
#27 0x0000556ed356cd9b in ?? ()
#28 0x00007f87f15a646a in __libc_start_main () from /lib64/libc.so.6
#29 0x0000556ed356d11a in _start ()

This is printed on stderr: "error marshalling arguments for set_transient (signature oiiu): null value passed for arg 0"

Not sure whether this is a bug in plasma-framework or Qt 5.9.1.

Workaround:

[PlasmaToolTips]
Delay=0

in ~/.config/plasmarc

Comment 1 David Edmundson 2017-07-28 09:05:21 UTC

>Not sure whether this is a bug in plasma-framework or Qt 5.9.1.

Bit of both.

Spec says       <arg name="parent" type="object" interface="wl_surface" summary="parent surface"/>


It doesn't have: allow-null="true"

Qt should check that and do nothing rather than make a protocol error.
It implies Plasma is setting an invisible window as a transient parent which doesn't make sense either.

But we'lll be switching to XdgV6 soon as not running this code anyway.

Comment 2 Fabian Vogt 2017-09-14 15:13:11 UTC

This bug is still 100% reproducible in Plasma 5.11.
So I would suggest fixing this for pre-xdg-shell as well, as plasma is practically unusuable without this.

Comment 3 David Edmundson 2017-09-14 16:09:27 UTC

ok.

Comment 4 Christoph Feck 2017-10-04 13:49:54 UTC


*** This bug has been marked as a duplicate of bug 381032 ***