Bug 353030

Summary:	Drkonqi fails to show window or restart kwin after a crash
Product:	[Applications] drkonqi	Reporter:	Andrew Chen <andrew>
Component:	general	Assignee:	Unassigned bugs mailing-list <unassigned-bugs>
Status:	RESOLVED FIXED
Severity:	crash	CC:	travneff
Priority:	NOR
Version:	5.4.1
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
See Also:	https://bugs.kde.org/show_bug.cgi?id=353428
Latest Commit:	http://commits.kde.org/kwin/69aa80750f8d61a5db6311c33751461041a260d5	Version Fixed In:	5.6
Sentry Crash Report:

Description Andrew Chen 2015-09-22 06:56:58 UTC

Drkonqi is called after the kwin crash. However, it does not it's window appear or restart kwin.

Reproducible: Always

Steps to Reproduce:
1. $ killall -SIGSEGV kwin_x11 (to simulate a segfault and trigger drkonqi)

Actual Results:  
Drkonqi doesn't show, kwin is not restarted. The system is dysfunctional due to the absence of kwin.

Expected Results:  
Drkonqi appears, kwin is restarted. Things starts functioning normally again.

Log after (simulated) crash:
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: Using /proc to determine executable path
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: Executable is: "/usr/bin/kwin_x11"
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: Executable exists: true
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: Enabling drkonqi crash catching
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: Invalid pixmap specified.
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: No frame loaded
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: No frame loaded
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: No frame loaded
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: No frame loaded
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: No frame loaded
Sep 22 18:54:20 ANDREW-LAPTOP-ARCH drkonqi[11349]: No frame loaded
Sep 22 18:54:22 ANDREW-LAPTOP-ARCH drkonqi[11349]: Sending SIGSTOP to process

Comment 1 Thomas Lübking 2016-01-14 23:18:57 UTC

Git commit 69aa80750f8d61a5db6311c33751461041a260d5 by Thomas Lübking.
Committed on 14/01/2016 at 22:40.
Pushed by luebking into branch 'master'.

force restart on crash

We don't want to actively release claims on segfaults, but then
drkonqi can stop us while we're still holding the WM privs.

=> If KWin performs a crash-restart, it forcefully takes WM privs
(since the old instance shall be replaced for quite sure)
Related: bug 348834, bug 353428

REVIEW: 126741
FIXED-IN: 5.6

M  +3    -2    main_x11.cpp

http://commits.kde.org/kwin/69aa80750f8d61a5db6311c33751461041a260d5