It appears that the NaCl exception handling method is broken in general. I tried the following patch:

diff --git a/src/common/System.cpp b/src/common/System.cpp
index 6e598f2ff..843d2681c 100644
--- a/src/common/System.cpp
+++ b/src/common/System.cpp
@@ -335,12 +335,15 @@ static void CrashHandler(const void* data, size_t n)
     Sys::Error("Crashed with NaCl exception");
 }

+void Crash(struct NaClExceptionContext *)
+{
+       Log::Warn("nacl crash!");
+       _exit(99);
+}
+
 void SetupCrashHandler()
 {
-#if !defined(__saigo__)
-    nacl_minidump_register_crash_handler();
-    nacl_minidump_set_callback(CrashHandler);
-#endif
+        nacl_exception_set_handler(Crash);
 }
 #else
 NORETURN static void CrashHandler(int sig)

Testing like ./daemonded -set vm.sgame.type 1 +map vega +delay 50f sgame.injectFault intdiv, the nacl crash! message appears on Windows, but not on Linux. All testing is with PNaCl.

P.S. for testing crashes on older Unvanquished versions with no injectFault, these behavior tree snippets are nice.

NaCl amd64 crash dumps no longer work on Linux #1504

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions