- djm@cvs.openbsd.org 2014/01/16 07:31:09

     [sftp-client.c]
     needless and incorrect cast to size_t can break resumption of
     large download; patch from tobias@

2 files changed, 9 insertions, 2 deletions

diff --git a/ChangeLog b/ChangeLog
index e30cec1a4..f8f432e7a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+20140118
+ - (djm) OpenBSD CVS Sync
+   - djm@cvs.openbsd.org 2014/01/16 07:31:09
+     [sftp-client.c]
+     needless and incorrect cast to size_t can break resumption of
+     large download; patch from tobias@
+
 20140112
  - (djm) OpenBSD CVS Sync
    - djm@cvs.openbsd.org 2014/01/10 05:59:19
diff --git a/sftp-client.c b/sftp-client.c
index 1eb821086..cb4e0c4b1 100644
--- a/sftp-client.c
+++ b/sftp-client.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: sftp-client.c,v 1.111 2013/12/05 22:59:45 djm Exp $ */
+/* $OpenBSD: sftp-client.c,v 1.112 2014/01/16 07:31:09 djm Exp $ */
 /*
  * Copyright (c) 2001-2004 Damien Miller <djm@openbsd.org>
  *
@@ -1104,7 +1104,7 @@ do_download(struct sftp_conn *conn, char *remote_path, char *local_path,
                             local_path, strerror(errno));
                         goto fail;
                 }
-                if ((size_t)st.st_size > size) {
+                if (st.st_size > size) {
                         error("Unable to resume download of \"%s\": "
                             "local file is larger than remote", local_path);
  fail: