diff options
author | Matt Johnston <matt@ucc.asn.au> | 2015-02-24 20:53:32 +0800 |
---|---|---|
committer | Matt Johnston <matt@ucc.asn.au> | 2015-02-24 20:53:32 +0800 |
commit | 4b1f5ec7c29043a7d4ec0dd75f10479ed909a187 (patch) | |
tree | 66366d412ebceb7b2a9f473729c136b9845cfcd3 /common-channel.c | |
parent | ab9439519ac3ca1ce5b273587544da3494f87308 (diff) |
Use m_close() which will avoid close(-1)
Diffstat (limited to 'common-channel.c')
-rw-r--r-- | common-channel.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/common-channel.c b/common-channel.c index db47695..a683448 100644 --- a/common-channel.c +++ b/common-channel.c @@ -586,11 +586,11 @@ static void remove_channel(struct Channel * channel) { /* close the FDs in case they haven't been done * yet (they might have been shutdown etc) */ TRACE(("CLOSE writefd %d", channel->writefd)) - close(channel->writefd); + m_close(channel->writefd); TRACE(("CLOSE readfd %d", channel->readfd)) - close(channel->readfd); + m_close(channel->readfd); TRACE(("CLOSE errfd %d", channel->errfd)) - close(channel->errfd); + m_close(channel->errfd); } if (!channel->close_handler_done @@ -1001,7 +1001,7 @@ static void close_chan_fd(struct Channel *channel, int fd, int how) { } } else { TRACE(("CLOSE some fd %d", fd)) - close(fd); + m_close(fd); closein = closeout = 1; } @@ -1024,7 +1024,7 @@ static void close_chan_fd(struct Channel *channel, int fd, int how) { if (channel->type->sepfds && channel->readfd == FD_CLOSED && channel->writefd == FD_CLOSED && channel->errfd == FD_CLOSED) { TRACE(("CLOSE (finally) of %d", fd)) - close(fd); + m_close(fd); } } |