Abort after handling SIGSEGV when chaining fails

ivanpovazan · ivanpovazan · commit de59b5b6a5f1 · 2024-12-16T14:51:30.000+01:00
diff --git a/src/mono/mono/mini/mini-runtime.c b/src/mono/mono/mini/mini-runtime.c
@@ -3908,7 +3908,8 @@ MONO_SIG_HANDLER_FUNC (, mono_sigsegv_signal_handler)
 		mono_handle_native_crash (mono_get_signame (SIGSEGV), &mctx, (MONO_SIG_HANDLER_INFO_TYPE*)info);
 
 		if (mono_do_crash_chaining) {
-			mono_chain_signal (MONO_SIG_HANDLER_PARAMS);
+			if (!mono_chain_signal (MONO_SIG_HANDLER_PARAMS))
+				abort ();
 			return;
 		}
 	}
@@ -3918,7 +3919,8 @@ MONO_SIG_HANDLER_FUNC (, mono_sigsegv_signal_handler)
 	} else {
 		mono_handle_native_crash (mono_get_signame (SIGSEGV), &mctx, (MONO_SIG_HANDLER_INFO_TYPE*)info);
 		if (mono_do_crash_chaining) {
-			mono_chain_signal (MONO_SIG_HANDLER_PARAMS);
+			if (!mono_chain_signal (MONO_SIG_HANDLER_PARAMS))
+				abort ();
 			return;
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -3908,7 +3908,8 @@ MONO_SIG_HANDLER_FUNC (, mono_sigsegv_signal_handler)`
`3908`	`3908`	`mono_handle_native_crash (mono_get_signame (SIGSEGV), &mctx, (MONO_SIG_HANDLER_INFO_TYPE*)info);`
`3909`	`3909`
`3910`	`3910`	`if (mono_do_crash_chaining) {`
`3911`		`- mono_chain_signal (MONO_SIG_HANDLER_PARAMS);`
	`3911`	`+ if (!mono_chain_signal (MONO_SIG_HANDLER_PARAMS))`
	`3912`	`+ abort ();`
`3912`	`3913`	`return;`
`3913`	`3914`	`}`
`3914`	`3915`	`}`
`@@ -3918,7 +3919,8 @@ MONO_SIG_HANDLER_FUNC (, mono_sigsegv_signal_handler)`
`3918`	`3919`	`} else {`
`3919`	`3920`	`mono_handle_native_crash (mono_get_signame (SIGSEGV), &mctx, (MONO_SIG_HANDLER_INFO_TYPE*)info);`
`3920`	`3921`	`if (mono_do_crash_chaining) {`
`3921`		`- mono_chain_signal (MONO_SIG_HANDLER_PARAMS);`
	`3922`	`+ if (!mono_chain_signal (MONO_SIG_HANDLER_PARAMS))`
	`3923`	`+ abort ();`
`3922`	`3924`	`return;`
`3923`	`3925`	`}`
`3924`	`3926`	`}`