From 90a53aed59be6bac45c13eac954471d56ac23375 Mon Sep 17 00:00:00 2001 From: virchau13 Date: Sun, 28 Apr 2024 00:38:48 +0800 Subject: CrashReporter: fix deadlocks by making it mostly async-signal-safe (#5771) `CrashReporter::createAndSaveCrash()` is not async-signal-safe, resulting in random deadlocks/double-crashes during Hyprland crashes. This changes the function to be (mostly) async-signal-safe. --- src/Compositor.cpp | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'src/Compositor.cpp') diff --git a/src/Compositor.cpp b/src/Compositor.cpp index c627fe72..17786cb2 100644 --- a/src/Compositor.cpp +++ b/src/Compositor.cpp @@ -36,6 +36,14 @@ void handleUnrecoverableSignal(int sig) { return; } + // Kill the program if the crash-reporter is caught in a deadlock. + signal(SIGALRM, [](int _) { + char const* msg = "\nCrashReporter exceeded timeout, forcefully exiting\n"; + write(2, msg, strlen(msg)); + abort(); + }); + alarm(15); + CrashReporter::createAndSaveCrash(sig); abort(); -- cgit v1.2.3