Merge pull request #10416 from Icinga/jschmidt/sigabrt-handler-fix

jschmidt-icinga · web-flow · commit 235a3e1aa59c · 2025-04-23T16:08:13.000+02:00
Fix SIGABRT not causing a core dump
diff --git a/AUTHORS b/AUTHORS
@@ -135,6 +135,7 @@ Jesse Morgan <morgajel@gmail.com>
 Jo Goossens <jo.goossens@hosted-power.com>
 Jochen Friedrich <j.friedrich@nwe.de>
 Johannes Meyer <johannes.meyer@icinga.com>
+Johannes Schmidt <johannes.schmidt@icinga.com>
 Jonas Meurer <jonas@freesources.org>
 Jordi van Scheijen <jordi.vanscheijen@solvinity.com>
 Josef Friedrich <josef@friedrich.rocks>
diff --git a/lib/base/application.cpp b/lib/base/application.cpp
@@ -776,6 +776,12 @@ void Application::SigAbrtHandler(int)
 	}
 
 	AttachDebugger(fname, interactive_debugger);
+	
+#ifdef __linux__
+	prctl(PR_SET_DUMPABLE, 1);
+#endif /* __linux __ */
+	
+	abort();
 }
 
 #ifdef _WIN32

Original file line number	Diff line number	Diff line change
`@@ -776,6 +776,12 @@ void Application::SigAbrtHandler(int)`
`776`	`776`	`}`
`777`	`777`
`778`	`778`	`AttachDebugger(fname, interactive_debugger);`
	`779`	`+`
	`780`	`+#ifdef __linux__`
	`781`	`+ prctl(PR_SET_DUMPABLE, 1);`
	`782`	`+#endif /* __linux __ */`
	`783`	`+`
	`784`	`+ abort();`
`779`	`785`	`}`
`780`	`786`
`781`	`787`	`#ifdef _WIN32`