Use ssize_t which is a signed int and test if read() returns -1.

Thanks to David Wu.

diff --git a/signals.c b/signals.c
index daf0347616aa9251298e0afe691e43c9d6fcffb0..648bd311fae1a8fcdf11f5ded6ddc6024cf57840 100644 (file)
--- a/signals.c
+++ b/signals.c
@@ -71,11 +71,11 @@ signal_read(void)
 {
        int sig = -1;
        char buf[16];
-       size_t bytes;
+       ssize_t bytes;
 
        memset(buf, 0, sizeof(buf));
        bytes = read(signal_pipe[0], buf, sizeof(buf));
-       if (bytes >= sizeof(sig))
+       if (bytes >= 0 && bytes >= sizeof(sig))
                memcpy(&sig, buf, sizeof(sig));
        return sig;
 }