diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2003-01-18 10:12:37 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2003-01-18 10:12:37 +0000 |
| commit | 039c1b75276118560dae730b033261a1ae8df7fa (patch) | |
| tree | 60a177af1f3c1ab6f8f79a6f0c89952eaa72a7b6 | |
| parent | e76afb7be1dbbd9337901dee42569c495ffa32ba (diff) | |
| download | glibc-039c1b75276118560dae730b033261a1ae8df7fa.tar glibc-039c1b75276118560dae730b033261a1ae8df7fa.tar.gz glibc-039c1b75276118560dae730b033261a1ae8df7fa.tar.bz2 glibc-039c1b75276118560dae730b033261a1ae8df7fa.zip | |
Update.
2003-01-18 Ulrich Drepper <drepper@redhat.com>
* inet/rexec.c (rexec_af): Protect socket operations with
TEMP_FAILURE_RETRY. Use writev instead of multiple write.
* inet/rcmd.c (rcmd_af): Likewise.
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | inet/rexec.c | 17 |
2 files changed, 18 insertions, 5 deletions
@@ -1,3 +1,9 @@ +2003-01-18 Ulrich Drepper <drepper@redhat.com> + + * inet/rexec.c (rexec_af): Protect socket operations with + TEMP_FAILURE_RETRY. Use writev instead of multiple write. + * inet/rcmd.c (rcmd_af): Likewise. + 2003-01-17 Roland McGrath <roland@redhat.com> * math/libm-test.inc (sin_test, cos_test, sincos_test) [TEST_DOUBLE]: diff --git a/inet/rexec.c b/inet/rexec.c index c6b432a6f8..3c14836aa2 100644 --- a/inet/rexec.c +++ b/inet/rexec.c @@ -43,6 +43,7 @@ static char sccsid[] = "@(#)rexec.c 8.1 (Berkeley) 6/4/93"; #include <stdlib.h> #include <string.h> #include <unistd.h> +#include <sys/uio.h> int rexecoptions; libc_freeres_ptr (static char *ahostbuf); @@ -136,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"); @@ -146,10 +148,15 @@ 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. */ |