aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2008-03-26 10:37:06 +0000
committerEric Wong <normalperson@yhbt.net>2008-03-26 10:37:06 +0000
commit33e88ff8f96627920c63b2d05dc16cb7ae15f318 (patch)
treeef152b410a24afceb409f5cd4aac3415b333e4a8
parentc6ceceae8a7f8b592c63086daa8f2269fb71ca08 (diff)
downloadmpd-33e88ff8f96627920c63b2d05dc16cb7ae15f318.tar.gz
mpd-33e88ff8f96627920c63b2d05dc16cb7ae15f318.tar.xz
mpd-33e88ff8f96627920c63b2d05dc16cb7ae15f318.zip
zero is a valid file descriptor
Although it may not happen in mpd code, it is perfectly possible for a newly allocated file descriptor to be zero. For theoretical correctness, allow 0. git-svn-id: https://svn.musicpd.org/mpd/trunk@7194 09075e82-0dd4-0310-85a5-a0d7c8717e4f
-rw-r--r--src/interface.c6
-rw-r--r--src/log.c8
2 files changed, 7 insertions, 7 deletions
diff --git a/src/interface.c b/src/interface.c
index 7dbcd2dc3..9a79f102e 100644
--- a/src/interface.c
+++ b/src/interface.c
@@ -604,7 +604,7 @@ static void closeAllInterfaces(void)
int i;
for (i = 0; i < interface_max_connections; i++) {
- if (interfaces[i].fd > 0)
+ if (interfaces[i].fd >= 0)
closeInterface(&(interfaces[i]));
if (interfaces[i].send_buf)
free(interfaces[i].send_buf);
@@ -626,7 +626,7 @@ void closeOldInterfaces(void)
int i;
for (i = 0; i < interface_max_connections; i++) {
- if (interfaces[i].fd > 0) {
+ if (interfaces[i].fd >= 0) {
if (interfaces[i].expired) {
DEBUG("interface %i: expired\n", i);
closeInterface(&(interfaces[i]));
@@ -690,7 +690,7 @@ int interfacePrintWithFD(int fd, char *buffer, int buflen)
int copylen;
Interface *interface;
- assert(fd > 0);
+ assert(fd >= 0);
if (i >= interface_max_connections ||
interfaces[i].fd < 0 || interfaces[i].fd != fd) {
diff --git a/src/log.c b/src/log.c
index 0d2d0b0b2..eb1d5a94d 100644
--- a/src/log.c
+++ b/src/log.c
@@ -36,8 +36,8 @@ static const char *err_filename;
static void redirect_logs(void)
{
- assert(out_fd > 0);
- assert(err_fd > 0);
+ assert(out_fd >= 0);
+ assert(err_fd >= 0);
if (dup2(out_fd, STDOUT_FILENO) < 0)
FATAL("problems dup2 stdout : %s\n", strerror(errno));
if (dup2(err_fd, STDERR_FILENO) < 0)
@@ -241,8 +241,8 @@ void close_log_files(void)
{
if (stdout_mode)
return;
- assert(out_fd > 0);
- assert(err_fd > 0);
+ assert(out_fd >= 0);
+ assert(err_fd >= 0);
xclose(out_fd);
xclose(err_fd);
}