Sync with Linux Kernel 2.5.69 and *BSD.
[kopensolaris-gnu/glibc.git] / inet / rexec.c
index 5cab324..3c14836 100644 (file)
@@ -40,11 +40,13 @@ static char sccsid[] = "@(#)rexec.c 8.1 (Berkeley) 6/4/93";
 #include <stdio.h>
 #include <netdb.h>
 #include <errno.h>
+#include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
+#include <sys/uio.h>
 
 int    rexecoptions;
-char   ahostbuf[NI_MAXHOST];
+libc_freeres_ptr (static char *ahostbuf);
 
 int
 rexec_af(ahost, rport, name, pass, cmd, fd2p, af)
@@ -58,33 +60,35 @@ rexec_af(ahost, rport, name, pass, cmd, fd2p, af)
        struct addrinfo hints, *res0;
        const char *orig_name = name;
        const char *orig_pass = pass;
-       u_short port;
+       u_short port = 0;
        int s, timo = 1, s3;
        char c;
-       int herr;
-       int gai, ok;
+       int gai;
        char servbuff[NI_MAXSERV];
 
-       snprintf(servbuff, sizeof(servbuff), "%d", rport);
+       __snprintf(servbuff, sizeof(servbuff), "%d", ntohs(rport));
        servbuff[sizeof(servbuff) - 1] = '\0';
 
-       memset(&hints, 0, sizeof(hints));
+       memset(&hints, '\0', sizeof(hints));
        hints.ai_family = af;
        hints.ai_socktype = SOCK_STREAM;
        hints.ai_flags = AI_CANONNAME;
-       if (gai = getaddrinfo(*ahost, servbuff, &hints, &res0)){
+       gai = getaddrinfo(*ahost, servbuff, &hints, &res0);
+       if (gai){
                /* XXX: set errno? */
                return -1;
        }
 
        if (res0->ai_canonname){
-               strncpy(ahostbuf, res0->ai_canonname, sizeof(ahostbuf));
-               ahostbuf[sizeof(ahostbuf)-1] = '\0';
+               free (ahostbuf);
+               ahostbuf = strdup (res0->ai_canonname);
+               if (ahostbuf == NULL) {
+                       perror ("rexec: strdup");
+                       return (-1);
+               }
                *ahost = ahostbuf;
-       }
-       else{
+       } else
                *ahost = NULL;
-       }
        ruserpass(res0->ai_canonname, &name, &pass);
 retry:
        s = __socket(res0->ai_family, res0->ai_socktype, 0);
@@ -114,13 +118,16 @@ retry:
                        (void) __close(s);
                        return (-1);
                }
-               listen(s2, 1);
+               __listen(s2, 1);
                sa2len = sizeof (sa2);
-               if (getsockname(s2, (struct sockaddr *)&sa2, &sa2len) < 0 ||
-                   sa2len != __libc_sa_len(sa2.__ss_family)) {
+               if (__getsockname(s2, (struct sockaddr *)&sa2, &sa2len) < 0) {
                        perror("getsockname");
                        (void) __close(s2);
                        goto bad;
+               } else if (sa2len != SA_LEN((struct sockaddr *)&sa2)) {
+                       __set_errno(EINVAL);
+                       (void) __close(s2);
+                       goto bad;
                }
                port = 0;
                if (!getnameinfo((struct sockaddr *)&sa2, sa2len,
@@ -130,7 +137,8 @@ retry:
                (void) sprintf(num, "%u", port);
                (void) __write(s, num, strlen(num)+1);
                { int len = sizeof (from);
-                 s3 = accept(s2, (struct sockaddr *)&from, &len);
+                 s3 = TEMP_FAILURE_RETRY (accept(s2, (struct sockaddr *)&from,
+                                                 &len));
                  __close(s2);
                  if (s3 < 0) {
                        perror("accept");
@@ -140,17 +148,22 @@ retry:
                }
                *fd2p = s3;
        }
-       (void) __write(s, name, strlen(name) + 1);
-       /* should public key encypt the password here */
-       (void) __write(s, pass, strlen(pass) + 1);
-       (void) __write(s, cmd, strlen(cmd) + 1);
+
+       struct iovec iov[3] =
+         {
+           [0] = { .iov_base = (void *) name, .iov_len = strlen (name) + 1 },
+           /* should public key encypt the password here */
+           [1] = { .iov_base = (void *) pass, .iov_len = strlen (pass) + 1 },
+           [2] = { .iov_base = (void *) cmd, .iov_len = strlen (cmd) + 1 }
+         };
+       (void) TEMP_FAILURE_RETRY (__writev (s, iov, 3));
 
        /* We don't need the memory allocated for the name and the password
           in ruserpass anymore.  */
        if (name != orig_name)
-         free (name);
+         free ((char *) name);
        if (pass != orig_pass)
-         free (pass);
+         free ((char *) pass);
 
        if (__read(s, &c, 1) != 1) {
                perror(*ahost);
@@ -173,6 +186,7 @@ bad:
        freeaddrinfo(res0);
        return (-1);
 }
+libc_hidden_def (rexec_af)
 
 int
 rexec(ahost, rport, name, pass, cmd, fd2p)