- 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@

diff --git a/ChangeLog b/ChangeLog
index e30cec1a42a027abbe43d6f79b98ded4690557b0..f8f432e7a7f2f9819c0ae27dd0edf5b61a3c4ac1 100644 (file)
--- 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 1eb821086f8f6169042e7f79030e12c1653dad33..cb4e0c4b1aa69368aba678e980e37f61645ed085 100644 (file)
--- 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: