Fix cleanup Windows' CatchInterrupts implementation.

diff --git a/os_win32.cpp b/os_win32.cpp
index 4bfdf5e00d96373c1a7e003f42c2dfd521ef935c..a35e2a3c0229aec6c7a6646c8893790be5c5f997 100644 (file)
--- a/os_win32.cpp
+++ b/os_win32.cpp
@@ -24,6 +24,7 @@
  **************************************************************************/
 
 #include <windows.h>
+#include <assert.h>
 #include <signal.h>
 #include <string.h>
 #include <stdio.h>
@@ -135,48 +136,42 @@ Abort(void)
 }
 
 
-struct Interrupts
-{
-    Interrupts()
-        : set(false),
-          prevfilter(NULL),
-          handler(NULL)
-    {}
-
-    bool set;
-    LPTOP_LEVEL_EXCEPTION_FILTER prevFilter;
+static LPTOP_LEVEL_EXCEPTION_FILTER prevExceptionFilter = NULL;
 
-    void (*handler)(int);
-};
-static Interrupts interrupts;
+static void (*handler)(int) = NULL;
 
-LONG WINAPI InterruptHandler(EXCEPTION_POINTERS *exceptionInfo)
+static LONG WINAPI UnhandledExceptionFilter(PEXCEPTION_POINTERS pExceptionInfo)
 {
-    if (interrupts.handler) {
+    if (handler) {
         int exceptionCode = 0;
-        if (exceptionInfo) {
-            exceptionCode = exceptionInfo->ExceptionRecord.ExceptionCode;
+        if (pExceptionInfo) {
+            PEXCEPTION_RECORD pExceptionRecord = pExceptionInfo->ExceptionRecord;
+            if (pExceptionRecord) {
+                exceptionCode = pExceptionRecord->ExceptionCode;
+            }
         }
 
-        interrupts.handler(exceptionCode);
+        handler(exceptionCode);
     }
 
-    if (interrupts.prevFilter) {
-        return interrupts.prevFilter(exceptionInfo);
+       if (prevExceptionFilter) {
+               return prevExceptionFilter(pExceptionInfo);
     } else {
-        return EXCEPTION_CONTINUE_SEARCH;
+               return EXCEPTION_CONTINUE_SEARCH;
     }
 }
 
 void
 CatchInterrupts(void (*func)(int))
 {
-    interrupts.handler = func;
+    assert(!handler);
+    assert(!prevExceptionFilter);
+
+    handler = func;
 
-    if (!interrupts.set) {
-        interrupts.prevFilter =
-            SetUnhandledExceptionFilter(InterruptHandler);
-        interrupts.set = true;
+    if (handler && !prevExceptionFilter) {
+        prevExceptionFilter =
+            SetUnhandledExceptionFilter(UnhandledExceptionFilter);
     }
 }