src/usr.bin/ssh/sshconnect.c - diff

Return to sshconnect.c CVS log

Up to [local] / src / usr.bin / ssh

Diff for /src/usr.bin/ssh/sshconnect.c between version 1.239 and 1.240

-version 1.239, 2013/08/20 00:11:38
+version 1.240, 2013/09/19 01:26:29
 Line 272
 Line 272
 Line 272
  static int
  ssh_create_socket(int privileged, struct addrinfo *ai)
  {
-         int sock, gaierr;
+         int sock, r, gaierr;
          struct addrinfo hints, *res;
-         /*
-          * If we are running as root and want to connect to a privileged
-          * port, bind our own socket to a privileged port.
-          */
-         if (privileged) {
-                 int p = IPPORT_RESERVED - 1;
-                 PRIV_START;
-                 sock = rresvport_af(&p, ai->ai_family);
-                 PRIV_END;
-                 if (sock < 0)
-                         error("rresvport: af=%d %.100s", ai->ai_family,
-                             strerror(errno));
-                 else
-                         debug("Allocated local port %d.", p);
-                 return sock;
-         }
          sock = socket(ai->ai_family, ai->ai_socktype, ai->ai_protocol);
          if (sock < 0) {
-                 error("socket: %.100s", strerror(errno));
+                 error("socket: %s", strerror(errno));
                  return -1;
          }
          fcntl(sock, F_SETFD, FD_CLOEXEC);
          /* Bind the socket to an alternative local IP address */
-         if (options.bind_address == NULL)
+         if (options.bind_address == NULL && !privileged)
                  return sock;
          memset(&hints, 0, sizeof(hints));
-Line 314
+Line 298
 Line 314
 Line 298
                  close(sock);
                  return -1;
          }
-         if (bind(sock, res->ai_addr, res->ai_addrlen) < 0) {
+         /*
-                 error("bind: %s: %s", options.bind_address, strerror(errno));
+          * If we are running as root and want to connect to a privileged
-                 close(sock);
+          * port, bind our own socket to a privileged port.
-                 freeaddrinfo(res);
+          */
-                 return -1;
+         if (privileged) {
+                 PRIV_START;
+                 r = bindresvport_sa(sock, res->ai_addr);
+                 PRIV_END;
+                 if (r < 0) {
+                         error("bindresvport_sa: af=%d %s", ai->ai_family,
+                             strerror(errno));
+                         goto fail;
+                 }
+         } else {
+                 if (bind(sock, res->ai_addr, res->ai_addrlen) < 0) {
+                         error("bind: %s: %s", options.bind_address,
+                             strerror(errno));
+  fail:
+                         close(sock);
+                         freeaddrinfo(res);
+                         return -1;
+                 }
          }
          freeaddrinfo(res);
          return sock;