* New upstream release (closes: #474301). Important changes not previously

  backported to 4.7p1:
  - 4.9/4.9p1 (http://www.openssh.com/txt/release-4.9):
    + Added chroot(2) support for sshd(8), controlled by a new option
      "ChrootDirectory" (closes: #139047, LP: #24777).
    + Linked sftp-server(8) into sshd(8). The internal sftp server is used
      when the command "internal-sftp" is specified in a Subsystem or
      ForceCommand declaration. When used with ChrootDirectory, the
      internal sftp server requires no special configuration of files
      inside the chroot environment.
    + Added a protocol extension method "posix-rename@openssh.com" for
      sftp-server(8) to perform POSIX atomic rename() operations; sftp(1)
      prefers this if available (closes: #308561).
    + Removed the fixed limit of 100 file handles in sftp-server(8).
    + ssh(8) will now skip generation of SSH protocol 1 ephemeral server
      keys when in inetd mode and protocol 2 connections are negotiated.
      This speeds up protocol 2 connections to inetd-mode servers that
      also allow Protocol 1.
    + Accept the PermitRootLogin directive in a sshd_config(5) Match
      block. Allows for, e.g. permitting root only from the local network.
    + Reworked sftp(1) argument splitting and escaping to be more
      internally consistent (i.e. between sftp commands) and more
      consistent with sh(1). Please note that this will change the
      interpretation of some quoted strings, especially those with
      embedded backslash escape sequences.
    + Support "Banner=none" in sshd_config(5) to disable sending of a
      pre-login banner (e.g. in a Match block).
    + ssh(1) ProxyCommands are now executed with $SHELL rather than
      /bin/sh.
    + ssh(1)'s ConnectTimeout option is now applied to both the TCP
      connection and the SSH banner exchange (previously it just covered
      the TCP connection). This allows callers of ssh(1) to better detect
      and deal with stuck servers that accept a TCP connection but don't
      progress the protocol, and also makes ConnectTimeout useful for
      connections via a ProxyCommand.
    + scp(1) incorrectly reported "stalled" on slow copies (closes:
      #140828).
    + scp(1) date underflow for timestamps before epoch.
    + ssh(1) used the obsolete SIG DNS RRtype for host keys in DNS,
      instead of the current standard RRSIG.
    + Correctly drain ACKs when a sftp(1) upload write fails midway,
      avoids a fatal() exit from what should be a recoverable condition.
    + Fixed ssh-keygen(1) selective host key hashing (i.e. "ssh-keygen -HF
      hostname") to not include any IP address in the data to be hashed.
    + Make ssh(1) skip listening on the IPv6 wildcard address when a
      binding address of 0.0.0.0 is used against an old SSH server that
      does not support the RFC4254 syntax for wildcard bind addresses.
    + Enable IPV6_V6ONLY socket option on sshd(8) listen socket, as is
      already done for X11/TCP forwarding sockets (closes: #439661).
    + Fix FD leak that could hang a ssh(1) connection multiplexing master.
    + Make ssh(1) -q option documentation consistent with reality.
    + Fixed sshd(8) PAM support not calling pam_session_close(), or
      failing to call it with root privileges (closes: #372680).
    + Fix activation of OpenSSL engine support when requested in configure
      (LP: #119295).
  - 5.1/5.1p1 (http://www.openssh.com/txt/release-5.1):
    + Introduce experimental SSH Fingerprint ASCII Visualisation to ssh(1)
      and ssh-keygen(1). Visual fingerprint display is controlled by a new
      ssh_config(5) option "VisualHostKey". The intent is to render SSH
      host keys in a visual form that is amenable to easy recall and
      rejection of changed host keys.
    + sshd_config(5) now supports CIDR address/masklen matching in "Match
      address" blocks, with a fallback to classic wildcard matching.
    + sshd(8) now supports CIDR matching in ~/.ssh/authorized_keys
      from="..." restrictions, also with a fallback to classic wildcard
      matching.
    + Added an extended test mode (-T) to sshd(8) to request that it write
      its effective configuration to stdout and exit. Extended test mode
      also supports the specification of connection parameters (username,
      source address and hostname) to test the application of
      sshd_config(5) Match rules.
    + ssh(1) now prints the number of bytes transferred and the overall
      connection throughput for SSH protocol 2 sessions when in verbose
      mode (previously these statistics were displayed for protocol 1
      connections only).
    + sftp-server(8) now supports extension methods statvfs@openssh.com
      and fstatvfs@openssh.com that implement statvfs(2)-like operations.
    + sftp(1) now has a "df" command to the sftp client that uses the
      statvfs@openssh.com to produce a df(1)-like display of filesystem
      space and inode utilisation (requires statvfs@openssh.com support on
      the server).
    + Added a MaxSessions option to sshd_config(5) to allow control of the
      number of multiplexed sessions supported over a single TCP
      connection. This allows increasing the number of allowed sessions
      above the previous default of 10, disabling connection multiplexing
      (MaxSessions=1) or disallowing login/shell/subsystem sessions
      entirely (MaxSessions=0).
    + Added a no-more-sessions@openssh.com global request extension that
      is sent from ssh(1) to sshd(8) when the client knows that it will
      never request another session (i.e. when session multiplexing is
      disabled). This allows a server to disallow further session requests
      and terminate the session in cases where the client has been
      hijacked.
    + ssh-keygen(1) now supports the use of the -l option in combination
      with -F to search for a host in ~/.ssh/known_hosts and display its
      fingerprint.
    + ssh-keyscan(1) now defaults to "rsa" (protocol 2) keys, instead of
      "rsa1".
    + Added an AllowAgentForwarding option to sshd_config(8) to control
      whether authentication agent forwarding is permitted. Note that this
      is a loose control, as a client may install their own unofficial
      forwarder.
    + ssh(1) and sshd(8): avoid unnecessary malloc/copy/free when
      receiving network data, resulting in a ~10% speedup.
    + ssh(1) and sshd(8) will now try additional addresses when connecting
      to a port forward destination whose DNS name resolves to more than
      one address. The previous behaviour was to try the only first
      address and give up if that failed.
    + ssh(1) and sshd(8) now support signalling that channels are
      half-closed for writing, through a channel protocol extension
      notification "eow@openssh.com". This allows propagation of closed
      file descriptors, so that commands such as "ssh -2 localhost od
      /bin/ls | true" do not send unnecessary data over the wire.
    + sshd(8): increased the default size of ssh protocol 1 ephemeral keys
      from 768 to 1024 bits.
    + When ssh(1) has been requested to fork after authentication ("ssh
      -f") with ExitOnForwardFailure enabled, delay the fork until after
      replies for any -R forwards have been seen. Allows for robust
      detection of -R forward failure when using -f.
    + "Match group" blocks in sshd_config(5) now support negation of
      groups. E.g. "Match group staff,!guests".
    + sftp(1) and sftp-server(8) now allow chmod-like operations to set
      set[ug]id/sticky bits.
    + The MaxAuthTries option is now permitted in sshd_config(5) match
      blocks.
    + Multiplexed ssh(1) sessions now support a subset of the ~ escapes
      that are available to a primary connection.
    + ssh(1) connection multiplexing will now fall back to creating a new
      connection in most error cases (closes: #352830).
    + Make ssh(1) deal more gracefully with channel requests that fail.
      Previously it would optimistically assume that requests would always
      succeed, which could cause hangs if they did not (e.g. when the
      server runs out of file descriptors).
    + ssh(1) now reports multiplexing errors via the multiplex slave's
      stderr where possible (subject to LogLevel in the mux master).
    + Prevent sshd(8) from erroneously applying public key restrictions
      leaned from ~/.ssh/authorized_keys to other authentication methods
      when public key authentication subsequently fails (LP: #161047).
    + Fixed an UMAC alignment problem that manifested on Itanium
      platforms.

1 files changed, 395 insertions, 181 deletions

diff --git a/sftp.c b/sftp.c
index f0d5dd557..e1aa49d0f 100644
--- a/sftp.c
+++ b/sftp.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: sftp.c,v 1.96 2007/01/03 04:09:15 stevesk Exp $ */
+/* $OpenBSD: sftp.c,v 1.103 2008/07/13 22:16:03 djm Exp $ */
 /*
  * Copyright (c) 2001-2004 Damien Miller <djm@openbsd.org>
  *
@@ -25,7 +25,11 @@
 #include <sys/param.h>
 #include <sys/socket.h>
 #include <sys/wait.h>
+#ifdef HAVE_SYS_STATVFS_H
+#include <sys/statvfs.h>
+#endif
 
+#include <ctype.h>
 #include <errno.h>
 
 #ifdef HAVE_PATHS_H
@@ -43,6 +47,14 @@ typedef void EditLine;
 #include <unistd.h>
 #include <stdarg.h>
 
+#ifdef HAVE_UTIL_H
+# include <util.h>
+#endif
+
+#ifdef HAVE_LIBUTIL_H
+# include <libutil.h>
+#endif
+
 #include "xmalloc.h"
 #include "log.h"
 #include "pathnames.h"
@@ -63,7 +75,7 @@ int batchmode = 0;
 size_t copy_buffer_len = 32768;
 
 /* Number of concurrent outstanding requests */
-size_t num_requests = 16;
+size_t num_requests = 64;
 
 /* PID of ssh transport process */
 static pid_t sshpid = -1;
@@ -103,6 +115,7 @@ extern char *__progname;
 #define I_CHGRP         2
 #define I_CHMOD         3
 #define I_CHOWN         4
+#define I_DF            24
 #define I_GET           5
 #define I_HELP          6
 #define I_LCHDIR        7
@@ -135,6 +148,7 @@ static const struct CMD cmds[] = {
         { "chgrp",      I_CHGRP },
         { "chmod",      I_CHMOD },
         { "chown",      I_CHOWN },
+        { "df",         I_DF },
         { "dir",        I_LS },
         { "exit",       I_QUIT },
         { "get",        I_GET },
@@ -199,6 +213,8 @@ help(void)
         printf("chgrp grp path                Change group of file 'path' to 'grp'\n");
         printf("chmod mode path               Change permissions of file 'path' to 'mode'\n");
         printf("chown own path                Change owner of file 'path' to 'own'\n");
+        printf("df [path]                     Display statistics for current directory or\n");
+        printf("                              filesystem containing 'path'\n");
         printf("help                          Display this help text\n");
         printf("get remote-path [local-path]  Download file\n");
         printf("lls [ls-options [path]]       Display local directory listing\n");
@@ -346,144 +362,105 @@ infer_path(const char *p, char **ifp)
 }
 
 static int
-parse_getput_flags(const char **cpp, int *pflag)
+parse_getput_flags(const char *cmd, char **argv, int argc, int *pflag)
 {
-        const char *cp = *cpp;
+        extern int opterr, optind, optopt, optreset;
+        int ch;
 
-        /* Check for flags */
-        if (cp[0] == '-' && cp[1] && strchr(WHITESPACE, cp[2])) {
-                switch (cp[1]) {
+        optind = optreset = 1;
+        opterr = 0;
+
+        *pflag = 0;
+        while ((ch = getopt(argc, argv, "Pp")) != -1) {
+                switch (ch) {
                 case 'p':
                 case 'P':
                         *pflag = 1;
                         break;
                 default:
-                        error("Invalid flag -%c", cp[1]);
-                        return(-1);
+                        error("%s: Invalid flag -%c", cmd, optopt);
+                        return -1;
                 }
-                cp += 2;
-                *cpp = cp + strspn(cp, WHITESPACE);
         }
 
-        return(0);
+        return optind;
 }
 
 static int
-parse_ls_flags(const char **cpp, int *lflag)
+parse_ls_flags(char **argv, int argc, int *lflag)
 {
-        const char *cp = *cpp;
+        extern int opterr, optind, optopt, optreset;
+        int ch;
 
-        /* Defaults */
-        *lflag = LS_NAME_SORT;
+        optind = optreset = 1;
+        opterr = 0;
 
-        /* Check for flags */
-        if (cp++[0] == '-') {
-                for (; strchr(WHITESPACE, *cp) == NULL; cp++) {
-                        switch (*cp) {
-                        case 'l':
-                                *lflag &= ~VIEW_FLAGS;
-                                *lflag |= LS_LONG_VIEW;
-                                break;
-                        case '1':
-                                *lflag &= ~VIEW_FLAGS;
-                                *lflag |= LS_SHORT_VIEW;
-                                break;
-                        case 'n':
-                                *lflag &= ~VIEW_FLAGS;
-                                *lflag |= LS_NUMERIC_VIEW|LS_LONG_VIEW;
-                                break;
-                        case 'S':
-                                *lflag &= ~SORT_FLAGS;
-                                *lflag |= LS_SIZE_SORT;
-                                break;
-                        case 't':
-                                *lflag &= ~SORT_FLAGS;
-                                *lflag |= LS_TIME_SORT;
-                                break;
-                        case 'r':
-                                *lflag |= LS_REVERSE_SORT;
-                                break;
-                        case 'f':
-                                *lflag &= ~SORT_FLAGS;
-                                break;
-                        case 'a':
-                                *lflag |= LS_SHOW_ALL;
-                                break;
-                        default:
-                                error("Invalid flag -%c", *cp);
-                                return(-1);
-                        }
+        *lflag = LS_NAME_SORT;
+        while ((ch = getopt(argc, argv, "1Saflnrt")) != -1) {
+                switch (ch) {
+                case '1':
+                        *lflag &= ~VIEW_FLAGS;
+                        *lflag |= LS_SHORT_VIEW;
+                        break;
+                case 'S':
+                        *lflag &= ~SORT_FLAGS;
+                        *lflag |= LS_SIZE_SORT;
+                        break;
+                case 'a':
+                        *lflag |= LS_SHOW_ALL;
+                        break;
+                case 'f':
+                        *lflag &= ~SORT_FLAGS;
+                        break;
+                case 'l':
+                        *lflag &= ~VIEW_FLAGS;
+                        *lflag |= LS_LONG_VIEW;
+                        break;
+                case 'n':
+                        *lflag &= ~VIEW_FLAGS;
+                        *lflag |= LS_NUMERIC_VIEW|LS_LONG_VIEW;
+                        break;
+                case 'r':
+                        *lflag |= LS_REVERSE_SORT;
+                        break;
+                case 't':
+                        *lflag &= ~SORT_FLAGS;
+                        *lflag |= LS_TIME_SORT;
+                        break;
+                default:
+                        error("ls: Invalid flag -%c", optopt);
+                        return -1;
                 }
-                *cpp = cp + strspn(cp, WHITESPACE);
         }
 
-        return(0);
+        return optind;
 }
 
 static int
-get_pathname(const char **cpp, char **path)
+parse_df_flags(const char *cmd, char **argv, int argc, int *hflag, int *iflag)
 {
-        const char *cp = *cpp, *end;
-        char quot;
-        u_int i, j;
-
-        cp += strspn(cp, WHITESPACE);
-        if (!*cp) {
-                *cpp = cp;
-                *path = NULL;
-                return (0);
-        }
-
-        *path = xmalloc(strlen(cp) + 1);
-
-        /* Check for quoted filenames */
-        if (*cp == '\"' || *cp == '\'') {
-                quot = *cp++;
+        extern int opterr, optind, optopt, optreset;
+        int ch;
 
-                /* Search for terminating quote, unescape some chars */
-                for (i = j = 0; i <= strlen(cp); i++) {
-                        if (cp[i] == quot) {    /* Found quote */
-                                i++;
-                                (*path)[j] = '\0';
-                                break;
-                        }
-                        if (cp[i] == '\0') {    /* End of string */
-                                error("Unterminated quote");
-                                goto fail;
-                        }
-                        if (cp[i] == '\\') {    /* Escaped characters */
-                                i++;
-                                if (cp[i] != '\'' && cp[i] != '\"' &&
-                                    cp[i] != '\\') {
-                                        error("Bad escaped character '\\%c'",
-                                            cp[i]);
-                                        goto fail;
-                                }
-                        }
-                        (*path)[j++] = cp[i];
-                }
+        optind = optreset = 1;
+        opterr = 0;
 
-                if (j == 0) {
-                        error("Empty quotes");
-                        goto fail;
+        *hflag = *iflag = 0;
+        while ((ch = getopt(argc, argv, "hi")) != -1) {
+                switch (ch) {
+                case 'h':
+                        *hflag = 1;
+                        break;
+                case 'i':
+                        *iflag = 1;
+                        break;
+                default:
+                        error("%s: Invalid flag -%c", cmd, optopt);
+                        return -1;
                 }
-                *cpp = cp + i + strspn(cp + i, WHITESPACE);
-        } else {
-                /* Read to end of filename */
-                end = strpbrk(cp, WHITESPACE);
-                if (end == NULL)
-                        end = strchr(cp, '\0');
-                *cpp = end + strspn(end, WHITESPACE);
-
-                memcpy(*path, cp, end - cp);
-                (*path)[end - cp] = '\0';
         }
-        return (0);
 
- fail:
-        xfree(*path);
-        *path = NULL;
-        return (-1);
+        return optind;
 }
 
 static int
@@ -499,17 +476,6 @@ is_dir(char *path)
 }
 
 static int
-is_reg(char *path)
-{
-        struct stat sb;
-
-        if (stat(path, &sb) == -1)
-                fatal("stat %s: %s", path, strerror(errno));
-
-        return(S_ISREG(sb.st_mode));
-}
-
-static int
 remote_is_dir(struct sftp_conn *conn, char *path)
 {
         Attrib *a;
@@ -597,6 +563,7 @@ process_put(struct sftp_conn *conn, char *src, char *dst, char *pwd, int pflag)
         glob_t g;
         int err = 0;
         int i;
+        struct stat sb;
 
         if (dst) {
                 tmp_dst = xstrdup(dst);
@@ -605,7 +572,7 @@ process_put(struct sftp_conn *conn, char *src, char *dst, char *pwd, int pflag)
 
         memset(&g, 0, sizeof(g));
         debug3("Looking up %s", src);
-        if (glob(src, 0, NULL, &g)) {
+        if (glob(src, GLOB_NOCHECK, NULL, &g)) {
                 error("File \"%s\" not found.", src);
                 err = -1;
                 goto out;
@@ -620,7 +587,13 @@ process_put(struct sftp_conn *conn, char *src, char *dst, char *pwd, int pflag)
         }
 
         for (i = 0; g.gl_pathv[i] && !interrupted; i++) {
-                if (!is_reg(g.gl_pathv[i])) {
+                if (stat(g.gl_pathv[i], &sb) == -1) {
+                        err = -1;
+                        error("stat %s: %s", g.gl_pathv[i], strerror(errno));
+                        continue;
+                }
+
+                if (!S_ISREG(sb.st_mode)) {
                         error("skipping non-regular file %s",
                             g.gl_pathv[i]);
                         continue;
@@ -867,14 +840,238 @@ do_globbed_ls(struct sftp_conn *conn, char *path, char *strip_path,
 }
 
 static int
-parse_args(const char **cpp, int *pflag, int *lflag, int *iflag,
+do_df(struct sftp_conn *conn, char *path, int hflag, int iflag)
+{
+        struct sftp_statvfs st;
+        char s_used[FMT_SCALED_STRSIZE];
+        char s_avail[FMT_SCALED_STRSIZE];
+        char s_root[FMT_SCALED_STRSIZE];
+        char s_total[FMT_SCALED_STRSIZE];
+
+        if (do_statvfs(conn, path, &st, 1) == -1)
+                return -1;
+        if (iflag) {
+                printf("     Inodes        Used       Avail      "
+                    "(root)    %%Capacity\n");
+                printf("%11llu %11llu %11llu %11llu         %3llu%%\n",
+                    (unsigned long long)st.f_files,
+                    (unsigned long long)(st.f_files - st.f_ffree),
+                    (unsigned long long)st.f_favail,
+                    (unsigned long long)st.f_ffree,
+                    (unsigned long long)(100 * (st.f_files - st.f_ffree) /
+                    st.f_files));
+        } else if (hflag) {
+                strlcpy(s_used, "error", sizeof(s_used));
+                strlcpy(s_avail, "error", sizeof(s_avail));
+                strlcpy(s_root, "error", sizeof(s_root));
+                strlcpy(s_total, "error", sizeof(s_total));
+                fmt_scaled((st.f_blocks - st.f_bfree) * st.f_frsize, s_used);
+                fmt_scaled(st.f_bavail * st.f_frsize, s_avail);
+                fmt_scaled(st.f_bfree * st.f_frsize, s_root);
+                fmt_scaled(st.f_blocks * st.f_frsize, s_total);
+                printf("    Size     Used    Avail   (root)    %%Capacity\n");
+                printf("%7sB %7sB %7sB %7sB         %3llu%%\n",
+                    s_total, s_used, s_avail, s_root,
+                    (unsigned long long)(100 * (st.f_blocks - st.f_bfree) /
+                    st.f_blocks));
+        } else {
+                printf("        Size         Used        Avail       "
+                    "(root)    %%Capacity\n");
+                printf("%12llu %12llu %12llu %12llu         %3llu%%\n",
+                    (unsigned long long)(st.f_frsize * st.f_blocks / 1024),
+                    (unsigned long long)(st.f_frsize *
+                    (st.f_blocks - st.f_bfree) / 1024),
+                    (unsigned long long)(st.f_frsize * st.f_bavail / 1024),
+                    (unsigned long long)(st.f_frsize * st.f_bfree / 1024),
+                    (unsigned long long)(100 * (st.f_blocks - st.f_bfree) /
+                    st.f_blocks));
+        }
+        return 0;
+}
+
+/*
+ * Undo escaping of glob sequences in place. Used to undo extra escaping
+ * applied in makeargv() when the string is destined for a function that
+ * does not glob it.
+ */
+static void
+undo_glob_escape(char *s)
+{
+        size_t i, j;
+
+        for (i = j = 0;;) {
+                if (s[i] == '\0') {
+                        s[j] = '\0';
+                        return;
+                }
+                if (s[i] != '\\') {
+                        s[j++] = s[i++];
+                        continue;
+                }
+                /* s[i] == '\\' */
+                ++i;
+                switch (s[i]) {
+                case '?':
+                case '[':
+                case '*':
+                case '\\':
+                        s[j++] = s[i++];
+                        break;
+                case '\0':
+                        s[j++] = '\\';
+                        s[j] = '\0';
+                        return;
+                default:
+                        s[j++] = '\\';
+                        s[j++] = s[i++];
+                        break;
+                }
+        }
+}
+
+/*
+ * Split a string into an argument vector using sh(1)-style quoting,
+ * comment and escaping rules, but with some tweaks to handle glob(3)
+ * wildcards.
+ * Returns NULL on error or a NULL-terminated array of arguments.
+ */
+#define MAXARGS         128
+#define MAXARGLEN       8192
+static char **
+makeargv(const char *arg, int *argcp)
+{
+        int argc, quot;
+        size_t i, j;
+        static char argvs[MAXARGLEN];
+        static char *argv[MAXARGS + 1];
+        enum { MA_START, MA_SQUOTE, MA_DQUOTE, MA_UNQUOTED } state, q;
+
+        *argcp = argc = 0;
+        if (strlen(arg) > sizeof(argvs) - 1) {
+ args_too_longs:
+                error("string too long");
+                return NULL;
+        }
+        state = MA_START;
+        i = j = 0;
+        for (;;) {
+                if (isspace(arg[i])) {
+                        if (state == MA_UNQUOTED) {
+                                /* Terminate current argument */
+                                argvs[j++] = '\0';
+                                argc++;
+                                state = MA_START;
+                        } else if (state != MA_START)
+                                argvs[j++] = arg[i];
+                } else if (arg[i] == '"' || arg[i] == '\'') {
+                        q = arg[i] == '"' ? MA_DQUOTE : MA_SQUOTE;
+                        if (state == MA_START) {
+                                argv[argc] = argvs + j;
+                                state = q;
+                        } else if (state == MA_UNQUOTED) 
+                                state = q;
+                        else if (state == q)
+                                state = MA_UNQUOTED;
+                        else
+                                argvs[j++] = arg[i];
+                } else if (arg[i] == '\\') {
+                        if (state == MA_SQUOTE || state == MA_DQUOTE) {
+                                quot = state == MA_SQUOTE ? '\'' : '"';
+                                /* Unescape quote we are in */
+                                /* XXX support \n and friends? */
+                                if (arg[i + 1] == quot) {
+                                        i++;
+                                        argvs[j++] = arg[i];
+                                } else if (arg[i + 1] == '?' ||
+                                    arg[i + 1] == '[' || arg[i + 1] == '*') {
+                                        /*
+                                         * Special case for sftp: append
+                                         * double-escaped glob sequence -
+                                         * glob will undo one level of
+                                         * escaping. NB. string can grow here.
+                                         */
+                                        if (j >= sizeof(argvs) - 5)
+                                                goto args_too_longs;
+                                        argvs[j++] = '\\';
+                                        argvs[j++] = arg[i++];
+                                        argvs[j++] = '\\';
+                                        argvs[j++] = arg[i];
+                                } else {
+                                        argvs[j++] = arg[i++];
+                                        argvs[j++] = arg[i];
+                                }
+                        } else {
+                                if (state == MA_START) {
+                                        argv[argc] = argvs + j;
+                                        state = MA_UNQUOTED;
+                                }
+                                if (arg[i + 1] == '?' || arg[i + 1] == '[' ||
+                                    arg[i + 1] == '*' || arg[i + 1] == '\\') {
+                                        /*
+                                         * Special case for sftp: append
+                                         * escaped glob sequence -
+                                         * glob will undo one level of
+                                         * escaping.
+                                         */
+                                        argvs[j++] = arg[i++];
+                                        argvs[j++] = arg[i];
+                                } else {
+                                        /* Unescape everything */
+                                        /* XXX support \n and friends? */
+                                        i++;
+                                        argvs[j++] = arg[i];
+                                }
+                        }
+                } else if (arg[i] == '#') {
+                        if (state == MA_SQUOTE || state == MA_DQUOTE)
+                                argvs[j++] = arg[i];
+                        else
+                                goto string_done;
+                } else if (arg[i] == '\0') {
+                        if (state == MA_SQUOTE || state == MA_DQUOTE) {
+                                error("Unterminated quoted argument");
+                                return NULL;
+                        }
+ string_done:
+                        if (state == MA_UNQUOTED) {
+                                argvs[j++] = '\0';
+                                argc++;
+                        }
+                        break;
+                } else {
+                        if (state == MA_START) {
+                                argv[argc] = argvs + j;
+                                state = MA_UNQUOTED;
+                        }
+                        if ((state == MA_SQUOTE || state == MA_DQUOTE) &&
+                            (arg[i] == '?' || arg[i] == '[' || arg[i] == '*')) {
+                                /*
+                                 * Special case for sftp: escape quoted
+                                 * glob(3) wildcards. NB. string can grow
+                                 * here.
+                                 */
+                                if (j >= sizeof(argvs) - 3)
+                                        goto args_too_longs;
+                                argvs[j++] = '\\';
+                                argvs[j++] = arg[i];
+                        } else
+                                argvs[j++] = arg[i];
+                }
+                i++;
+        }
+        *argcp = argc;
+        return argv;
+}
+
+static int
+parse_args(const char **cpp, int *pflag, int *lflag, int *iflag, int *hflag,
     unsigned long *n_arg, char **path1, char **path2)
 {
         const char *cmd, *cp = *cpp;
-        char *cp2;
+        char *cp2, **argv;
         int base = 0;
         long l;
-        int i, cmdnum;
+        int i, cmdnum, optidx, argc;
 
         /* Skip leading whitespace */
         cp = cp + strspn(cp, WHITESPACE);
@@ -890,17 +1087,13 @@ parse_args(const char **cpp, int *pflag, int *lflag, int *iflag,
                 cp++;
         }
 
+        if ((argv = makeargv(cp, &argc)) == NULL)
+                return -1;
+
         /* Figure out which command we have */
-        for (i = 0; cmds[i].c; i++) {
-                int cmdlen = strlen(cmds[i].c);
-
-                /* Check for command followed by whitespace */
-                if (!strncasecmp(cp, cmds[i].c, cmdlen) &&
-                    strchr(WHITESPACE, cp[cmdlen])) {
-                        cp += cmdlen;
-                        cp = cp + strspn(cp, WHITESPACE);
+        for (i = 0; cmds[i].c != NULL; i++) {
+                if (strcasecmp(cmds[i].c, argv[0]) == 0)
                         break;
-                }
         }
         cmdnum = cmds[i].n;
         cmd = cmds[i].c;
@@ -911,40 +1104,44 @@ parse_args(const char **cpp, int *pflag, int *lflag, int *iflag,
                 cmdnum = I_SHELL;
         } else if (cmdnum == -1) {
                 error("Invalid command.");
-                return (-1);
+                return -1;
         }
 
         /* Get arguments and parse flags */
-        *lflag = *pflag = *n_arg = 0;
+        *lflag = *pflag = *hflag = *n_arg = 0;
         *path1 = *path2 = NULL;
+        optidx = 1;
         switch (cmdnum) {
         case I_GET:
         case I_PUT:
-                if (parse_getput_flags(&cp, pflag))
-                        return(-1);
+                if ((optidx = parse_getput_flags(cmd, argv, argc, pflag)) == -1)
+                        return -1;
                 /* Get first pathname (mandatory) */
-                if (get_pathname(&cp, path1))
-                        return(-1);
-                if (*path1 == NULL) {
+                if (argc - optidx < 1) {
                         error("You must specify at least one path after a "
                             "%s command.", cmd);
-                        return(-1);
+                        return -1;
+                }
+                *path1 = xstrdup(argv[optidx]);
+                /* Get second pathname (optional) */
+                if (argc - optidx > 1) {
+                        *path2 = xstrdup(argv[optidx + 1]);
+                        /* Destination is not globbed */
+                        undo_glob_escape(*path2);
                 }
-                /* Try to get second pathname (optional) */
-                if (get_pathname(&cp, path2))
-                        return(-1);
                 break;
         case I_RENAME:
         case I_SYMLINK:
-                if (get_pathname(&cp, path1))
-                        return(-1);
-                if (get_pathname(&cp, path2))
-                        return(-1);
-                if (!*path1 || !*path2) {
+                if (argc - optidx < 2) {
                         error("You must specify two paths after a %s "
                             "command.", cmd);
-                        return(-1);
+                        return -1;
                 }
+                *path1 = xstrdup(argv[optidx]);
+                *path2 = xstrdup(argv[optidx + 1]);
+                /* Paths are not globbed */
+                undo_glob_escape(*path1);
+                undo_glob_escape(*path2);
                 break;
         case I_RM:
         case I_MKDIR:
@@ -953,59 +1150,69 @@ parse_args(const char **cpp, int *pflag, int *lflag, int *iflag,
         case I_LCHDIR:
         case I_LMKDIR:
                 /* Get pathname (mandatory) */
-                if (get_pathname(&cp, path1))
-                        return(-1);
-                if (*path1 == NULL) {
+                if (argc - optidx < 1) {
                         error("You must specify a path after a %s command.",
                             cmd);
-                        return(-1);
+                        return -1;
+                }
+                *path1 = xstrdup(argv[optidx]);
+                /* Only "rm" globs */
+                if (cmdnum != I_RM)
+                        undo_glob_escape(*path1);
+                break;
+        case I_DF:
+                if ((optidx = parse_df_flags(cmd, argv, argc, hflag,
+                    iflag)) == -1)
+                        return -1;
+                /* Default to current directory if no path specified */
+                if (argc - optidx < 1)
+                        *path1 = NULL;
+                else {
+                        *path1 = xstrdup(argv[optidx]);
+                        undo_glob_escape(*path1);
                 }
                 break;
         case I_LS:
-                if (parse_ls_flags(&cp, lflag))
+                if ((optidx = parse_ls_flags(argv, argc, lflag)) == -1)
                         return(-1);
                 /* Path is optional */
-                if (get_pathname(&cp, path1))
-                        return(-1);
+                if (argc - optidx > 0)
+                        *path1 = xstrdup(argv[optidx]);
                 break;
         case I_LLS:
+                /* Skip ls command and following whitespace */
+                cp = cp + strlen(cmd) + strspn(cp, WHITESPACE);
         case I_SHELL:
                 /* Uses the rest of the line */
                 break;
         case I_LUMASK:
-                base = 8;
         case I_CHMOD:
                 base = 8;
         case I_CHOWN:
         case I_CHGRP:
                 /* Get numeric arg (mandatory) */
+                if (argc - optidx < 1)
+                        goto need_num_arg;
                 errno = 0;
-                l = strtol(cp, &cp2, base);
-                if (cp2 == cp || ((l == LONG_MIN || l == LONG_MAX) &&
-                    errno == ERANGE) || l < 0) {
+                l = strtol(argv[optidx], &cp2, base);
+                if (cp2 == argv[optidx] || *cp2 != '\0' ||
+                    ((l == LONG_MIN || l == LONG_MAX) && errno == ERANGE) ||
+                    l < 0) {
+ need_num_arg:
                         error("You must supply a numeric argument "
                             "to the %s command.", cmd);
-                        return(-1);
+                        return -1;
                 }
-                cp = cp2;
                 *n_arg = l;
-                if (cmdnum == I_LUMASK && strchr(WHITESPACE, *cp))
+                if (cmdnum == I_LUMASK)
                         break;
-                if (cmdnum == I_LUMASK || !strchr(WHITESPACE, *cp)) {
-                        error("You must supply a numeric argument "
-                            "to the %s command.", cmd);
-                        return(-1);
-                }
-                cp += strspn(cp, WHITESPACE);
-
                 /* Get pathname (mandatory) */
-                if (get_pathname(&cp, path1))
-                        return(-1);
-                if (*path1 == NULL) {
+                if (argc - optidx < 2) {
                         error("You must specify a path after a %s command.",
                             cmd);
-                        return(-1);
+                        return -1;
                 }
+                *path1 = xstrdup(argv[optidx + 1]);
                 break;
         case I_QUIT:
         case I_PWD:
@@ -1027,7 +1234,7 @@ parse_dispatch_command(struct sftp_conn *conn, const char *cmd, char **pwd,
     int err_abort)
 {
         char *path1, *path2, *tmp;
-        int pflag, lflag, iflag, cmdnum, i;
+        int pflag, lflag, iflag, hflag, cmdnum, i;
         unsigned long n_arg;
         Attrib a, *aa;
         char path_buf[MAXPATHLEN];
@@ -1035,7 +1242,7 @@ parse_dispatch_command(struct sftp_conn *conn, const char *cmd, char **pwd,
         glob_t g;
 
         path1 = path2 = NULL;
-        cmdnum = parse_args(&cmd, &pflag, &lflag, &iflag, &n_arg,
+        cmdnum = parse_args(&cmd, &pflag, &lflag, &iflag, &hflag, &n_arg,
             &path1, &path2);
 
         if (iflag != 0)
@@ -1129,6 +1336,13 @@ parse_dispatch_command(struct sftp_conn *conn, const char *cmd, char **pwd,
                 path1 = make_absolute(path1, *pwd);
                 err = do_globbed_ls(conn, path1, tmp, lflag);
                 break;
+        case I_DF:
+                /* Default to current directory if no path specified */
+                if (path1 == NULL)
+                        path1 = xstrdup(*pwd);
+                path1 = make_absolute(path1, *pwd);
+                err = do_df(conn, path1, hflag, iflag);
+                break;
         case I_LCHDIR:
                 if (chdir(path1) == -1) {
                         error("Couldn't change local directory to "