Index: components/nacl/loader/nonsfi/irt_exception_handling.cc |
diff --git a/components/nacl/loader/nonsfi/irt_exception_handling.cc b/components/nacl/loader/nonsfi/irt_exception_handling.cc |
index 327f6a40e598dfcb69e99f791bd822d103df09e6..cc5ad9b9881b94dfef5baf55a246b3c4fd5d57d2 100644 |
--- a/components/nacl/loader/nonsfi/irt_exception_handling.cc |
+++ b/components/nacl/loader/nonsfi/irt_exception_handling.cc |
@@ -29,7 +29,7 @@ const int kSignals[] = { |
// This signal does not exist on MIPS. |
SIGSTKFLT, |
#endif |
- SIGINT, SIGQUIT, SIGILL, SIGTRAP, SIGBUS, SIGFPE, SIGSEGV, |
+ SIGHUP, SIGINT, SIGQUIT, SIGILL, SIGTRAP, SIGBUS, SIGFPE, SIGSEGV, SIGTERM, |
Mark Seaborn
2015/06/10 18:53:12
Do you need to add SIGHUP and SIGTERM, or is it ju
rickyz (no longer on Chrome)
2015/06/10 22:42:34
I want to add any signals that might be sent to th
|
// Handle SIGABRT in case someone sends it asynchronously using kill(). |
SIGABRT |
}; |
@@ -49,7 +49,7 @@ void SignalCatch(int sig, siginfo_t* info, void* uc) { |
not useful for NonSFI NaCl. */); |
signal_handler_function_pointer(&exception_frame.context); |
} |
- _exit(-1); |
+ _exit(-sig); |
rickyz (no longer on Chrome)
2015/06/10 01:31:43
I didn't want to change this file a ton in this CL
Mark Seaborn
2015/06/10 18:53:12
Yes, that's all intentional.
This implements the
rickyz (no longer on Chrome)
2015/06/10 22:42:34
Ah, in that case, is it incorrect to run the excep
Mark Seaborn
2015/06/15 22:29:04
Yes, it's technically incorrect, although it might
rickyz (no longer on Chrome)
2015/06/15 23:23:51
Ah, thank you for the detailed explanation. In tha
|
} |
int IrtExceptionHandler(NaClExceptionHandler handler, |