fix dwarf-based unwinding to the end of stack (#42620)
authorYaroslav Yamshchikov <y.yamshchiko@samsung.com>
Wed, 23 Sep 2020 19:26:18 +0000 (22:26 +0300)
committerGitHub <noreply@github.com>
Wed, 23 Sep 2020 19:26:18 +0000 (21:26 +0200)
We experience CLR crash on some architectures (at least in x86) in case
of unhandled managed exception. libunwind steps to the very end of a
stack, and if .eh_frame info is correct, it returns with retcode 0 and
ip=0 from unw_step, then PAL calls unw_is_signal_frame with
c->validate==0 which in turn dereferences zeroed ip in access_mem.
libunwind spec says that retcode 0 from unw_step means very end of a
stack, so PAL should not expect any frames, signal or not. It should
convert cursor back to SEH representation and return with TRUE.

src/coreclr/src/pal/src/exception/seh-unwind.cpp

index 524d725..a63f56c 100644 (file)
@@ -357,7 +357,7 @@ BOOL PAL_VirtualUnwind(CONTEXT *context, KNONVOLATILE_CONTEXT_POINTERS *contextP
     // Check if the frame we have unwound to is a frame that caused
     // synchronous signal, like a hardware exception and record it
     // in the context flags.
-    if (unw_is_signal_frame(&cursor) > 0)
+    if ((st != 0) && (unw_is_signal_frame(&cursor) > 0))
     {
         context->ContextFlags |= CONTEXT_EXCEPTION_ACTIVE;
 #if defined(CONTEXT_UNWOUND_TO_CALL)