Code

Check the return value of write() to satisfy GCC's unused-result checks.
authorSebastian Harl <sh@tokkee.org>
Fri, 27 Feb 2015 16:31:29 +0000 (17:31 +0100)
committerSebastian Harl <sh@tokkee.org>
Fri, 27 Feb 2015 16:31:29 +0000 (17:31 +0100)
src/frontend/sock.c

index cf187fcb42adc4bffc229f6c20ed208105c9bde5..1f3ec5fd74b70ee903a68ce8678bd2fb69c4a87b 100644 (file)
@@ -570,7 +570,11 @@ connection_handler(void *data)
                                        "connection %s to list of open connections",
                                        SDB_OBJ(conn)->name);
                }
-               write(sock->trigger[TRIGGER_WRITE], "", 1);
+               if (write(sock->trigger[TRIGGER_WRITE], "", 1) <= 0) {
+                       /* This shouldn't happen and it's not critical; in the worst cases
+                        * it slows us down. */
+                       sdb_log(SDB_LOG_WARNING, "frontend: Failed to trigger main loop");
+               }
 
                /* pass ownership back to list; or destroy in case of an error */
                sdb_object_deref(SDB_OBJ(conn));