1 files changed, 7 insertions, 5 deletions

diff --git a/sftp-client.c b/sftp-client.c
index 73e3c2f53..72f7fff7a 100644
--- a/sftp-client.c
+++ b/sftp-client.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: sftp-client.c,v 1.133 2019/01/24 16:52:17 dtucker Exp $ */
+/* $OpenBSD: sftp-client.c,v 1.135 2019/10/04 04:31:59 djm Exp $ */
 /*
  * Copyright (c) 2001-2004 Damien Miller <djm@openbsd.org>
  *
@@ -631,8 +631,7 @@ do_lsreaddir(struct sftp_conn *conn, const char *path, int print_flag,
                                     __func__, ssh_err(r));
                                 free(filename);
                                 free(longname);
-                                sshbuf_free(msg);
-                                return -1;
+                                goto out;
                         }
 
                         if (print_flag)
@@ -1203,7 +1202,7 @@ do_download(struct sftp_conn *conn, const char *remote_path,
         struct sshbuf *msg;
         u_char *handle;
         int local_fd = -1, write_error;
-        int read_error, write_errno, reordered = 0, r;
+        int read_error, write_errno, lmodified = 0, reordered = 0, r;
         u_int64_t offset = 0, size, highwater;
         u_int mode, id, buflen, num_req, max_req, status = SSH2_FX_OK;
         off_t progress_counter;
@@ -1373,6 +1372,7 @@ do_download(struct sftp_conn *conn, const char *remote_path,
                         if (len > req->len)
                                 fatal("Received more data than asked for "
                                     "%zu > %zu", len, req->len);
+                        lmodified = 1;
                         if ((lseek(local_fd, req->offset, SEEK_SET) == -1 ||
                             atomicio(vwrite, local_fd, data, len) != len) &&
                             !write_error) {
@@ -1476,7 +1476,9 @@ do_download(struct sftp_conn *conn, const char *remote_path,
                                 error("Can't set times on \"%s\": %s",
                                     local_path, strerror(errno));
                 }
-                if (fsync_flag) {
+                if (resume_flag && !lmodified)
+                        logit("File \"%s\" was not modified", local_path);
+                else if (fsync_flag) {
                         debug("syncing \"%s\"", local_path);
                         if (fsync(local_fd) == -1)
                                 error("Couldn't sync file \"%s\": %s",