libnfs.a: move get_socket() function to utils/mount/network.c
authorChuck Lever <chuck.lever@oracle.com>
Sat, 28 Jul 2007 21:50:45 +0000 (17:50 -0400)
committerNeil Brown <neilb@suse.de>
Mon, 30 Jul 2007 06:12:53 +0000 (16:12 +1000)
Now we can address the real problem: that get_socket() depends on the
global variable "verbose" which is only available in the mount command.

Move get_socket() into utils/mount/network.c, and make it static.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Neil Brown <neilb@suse.de>
support/include/conn.h
support/nfs/conn.c
utils/mount/network.c

index ee1b686..2b1f07d 100644 (file)
@@ -28,7 +28,5 @@ typedef struct {
 static const struct timeval TIMEOUT = { 20, 0 };
 static const struct timeval RETRY_TIMEOUT = { 3, 0 };
 
-int get_socket(struct sockaddr_in *, u_int, int, int);
-
 #endif /* _CONN_H */
 
index d153aa2..db9185d 100644 (file)
 
 #include "conn.h"
 
-#if SIZEOF_SOCKLEN_T - 0 == 0
-#define socklen_t int
-#endif
-
-extern int verbose;
-
-/*
- * Create a socket that is locally bound to a 
- * reserve or non-reserve port. For any failures,
- * RPC_ANYSOCK is returned which will cause 
- * the RPC code to create the socket instead. 
- */
-int get_socket(struct sockaddr_in *saddr, u_int p_prot, int resvp, int conn)
-{
-       int so, cc, type;
-       struct sockaddr_in laddr;
-       socklen_t namelen = sizeof(laddr);
-
-       type = (p_prot == IPPROTO_UDP ? SOCK_DGRAM : SOCK_STREAM);
-       if ((so = socket (AF_INET, type, p_prot)) < 0) {
-               rpc_createerr.cf_stat = RPC_SYSTEMERROR;
-               rpc_createerr.cf_error.re_errno = errno;
-               if (verbose) {
-                       fprintf(stderr, 
-                               "mount: Unable to create %s socket: errno %d (%s)\n",
-                               p_prot == IPPROTO_UDP ? "UDP" : "TCP", 
-                               errno, strerror(errno));
-               }
-               return RPC_ANYSOCK;
-       }
-       laddr.sin_family = AF_INET;
-       laddr.sin_port = 0;
-       laddr.sin_addr.s_addr = htonl(INADDR_ANY);
-       if (resvp) {
-               if (bindresvport(so, &laddr) < 0) {
-                       rpc_createerr.cf_stat = RPC_SYSTEMERROR;
-                       rpc_createerr.cf_error.re_errno = errno;
-                       if (verbose) {
-                               fprintf(stderr, 
-                                       "mount: Unable to bindresvport %s socket: errno %d (%s)\n",
-                                       p_prot == IPPROTO_UDP ? "UDP" : "TCP", 
-                                       errno, strerror(errno));
-                       }
-                       close(so);
-                       return RPC_ANYSOCK;
-               }
-       } else {
-               cc = bind(so, (struct sockaddr *)&laddr, namelen);
-               if (cc < 0) {
-                       rpc_createerr.cf_stat = RPC_SYSTEMERROR;
-                       rpc_createerr.cf_error.re_errno = errno;
-                       if (verbose) {
-                               fprintf(stderr, 
-                                       "mount: Unable to bind to %s socket: errno %d (%s)\n",
-                                       p_prot == IPPROTO_UDP ? "UDP" : "TCP", 
-                                       errno, strerror(errno));
-                       }
-                       close(so);
-                       return RPC_ANYSOCK;
-               }
-       }
-       if (type == SOCK_STREAM || (conn && type == SOCK_DGRAM)) {
-               cc = connect(so, (struct sockaddr *)saddr, namelen);
-               if (cc < 0) {
-                       rpc_createerr.cf_stat = RPC_SYSTEMERROR;
-                       rpc_createerr.cf_error.re_errno = errno;
-                       if (verbose) {
-                               fprintf(stderr, 
-                                       "mount: Unable to connect to %s:%d, errno %d (%s)\n",
-                                       inet_ntoa(saddr->sin_addr), ntohs(saddr->sin_port),
-                                       errno, strerror(errno));
-                       }
-                       close(so);
-                       return RPC_ANYSOCK;
-               }
-       }
-       return so;
-}
index a5b0b71..4831990 100644 (file)
@@ -158,6 +158,79 @@ int nfs_gethostbyname(const char *hostname, struct sockaddr_in *saddr)
        return 1;
 }
 
+/*
+ * Create a socket that is locally bound to a reserved or non-reserved
+ * port. For any failures, RPC_ANYSOCK is returned which will cause 
+ * the RPC code to create the socket instead. 
+ */
+static int get_socket(struct sockaddr_in *saddr, unsigned int p_prot,
+                       int resvp, int conn)
+{
+       int so, cc, type;
+       struct sockaddr_in laddr;
+       socklen_t namelen = sizeof(laddr);
+
+       type = (p_prot == IPPROTO_UDP ? SOCK_DGRAM : SOCK_STREAM);
+       if ((so = socket (AF_INET, type, p_prot)) < 0) {
+               rpc_createerr.cf_stat = RPC_SYSTEMERROR;
+               rpc_createerr.cf_error.re_errno = errno;
+               if (verbose) {
+                       fprintf(stderr, 
+                               "mount: Unable to create %s socket: errno %d (%s)\n",
+                               p_prot == IPPROTO_UDP ? "UDP" : "TCP", 
+                               errno, strerror(errno));
+               }
+               return RPC_ANYSOCK;
+       }
+       laddr.sin_family = AF_INET;
+       laddr.sin_port = 0;
+       laddr.sin_addr.s_addr = htonl(INADDR_ANY);
+       if (resvp) {
+               if (bindresvport(so, &laddr) < 0) {
+                       rpc_createerr.cf_stat = RPC_SYSTEMERROR;
+                       rpc_createerr.cf_error.re_errno = errno;
+                       if (verbose) {
+                               fprintf(stderr, 
+                                       "mount: Unable to bindresvport %s socket: errno %d (%s)\n",
+                                       p_prot == IPPROTO_UDP ? "UDP" : "TCP", 
+                                       errno, strerror(errno));
+                       }
+                       close(so);
+                       return RPC_ANYSOCK;
+               }
+       } else {
+               cc = bind(so, (struct sockaddr *)&laddr, namelen);
+               if (cc < 0) {
+                       rpc_createerr.cf_stat = RPC_SYSTEMERROR;
+                       rpc_createerr.cf_error.re_errno = errno;
+                       if (verbose) {
+                               fprintf(stderr, 
+                                       "mount: Unable to bind to %s socket: errno %d (%s)\n",
+                                       p_prot == IPPROTO_UDP ? "UDP" : "TCP", 
+                                       errno, strerror(errno));
+                       }
+                       close(so);
+                       return RPC_ANYSOCK;
+               }
+       }
+       if (type == SOCK_STREAM || (conn && type == SOCK_DGRAM)) {
+               cc = connect(so, (struct sockaddr *)saddr, namelen);
+               if (cc < 0) {
+                       rpc_createerr.cf_stat = RPC_SYSTEMERROR;
+                       rpc_createerr.cf_error.re_errno = errno;
+                       if (verbose) {
+                               fprintf(stderr, 
+                                       "mount: Unable to connect to %s:%d, errno %d (%s)\n",
+                                       inet_ntoa(saddr->sin_addr), ntohs(saddr->sin_port),
+                                       errno, strerror(errno));
+                       }
+                       close(so);
+                       return RPC_ANYSOCK;
+               }
+       }
+       return so;
+}
+
 /*
  * getport() is very similar to pmap_getport() with the exception that
  * this version tries to use an ephemeral port, since reserved ports are