diff options
| author | Roy Marples <roy@marples.name> | 2007-07-18 11:35:50 +0000 |
|---|---|---|
| committer | Roy Marples <roy@marples.name> | 2007-07-18 11:35:50 +0000 |
| commit | d6be0166aa5874a273c4328cea8106aa92aab5d2 (patch) | |
| tree | cb6833a169724ff868a995d6c013f68d8cd6de33 /signals.c | |
| parent | 77659ce0359c096d8a2667541879f8a59fad8121 (diff) | |
| download | dhcpcd-d6be0166aa5874a273c4328cea8106aa92aab5d2.tar.xz | |
==-1 is cheaper than <0
Diffstat (limited to 'signals.c')
| -rw-r--r-- | signals.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -44,7 +44,7 @@ static void signal_handler (int sig) return; } - if (send (signal_pipe[1], &sig, sizeof (sig), MSG_DONTWAIT) < 0) + if (send (signal_pipe[1], &sig, sizeof (sig), MSG_DONTWAIT) == -1) logger (LOG_ERR, "Could not send signal: %s", strerror (errno)); } @@ -59,8 +59,8 @@ void signal_setup(void) /* Stop any scripts from inheriting us */ for (i = 0; i < 2; i++) - if ((flags = fcntl (signal_pipe[i], F_GETFD, 0)) < 0 || - fcntl (signal_pipe[i], F_SETFD, flags | FD_CLOEXEC) < 0) + if ((flags = fcntl (signal_pipe[i], F_GETFD, 0)) == -1 || + fcntl (signal_pipe[i], F_SETFD, flags | FD_CLOEXEC) == -1) logger (LOG_ERR ,"fcntl: %s", strerror (errno)); signal (SIGHUP, signal_handler); @@ -92,7 +92,7 @@ int signal_read (const fd_set *rfds) if (! FD_ISSET (signal_pipe[0], rfds)) return 0; - if (read (signal_pipe[0], &sig, sizeof (sig)) < 0) + if (read (signal_pipe[0], &sig, sizeof (sig)) == -1) return -1; return sig; |
