Fix unintentional exit on suspend

Suspending xorg on FreeBSD (and possibly other scenarios) results in a
recoverable INTR from select(). Only exit on signal after actually
having recieved a signal.

2 files changed, 8 insertions, 4 deletions

diff --git a/Makefile b/Makefile
index 460dca3..70a0921 100644
--- a/Makefile
+++ b/Makefile
@@ -1,4 +1,4 @@
-VERSION=20180725
+VERSION=20231108
 DIST=mcwm-$(VERSION)
 SRC=mcwm.c list.c config.h events.h list.h hidden.c
 DISTFILES=LICENSE Makefile NEWS README TODO WISHLIST mcwm.man hidden.man scripts $(SRC)
diff --git a/mcwm.c b/mcwm.c
index 6a2e6ce..771c997 100644
--- a/mcwm.c
+++ b/mcwm.c
@@ -3594,6 +3594,7 @@ void events(void)
     int fd;                         /* Our X file descriptor */
     fd_set in;                      /* For select */
     int found;                      /* Ditto. */
+    int err;
 
     /* Get the file descriptor so we can do select() on it. */
     fd = xcb_get_file_descriptor(conn);
@@ -3622,8 +3623,9 @@ void events(void)
              * Check if we have an unrecoverable connection error,
              * like a disconnected X server.
              */
-            if (xcb_connection_has_error(conn))
+            if ((err = xcb_connection_has_error(conn)))
             {
+                PDEBUG("xcb_connection_has_error() returned %d.\n", err);
                 cleanup(0);
                 exit(1);
             }
@@ -3633,8 +3635,10 @@ void events(void)
             {
                 if (EINTR == errno)
                 {
-                    /* We received a signal. Break out of loop. */
-                    break;
+                    if (sigcode != 0) {
+                        /* We received a signal. Break out of loop. */
+                        break;
+                    }
                 }
                 else
                 {