More upgrades to psftp: it now supports mv, chmod, reget and reput.
[u/mdw/putty] / psftp.c
diff --git a/psftp.c b/psftp.c
index f4c254a..edc4bed 100644 (file)
--- a/psftp.c
+++ b/psftp.c
@@ -8,6 +8,7 @@
 #include <stdlib.h>
 #include <stdarg.h>
 #include <assert.h>
+#include <limits.h>
 
 #define PUTTY_DO_GLOBALS
 #include "putty.h"
 #include "sftp.h"
 #include "int64.h"
 
+/*
+ * Since SFTP is a request-response oriented protocol, it requires
+ * no buffer management: when we send data, we stop and wait for an
+ * acknowledgement _anyway_, and so we can't possibly overfill our
+ * send buffer.
+ */
+
 /* ----------------------------------------------------------------------
  * String handling routines.
  */
 
-char *dupstr(char *s) {
+char *dupstr(char *s)
+{
     int len = strlen(s);
-    char *p = smalloc(len+1);
+    char *p = smalloc(len + 1);
     strcpy(p, s);
     return p;
 }
 
 /* Allocate the concatenation of N strings. Terminate arg list with NULL. */
-char *dupcat(char *s1, ...) {
+char *dupcat(char *s1, ...)
+{
     int len;
     char *p, *q, *sn;
     va_list ap;
@@ -43,7 +53,7 @@ char *dupcat(char *s1, ...) {
     }
     va_end(ap);
 
-    p = smalloc(len+1);
+    p = smalloc(len + 1);
     strcpy(p, s1);
     q = p + strlen(p);
 
@@ -75,14 +85,15 @@ char *pwd, *homedir;
  * canonification fails, at least fall back to returning a _valid_
  * pathname (though it may be ugly, eg /home/simon/../foobar).
  */
-char *canonify(char *name) {
+char *canonify(char *name)
+{
     char *fullname, *canonname;
 
     if (name[0] == '/') {
        fullname = dupstr(name);
     } else {
        char *slash;
-       if (pwd[strlen(pwd)-1] == '/')
+       if (pwd[strlen(pwd) - 1] == '/')
            slash = "";
        else
            slash = "/";
@@ -95,73 +106,73 @@ char *canonify(char *name) {
        sfree(fullname);
        return canonname;
     } else {
-        /*
-         * Attempt number 2. Some FXP_REALPATH implementations
-         * (glibc-based ones, in particular) require the _whole_
-         * path to point to something that exists, whereas others
-         * (BSD-based) only require all but the last component to
-         * exist. So if the first call failed, we should strip off
-         * everything from the last slash onwards and try again,
-         * then put the final component back on.
-         * 
-         * Special cases:
-         * 
-         *  - if the last component is "/." or "/..", then we don't
-         *    bother trying this because there's no way it can work.
-         * 
-         *  - if the thing actually ends with a "/", we remove it
-         *    before we start. Except if the string is "/" itself
-         *    (although I can't see why we'd have got here if so,
-         *    because surely "/" would have worked the first
-         *    time?), in which case we don't bother.
-         * 
-         *  - if there's no slash in the string at all, give up in
-         *    confusion (we expect at least one because of the way
-         *    we constructed the string).
-         */
-        
-        int i;
-        char *returnname;
+       /*
+        * Attempt number 2. Some FXP_REALPATH implementations
+        * (glibc-based ones, in particular) require the _whole_
+        * path to point to something that exists, whereas others
+        * (BSD-based) only require all but the last component to
+        * exist. So if the first call failed, we should strip off
+        * everything from the last slash onwards and try again,
+        * then put the final component back on.
+        * 
+        * Special cases:
+        * 
+        *  - if the last component is "/." or "/..", then we don't
+        *    bother trying this because there's no way it can work.
+        * 
+        *  - if the thing actually ends with a "/", we remove it
+        *    before we start. Except if the string is "/" itself
+        *    (although I can't see why we'd have got here if so,
+        *    because surely "/" would have worked the first
+        *    time?), in which case we don't bother.
+        * 
+        *  - if there's no slash in the string at all, give up in
+        *    confusion (we expect at least one because of the way
+        *    we constructed the string).
+        */
 
-        i = strlen(fullname);
-        if (i > 2 && fullname[i-1] == '/')
-            fullname[--i] = '\0';      /* strip trailing / unless at pos 0 */
-        while (i > 0 && fullname[--i] != '/');
+       int i;
+       char *returnname;
 
-        /*
-         * Give up on special cases.
-         */
-        if (fullname[i] != '/' ||      /* no slash at all */
-            !strcmp(fullname+i, "/.") ||   /* ends in /. */
-            !strcmp(fullname+i, "/..") ||   /* ends in /.. */
-            !strcmp(fullname, "/")) {
-            return fullname;
-        }
+       i = strlen(fullname);
+       if (i > 2 && fullname[i - 1] == '/')
+           fullname[--i] = '\0';      /* strip trailing / unless at pos 0 */
+       while (i > 0 && fullname[--i] != '/');
 
-        /*
-         * Now i points at the slash. Deal with the final special
-         * case i==0 (ie the whole path was "/nonexistentfile").
-         */
-        fullname[i] = '\0';            /* separate the string */
-        if (i == 0) {
-            canonname = fxp_realpath("/");
-        } else {
-            canonname = fxp_realpath(fullname);
-        }
+       /*
+        * Give up on special cases.
+        */
+       if (fullname[i] != '/' ||      /* no slash at all */
+           !strcmp(fullname + i, "/.") ||      /* ends in /. */
+           !strcmp(fullname + i, "/..") ||     /* ends in /.. */
+           !strcmp(fullname, "/")) {
+           return fullname;
+       }
 
-        if (!canonname)
-            return fullname;           /* even that failed; give up */
+       /*
+        * Now i points at the slash. Deal with the final special
+        * case i==0 (ie the whole path was "/nonexistentfile").
+        */
+       fullname[i] = '\0';            /* separate the string */
+       if (i == 0) {
+           canonname = fxp_realpath("/");
+       } else {
+           canonname = fxp_realpath(fullname);
+       }
 
-        /*
-         * We have a canonical name for all but the last path
-         * component. Concatenate the last component and return.
-         */
-        returnname = dupcat(canonname,
-                            canonname[strlen(canonname)-1] == '/' ? "" : "/",
-                            fullname+i+1, NULL);
-        sfree(fullname);
-        sfree(canonname);
-        return returnname;
+       if (!canonname)
+           return fullname;           /* even that failed; give up */
+
+       /*
+        * We have a canonical name for all but the last path
+        * component. Concatenate the last component and return.
+        */
+       returnname = dupcat(canonname,
+                           canonname[strlen(canonname) - 1] ==
+                           '/' ? "" : "/", fullname + i + 1, NULL);
+       sfree(fullname);
+       sfree(canonname);
+       return returnname;
     }
 }
 
@@ -171,19 +182,22 @@ char *canonify(char *name) {
 struct sftp_command {
     char **words;
     int nwords, wordssize;
-    int (*obey)(struct sftp_command *);/* returns <0 to quit */
+    int (*obey) (struct sftp_command *);       /* returns <0 to quit */
 };
 
-int sftp_cmd_null(struct sftp_command *cmd) {
+int sftp_cmd_null(struct sftp_command *cmd)
+{
     return 0;
 }
 
-int sftp_cmd_unknown(struct sftp_command *cmd) {
+int sftp_cmd_unknown(struct sftp_command *cmd)
+{
     printf("psftp: unknown command \"%s\"\n", cmd->words[0]);
     return 0;
 }
 
-int sftp_cmd_quit(struct sftp_command *cmd) {
+int sftp_cmd_quit(struct sftp_command *cmd)
+{
     return -1;
 }
 
@@ -191,12 +205,14 @@ int sftp_cmd_quit(struct sftp_command *cmd) {
  * List a directory. If no arguments are given, list pwd; otherwise
  * list the directory given in words[1].
  */
-static int sftp_ls_compare(const void *av, const void *bv) {
-    const struct fxp_name *a = (const struct fxp_name *)av;
-    const struct fxp_name *b = (const struct fxp_name *)bv;
+static int sftp_ls_compare(const void *av, const void *bv)
+{
+    const struct fxp_name *a = (const struct fxp_name *) av;
+    const struct fxp_name *b = (const struct fxp_name *) bv;
     return strcmp(a->filename, b->filename);
 }
-int sftp_cmd_ls(struct sftp_command *cmd) {
+int sftp_cmd_ls(struct sftp_command *cmd)
+{
     struct fxp_handle *dirh;
     struct fxp_names *names;
     struct fxp_name *ournames;
@@ -240,7 +256,8 @@ int sftp_cmd_ls(struct sftp_command *cmd) {
 
            if (nnames + names->nnames >= namesize) {
                namesize += names->nnames + 128;
-               ournames = srealloc(ournames, namesize * sizeof(*ournames));
+               ournames =
+                   srealloc(ournames, namesize * sizeof(*ournames));
            }
 
            for (i = 0; i < names->nnames; i++)
@@ -273,7 +290,8 @@ int sftp_cmd_ls(struct sftp_command *cmd) {
  * Change directories. We do this by canonifying the new name, then
  * trying to OPENDIR it. Only if that succeeds do we set the new pwd.
  */
-int sftp_cmd_cd(struct sftp_command *cmd) {
+int sftp_cmd_cd(struct sftp_command *cmd)
+{
     struct fxp_handle *dirh;
     char *dir;
 
@@ -304,9 +322,13 @@ int sftp_cmd_cd(struct sftp_command *cmd) {
 }
 
 /*
- * Get a file and save it at the local end.
+ * Get a file and save it at the local end. We have two very
+ * similar commands here: `get' and `reget', which differ in that
+ * `reget' checks for the existence of the destination file and
+ * starts from where a previous aborted transfer left off.
  */
-int sftp_cmd_get(struct sftp_command *cmd) {
+int sftp_general_get(struct sftp_command *cmd, int restart)
+{
     struct fxp_handle *fh;
     char *fname, *outfname;
     uint64 offset;
@@ -330,17 +352,31 @@ int sftp_cmd_get(struct sftp_command *cmd) {
        sfree(fname);
        return 0;
     }
-    fp = fopen(outfname, "wb");
+
+    if (restart) {
+       fp = fopen(outfname, "rb+");
+    } else {
+       fp = fopen(outfname, "wb");
+    }
+
     if (!fp) {
        printf("local: unable to open %s\n", outfname);
-        fxp_close(fh);
+       fxp_close(fh);
        sfree(fname);
        return 0;
     }
 
-    printf("remote:%s => local:%s\n", fname, outfname);
+    if (restart) {
+       long posn;
+       fseek(fp, 0L, SEEK_END);
+       posn = ftell(fp);
+       printf("reget: restarting at file position %ld\n", posn);
+       offset = uint64_make(0, posn);
+    } else {
+       offset = uint64_make(0, 0);
+    }
 
-    offset = uint64_make(0,0);
+    printf("remote:%s => local:%s\n", fname, outfname);
 
     /*
      * FIXME: we can use FXP_FSTAT here to get the file size, and
@@ -352,17 +388,16 @@ int sftp_cmd_get(struct sftp_command *cmd) {
        int wpos, wlen;
 
        len = fxp_read(fh, buffer, offset, sizeof(buffer));
-       if ((len == -1 && fxp_error_type() == SSH_FX_EOF) ||
-           len == 0)
+       if ((len == -1 && fxp_error_type() == SSH_FX_EOF) || len == 0)
            break;
        if (len == -1) {
            printf("error while reading: %s\n", fxp_error());
            break;
        }
-       
+
        wpos = 0;
        while (wpos < len) {
-           wlen = fwrite(buffer, 1, len-wpos, fp);
+           wlen = fwrite(buffer, 1, len - wpos, fp);
            if (wlen <= 0) {
                printf("error while writing local file\n");
                break;
@@ -380,11 +415,23 @@ int sftp_cmd_get(struct sftp_command *cmd) {
 
     return 0;
 }
+int sftp_cmd_get(struct sftp_command *cmd)
+{
+    return sftp_general_get(cmd, 0);
+}
+int sftp_cmd_reget(struct sftp_command *cmd)
+{
+    return sftp_general_get(cmd, 1);
+}
 
 /*
- * Send a file and store it at the remote end.
+ * Send a file and store it at the remote end. We have two very
+ * similar commands here: `put' and `reput', which differ in that
+ * `reput' checks for the existence of the destination file and
+ * starts from where a previous aborted transfer left off.
  */
-int sftp_cmd_put(struct sftp_command *cmd) {
+int sftp_general_put(struct sftp_command *cmd, int restart)
+{
     struct fxp_handle *fh;
     char *fname, *origoutfname, *outfname;
     uint64 offset;
@@ -406,20 +453,50 @@ int sftp_cmd_put(struct sftp_command *cmd) {
     fp = fopen(fname, "rb");
     if (!fp) {
        printf("local: unable to open %s\n", fname);
-        fxp_close(fh);
        sfree(outfname);
        return 0;
     }
-    fh = fxp_open(outfname, SSH_FXF_WRITE | SSH_FXF_CREAT | SSH_FXF_TRUNC);
+    if (restart) {
+       fh = fxp_open(outfname,
+                     SSH_FXF_WRITE);
+    } else {
+       fh = fxp_open(outfname,
+                     SSH_FXF_WRITE | SSH_FXF_CREAT | SSH_FXF_TRUNC);
+    }
     if (!fh) {
        printf("%s: %s\n", outfname, fxp_error());
        sfree(outfname);
        return 0;
     }
 
-    printf("local:%s => remote:%s\n", fname, outfname);
+    if (restart) {
+       char decbuf[30];
+       struct fxp_attrs attrs;
+       if (!fxp_fstat(fh, &attrs)) {
+           printf("read size of %s: %s\n", outfname, fxp_error());
+           sfree(outfname);
+           return 0;
+       }
+       if (!(attrs.flags & SSH_FILEXFER_ATTR_SIZE)) {
+           printf("read size of %s: size was not given\n", outfname);
+           sfree(outfname);
+           return 0;
+       }
+       offset = attrs.size;
+       uint64_decimal(offset, decbuf);
+       printf("reput: restarting at file position %s\n", decbuf);
+       if (uint64_compare(offset, uint64_make(0, LONG_MAX)) > 0) {
+           printf("reput: remote file is larger than we can deal with\n");
+           sfree(outfname);
+           return 0;
+       }
+       if (fseek(fp, offset.lo, SEEK_SET) != 0)
+           fseek(fp, 0, SEEK_END);    /* *shrug* */
+    } else {
+       offset = uint64_make(0, 0);
+    }
 
-    offset = uint64_make(0,0);
+    printf("local:%s => remote:%s\n", fname, outfname);
 
     /*
      * FIXME: we can use FXP_FSTAT here to get the file size, and
@@ -449,36 +526,358 @@ int sftp_cmd_put(struct sftp_command *cmd) {
 
     return 0;
 }
+int sftp_cmd_put(struct sftp_command *cmd)
+{
+    return sftp_general_put(cmd, 0);
+}
+int sftp_cmd_reput(struct sftp_command *cmd)
+{
+    return sftp_general_put(cmd, 1);
+}
+
+int sftp_cmd_mkdir(struct sftp_command *cmd)
+{
+    char *dir;
+    int result;
+
+
+    if (cmd->nwords < 2) {
+       printf("mkdir: expects a directory\n");
+       return 0;
+    }
+
+    dir = canonify(cmd->words[1]);
+    if (!dir) {
+       printf("%s: %s\n", dir, fxp_error());
+       return 0;
+    }
+
+    result = fxp_mkdir(dir);
+    if (!result) {
+       printf("mkdir %s: %s\n", dir, fxp_error());
+       sfree(dir);
+       return 0;
+    }
+
+    sfree(dir);
+    return 0;
+}
+
+int sftp_cmd_rmdir(struct sftp_command *cmd)
+{
+    char *dir;
+    int result;
+
+
+    if (cmd->nwords < 2) {
+       printf("rmdir: expects a directory\n");
+       return 0;
+    }
+
+    dir = canonify(cmd->words[1]);
+    if (!dir) {
+       printf("%s: %s\n", dir, fxp_error());
+       return 0;
+    }
+
+    result = fxp_rmdir(dir);
+    if (!result) {
+       printf("rmdir %s: %s\n", dir, fxp_error());
+       sfree(dir);
+       return 0;
+    }
+
+    sfree(dir);
+    return 0;
+}
+
+int sftp_cmd_rm(struct sftp_command *cmd)
+{
+    char *fname;
+    int result;
+
+    if (cmd->nwords < 2) {
+       printf("rm: expects a filename\n");
+       return 0;
+    }
+
+    fname = canonify(cmd->words[1]);
+    if (!fname) {
+       printf("%s: %s\n", fname, fxp_error());
+       return 0;
+    }
+
+    result = fxp_remove(fname);
+    if (!result) {
+       printf("rm %s: %s\n", fname, fxp_error());
+       sfree(fname);
+       return 0;
+    }
+
+    sfree(fname);
+    return 0;
+
+}
+
+int sftp_cmd_mv(struct sftp_command *cmd)
+{
+    char *srcfname, *dstfname;
+    int result;
+
+    if (cmd->nwords < 3) {
+       printf("mv: expects two filenames\n");
+       return 0;
+    }
+    srcfname = canonify(cmd->words[1]);
+    if (!srcfname) {
+       printf("%s: %s\n", srcfname, fxp_error());
+       return 0;
+    }
+
+    dstfname = canonify(cmd->words[2]);
+    if (!dstfname) {
+       printf("%s: %s\n", dstfname, fxp_error());
+       return 0;
+    }
+
+    result = fxp_rename(srcfname, dstfname);
+    if (!result) {
+       char const *error = fxp_error();
+       struct fxp_attrs attrs;
+
+       /*
+        * The move might have failed because dstfname pointed at a
+        * directory. We check this possibility now: if dstfname
+        * _is_ a directory, we re-attempt the move by appending
+        * the basename of srcfname to dstfname.
+        */
+       result = fxp_stat(dstfname, &attrs);
+       if (result &&
+           (attrs.flags & SSH_FILEXFER_ATTR_PERMISSIONS) &&
+           (attrs.permissions & 0040000)) {
+           char *p;
+           char *newname, *newcanon;
+           printf("(destination %s is a directory)\n", dstfname);
+           p = srcfname + strlen(srcfname);
+           while (p > srcfname && p[-1] != '/') p--;
+           newname = dupcat(dstfname, "/", p, NULL);
+           newcanon = canonify(newname);
+           sfree(newname);
+           if (newcanon) {
+               sfree(dstfname);
+               dstfname = newcanon;
+               result = fxp_rename(srcfname, dstfname);
+               error = result ? NULL : fxp_error();
+           }
+       }
+       if (error) {
+           printf("mv %s %s: %s\n", srcfname, dstfname, error);
+           sfree(srcfname);
+           sfree(dstfname);
+           return 0;
+       }
+    }
+    printf("%s -> %s\n", srcfname, dstfname);
+
+    sfree(srcfname);
+    sfree(dstfname);
+    return 0;
+}
+
+int sftp_cmd_chmod(struct sftp_command *cmd)
+{
+    char *fname, *mode;
+    int result;
+    struct fxp_attrs attrs;
+    unsigned attrs_clr, attrs_xor, oldperms, newperms;
+
+    if (cmd->nwords < 3) {
+       printf("chmod: expects a mode specifier and a filename\n");
+       return 0;
+    }
+
+    /*
+     * Attempt to parse the mode specifier in cmd->words[1]. We
+     * don't support the full horror of Unix chmod; instead we
+     * support a much simpler syntax in which the user can either
+     * specify an octal number, or a comma-separated sequence of
+     * [ugoa]*[-+=][rwxst]+. (The initial [ugoa] sequence may
+     * _only_ be omitted if the only attribute mentioned is t,
+     * since all others require a user/group/other specification.
+     * Additionally, the s attribute may not be specified for any
+     * [ugoa] specifications other than exactly u or exactly g.
+     */
+    attrs_clr = attrs_xor = 0;
+    mode = cmd->words[1];
+    if (mode[0] >= '0' && mode[0] <= '9') {
+       if (mode[strspn(mode, "01234567")]) {
+           printf("chmod: numeric file modes should"
+                  " contain digits 0-7 only\n");
+           return 0;
+       }
+       attrs_clr = 07777;
+       sscanf(mode, "%o", &attrs_xor);
+       attrs_xor &= attrs_clr;
+    } else {
+       while (*mode) {
+           char *modebegin = mode;
+           unsigned subset, perms;
+           int action;
+
+           subset = 0;
+           while (*mode && *mode != ',' &&
+                  *mode != '+' && *mode != '-' && *mode != '=') {
+               switch (*mode) {
+                 case 'u': subset |= 04700; break; /* setuid, user perms */
+                 case 'g': subset |= 02070; break; /* setgid, group perms */
+                 case 'o': subset |= 00007; break; /* just other perms */
+                 case 'a': subset |= 06777; break; /* all of the above */
+                 default:
+                   printf("chmod: file mode '%.*s' contains unrecognised"
+                          " user/group/other specifier '%c'\n",
+                          strcspn(modebegin, ","), modebegin, *mode);
+                   return 0;
+               }
+               mode++;
+           }
+           if (!*mode || *mode == ',') {
+               printf("chmod: file mode '%.*s' is incomplete\n",
+                      strcspn(modebegin, ","), modebegin);
+               return 0;
+           }
+           action = *mode++;
+           if (!*mode || *mode == ',') {
+               printf("chmod: file mode '%.*s' is incomplete\n",
+                      strcspn(modebegin, ","), modebegin);
+               return 0;
+           }
+           perms = 0;
+           while (*mode && *mode != ',') {
+               switch (*mode) {
+                 case 'r': perms |= 00444; break;
+                 case 'w': perms |= 00222; break;
+                 case 'x': perms |= 00111; break;
+                 case 't': perms |= 01000; subset |= 01000; break;
+                 case 's':
+                   if ((subset & 06777) != 04700 &&
+                       (subset & 06777) != 02070) {
+                       printf("chmod: file mode '%.*s': set[ug]id bit should"
+                              " be used with exactly one of u or g only\n",
+                              strcspn(modebegin, ","), modebegin);
+                       return 0;
+                   }
+                   perms |= 06000;
+                   break;
+                 default:
+                   printf("chmod: file mode '%.*s' contains unrecognised"
+                          " permission specifier '%c'\n",
+                          strcspn(modebegin, ","), modebegin, *mode);
+                   return 0;
+               }
+               mode++;
+           }
+           if (!(subset & 06777) && (perms &~ subset)) {
+               printf("chmod: file mode '%.*s' contains no user/group/other"
+                      " specifier and permissions other than 't' \n",
+                      strcspn(modebegin, ","), modebegin, *mode);
+               return 0;
+           }
+           perms &= subset;
+           switch (action) {
+             case '+':
+               attrs_clr |= perms;
+               attrs_xor |= perms;
+               break;
+             case '-':
+               attrs_clr |= perms;
+               attrs_xor &= ~perms;
+               break;
+             case '=':
+               attrs_clr |= subset;
+               attrs_xor |= perms;
+               break;
+           }
+           if (*mode) mode++;         /* eat comma */
+       }
+    }
+
+    fname = canonify(cmd->words[2]);
+    if (!fname) {
+       printf("%s: %s\n", fname, fxp_error());
+       return 0;
+    }
+
+    result = fxp_stat(fname, &attrs);
+    if (!result || !(attrs.flags & SSH_FILEXFER_ATTR_PERMISSIONS)) {
+       printf("get attrs for %s: %s\n", fname,
+              result ? "file permissions not provided" : fxp_error());
+       sfree(fname);
+       return 0;
+    }
+
+    attrs.flags = SSH_FILEXFER_ATTR_PERMISSIONS;   /* perms _only_ */
+    oldperms = attrs.permissions & 07777;
+    attrs.permissions &= ~attrs_clr;
+    attrs.permissions ^= attrs_xor;
+    newperms = attrs.permissions & 07777;
+
+    result = fxp_setstat(fname, attrs);
+
+    if (!result) {
+       printf("set attrs for %s: %s\n", fname, fxp_error());
+       sfree(fname);
+       return 0;
+    }
+
+    printf("%s: %04o -> %04o\n", fname, oldperms, newperms);
+
+    sfree(fname);
+    return 0;
+}
 
 static struct sftp_cmd_lookup {
     char *name;
-    int (*obey)(struct sftp_command *);
+    int (*obey) (struct sftp_command *);
 } sftp_lookup[] = {
     /*
      * List of sftp commands. This is binary-searched so it MUST be
      * in ASCII order.
      */
-    {"bye", sftp_cmd_quit},
-    {"cd", sftp_cmd_cd},
-    {"dir", sftp_cmd_ls},
-    {"exit", sftp_cmd_quit},
-    {"get", sftp_cmd_get},
-    {"ls", sftp_cmd_ls},
-    {"put", sftp_cmd_put},
-    {"quit", sftp_cmd_quit},
-};
+    {
+    "bye", sftp_cmd_quit}, {
+    "cd", sftp_cmd_cd}, {
+    "chmod", sftp_cmd_chmod}, {
+    "del", sftp_cmd_rm}, {
+    "delete", sftp_cmd_rm}, {
+    "dir", sftp_cmd_ls}, {
+    "exit", sftp_cmd_quit}, {
+    "get", sftp_cmd_get}, {
+    "ls", sftp_cmd_ls}, {
+    "mkdir", sftp_cmd_mkdir}, {
+    "mv", sftp_cmd_mv}, {
+    "put", sftp_cmd_put}, {
+    "quit", sftp_cmd_quit}, {
+    "reget", sftp_cmd_reget}, {
+    "ren", sftp_cmd_mv}, {
+    "rename", sftp_cmd_mv}, {
+    "reput", sftp_cmd_reput}, {
+    "rm", sftp_cmd_rm}, {
+    "rmdir", sftp_cmd_rmdir},};
 
 /* ----------------------------------------------------------------------
  * Command line reading and parsing.
  */
-struct sftp_command *sftp_getcmd(void) {
+struct sftp_command *sftp_getcmd(FILE *fp, int mode, int modeflags)
+{
     char *line;
     int linelen, linesize;
     struct sftp_command *cmd;
     char *p, *q, *r;
     int quoting;
 
-    printf("psftp> ");
+       if ((mode == 0) || (modeflags & 1)) {
+           printf("psftp> ");
+       }
     fflush(stdout);
 
     cmd = smalloc(sizeof(struct sftp_command));
@@ -494,16 +893,19 @@ struct sftp_command *sftp_getcmd(void) {
 
        linesize += 512;
        line = srealloc(line, linesize);
-       ret = fgets(line+linelen, linesize-linelen, stdin);
+       ret = fgets(line + linelen, linesize - linelen, fp);
+       if (modeflags & 1) {
+               printf("%s", ret);
+       }
 
        if (!ret || (linelen == 0 && line[0] == '\0')) {
            cmd->obey = sftp_cmd_quit;
            printf("quit\n");
            return cmd;                /* eof */
        }
-       len = linelen + strlen(line+linelen);
+       len = linelen + strlen(line + linelen);
        linelen += len;
-       if (line[linelen-1] == '\n') {
+       if (line[linelen - 1] == '\n') {
            linelen--;
            line[linelen] = '\0';
            break;
@@ -529,7 +931,8 @@ struct sftp_command *sftp_getcmd(void) {
     p = line;
     while (*p) {
        /* skip whitespace */
-       while (*p && (*p == ' ' || *p == '\t')) p++;
+       while (*p && (*p == ' ' || *p == '\t'))
+           p++;
        /* mark start of word */
        q = r = p;                     /* q sits at start, r writes word */
        quoting = 0;
@@ -537,17 +940,19 @@ struct sftp_command *sftp_getcmd(void) {
            if (!quoting && (*p == ' ' || *p == '\t'))
                break;                 /* reached end of word */
            else if (*p == '"' && p[1] == '"')
-               p+=2, *r++ = '"';      /* a literal quote */
+               p += 2, *r++ = '"';    /* a literal quote */
            else if (*p == '"')
                p++, quoting = !quoting;
            else
                *r++ = *p++;
        }
-       if (*p) p++;                           /* skip over the whitespace */
+       if (*p)
+           p++;                       /* skip over the whitespace */
        *r = '\0';
        if (cmd->nwords >= cmd->wordssize) {
            cmd->wordssize = cmd->nwords + 16;
-           cmd->words = srealloc(cmd->words, cmd->wordssize*sizeof(char *));
+           cmd->words =
+               srealloc(cmd->words, cmd->wordssize * sizeof(char *));
        }
        cmd->words[cmd->nwords++] = q;
     }
@@ -582,14 +987,17 @@ struct sftp_command *sftp_getcmd(void) {
     return cmd;
 }
 
-void do_sftp(void) {
+void do_sftp(int mode, int modeflags, char *batchfile)
+{
+    FILE *fp;
+
     /*
      * Do protocol initialisation. 
      */
     if (!fxp_init()) {
        fprintf(stderr,
-               "Fatal: unable to initialise SFTP: %s\n",
-               fxp_error());
+               "Fatal: unable to initialise SFTP: %s\n", fxp_error());
+       return;
     }
 
     /*
@@ -606,16 +1014,42 @@ void do_sftp(void) {
     }
     pwd = dupstr(homedir);
 
-    /* ------------------------------------------------------------------
-     * Now we're ready to do Real Stuff.
+    /*
+     * Batch mode?
      */
-    while (1) {
-       struct sftp_command *cmd;
-       cmd = sftp_getcmd();
-       if (!cmd)
-           break;
-       if (cmd->obey(cmd) < 0)
-           break;
+    if (mode == 0) {
+
+        /* ------------------------------------------------------------------
+         * Now we're ready to do Real Stuff.
+         */
+        while (1) {
+       struct sftp_command *cmd;
+       cmd = sftp_getcmd(stdin, 0, 0);
+       if (!cmd)
+           break;
+               if (cmd->obey(cmd) < 0)
+                   break;
+           }
+    } else {
+        fp = fopen(batchfile, "r");
+        if (!fp) {
+        printf("Fatal: unable to open %s\n", batchfile);
+        return;
+        }
+        while (1) {
+       struct sftp_command *cmd;
+       cmd = sftp_getcmd(fp, mode, modeflags);
+       if (!cmd)
+           break;
+               if (cmd->obey(cmd) < 0)
+                   break;
+               if (fxp_error() != NULL) {
+                       if (!(modeflags & 2))
+                               break;
+               }
+        }
+           fclose(fp);
+
     }
 }
 
@@ -626,38 +1060,43 @@ void do_sftp(void) {
 static int verbose = 0;
 
 void verify_ssh_host_key(char *host, int port, char *keytype,
-                         char *keystr, char *fingerprint) {
+                        char *keystr, char *fingerprint)
+{
     int ret;
+    HANDLE hin;
+    DWORD savemode, i;
 
     static const char absentmsg[] =
-        "The server's host key is not cached in the registry. You\n"
-        "have no guarantee that the server is the computer you\n"
-        "think it is.\n"
-        "The server's key fingerprint is:\n"
-        "%s\n"
-        "If you trust this host, enter \"y\" to add the key to\n"
-        "PuTTY's cache and carry on connecting.\n"
-        "If you do not trust this host, enter \"n\" to abandon the\n"
-        "connection.\n"
-        "Continue connecting? (y/n) ";
+       "The server's host key is not cached in the registry. You\n"
+       "have no guarantee that the server is the computer you\n"
+       "think it is.\n"
+       "The server's key fingerprint is:\n"
+       "%s\n"
+       "If you trust this host, enter \"y\" to add the key to\n"
+       "PuTTY's cache and carry on connecting.\n"
+       "If you want to carry on connecting just once, without\n"
+       "adding the key to the cache, enter \"n\".\n"
+       "If you do not trust this host, press Return to abandon the\n"
+       "connection.\n"
+       "Store key in cache? (y/n) ";
 
     static const char wrongmsg[] =
-        "WARNING - POTENTIAL SECURITY BREACH!\n"
-        "The server's host key does not match the one PuTTY has\n"
-        "cached in the registry. This means that either the\n"
-        "server administrator has changed the host key, or you\n"
-        "have actually connected to another computer pretending\n"
-        "to be the server.\n"
-        "The new key fingerprint is:\n"
-        "%s\n"
-        "If you were expecting this change and trust the new key,\n"
-        "enter Yes to update PuTTY's cache and continue connecting.\n"
-        "If you want to carry on connecting but without updating\n"
-        "the cache, enter No.\n"
-        "If you want to abandon the connection completely, press\n"
-        "Return to cancel. Pressing Return is the ONLY guaranteed\n"
-        "safe choice.\n"
-        "Update cached key? (y/n, Return cancels connection) ";
+       "WARNING - POTENTIAL SECURITY BREACH!\n"
+       "The server's host key does not match the one PuTTY has\n"
+       "cached in the registry. This means that either the\n"
+       "server administrator has changed the host key, or you\n"
+       "have actually connected to another computer pretending\n"
+       "to be the server.\n"
+       "The new key fingerprint is:\n"
+       "%s\n"
+       "If you were expecting this change and trust the new key,\n"
+       "enter \"y\" to update PuTTY's cache and continue connecting.\n"
+       "If you want to carry on connecting but without updating\n"
+       "the cache, enter \"n\".\n"
+       "If you want to abandon the connection completely, press\n"
+       "Return to cancel. Pressing Return is the ONLY guaranteed\n"
+       "safe choice.\n"
+       "Update cached key? (y/n, Return cancels connection) ";
 
     static const char abandoned[] = "Connection abandoned.\n";
 
@@ -668,28 +1107,71 @@ void verify_ssh_host_key(char *host, int port, char *keytype,
      */
     ret = verify_host_key(host, port, keytype, keystr);
 
-    if (ret == 0)                      /* success - key matched OK */
-        return;
-    if (ret == 2) {                    /* key was different */
-        fprintf(stderr, wrongmsg, fingerprint);
-        if (fgets(line, sizeof(line), stdin) &&
-            line[0] != '\0' && line[0] != '\n') {
-            if (line[0] == 'y' || line[0] == 'Y')
-                store_host_key(host, port, keytype, keystr);
-        } else {
-            fprintf(stderr, abandoned);
-            exit(0);
-        }
+    if (ret == 0)                     /* success - key matched OK */
+       return;
+
+    if (ret == 2) {                   /* key was different */
+       fprintf(stderr, wrongmsg, fingerprint);
+       fflush(stderr);
     }
-    if (ret == 1) {                    /* key was absent */
-        fprintf(stderr, absentmsg, fingerprint);
-        if (fgets(line, sizeof(line), stdin) &&
-            (line[0] == 'y' || line[0] == 'Y'))
-            store_host_key(host, port, keytype, keystr);
-        else {
-            fprintf(stderr, abandoned);
-            exit(0);
-        }
+    if (ret == 1) {                   /* key was absent */
+       fprintf(stderr, absentmsg, fingerprint);
+       fflush(stderr);
+    }
+
+    hin = GetStdHandle(STD_INPUT_HANDLE);
+    GetConsoleMode(hin, &savemode);
+    SetConsoleMode(hin, (savemode | ENABLE_ECHO_INPUT |
+                        ENABLE_PROCESSED_INPUT | ENABLE_LINE_INPUT));
+    ReadFile(hin, line, sizeof(line) - 1, &i, NULL);
+    SetConsoleMode(hin, savemode);
+
+    if (line[0] != '\0' && line[0] != '\r' && line[0] != '\n') {
+       if (line[0] == 'y' || line[0] == 'Y')
+           store_host_key(host, port, keytype, keystr);
+    } else {
+       fprintf(stderr, abandoned);
+       exit(0);
+    }
+}
+
+/*
+ * Ask whether the selected cipher is acceptable (since it was
+ * below the configured 'warn' threshold).
+ * cs: 0 = both ways, 1 = client->server, 2 = server->client
+ */
+void askcipher(char *ciphername, int cs)
+{
+    HANDLE hin;
+    DWORD savemode, i;
+
+    static const char msg[] =
+       "The first %scipher supported by the server is\n"
+       "%s, which is below the configured warning threshold.\n"
+       "Continue with connection? (y/n) ";
+    static const char abandoned[] = "Connection abandoned.\n";
+
+    char line[32];
+
+    fprintf(stderr, msg,
+           (cs == 0) ? "" :
+           (cs == 1) ? "client-to-server " :
+                       "server-to-client ",
+           ciphername);
+    fflush(stderr);
+
+    hin = GetStdHandle(STD_INPUT_HANDLE);
+    GetConsoleMode(hin, &savemode);
+    SetConsoleMode(hin, (savemode | ENABLE_ECHO_INPUT |
+                        ENABLE_PROCESSED_INPUT | ENABLE_LINE_INPUT));
+    ReadFile(hin, line, sizeof(line) - 1, &i, NULL);
+    SetConsoleMode(hin, savemode);
+
+    if (line[0] == 'y' || line[0] == 'Y') {
+       return;
+    } else {
+       fprintf(stderr, abandoned);
+       exit(0);
     }
 }
 
@@ -698,11 +1180,11 @@ void verify_ssh_host_key(char *host, int port, char *keytype,
  */
 void fatalbox(char *fmt, ...)
 {
-    char str[0x100]; /* Make the size big enough */
+    char str[0x100];                  /* Make the size big enough */
     va_list ap;
     va_start(ap, fmt);
     strcpy(str, "Fatal:");
-    vsprintf(str+strlen(str), fmt, ap);
+    vsprintf(str + strlen(str), fmt, ap);
     va_end(ap);
     strcat(str, "\n");
     fprintf(stderr, str);
@@ -711,11 +1193,11 @@ void fatalbox(char *fmt, ...)
 }
 void connection_fatal(char *fmt, ...)
 {
-    char str[0x100]; /* Make the size big enough */
+    char str[0x100];                  /* Make the size big enough */
     va_list ap;
     va_start(ap, fmt);
     strcpy(str, "Fatal:");
-    vsprintf(str+strlen(str), fmt, ap);
+    vsprintf(str + strlen(str), fmt, ap);
     va_end(ap);
     strcat(str, "\n");
     fprintf(stderr, str);
@@ -723,9 +1205,12 @@ void connection_fatal(char *fmt, ...)
     exit(1);
 }
 
-void logevent(char *string) { }
+void logevent(char *string)
+{
+}
 
-void ldisc_send(char *buf, int len) {
+void ldisc_send(char *buf, int len)
+{
     /*
      * This is only here because of the calls to ldisc_send(NULL,
      * 0) in ssh.c. Nothing in PSFTP actually needs to use the
@@ -739,7 +1224,8 @@ void ldisc_send(char *buf, int len) {
  * Be told what socket we're supposed to be using.
  */
 static SOCKET sftp_ssh_socket;
-char *do_select(SOCKET skt, int startup) {
+char *do_select(SOCKET skt, int startup)
+{
     if (startup)
        sftp_ssh_socket = skt;
     else
@@ -757,13 +1243,14 @@ extern int select_result(WPARAM, LPARAM);
  * do this until we have enough data.
  */
 
-static unsigned char *outptr;          /* where to put the data */
-static unsigned outlen;                /* how much data required */
+static unsigned char *outptr;         /* where to put the data */
+static unsigned outlen;                       /* how much data required */
 static unsigned char *pending = NULL;  /* any spare data */
-static unsigned pendlen=0, pendsize=0; /* length and phys. size of buffer */
-void from_backend(int is_stderr, char *data, int datalen) {
-    unsigned char *p = (unsigned char *)data;
-    unsigned len = (unsigned)datalen;
+static unsigned pendlen = 0, pendsize = 0;     /* length and phys. size of buffer */
+int from_backend(int is_stderr, char *data, int datalen)
+{
+    unsigned char *p = (unsigned char *) data;
+    unsigned len = (unsigned) datalen;
 
     /*
      * stderr data is just spouted to local stderr and otherwise
@@ -771,37 +1258,43 @@ void from_backend(int is_stderr, char *data, int datalen) {
      */
     if (is_stderr) {
        fwrite(data, 1, len, stderr);
-       return;
+       return 0;
     }
 
     /*
      * If this is before the real session begins, just return.
      */
     if (!outptr)
-        return;
+       return 0;
 
     if (outlen > 0) {
-        unsigned used = outlen;
-        if (used > len) used = len;
-        memcpy(outptr, p, used);
-        outptr += used; outlen -= used;
-        p += used; len -= used;
+       unsigned used = outlen;
+       if (used > len)
+           used = len;
+       memcpy(outptr, p, used);
+       outptr += used;
+       outlen -= used;
+       p += used;
+       len -= used;
     }
 
     if (len > 0) {
-        if (pendsize < pendlen + len) {
-            pendsize = pendlen + len + 4096;
-            pending = (pending ? srealloc(pending, pendsize) :
-                       smalloc(pendsize));
-            if (!pending)
-                fatalbox("Out of memory");
-        }
-        memcpy(pending+pendlen, p, len);
-        pendlen += len;
+       if (pendsize < pendlen + len) {
+           pendsize = pendlen + len + 4096;
+           pending = (pending ? srealloc(pending, pendsize) :
+                      smalloc(pendsize));
+           if (!pending)
+               fatalbox("Out of memory");
+       }
+       memcpy(pending + pendlen, p, len);
+       pendlen += len;
     }
+
+    return 0;
 }
-int sftp_recvdata(char *buf, int len) {
-    outptr = (unsigned char *)buf;
+int sftp_recvdata(char *buf, int len)
+{
+    outptr = (unsigned char *) buf;
     outlen = len;
 
     /*
@@ -809,73 +1302,75 @@ int sftp_recvdata(char *buf, int len) {
      * need.
      */
     if (pendlen > 0) {
-        unsigned pendused = pendlen;
-        if (pendused > outlen)
-            pendused = outlen;
+       unsigned pendused = pendlen;
+       if (pendused > outlen)
+           pendused = outlen;
        memcpy(outptr, pending, pendused);
-        memmove(pending, pending+pendused, pendlen-pendused);
+       memmove(pending, pending + pendused, pendlen - pendused);
        outptr += pendused;
        outlen -= pendused;
-        pendlen -= pendused;
-        if (pendlen == 0) {
-            pendsize = 0;
-            sfree(pending);
-            pending = NULL;
-        }
-        if (outlen == 0)
-            return 1;
+       pendlen -= pendused;
+       if (pendlen == 0) {
+           pendsize = 0;
+           sfree(pending);
+           pending = NULL;
+       }
+       if (outlen == 0)
+           return 1;
     }
 
     while (outlen > 0) {
-        fd_set readfds;
+       fd_set readfds;
 
-        FD_ZERO(&readfds);
-        FD_SET(sftp_ssh_socket, &readfds);
-        if (select(1, &readfds, NULL, NULL, NULL) < 0)
-            return 0;                  /* doom */
-        select_result((WPARAM)sftp_ssh_socket, (LPARAM)FD_READ);
+       FD_ZERO(&readfds);
+       FD_SET(sftp_ssh_socket, &readfds);
+       if (select(1, &readfds, NULL, NULL, NULL) < 0)
+           return 0;                  /* doom */
+       select_result((WPARAM) sftp_ssh_socket, (LPARAM) FD_READ);
     }
 
     return 1;
 }
-int sftp_senddata(char *buf, int len) {
-    back->send((unsigned char *)buf, len);
+int sftp_senddata(char *buf, int len)
+{
+    back->send((unsigned char *) buf, len);
     return 1;
 }
 
 /*
  * Loop through the ssh connection and authentication process.
  */
-static void ssh_sftp_init(void) {
+static void ssh_sftp_init(void)
+{
     if (sftp_ssh_socket == INVALID_SOCKET)
        return;
     while (!back->sendok()) {
-        fd_set readfds;
-        FD_ZERO(&readfds);
-        FD_SET(sftp_ssh_socket, &readfds);
-        if (select(1, &readfds, NULL, NULL, NULL) < 0)
-            return;                    /* doom */
-        select_result((WPARAM)sftp_ssh_socket, (LPARAM)FD_READ);
+       fd_set readfds;
+       FD_ZERO(&readfds);
+       FD_SET(sftp_ssh_socket, &readfds);
+       if (select(1, &readfds, NULL, NULL, NULL) < 0)
+           return;                    /* doom */
+       select_result((WPARAM) sftp_ssh_socket, (LPARAM) FD_READ);
     }
 }
 
 static char *password = NULL;
-static int get_password(const char *prompt, char *str, int maxlen)
+static int get_line(const char *prompt, char *str, int maxlen, int is_pw)
 {
     HANDLE hin, hout;
-    DWORD savemode, i;
+    DWORD savemode, newmode, i;
 
     if (password) {
-        static int tried_once = 0;
-
-        if (tried_once) {
-            return 0;
-        } else {
-            strncpy(str, password, maxlen);
-            str[maxlen-1] = '\0';
-            tried_once = 1;
-            return 1;
-        }
+       static int tried_once = 0;
+
+       if (tried_once) {
+           return 0;
+       } else {
+           strncpy(str, password, maxlen);
+           str[maxlen - 1] = '\0';
+           tried_once = 1;
+           return 1;
+       }
     }
 
     hin = GetStdHandle(STD_INPUT_HANDLE);
@@ -886,18 +1381,26 @@ static int get_password(const char *prompt, char *str, int maxlen)
     }
 
     GetConsoleMode(hin, &savemode);
-    SetConsoleMode(hin, (savemode & (~ENABLE_ECHO_INPUT)) |
-                  ENABLE_PROCESSED_INPUT | ENABLE_LINE_INPUT);
+    newmode = savemode | ENABLE_PROCESSED_INPUT | ENABLE_LINE_INPUT;
+    if (is_pw)
+       newmode &= ~ENABLE_ECHO_INPUT;
+    else
+       newmode |= ENABLE_ECHO_INPUT;
+    SetConsoleMode(hin, newmode);
 
     WriteFile(hout, prompt, strlen(prompt), &i, NULL);
-    ReadFile(hin, str, maxlen-1, &i, NULL);
+    ReadFile(hin, str, maxlen - 1, &i, NULL);
 
     SetConsoleMode(hin, savemode);
 
-    if ((int)i > maxlen) i = maxlen-1; else i = i - 2;
+    if ((int) i > maxlen)
+       i = maxlen - 1;
+    else
+       i = i - 2;
     str[i] = '\0';
 
-    WriteFile(hout, "\r\n", 2, &i, NULL);
+    if (is_pw)
+       WriteFile(hout, "\r\n", 2, &i, NULL);
 
     return 1;
 }
@@ -915,8 +1418,7 @@ static void init_winsock(void)
        fprintf(stderr, "Unable to initialise WinSock");
        exit(1);
     }
-    if (LOBYTE(wsadata.wVersion) != 1 ||
-       HIBYTE(wsadata.wVersion) != 1) {
+    if (LOBYTE(wsadata.wVersion) != 1 || HIBYTE(wsadata.wVersion) != 1) {
        fprintf(stderr, "WinSock version is incompatible with 1.1");
        exit(1);
     }
@@ -931,6 +1433,9 @@ static void usage(void)
     printf("%s\n", ver);
     printf("Usage: psftp [options] user@host\n");
     printf("Options:\n");
+    printf("  -b file   use specified batchfile\n");
+    printf("  -bc       output batchfile commands\n");
+    printf("  -be       don't stop batchfile processing if errors\n");
     printf("  -v        show verbose messages\n");
     printf("  -P port   connect to specified port\n");
     printf("  -pw passw login with specified password\n");
@@ -946,9 +1451,12 @@ int main(int argc, char *argv[])
     int portnumber = 0;
     char *user, *host, *userhost, *realhost;
     char *err;
+    int mode = 0;
+    int modeflags = 0;
+    char *batchfile = NULL;
 
     flags = FLAG_STDERR;
-    ssh_get_password = &get_password;
+    ssh_get_line = &get_line;
     init_winsock();
     sk_init();
 
@@ -965,12 +1473,19 @@ int main(int argc, char *argv[])
        } else if (strcmp(argv[i], "-h") == 0 ||
                   strcmp(argv[i], "-?") == 0) {
            usage();
-       } else if (strcmp(argv[i], "-l") == 0 && i+1 < argc) {
+       } else if (strcmp(argv[i], "-l") == 0 && i + 1 < argc) {
            user = argv[++i];
-       } else if (strcmp(argv[i], "-P") == 0 && i+1 < argc) {
+       } else if (strcmp(argv[i], "-P") == 0 && i + 1 < argc) {
            portnumber = atoi(argv[++i]);
-       } else if (strcmp(argv[i], "-pw") == 0 && i+1 < argc) {
+       } else if (strcmp(argv[i], "-pw") == 0 && i + 1 < argc) {
            password = argv[++i];
+    } else if (strcmp(argv[i], "-b") == 0 && i + 1 < argc) {
+           mode = 1;
+        batchfile = argv[++i];
+    } else if (strcmp(argv[i], "-bc") == 0 && i + 1 < argc) {
+           modeflags = modeflags | 1;
+    } else if (strcmp(argv[i], "-be") == 0 && i + 1 < argc) {
+           modeflags = modeflags | 2;
        } else if (strcmp(argv[i], "--") == 0) {
            i++;
            break;
@@ -993,7 +1508,8 @@ int main(int argc, char *argv[])
     } else {
        *host++ = '\0';
        if (user) {
-           printf("psftp: multiple usernames specified; using \"%s\"\n", user);
+           printf("psftp: multiple usernames specified; using \"%s\"\n",
+                  user);
        } else
            user = userhost;
     }
@@ -1002,16 +1518,16 @@ int main(int argc, char *argv[])
     do_defaults(host, &cfg);
     if (cfg.host[0] == '\0') {
        /* No settings for this host; use defaults */
-        do_defaults(NULL, &cfg);
-       strncpy(cfg.host, host, sizeof(cfg.host)-1);
-       cfg.host[sizeof(cfg.host)-1] = '\0';
+       do_defaults(NULL, &cfg);
+       strncpy(cfg.host, host, sizeof(cfg.host) - 1);
+       cfg.host[sizeof(cfg.host) - 1] = '\0';
        cfg.port = 22;
     }
 
     /* Set username */
     if (user != NULL && user[0] != '\0') {
-       strncpy(cfg.username, user, sizeof(cfg.username)-1);
-       cfg.username[sizeof(cfg.username)-1] = '\0';
+       strncpy(cfg.username, user, sizeof(cfg.username) - 1);
+       cfg.username[sizeof(cfg.username) - 1] = '\0';
     }
     if (!cfg.username[0]) {
        printf("login as: ");
@@ -1020,8 +1536,8 @@ int main(int argc, char *argv[])
            exit(1);
        } else {
            int len = strlen(cfg.username);
-           if (cfg.username[len-1] == '\n')
-               cfg.username[len-1] = '\0';
+           if (cfg.username[len - 1] == '\n')
+               cfg.username[len - 1] = '\0';
        }
     }
 
@@ -1050,7 +1566,7 @@ int main(int argc, char *argv[])
     if (verbose && realhost != NULL)
        printf("Connected to %s\n", realhost);
 
-    do_sftp();
+    do_sftp(mode, modeflags, batchfile);
 
     if (back != NULL && back->socket() != NULL) {
        char ch;