Attachment 160853 Details for Bug 247802 – Proposed patch for sftp client

[patch] Proposed patch for sftp client

openssh-3.9p1-sftp-drain-acks.patch (text/plain), 1.98 KB, created by Tomas Mraz on 2007-08-07 20:33:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomas Mraz

Created: 2007-08-07 20:33:38 UTC

Size: 1.98 KB

patch

obsolete

>diff -up openssh-3.9p1/sftp-client.c.drain-acks openssh-3.9p1/sftp-client.c
>--- openssh-3.9p1/sftp-client.c.drain-acks	2007-08-07 12:22:45.000000000 +0200
>+++ openssh-3.9p1/sftp-client.c	2007-08-07 22:15:36.000000000 +0200
>@@ -978,7 +978,8 @@ int
> do_upload(struct sftp_conn *conn, char *local_path, char *remote_path,
>     int pflag)
> {
>-	int local_fd, status;
>+	int local_fd;
>+	int status = SSH2_FX_OK;
> 	u_int handle_len, id, type;
> 	u_int64_t offset;
> 	char *handle, *data;
>@@ -1060,7 +1061,7 @@ do_upload(struct sftp_conn *conn, char *
> 		 * Simulate an EOF on interrupt, allowing ACKs from the
> 		 * server to drain.
> 		 */
>-		if (interrupted)
>+		if (interrupted || status != SSH2_FX_OK)
> 			len = 0;
> 		else do
> 			len = read(local_fd, data, conn->transfer_buflen);
>@@ -1117,15 +1118,6 @@ do_upload(struct sftp_conn *conn, char *
> 				fatal("Can't find request for ID %u", r_id);
> 			TAILQ_REMOVE(&acks, ack, tq);
> 
>-			if (status != SSH2_FX_OK) {
>-				error("Couldn't write to remote file \"%s\": %s",
>-				    remote_path, fx2txt(status));
>-				do_close(conn, handle, handle_len);
>-				close(local_fd);
>-				xfree(data);
>-				xfree(ack);
>-				goto done;
>-			}
> 			debug3("In write loop, ack for %u %u bytes at %llu",
> 			   ack->id, ack->len, (unsigned long long)ack->offset);
> 			++ackid;
>@@ -1137,21 +1129,25 @@ do_upload(struct sftp_conn *conn, char *
> 		stop_progress_meter();
> 	xfree(data);
> 
>+	if (status != SSH2_FX_OK) {
>+		error("Couldn't write to remote file \"%s\": %s",
>+		    remote_path, fx2txt(status));
>+		status = -1;
>+	}
>+
> 	if (close(local_fd) == -1) {
> 		error("Couldn't close local file \"%s\": %s", local_path,
> 		    strerror(errno));
>-		do_close(conn, handle, handle_len);
> 		status = -1;
>-		goto done;
> 	}
> 
> 	/* Override umask and utimes if asked */
> 	if (pflag)
> 		do_fsetstat(conn, handle, handle_len, &a);
> 
>-	status = do_close(conn, handle, handle_len);
>+	if (do_close(conn, handle, handle_len) != SSH2_FX_OK)
>+		status = -1;
> 
>-done:
> 	xfree(handle);
> 	buffer_free(&msg);
> 	return(status);

Actions: View | Diff

Attachments on bug 247802: 160805 | 160853