Attachment 1670 Details for Bug 1633 – fall back to select() on read/write interruptions

[patch] fall back to select() on read/write interruptions

ssh-agent-retry.diff (text/plain), 1.60 KB, created by Damien Miller on 2009-08-19 06:37:05 AEST

(hide)

Description:

Filename:

MIME Type:

Creator: Damien Miller

Created: 2009-08-19 06:37:05 AEST

Size: 1.60 KB

patch

obsolete

>Index: ssh-agent.c
>===================================================================
>RCS file: /var/cvs/openssh/ssh-agent.c,v
>retrieving revision 1.179
>diff -u -p -r1.179 ssh-agent.c
>--- ssh-agent.c	21 Jun 2009 07:50:15 -0000	1.179
>+++ ssh-agent.c	18 Aug 2009 20:32:52 -0000
>@@ -956,16 +956,13 @@ after_select(fd_set *readset, fd_set *wr
> 		case AUTH_CONNECTION:
> 			if (buffer_len(&sockets[i].output) > 0 &&
> 			    FD_ISSET(sockets[i].fd, writeset)) {
>-				do {
>-					len = write(sockets[i].fd,
>-					    buffer_ptr(&sockets[i].output),
>-					    buffer_len(&sockets[i].output));
>-					if (len == -1 && (errno == EAGAIN ||
>-					    errno == EINTR ||
>-					    errno == EWOULDBLOCK))
>-						continue;
>-					break;
>-				} while (1);
>+				len = write(sockets[i].fd,
>+				    buffer_ptr(&sockets[i].output),
>+				    buffer_len(&sockets[i].output));
>+				if (len == -1 && (errno == EAGAIN ||
>+				    errno == EINTR ||
>+				    errno == EWOULDBLOCK))
>+					continue;
> 				if (len <= 0) {
> 					close_socket(&sockets[i]);
> 					break;
>@@ -973,14 +970,11 @@ after_select(fd_set *readset, fd_set *wr
> 				buffer_consume(&sockets[i].output, len);
> 			}
> 			if (FD_ISSET(sockets[i].fd, readset)) {
>-				do {
>-					len = read(sockets[i].fd, buf, sizeof(buf));
>-					if (len == -1 && (errno == EAGAIN ||
>-					    errno == EINTR ||
>-					    errno == EWOULDBLOCK))
>-						continue;
>-					break;
>-				} while (1);
>+				len = read(sockets[i].fd, buf, sizeof(buf));
>+				if (len == -1 && (errno == EAGAIN ||
>+				    errno == EINTR ||
>+				    errno == EWOULDBLOCK))
>+					continue;
> 				if (len <= 0) {
> 					close_socket(&sockets[i]);
> 					break;

Actions: View | Diff

Attachments on bug 1633: 1670 | 1671