From e03ee85ef434f307500a71927dfb3e876161847a Mon Sep 17 00:00:00 2001 From: Max Altgelt Date: Fri, 10 Feb 2023 11:38:42 +0100 Subject: [PATCH] runtime: Allow handling of EXCEPTION_IN_PAGE_ERROR Currently, access faults on memory mapped files on Windows (e.g. from the drive the memory mapped file is on being ejected) cause a runtime fault that can not be caught by debug.SetPanicOnFault. On Unix systems, on the other hand, this causes a SIGBUS signal, which can be caught by debug.SetPanicOnFault. Given that the documentation of debug.SetPanicOnFault mentions handling memory mapped files, this is arguably the correct behaviour. Add handling, analogous to SIGBUS, to EXCEPTION_IN_PAGE_ERROR on Windows, to allow for users to handle this error. Fixes #58457 Change-Id: Ic7695fc01271f3552782089ac75c403d5279811f Reviewed-on: https://go-review.googlesource.com/c/go/+/467195 Reviewed-by: David Chase Reviewed-by: Quim Muntal Run-TryBot: David Chase TryBot-Result: Gopher Robot Reviewed-by: Russ Cox Reviewed-by: Alex Brainman --- src/runtime/defs_windows.go | 1 + src/runtime/signal_windows.go | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/runtime/defs_windows.go b/src/runtime/defs_windows.go index 7e5d9570f80e64..60f20a5c2cbe91 100644 --- a/src/runtime/defs_windows.go +++ b/src/runtime/defs_windows.go @@ -27,6 +27,7 @@ const ( _CTRL_SHUTDOWN_EVENT = 0x6 _EXCEPTION_ACCESS_VIOLATION = 0xc0000005 + _EXCEPTION_IN_PAGE_ERROR = 0xc0000006 _EXCEPTION_BREAKPOINT = 0x80000003 _EXCEPTION_ILLEGAL_INSTRUCTION = 0xc000001d _EXCEPTION_FLT_DENORMAL_OPERAND = 0xc000008d diff --git a/src/runtime/signal_windows.go b/src/runtime/signal_windows.go index 1ea0d1e06c470d..e4258f01b0865e 100644 --- a/src/runtime/signal_windows.go +++ b/src/runtime/signal_windows.go @@ -76,6 +76,7 @@ func isgoexception(info *exceptionrecord, r *context) bool { default: return false case _EXCEPTION_ACCESS_VIOLATION: + case _EXCEPTION_IN_PAGE_ERROR: case _EXCEPTION_INT_DIVIDE_BY_ZERO: case _EXCEPTION_INT_OVERFLOW: case _EXCEPTION_FLT_DENORMAL_OPERAND: @@ -345,7 +346,7 @@ func sigpanic() { } switch gp.sig { - case _EXCEPTION_ACCESS_VIOLATION: + case _EXCEPTION_ACCESS_VIOLATION, _EXCEPTION_IN_PAGE_ERROR: if gp.sigcode1 < 0x1000 { panicmem() }