From 38d296fd2fe1c5dd13b0b700b7c3900540475111 Mon Sep 17 00:00:00 2001 From: Sean Hefty Date: Fri, 25 May 2012 12:23:10 -0700 Subject: [PATCH] acm: Use -1 to indicate an invalid socket rather than 0 socket() can return 0 as a valid socket. This can happen when using a daemon that closes stdin/out/err. Signed-off-by: Sean Hefty --- src/acm.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/acm.c b/src/acm.c index bcf11dad..9c659195 100755 --- a/src/acm.c +++ b/src/acm.c @@ -61,7 +61,7 @@ typedef struct acm_msg cma_acm_msg_t; #endif static pthread_mutex_t acm_lock = PTHREAD_MUTEX_INITIALIZER; -static int sock; +static int sock = -1; static short server_port = 6125; struct ib_connect_hdr { @@ -108,12 +108,12 @@ void ucma_ib_init(void) err: close(sock); - sock = 0; + sock = -1; } void ucma_ib_cleanup(void) { - if (sock > 0) { + if (sock >= 0) { shutdown(sock, SHUT_RDWR); close(sock); } @@ -322,7 +322,7 @@ void ucma_ib_resolve(struct rdma_addrinfo **rai, struct rdma_addrinfo *hints) struct acm_ep_addr_data *data; int ret; - if (sock <= 0) + if (sock < 0) return; memset(&msg, 0, sizeof msg); -- 2.45.2