avoid close(-1) cases
authorderaadt <deraadt@openbsd.org>
Tue, 10 Dec 1996 07:46:33 +0000 (07:46 +0000)
committerderaadt <deraadt@openbsd.org>
Tue, 10 Dec 1996 07:46:33 +0000 (07:46 +0000)
lib/libc/rpc/clnt_simple.c
lib/libc/rpc/clnt_tcp.c
lib/libc/rpc/pmap_getmaps.c
lib/libc/rpc/pmap_getport.c
lib/libc/rpc/pmap_rmt.c

index 20071c9..bc127b0 100644 (file)
@@ -28,7 +28,7 @@
  */
 
 #if defined(LIBC_SCCS) && !defined(lint)
-static char *rcsid = "$OpenBSD: clnt_simple.c,v 1.5 1996/12/10 07:43:34 deraadt Exp $";
+static char *rcsid = "$OpenBSD: clnt_simple.c,v 1.6 1996/12/10 07:46:33 deraadt Exp $";
 #endif /* LIBC_SCCS and not lint */
 
 /* 
@@ -80,7 +80,8 @@ callrpc(host, prognum, versnum, procnum, inproc, in, outproc, out)
                /* reuse old client */          
        } else {
                crp->valid = 0;
-               (void)close(crp->socket);
+               if (crp->socket != -1)
+                       (void)close(crp->socket);
                crp->socket = RPC_ANYSOCK;
                if (crp->client) {
                        clnt_destroy(crp->client);
index 694ebeb..3cc3322 100644 (file)
@@ -28,7 +28,7 @@
  */
 
 #if defined(LIBC_SCCS) && !defined(lint)
-static char *rcsid = "$OpenBSD: clnt_tcp.c,v 1.10 1996/11/14 06:17:36 etheisen Exp $";
+static char *rcsid = "$OpenBSD: clnt_tcp.c,v 1.11 1996/12/10 07:46:37 deraadt Exp $";
 #endif /* LIBC_SCCS and not lint */
  
 /*
@@ -163,7 +163,8 @@ clnttcp_create(raddr, prog, vers, sockp, sendsz, recvsz)
                    sizeof(*raddr)) < 0)) {
                        rpc_createerr.cf_stat = RPC_SYSTEMERROR;
                        rpc_createerr.cf_error.re_errno = errno;
-                       (void)close(*sockp);
+                       if (*sockp != -1)
+                               (void)close(*sockp);
                        goto fooy;
                }
                ct->ct_closeit = TRUE;
index be6cd8f..b7a822b 100644 (file)
@@ -28,7 +28,7 @@
  */
 
 #if defined(LIBC_SCCS) && !defined(lint)
-static char *rcsid = "$OpenBSD: pmap_getmaps.c,v 1.3 1996/08/19 08:31:37 tholo Exp $";
+static char *rcsid = "$OpenBSD: pmap_getmaps.c,v 1.4 1996/12/10 07:46:39 deraadt Exp $";
 #endif /* LIBC_SCCS and not lint */
 
 /*
@@ -76,7 +76,8 @@ pmap_getmaps(address)
                }
                CLNT_DESTROY(client);
        }
-       (void)close(socket);
+       if (socket != -1)
+               (void)close(socket);
        address->sin_port = 0;
        return (head);
 }
index 89e867e..07e2191 100644 (file)
@@ -28,7 +28,7 @@
  */
 
 #if defined(LIBC_SCCS) && !defined(lint)
-static char *rcsid = "$OpenBSD: pmap_getport.c,v 1.3 1996/08/19 08:31:38 tholo Exp $";
+static char *rcsid = "$OpenBSD: pmap_getport.c,v 1.4 1996/12/10 07:46:41 deraadt Exp $";
 #endif /* LIBC_SCCS and not lint */
 
 /*
@@ -81,7 +81,8 @@ pmap_getport(address, program, version, protocol)
                }
                CLNT_DESTROY(client);
        }
-       (void)close(socket);
+       if (socket != -1)
+               (void)close(socket);
        address->sin_port = 0;
        return (port);
 }
index 5d205e7..00c0254 100644 (file)
@@ -28,7 +28,7 @@
  */
 
 #if defined(LIBC_SCCS) && !defined(lint)
-static char *rcsid = "$OpenBSD: pmap_rmt.c,v 1.11 1996/11/14 06:23:50 etheisen Exp $";
+static char *rcsid = "$OpenBSD: pmap_rmt.c,v 1.12 1996/12/10 07:46:43 deraadt Exp $";
 #endif /* LIBC_SCCS and not lint */
 
 /*
@@ -96,7 +96,8 @@ pmap_rmtcall(addr, prog, vers, proc, xdrargs, argsp, xdrres, resp, tout, port_pt
        } else {
                stat = RPC_FAILED;
        }
-       (void)close(socket);
+       if (socket != -1)
+               (void)close(socket);
        addr->sin_port = 0;
        return (stat);
 }