Sebastian Kuschel reports that pfd_closing can be called for a socket
[u/mdw/putty] / proxy.c
diff --git a/proxy.c b/proxy.c
index b773f25..8a14623 100644 (file)
--- a/proxy.c
+++ b/proxy.c
@@ -5,13 +5,20 @@
  * code and the higher level backend.
  */
 
-#include <windows.h>
+#include <assert.h>
+#include <ctype.h>
+#include <string.h>
 
 #define DEFINE_PLUG_METHOD_MACROS
 #include "putty.h"
 #include "network.h"
 #include "proxy.h"
 
+#define do_proxy_dns(conf) \
+    (conf_get_int(conf, CONF_proxy_dns) == FORCE_ON || \
+        (conf_get_int(conf, CONF_proxy_dns) == AUTO && \
+             conf_get_int(conf, CONF_proxy_type) != PROXY_SOCKS4))
+
 /*
  * Call this when proxy negotiation is complete, so that this
  * socket can begin working normally.
@@ -20,58 +27,53 @@ void proxy_activate (Proxy_Socket p)
 {
     void *data;
     int len;
-
-    p->lock_close =
-       p->lock_write =
-       p->lock_write_oob =
-       p->lock_receive =
-       p->lock_flush =
-       p->lock_closing =
-       p->lock_sent =
-       p->lock_accepting =
-       p->lock_freeze = 1;
-
+    long output_before, output_after;
+    
     p->state = PROXY_STATE_ACTIVE;
 
-    /* let's try to keep extra receive events from coming through */
+    /* we want to ignore new receive events until we have sent
+     * all of our buffered receive data.
+     */
     sk_set_frozen(p->sub_socket, 1);
 
+    /* how many bytes of output have we buffered? */
+    output_before = bufchain_size(&p->pending_oob_output_data) +
+       bufchain_size(&p->pending_output_data);
+    /* and keep track of how many bytes do not get sent. */
+    output_after = 0;
+    
+    /* send buffered OOB writes */
     while (bufchain_size(&p->pending_oob_output_data) > 0) {
        bufchain_prefix(&p->pending_oob_output_data, &data, &len);
-       sk_write_oob(p->sub_socket, data, len);
+       output_after += sk_write_oob(p->sub_socket, data, len);
        bufchain_consume(&p->pending_oob_output_data, len);
     }
-    bufchain_clear(&p->pending_oob_output_data);
 
+    /* send buffered normal writes */
     while (bufchain_size(&p->pending_output_data) > 0) {
        bufchain_prefix(&p->pending_output_data, &data, &len);
-       sk_write(p->sub_socket, data, len);
+       output_after += sk_write(p->sub_socket, data, len);
        bufchain_consume(&p->pending_output_data, len);
     }
-    bufchain_clear(&p->pending_output_data);
 
-    p->lock_write_oob = 0;
-    p->lock_write = 0;
+    /* if we managed to send any data, let the higher levels know. */
+    if (output_after < output_before)
+       plug_sent(p->plug, output_after);
 
+    /* if we were asked to flush the output during
+     * the proxy negotiation process, do so now.
+     */
     if (p->pending_flush) sk_flush(p->sub_socket);
-    p->lock_flush = 0;
 
-    while (bufchain_size(&p->pending_input_data) > 0) {
-       bufchain_prefix(&p->pending_input_data, &data, &len);
-       plug_receive(p->plug, 0, data, len);
-       bufchain_consume(&p->pending_input_data, len);
-    }
-    bufchain_clear(&p->pending_input_data);
-    p->lock_receive = 0;
+    /* if we have a pending EOF to send, send it */
+    if (p->pending_eof) sk_write_eof(p->sub_socket);
 
-    /* now set the underlying socket to whatever freeze state they wanted */
-    sk_set_frozen(p->sub_socket, p->freeze);
-    p->lock_freeze = 0;
-
-    p->lock_sent = 0;
-    p->lock_accepting = 0;
-    p->lock_closing = 0;
-    p->lock_close = 0;
+    /* if the backend wanted the socket unfrozen, try to unfreeze.
+     * our set_frozen handler will flush buffered receive data before
+     * unfreezing the actual underlying socket.
+     */
+    if (!p->freeze)
+       sk_set_frozen((Socket)p, 0);
 }
 
 /* basic proxy socket functions */
@@ -89,16 +91,15 @@ static void sk_proxy_close (Socket s)
 {
     Proxy_Socket ps = (Proxy_Socket) s;
 
-    while (ps->lock_close) ;
     sk_close(ps->sub_socket);
+    sk_addr_free(ps->remote_addr);
     sfree(ps);
 }
 
-static int sk_proxy_write (Socket s, char *data, int len)
+static int sk_proxy_write (Socket s, const char *data, int len)
 {
     Proxy_Socket ps = (Proxy_Socket) s;
 
-    while (ps->lock_write) ;
     if (ps->state != PROXY_STATE_ACTIVE) {
        bufchain_add(&ps->pending_output_data, data, len);
        return bufchain_size(&ps->pending_output_data);
@@ -106,11 +107,10 @@ static int sk_proxy_write (Socket s, char *data, int len)
     return sk_write(ps->sub_socket, data, len);
 }
 
-static int sk_proxy_write_oob (Socket s, char *data, int len)
+static int sk_proxy_write_oob (Socket s, const char *data, int len)
 {
     Proxy_Socket ps = (Proxy_Socket) s;
 
-    while (ps->lock_write_oob) ;
     if (ps->state != PROXY_STATE_ACTIVE) {
        bufchain_clear(&ps->pending_output_data);
        bufchain_clear(&ps->pending_oob_output_data);
@@ -120,11 +120,21 @@ static int sk_proxy_write_oob (Socket s, char *data, int len)
     return sk_write_oob(ps->sub_socket, data, len);
 }
 
+static void sk_proxy_write_eof (Socket s)
+{
+    Proxy_Socket ps = (Proxy_Socket) s;
+
+    if (ps->state != PROXY_STATE_ACTIVE) {
+        ps->pending_eof = 1;
+       return;
+    }
+    sk_write_eof(ps->sub_socket);
+}
+
 static void sk_proxy_flush (Socket s)
 {
     Proxy_Socket ps = (Proxy_Socket) s;
 
-    while (ps->lock_flush) ;
     if (ps->state != PROXY_STATE_ACTIVE) {
        ps->pending_flush = 1;
        return;
@@ -148,15 +158,42 @@ static void sk_proxy_set_frozen (Socket s, int is_frozen)
 {
     Proxy_Socket ps = (Proxy_Socket) s;
 
-    while (ps->lock_freeze) ;
     if (ps->state != PROXY_STATE_ACTIVE) {
        ps->freeze = is_frozen;
        return;
     }
+    
+    /* handle any remaining buffered recv data first */
+    if (bufchain_size(&ps->pending_input_data) > 0) {
+       ps->freeze = is_frozen;
+
+       /* loop while we still have buffered data, and while we are
+        * unfrozen. the plug_receive call in the loop could result 
+        * in a call back into this function refreezing the socket, 
+        * so we have to check each time.
+        */
+        while (!ps->freeze && bufchain_size(&ps->pending_input_data) > 0) {
+           void *data;
+           char databuf[512];
+           int len;
+           bufchain_prefix(&ps->pending_input_data, &data, &len);
+           if (len > lenof(databuf))
+               len = lenof(databuf);
+           memcpy(databuf, data, len);
+           bufchain_consume(&ps->pending_input_data, len);
+           plug_receive(ps->plug, 0, databuf, len);
+       }
+
+       /* if we're still frozen, we'll have to wait for another
+        * call from the backend to finish unbuffering the data.
+        */
+       if (ps->freeze) return;
+    }
+    
     sk_set_frozen(ps->sub_socket, is_frozen);
 }
 
-static char * sk_proxy_socket_error (Socket s)
+static const char * sk_proxy_socket_error (Socket s)
 {
     Proxy_Socket ps = (Proxy_Socket) s;
     if (ps->error != NULL || ps->sub_socket == NULL) {
@@ -167,13 +204,21 @@ static char * sk_proxy_socket_error (Socket s)
 
 /* basic proxy plug functions */
 
-static int plug_proxy_closing (Plug p, char *error_msg,
+static void plug_proxy_log(Plug plug, int type, SockAddr addr, int port,
+                          const char *error_msg, int error_code)
+{
+    Proxy_Plug pp = (Proxy_Plug) plug;
+    Proxy_Socket ps = pp->proxy_socket;
+
+    plug_log(ps->plug, type, addr, port, error_msg, error_code);
+}
+
+static int plug_proxy_closing (Plug p, const char *error_msg,
                               int error_code, int calling_back)
 {
     Proxy_Plug pp = (Proxy_Plug) p;
     Proxy_Socket ps = pp->proxy_socket;
 
-    while (ps->lock_closing) ;
     if (ps->state != PROXY_STATE_ACTIVE) {
        ps->closing_error_msg = error_msg;
        ps->closing_error_code = error_code;
@@ -189,7 +234,6 @@ static int plug_proxy_receive (Plug p, int urgent, char *data, int len)
     Proxy_Plug pp = (Proxy_Plug) p;
     Proxy_Socket ps = pp->proxy_socket;
 
-    while (ps->lock_receive) ;
     if (ps->state != PROXY_STATE_ACTIVE) {
        /* we will lose the urgentness of this data, but since most,
         * if not all, of this data will be consumed by the negotiation
@@ -209,7 +253,6 @@ static void plug_proxy_sent (Plug p, int bufsize)
     Proxy_Plug pp = (Proxy_Plug) p;
     Proxy_Socket ps = pp->proxy_socket;
 
-    while (ps->lock_sent) ;
     if (ps->state != PROXY_STATE_ACTIVE) {
        ps->sent_bufsize = bufsize;
        ps->negotiate(ps, PROXY_CHANGE_SENT);
@@ -218,12 +261,11 @@ static void plug_proxy_sent (Plug p, int bufsize)
     plug_sent(ps->plug, bufsize);
 }
 
-static int plug_proxy_accepting (Plug p, void *sock)
+static int plug_proxy_accepting (Plug p, OSSocket sock)
 {
     Proxy_Plug pp = (Proxy_Plug) p;
     Proxy_Socket ps = pp->proxy_socket;
 
-    while (ps->lock_accepting) ;
     if (ps->state != PROXY_STATE_ACTIVE) {
        ps->accepting_sock = sock;
        return ps->negotiate(ps, PROXY_CHANGE_ACCEPTING);
@@ -231,20 +273,46 @@ static int plug_proxy_accepting (Plug p, void *sock)
     return plug_accepting(ps->plug, sock);
 }
 
-static int proxy_for_destination (SockAddr addr, char * hostname, int port)
+/*
+ * This function can accept a NULL pointer as `addr', in which case
+ * it will only check the host name.
+ */
+static int proxy_for_destination (SockAddr addr, const char *hostname,
+                                  int port, Conf *conf)
 {
     int s = 0, e = 0;
     char hostip[64];
     int hostip_len, hostname_len;
-    char * exclude_list;
+    const char *exclude_list;
+
+    /*
+     * Special local connections such as Unix-domain sockets
+     * unconditionally cannot be proxied, even in proxy-localhost
+     * mode. There just isn't any way to ask any known proxy type for
+     * them.
+     */
+    if (addr && sk_address_is_special_local(addr))
+        return 0;                      /* do not proxy */
+
+    /*
+     * Check the host name and IP against the hard-coded
+     * representations of `localhost'.
+     */
+    if (!conf_get_int(conf, CONF_even_proxy_localhost) &&
+       (sk_hostname_is_local(hostname) ||
+        (addr && sk_address_is_local(addr))))
+       return 0;                      /* do not proxy */
 
     /* we want a string representation of the IP address for comparisons */
-    sk_getaddr(addr, hostip, 64);
+    if (addr) {
+       sk_getaddr(addr, hostip, 64);
+       hostip_len = strlen(hostip);
+    } else
+       hostip_len = 0;                /* placate gcc; shouldn't be required */
 
-    hostip_len = strlen(hostip);
     hostname_len = strlen(hostname);
 
-    exclude_list = cfg.proxy_exclude_list;
+    exclude_list = conf_get_str(conf, CONF_proxy_exclude_list);
 
     /* now parse the exclude list, and see if either our IP
      * or hostname matches anything in it.
@@ -252,7 +320,7 @@ static int proxy_for_destination (SockAddr addr, char * hostname, int port)
 
     while (exclude_list[s]) {
        while (exclude_list[s] &&
-              (isspace(exclude_list[s]) ||
+              (isspace((unsigned char)exclude_list[s]) ||
                exclude_list[s] == ',')) s++;
 
        if (!exclude_list[s]) break;
@@ -260,7 +328,7 @@ static int proxy_for_destination (SockAddr addr, char * hostname, int port)
        e = s;
 
        while (exclude_list[e] &&
-              (isalnum(exclude_list[e]) ||
+              (isalnum((unsigned char)exclude_list[e]) ||
                exclude_list[e] == '-' ||
                exclude_list[e] == '.' ||
                exclude_list[e] == '*')) e++;
@@ -268,8 +336,8 @@ static int proxy_for_destination (SockAddr addr, char * hostname, int port)
        if (exclude_list[s] == '*') {
            /* wildcard at beginning of entry */
 
-           if (strnicmp(hostip + hostip_len - (e - s - 1),
-                        exclude_list + s + 1, e - s - 1) == 0 ||
+           if ((addr && strnicmp(hostip + hostip_len - (e - s - 1),
+                                 exclude_list + s + 1, e - s - 1) == 0) ||
                strnicmp(hostname + hostname_len - (e - s - 1),
                         exclude_list + s + 1, e - s - 1) == 0)
                return 0; /* IP/hostname range excluded. do not use proxy. */
@@ -277,7 +345,7 @@ static int proxy_for_destination (SockAddr addr, char * hostname, int port)
        } else if (exclude_list[e-1] == '*') {
            /* wildcard at end of entry */
 
-           if (strnicmp(hostip, exclude_list + s, e - s - 1) == 0 ||
+           if ((addr && strnicmp(hostip, exclude_list + s, e - s - 1) == 0) ||
                strnicmp(hostname, exclude_list + s, e - s - 1) == 0)
                return 0; /* IP/hostname range excluded. do not use proxy. */
 
@@ -286,28 +354,48 @@ static int proxy_for_destination (SockAddr addr, char * hostname, int port)
             * match (ie. a specific IP)
             */
 
-           if (stricmp(hostip, exclude_list + s) == 0)
+           if (addr && strnicmp(hostip, exclude_list + s, e - s) == 0)
                return 0; /* IP/hostname excluded. do not use proxy. */
-           if (stricmp(hostname, exclude_list + s) == 0)
+           if (strnicmp(hostname, exclude_list + s, e - s) == 0)
                return 0; /* IP/hostname excluded. do not use proxy. */
        }
 
        s = e;
+
+       /* Make sure we really have reached the next comma or end-of-string */
+       while (exclude_list[s] &&
+              !isspace((unsigned char)exclude_list[s]) &&
+              exclude_list[s] != ',') s++;
     }
 
     /* no matches in the exclude list, so use the proxy */
     return 1;
 }
 
+SockAddr name_lookup(char *host, int port, char **canonicalname,
+                    Conf *conf, int addressfamily)
+{
+    if (conf_get_int(conf, CONF_proxy_type) != PROXY_NONE &&
+       do_proxy_dns(conf) &&
+       proxy_for_destination(NULL, host, port, conf)) {
+       *canonicalname = dupstr(host);
+       return sk_nonamelookup(host);
+    }
+
+    return sk_namelookup(host, canonicalname, addressfamily);
+}
+
 Socket new_connection(SockAddr addr, char *hostname,
                      int port, int privport,
-                     int oobinline, int nodelay, Plug plug)
+                     int oobinline, int nodelay, int keepalive,
+                     Plug plug, Conf *conf)
 {
-    static struct socket_function_table socket_fn_table = {
+    static const struct socket_function_table socket_fn_table = {
        sk_proxy_plug,
        sk_proxy_close,
        sk_proxy_write,
        sk_proxy_write_oob,
+       sk_proxy_write_eof,
        sk_proxy_flush,
        sk_proxy_set_private_ptr,
        sk_proxy_get_private_ptr,
@@ -315,76 +403,92 @@ Socket new_connection(SockAddr addr, char *hostname,
        sk_proxy_socket_error
     };
 
-    static struct plug_function_table plug_fn_table = {
+    static const struct plug_function_table plug_fn_table = {
+       plug_proxy_log,
        plug_proxy_closing,
        plug_proxy_receive,
        plug_proxy_sent,
        plug_proxy_accepting
     };
 
-    if (cfg.proxy_type != PROXY_NONE &&
-       proxy_for_destination(addr, hostname, port))
+    if (conf_get_int(conf, CONF_proxy_type) != PROXY_NONE &&
+       proxy_for_destination(addr, hostname, port, conf))
     {
        Proxy_Socket ret;
        Proxy_Plug pplug;
        SockAddr proxy_addr;
-       char * proxy_canonical_name;
+       char *proxy_canonical_name;
+       Socket sret;
+       int type;
 
-       ret = smalloc(sizeof(struct Socket_proxy_tag));
+       if ((sret = platform_new_connection(addr, hostname, port, privport,
+                                           oobinline, nodelay, keepalive,
+                                           plug, conf)) !=
+           NULL)
+           return sret;
+
+       ret = snew(struct Socket_proxy_tag);
        ret->fn = &socket_fn_table;
+       ret->conf = conf_copy(conf);
        ret->plug = plug;
-       ret->remote_addr = addr;
+       ret->remote_addr = addr;       /* will need to be freed on close */
        ret->remote_port = port;
 
+       ret->error = NULL;
+       ret->pending_flush = 0;
+       ret->pending_eof = 0;
+       ret->freeze = 0;
+
        bufchain_init(&ret->pending_input_data);
        bufchain_init(&ret->pending_output_data);
        bufchain_init(&ret->pending_oob_output_data);
 
-       ret->lock_close =
-           ret->lock_write =
-           ret->lock_write_oob =
-           ret->lock_receive =
-           ret->lock_flush =
-           ret->lock_closing =
-           ret->lock_sent =
-           ret->lock_accepting = 0;
-
        ret->sub_socket = NULL;
        ret->state = PROXY_STATE_NEW;
+       ret->negotiate = NULL;
 
-       if (cfg.proxy_type == PROXY_HTTP) {
+       type = conf_get_int(conf, CONF_proxy_type);
+       if (type == PROXY_HTTP) {
            ret->negotiate = proxy_http_negotiate;
-       } else if (cfg.proxy_type == PROXY_SOCKS) {
-           ret->negotiate = proxy_socks_negotiate;
-       } else if (cfg.proxy_type == PROXY_TELNET) {
+       } else if (type == PROXY_SOCKS4) {
+            ret->negotiate = proxy_socks4_negotiate;
+       } else if (type == PROXY_SOCKS5) {
+            ret->negotiate = proxy_socks5_negotiate;
+       } else if (type == PROXY_TELNET) {
            ret->negotiate = proxy_telnet_negotiate;
        } else {
-           ret->error = "Network error: Unknown proxy method";
+           ret->error = "Proxy error: Unknown proxy method";
            return (Socket) ret;
        }
 
        /* create the proxy plug to map calls from the actual
         * socket into our proxy socket layer */
-       pplug = smalloc(sizeof(struct Plug_proxy_tag));
+       pplug = snew(struct Plug_proxy_tag);
        pplug->fn = &plug_fn_table;
        pplug->proxy_socket = ret;
 
        /* look-up proxy */
-       proxy_addr = sk_namelookup(cfg.proxy_host,
-                                  &proxy_canonical_name);
+       proxy_addr = sk_namelookup(conf_get_str(conf, CONF_proxy_host),
+                                  &proxy_canonical_name,
+                                  conf_get_int(conf, CONF_addressfamily));
+       if (sk_addr_error(proxy_addr) != NULL) {
+           ret->error = "Proxy error: Unable to resolve proxy host name";
+            sfree(pplug);
+            sk_addr_free(proxy_addr);
+           return (Socket)ret;
+       }
        sfree(proxy_canonical_name);
 
        /* create the actual socket we will be using,
         * connected to our proxy server and port.
         */
-       ret->sub_socket = sk_new(proxy_addr, cfg.proxy_port,
+       ret->sub_socket = sk_new(proxy_addr,
+                                conf_get_int(conf, CONF_proxy_port),
                                 privport, oobinline,
-                                nodelay, (Plug) pplug);
+                                nodelay, keepalive, (Plug) pplug);
        if (sk_socket_error(ret->sub_socket) != NULL)
            return (Socket) ret;
 
-       sk_addr_free(proxy_addr);
-
        /* start the proxy negotiation process... */
        sk_set_frozen(ret->sub_socket, 0);
        ret->negotiate(ret, PROXY_CHANGE_NEW);
@@ -393,16 +497,17 @@ Socket new_connection(SockAddr addr, char *hostname,
     }
 
     /* no proxy, so just return the direct socket */
-    return sk_new(addr, port, privport, oobinline, nodelay, plug);
+    return sk_new(addr, port, privport, oobinline, nodelay, keepalive, plug);
 }
 
-Socket new_listener(int port, Plug plug, int local_host_only)
+Socket new_listener(char *srcaddr, int port, Plug plug, int local_host_only,
+                   Conf *conf, int addressfamily)
 {
     /* TODO: SOCKS (and potentially others) support inbound
      * TODO: connections via the proxy. support them.
      */
 
-    return sk_newlistener(port, plug, local_host_only);
+    return sk_newlistener(srcaddr, port, plug, local_host_only, addressfamily);
 }
 
 /* ----------------------------------------------------------------------
@@ -451,16 +556,37 @@ int proxy_http_negotiate (Proxy_Socket p, int change)
         * for this proxy method, it's just a simple HTTP
         * request
         */
-       char buf[1024], dest[21];
+       char *buf, dest[512];
+       char *username, *password;
 
-       sk_getaddr(p->remote_addr, dest, 20);
+       sk_getaddr(p->remote_addr, dest, lenof(dest));
 
-       sprintf(buf, "CONNECT %s:%i HTTP/1.1\r\nHost: %s:%i\r\n\r\n",
-               dest, p->remote_port, dest, p->remote_port);
+       buf = dupprintf("CONNECT %s:%i HTTP/1.1\r\nHost: %s:%i\r\n",
+                       dest, p->remote_port, dest, p->remote_port);
        sk_write(p->sub_socket, buf, strlen(buf));
+       sfree(buf);
+
+       username = conf_get_str(p->conf, CONF_proxy_username);
+       password = conf_get_str(p->conf, CONF_proxy_password);
+       if (username[0] || password[0]) {
+           char *buf, *buf2;
+           int i, j, len;
+           buf = dupprintf("%s:%s", username, password);
+           len = strlen(buf);
+           buf2 = snewn(len * 4 / 3 + 100, char);
+           sprintf(buf2, "Proxy-Authorization: Basic ");
+           for (i = 0, j = strlen(buf2); i < len; i += 3, j += 4)
+               base64_encode_atom((unsigned char *)(buf+i),
+                                  (len-i > 3 ? 3 : len-i), buf2+j);
+           strcpy(buf2+j, "\r\n");
+           sk_write(p->sub_socket, buf2, strlen(buf2));
+           sfree(buf);
+           sfree(buf2);
+       }
 
-       p->state = 1;
+       sk_write(p->sub_socket, "\r\n", 2);
 
+       p->state = 1;
        return 0;
     }
 
@@ -499,7 +625,7 @@ int proxy_http_negotiate (Proxy_Socket p, int change)
         * we'll need to parse, process, and respond to appropriately.
         */
 
-       void *data;
+       char *data, *datap;
        int len;
        int eol;
 
@@ -508,27 +634,51 @@ int proxy_http_negotiate (Proxy_Socket p, int change)
            int min_ver, maj_ver, status;
 
            /* get the status line */
-           bufchain_prefix(&p->pending_input_data, &data, &len);
+           len = bufchain_size(&p->pending_input_data);
+           assert(len > 0);           /* or we wouldn't be here */
+           data = snewn(len+1, char);
+           bufchain_fetch(&p->pending_input_data, data, len);
+           /*
+            * We must NUL-terminate this data, because Windows
+            * sscanf appears to require a NUL at the end of the
+            * string because it strlens it _first_. Sigh.
+            */
+           data[len] = '\0';
+
            eol = get_line_end(data, len);
-           if (eol < 0) return 1;
+           if (eol < 0) {
+               sfree(data);
+               return 1;
+           }
 
-           sscanf((char *)data, "HTTP/%i.%i %i", &maj_ver, &min_ver, &status);
+           status = -1;
+           /* We can't rely on whether the %n incremented the sscanf return */
+           if (sscanf((char *)data, "HTTP/%i.%i %n",
+                      &maj_ver, &min_ver, &status) < 2 || status == -1) {
+               plug_closing(p->plug, "Proxy error: HTTP response was absent",
+                            PROXY_ERROR_GENERAL, 0);
+               sfree(data);
+               return 1;
+           }
 
            /* remove the status line from the input buffer. */
            bufchain_consume(&p->pending_input_data, eol);
-
-           /* TODO: we need to support Proxy-Auth headers */
-
-           if (status < 200 || status > 299) {
+           if (data[status] != '2') {
                /* error */
-               /* TODO: return a more specific error message,
-                * TODO: based on the status code.
-                */
-               plug_closing(p->plug, "Network error: Error while communicating with proxy",
-                           PROXY_ERROR_GENERAL, 0);
+               char *buf;
+               data[eol] = '\0';
+               while (eol > status &&
+                      (data[eol-1] == '\r' || data[eol-1] == '\n'))
+                   data[--eol] = '\0';
+               buf = dupprintf("Proxy error: %s", data+status);
+               plug_closing(p->plug, buf, PROXY_ERROR_GENERAL, 0);
+               sfree(buf);
+               sfree(data);
                return 1;
            }
 
+           sfree(data);
+
            p->state = 2;
        }
 
@@ -538,16 +688,23 @@ int proxy_http_negotiate (Proxy_Socket p, int change)
             * header of length 2, (ie. just "\r\n")
             */
 
-           bufchain_prefix(&p->pending_input_data, &data, &len);
-           eol = get_line_end(data, len);
+           len = bufchain_size(&p->pending_input_data);
+           assert(len > 0);           /* or we wouldn't be here */
+           data = snewn(len, char);
+           datap = data;
+           bufchain_fetch(&p->pending_input_data, data, len);
+
+           eol = get_line_end(datap, len);
+           if (eol < 0) {
+               sfree(data);
+               return 1;
+           }
            while (eol > 2)
            {
-               /* TODO: Proxy-Auth stuff. in some cases, we will
-                * TODO: need to extract information from headers.
-                */
                bufchain_consume(&p->pending_input_data, eol);
-               bufchain_prefix(&p->pending_input_data, &data, &len);
-               eol = get_line_end(data, len);
+               datap += eol;
+               len   -= eol;
+               eol = get_line_end(datap, len);
            }
 
            if (eol == 2) {
@@ -556,30 +713,522 @@ int proxy_http_negotiate (Proxy_Socket p, int change)
                proxy_activate(p);
                /* proxy activate will have dealt with
                 * whatever is left of the buffer */
+               sfree(data);
                return 1;
            }
 
+           sfree(data);
            return 1;
        }
     }
 
-    plug_closing(p->plug, "Network error: Unexpected proxy error",
+    plug_closing(p->plug, "Proxy error: unexpected proxy error",
                 PROXY_ERROR_UNEXPECTED, 0);
-    return 0;
+    return 1;
 }
 
 /* ----------------------------------------------------------------------
- * SOCKS proxy type (as yet unimplemented).
+ * SOCKS proxy type.
  */
 
-int proxy_socks_negotiate (Proxy_Socket p, int change)
+/* SOCKS version 4 */
+int proxy_socks4_negotiate (Proxy_Socket p, int change)
 {
-    p->error = "Network error: SOCKS proxy implementation is incomplete";
-    return 0;
+    if (p->state == PROXY_CHANGE_NEW) {
+
+       /* request format:
+        *  version number (1 byte) = 4
+        *  command code (1 byte)
+        *    1 = CONNECT
+        *    2 = BIND
+        *  dest. port (2 bytes) [network order]
+        *  dest. address (4 bytes)
+        *  user ID (variable length, null terminated string)
+        */
+
+       int length, type, namelen;
+       char *command, addr[4], hostname[512];
+       char *username;
+
+       type = sk_addrtype(p->remote_addr);
+       if (type == ADDRTYPE_IPV6) {
+            p->error = "Proxy error: SOCKS version 4 does not support IPv6";
+           return 1;
+       } else if (type == ADDRTYPE_IPV4) {
+           namelen = 0;
+           sk_addrcopy(p->remote_addr, addr);
+       } else {                       /* type == ADDRTYPE_NAME */
+           assert(type == ADDRTYPE_NAME);
+           sk_getaddr(p->remote_addr, hostname, lenof(hostname));
+           namelen = strlen(hostname) + 1;   /* include the NUL */
+           addr[0] = addr[1] = addr[2] = 0;
+           addr[3] = 1;
+       }
+
+       username = conf_get_str(p->conf, CONF_proxy_username);
+       length = strlen(username) + namelen + 9;
+       command = snewn(length, char);
+       strcpy(command + 8, username);
+
+       command[0] = 4; /* version 4 */
+       command[1] = 1; /* CONNECT command */
+
+       /* port */
+       command[2] = (char) (p->remote_port >> 8) & 0xff;
+       command[3] = (char) p->remote_port & 0xff;
+
+       /* address */
+       memcpy(command + 4, addr, 4);
+
+       /* hostname */
+       memcpy(command + 8 + strlen(username) + 1,
+              hostname, namelen);
+
+       sk_write(p->sub_socket, command, length);
+       sfree(username);
+       sfree(command);
+
+       p->state = 1;
+       return 0;
+    }
+
+    if (change == PROXY_CHANGE_CLOSING) {
+       /* if our proxy negotiation process involves closing and opening
+        * new sockets, then we would want to intercept this closing
+        * callback when we were expecting it. if we aren't anticipating
+        * a socket close, then some error must have occurred. we'll
+        * just pass those errors up to the backend.
+        */
+       return plug_closing(p->plug, p->closing_error_msg,
+                           p->closing_error_code,
+                           p->closing_calling_back);
+    }
+
+    if (change == PROXY_CHANGE_SENT) {
+       /* some (or all) of what we wrote to the proxy was sent.
+        * we don't do anything new, however, until we receive the
+        * proxy's response. we might want to set a timer so we can
+        * timeout the proxy negotiation after a while...
+        */
+       return 0;
+    }
+
+    if (change == PROXY_CHANGE_ACCEPTING) {
+       /* we should _never_ see this, as we are using our socket to
+        * connect to a proxy, not accepting inbound connections.
+        * what should we do? close the socket with an appropriate
+        * error message?
+        */
+       return plug_accepting(p->plug, p->accepting_sock);
+    }
+
+    if (change == PROXY_CHANGE_RECEIVE) {
+       /* we have received data from the underlying socket, which
+        * we'll need to parse, process, and respond to appropriately.
+        */
+
+       if (p->state == 1) {
+           /* response format:
+            *  version number (1 byte) = 4
+            *  reply code (1 byte)
+            *    90 = request granted
+            *    91 = request rejected or failed
+            *    92 = request rejected due to lack of IDENTD on client
+            *    93 = request rejected due to difference in user ID 
+            *         (what we sent vs. what IDENTD said)
+            *  dest. port (2 bytes)
+            *  dest. address (4 bytes)
+            */
+
+           char data[8];
+
+           if (bufchain_size(&p->pending_input_data) < 8)
+               return 1;              /* not got anything yet */
+           
+           /* get the response */
+           bufchain_fetch(&p->pending_input_data, data, 8);
+
+           if (data[0] != 0) {
+               plug_closing(p->plug, "Proxy error: SOCKS proxy responded with "
+                                     "unexpected reply code version",
+                            PROXY_ERROR_GENERAL, 0);
+               return 1;
+           }
+
+           if (data[1] != 90) {
+
+               switch (data[1]) {
+                 case 92:
+                   plug_closing(p->plug, "Proxy error: SOCKS server wanted IDENTD on client",
+                                PROXY_ERROR_GENERAL, 0);
+                   break;
+                 case 93:
+                   plug_closing(p->plug, "Proxy error: Username and IDENTD on client don't agree",
+                                PROXY_ERROR_GENERAL, 0);
+                   break;
+                 case 91:
+                 default:
+                   plug_closing(p->plug, "Proxy error: Error while communicating with proxy",
+                                PROXY_ERROR_GENERAL, 0);
+                   break;
+               }
+
+               return 1;
+           }
+           bufchain_consume(&p->pending_input_data, 8);
+
+           /* we're done */
+           proxy_activate(p);
+           /* proxy activate will have dealt with
+            * whatever is left of the buffer */
+           return 1;
+       }
+    }
+
+    plug_closing(p->plug, "Proxy error: unexpected proxy error",
+                PROXY_ERROR_UNEXPECTED, 0);
+    return 1;
+}
+
+/* SOCKS version 5 */
+int proxy_socks5_negotiate (Proxy_Socket p, int change)
+{
+    if (p->state == PROXY_CHANGE_NEW) {
+
+       /* initial command:
+        *  version number (1 byte) = 5
+        *  number of available authentication methods (1 byte)
+        *  available authentication methods (1 byte * previous value)
+        *    authentication methods:
+        *     0x00 = no authentication
+        *     0x01 = GSSAPI
+        *     0x02 = username/password
+        *     0x03 = CHAP
+        */
+
+       char command[5];
+       char *username, *password;
+       int len;
+
+       command[0] = 5; /* version 5 */
+       username = conf_get_str(p->conf, CONF_proxy_username);
+       password = conf_get_str(p->conf, CONF_proxy_password);
+       if (username[0] || password[0]) {
+           command[2] = 0x00;         /* no authentication */
+           len = 3;
+           proxy_socks5_offerencryptedauth (command, &len);
+           command[len++] = 0x02;             /* username/password */
+           command[1] = len - 2;       /* Number of methods supported */
+       } else {
+           command[1] = 1;            /* one methods supported: */
+           command[2] = 0x00;         /* no authentication */
+           len = 3;
+       }
+
+       sk_write(p->sub_socket, command, len);
+
+       p->state = 1;
+       return 0;
+    }
+
+    if (change == PROXY_CHANGE_CLOSING) {
+       /* if our proxy negotiation process involves closing and opening
+        * new sockets, then we would want to intercept this closing
+        * callback when we were expecting it. if we aren't anticipating
+        * a socket close, then some error must have occurred. we'll
+        * just pass those errors up to the backend.
+        */
+       return plug_closing(p->plug, p->closing_error_msg,
+                           p->closing_error_code,
+                           p->closing_calling_back);
+    }
+
+    if (change == PROXY_CHANGE_SENT) {
+       /* some (or all) of what we wrote to the proxy was sent.
+        * we don't do anything new, however, until we receive the
+        * proxy's response. we might want to set a timer so we can
+        * timeout the proxy negotiation after a while...
+        */
+       return 0;
+    }
+
+    if (change == PROXY_CHANGE_ACCEPTING) {
+       /* we should _never_ see this, as we are using our socket to
+        * connect to a proxy, not accepting inbound connections.
+        * what should we do? close the socket with an appropriate
+        * error message?
+        */
+       return plug_accepting(p->plug, p->accepting_sock);
+    }
+
+    if (change == PROXY_CHANGE_RECEIVE) {
+       /* we have received data from the underlying socket, which
+        * we'll need to parse, process, and respond to appropriately.
+        */
+
+       if (p->state == 1) {
+
+           /* initial response:
+            *  version number (1 byte) = 5
+            *  authentication method (1 byte)
+            *    authentication methods:
+            *     0x00 = no authentication
+            *     0x01 = GSSAPI
+            *     0x02 = username/password
+            *     0x03 = CHAP
+            *     0xff = no acceptable methods
+            */
+           char data[2];
+
+           if (bufchain_size(&p->pending_input_data) < 2)
+               return 1;              /* not got anything yet */
+
+           /* get the response */
+           bufchain_fetch(&p->pending_input_data, data, 2);
+
+           if (data[0] != 5) {
+               plug_closing(p->plug, "Proxy error: SOCKS proxy returned unexpected version",
+                            PROXY_ERROR_GENERAL, 0);
+               return 1;
+           }
+
+           if (data[1] == 0x00) p->state = 2; /* no authentication needed */
+           else if (data[1] == 0x01) p->state = 4; /* GSSAPI authentication */
+           else if (data[1] == 0x02) p->state = 5; /* username/password authentication */
+           else if (data[1] == 0x03) p->state = 6; /* CHAP authentication */
+           else {
+               plug_closing(p->plug, "Proxy error: SOCKS proxy did not accept our authentication",
+                            PROXY_ERROR_GENERAL, 0);
+               return 1;
+           }
+           bufchain_consume(&p->pending_input_data, 2);
+       }
+
+       if (p->state == 7) {
+
+           /* password authentication reply format:
+            *  version number (1 bytes) = 1
+            *  reply code (1 byte)
+            *    0 = succeeded
+            *    >0 = failed
+            */
+           char data[2];
+
+           if (bufchain_size(&p->pending_input_data) < 2)
+               return 1;              /* not got anything yet */
+
+           /* get the response */
+           bufchain_fetch(&p->pending_input_data, data, 2);
+
+           if (data[0] != 1) {
+               plug_closing(p->plug, "Proxy error: SOCKS password "
+                            "subnegotiation contained wrong version number",
+                            PROXY_ERROR_GENERAL, 0);
+               return 1;
+           }
+
+           if (data[1] != 0) {
+
+               plug_closing(p->plug, "Proxy error: SOCKS proxy refused"
+                            " password authentication",
+                            PROXY_ERROR_GENERAL, 0);
+               return 1;
+           }
+
+           bufchain_consume(&p->pending_input_data, 2);
+           p->state = 2;              /* now proceed as authenticated */
+       }
+
+       if (p->state == 8) {
+           int ret;
+           ret = proxy_socks5_handlechap(p);
+           if (ret) return ret;
+       }
+
+       if (p->state == 2) {
+
+           /* request format:
+            *  version number (1 byte) = 5
+            *  command code (1 byte)
+            *    1 = CONNECT
+            *    2 = BIND
+            *    3 = UDP ASSOCIATE
+            *  reserved (1 byte) = 0x00
+            *  address type (1 byte)
+            *    1 = IPv4
+            *    3 = domainname (first byte has length, no terminating null)
+            *    4 = IPv6
+            *  dest. address (variable)
+            *  dest. port (2 bytes) [network order]
+            */
+
+           char command[512];
+           int len;
+           int type;
+
+           type = sk_addrtype(p->remote_addr);
+           if (type == ADDRTYPE_IPV4) {
+               len = 10;              /* 4 hdr + 4 addr + 2 trailer */
+               command[3] = 1; /* IPv4 */
+               sk_addrcopy(p->remote_addr, command+4);
+           } else if (type == ADDRTYPE_IPV6) {
+               len = 22;              /* 4 hdr + 16 addr + 2 trailer */
+               command[3] = 4; /* IPv6 */
+               sk_addrcopy(p->remote_addr, command+4);
+           } else {
+               assert(type == ADDRTYPE_NAME);
+               command[3] = 3;
+               sk_getaddr(p->remote_addr, command+5, 256);
+               command[4] = strlen(command+5);
+               len = 7 + command[4];  /* 4 hdr, 1 len, N addr, 2 trailer */
+           }
+
+           command[0] = 5; /* version 5 */
+           command[1] = 1; /* CONNECT command */
+           command[2] = 0x00;
+
+           /* port */
+           command[len-2] = (char) (p->remote_port >> 8) & 0xff;
+           command[len-1] = (char) p->remote_port & 0xff;
+
+           sk_write(p->sub_socket, command, len);
+
+           p->state = 3;
+           return 1;
+       }
+
+       if (p->state == 3) {
+
+           /* reply format:
+            *  version number (1 bytes) = 5
+            *  reply code (1 byte)
+            *    0 = succeeded
+            *    1 = general SOCKS server failure
+            *    2 = connection not allowed by ruleset
+            *    3 = network unreachable
+            *    4 = host unreachable
+            *    5 = connection refused
+            *    6 = TTL expired
+            *    7 = command not supported
+            *    8 = address type not supported
+            * reserved (1 byte) = x00
+            * address type (1 byte)
+            *    1 = IPv4
+            *    3 = domainname (first byte has length, no terminating null)
+            *    4 = IPv6
+            * server bound address (variable)
+            * server bound port (2 bytes) [network order]
+            */
+           char data[5];
+           int len;
+
+           /* First 5 bytes of packet are enough to tell its length. */ 
+           if (bufchain_size(&p->pending_input_data) < 5)
+               return 1;              /* not got anything yet */
+
+           /* get the response */
+           bufchain_fetch(&p->pending_input_data, data, 5);
+
+           if (data[0] != 5) {
+               plug_closing(p->plug, "Proxy error: SOCKS proxy returned wrong version number",
+                            PROXY_ERROR_GENERAL, 0);
+               return 1;
+           }
+
+           if (data[1] != 0) {
+               char buf[256];
+
+               strcpy(buf, "Proxy error: ");
+
+               switch (data[1]) {
+                 case 1: strcat(buf, "General SOCKS server failure"); break;
+                 case 2: strcat(buf, "Connection not allowed by ruleset"); break;
+                 case 3: strcat(buf, "Network unreachable"); break;
+                 case 4: strcat(buf, "Host unreachable"); break;
+                 case 5: strcat(buf, "Connection refused"); break;
+                 case 6: strcat(buf, "TTL expired"); break;
+                 case 7: strcat(buf, "Command not supported"); break;
+                 case 8: strcat(buf, "Address type not supported"); break;
+                 default: sprintf(buf+strlen(buf),
+                                  "Unrecognised SOCKS error code %d",
+                                  data[1]);
+                   break;
+               }
+               plug_closing(p->plug, buf, PROXY_ERROR_GENERAL, 0);
+
+               return 1;
+           }
+
+           /*
+            * Eat the rest of the reply packet.
+            */
+           len = 6;                   /* first 4 bytes, last 2 */
+           switch (data[3]) {
+             case 1: len += 4; break; /* IPv4 address */
+             case 4: len += 16; break;/* IPv6 address */
+             case 3: len += (unsigned char)data[4]; break; /* domain name */
+             default:
+               plug_closing(p->plug, "Proxy error: SOCKS proxy returned "
+                            "unrecognised address format",
+                            PROXY_ERROR_GENERAL, 0);
+               return 1;
+           }
+           if (bufchain_size(&p->pending_input_data) < len)
+               return 1;              /* not got whole reply yet */
+           bufchain_consume(&p->pending_input_data, len);
+
+           /* we're done */
+           proxy_activate(p);
+           return 1;
+       }
+
+       if (p->state == 4) {
+           /* TODO: Handle GSSAPI authentication */
+           plug_closing(p->plug, "Proxy error: We don't support GSSAPI authentication",
+                        PROXY_ERROR_GENERAL, 0);
+           return 1;
+       }
+
+       if (p->state == 5) {
+           char *username = conf_get_str(p->conf, CONF_proxy_username);
+           char *password = conf_get_str(p->conf, CONF_proxy_password);
+           if (username[0] || password[0]) {
+               char userpwbuf[255 + 255 + 3];
+               int ulen, plen;
+               ulen = strlen(username);
+               if (ulen > 255) ulen = 255; if (ulen < 1) ulen = 1;
+               plen = strlen(password);
+               if (plen > 255) plen = 255; if (plen < 1) plen = 1;
+               userpwbuf[0] = 1;      /* version number of subnegotiation */
+               userpwbuf[1] = ulen;
+               memcpy(userpwbuf+2, username, ulen);
+               userpwbuf[ulen+2] = plen;
+               memcpy(userpwbuf+ulen+3, password, plen);
+               sk_write(p->sub_socket, userpwbuf, ulen + plen + 3);
+               p->state = 7;
+           } else 
+               plug_closing(p->plug, "Proxy error: Server chose "
+                            "username/password authentication but we "
+                            "didn't offer it!",
+                        PROXY_ERROR_GENERAL, 0);
+           return 1;
+       }
+
+       if (p->state == 6) {
+           int ret;
+           ret = proxy_socks5_selectchap(p);
+           if (ret) return ret;
+       }
+
+    }
+
+    plug_closing(p->plug, "Proxy error: Unexpected proxy error",
+                PROXY_ERROR_UNEXPECTED, 0);
+    return 1;
 }
 
 /* ----------------------------------------------------------------------
- * `Telnet' proxy type (as yet unimplemented).
+ * `Telnet' proxy type.
  *
  * (This is for ad-hoc proxies where you connect to the proxy's
  * telnet port and send a command such as `connect host port'. The
@@ -587,8 +1236,282 @@ int proxy_socks_negotiate (Proxy_Socket p, int change)
  * standardised or at all well-defined.)
  */
 
+char *format_telnet_command(SockAddr addr, int port, Conf *conf)
+{
+    char *fmt = conf_get_str(conf, CONF_proxy_telnet_command);
+    char *ret = NULL;
+    int retlen = 0, retsize = 0;
+    int so = 0, eo = 0;
+#define ENSURE(n) do { \
+    if (retsize < retlen + n) { \
+       retsize = retlen + n + 512; \
+       ret = sresize(ret, retsize, char); \
+    } \
+} while (0)
+
+    /* we need to escape \\, \%, \r, \n, \t, \x??, \0???, 
+     * %%, %host, %port, %user, and %pass
+     */
+
+    while (fmt[eo] != 0) {
+
+       /* scan forward until we hit end-of-line,
+        * or an escape character (\ or %) */
+       while (fmt[eo] != 0 && fmt[eo] != '%' && fmt[eo] != '\\')
+           eo++;
+
+       /* if we hit eol, break out of our escaping loop */
+       if (fmt[eo] == 0) break;
+
+       /* if there was any unescaped text before the escape
+        * character, send that now */
+       if (eo != so) {
+           ENSURE(eo - so);
+           memcpy(ret + retlen, fmt + so, eo - so);
+           retlen += eo - so;
+       }
+
+       so = eo++;
+
+       /* if the escape character was the last character of
+        * the line, we'll just stop and send it. */
+       if (fmt[eo] == 0) break;
+
+       if (fmt[so] == '\\') {
+
+           /* we recognize \\, \%, \r, \n, \t, \x??.
+            * anything else, we just send unescaped (including the \).
+            */
+
+           switch (fmt[eo]) {
+
+             case '\\':
+               ENSURE(1);
+               ret[retlen++] = '\\';
+               eo++;
+               break;
+
+             case '%':
+               ENSURE(1);
+               ret[retlen++] = '%';
+               eo++;
+               break;
+
+             case 'r':
+               ENSURE(1);
+               ret[retlen++] = '\r';
+               eo++;
+               break;
+
+             case 'n':
+               ENSURE(1);
+               ret[retlen++] = '\n';
+               eo++;
+               break;
+
+             case 't':
+               ENSURE(1);
+               ret[retlen++] = '\t';
+               eo++;
+               break;
+
+             case 'x':
+             case 'X':
+               {
+                   /* escaped hexadecimal value (ie. \xff) */
+                   unsigned char v = 0;
+                   int i = 0;
+
+                   for (;;) {
+                       eo++;
+                       if (fmt[eo] >= '0' && fmt[eo] <= '9')
+                           v += fmt[eo] - '0';
+                       else if (fmt[eo] >= 'a' && fmt[eo] <= 'f')
+                           v += fmt[eo] - 'a' + 10;
+                       else if (fmt[eo] >= 'A' && fmt[eo] <= 'F')
+                           v += fmt[eo] - 'A' + 10;
+                       else {
+                           /* non hex character, so we abort and just
+                            * send the whole thing unescaped (including \x)
+                            */
+                           ENSURE(1);
+                           ret[retlen++] = '\\';
+                           eo = so + 1;
+                           break;
+                       }
+
+                       /* we only extract two hex characters */
+                       if (i == 1) {
+                           ENSURE(1);
+                           ret[retlen++] = v;
+                           eo++;
+                           break;
+                       }
+
+                       i++;
+                       v <<= 4;
+                   }
+               }
+               break;
+
+             default:
+               ENSURE(2);
+               memcpy(ret+retlen, fmt + so, 2);
+               retlen += 2;
+               eo++;
+               break;
+           }
+       } else {
+
+           /* % escape. we recognize %%, %host, %port, %user, %pass.
+            * %proxyhost, %proxyport. Anything else we just send
+            * unescaped (including the %).
+            */
+
+           if (fmt[eo] == '%') {
+               ENSURE(1);
+               ret[retlen++] = '%';
+               eo++;
+           }
+           else if (strnicmp(fmt + eo, "host", 4) == 0) {
+               char dest[512];
+               int destlen;
+               sk_getaddr(addr, dest, lenof(dest));
+               destlen = strlen(dest);
+               ENSURE(destlen);
+               memcpy(ret+retlen, dest, destlen);
+               retlen += destlen;
+               eo += 4;
+           }
+           else if (strnicmp(fmt + eo, "port", 4) == 0) {
+               char portstr[8], portlen;
+               portlen = sprintf(portstr, "%i", port);
+               ENSURE(portlen);
+               memcpy(ret + retlen, portstr, portlen);
+               retlen += portlen;
+               eo += 4;
+           }
+           else if (strnicmp(fmt + eo, "user", 4) == 0) {
+               char *username = conf_get_str(conf, CONF_proxy_username);
+               int userlen = strlen(username);
+               ENSURE(userlen);
+               memcpy(ret+retlen, username, userlen);
+               retlen += userlen;
+               eo += 4;
+           }
+           else if (strnicmp(fmt + eo, "pass", 4) == 0) {
+               char *password = conf_get_str(conf, CONF_proxy_password);
+               int passlen = strlen(password);
+               ENSURE(passlen);
+               memcpy(ret+retlen, password, passlen);
+               retlen += passlen;
+               eo += 4;
+           }
+           else if (strnicmp(fmt + eo, "proxyhost", 9) == 0) {
+               char *host = conf_get_str(conf, CONF_proxy_host);
+               int phlen = strlen(host);
+               ENSURE(phlen);
+               memcpy(ret+retlen, host, phlen);
+               retlen += phlen;
+               eo += 9;
+           }
+           else if (strnicmp(fmt + eo, "proxyport", 9) == 0) {
+               int port = conf_get_int(conf, CONF_proxy_port);
+                char pport[50];
+               int pplen;
+                sprintf(pport, "%d", port);
+                pplen = strlen(pport);
+               ENSURE(pplen);
+               memcpy(ret+retlen, pport, pplen);
+               retlen += pplen;
+               eo += 9;
+           }
+           else {
+               /* we don't escape this, so send the % now, and
+                * don't advance eo, so that we'll consider the
+                * text immediately following the % as unescaped.
+                */
+               ENSURE(1);
+               ret[retlen++] = '%';
+           }
+       }
+
+       /* resume scanning for additional escapes after this one. */
+       so = eo;
+    }
+
+    /* if there is any unescaped text at the end of the line, send it */
+    if (eo != so) {
+       ENSURE(eo - so);
+       memcpy(ret + retlen, fmt + so, eo - so);
+       retlen += eo - so;
+    }
+
+    ENSURE(1);
+    ret[retlen] = '\0';
+    return ret;
+
+#undef ENSURE
+}
+
 int proxy_telnet_negotiate (Proxy_Socket p, int change)
 {
-    p->error = "Network error: Telnet proxy implementation is incomplete";
-    return 0;
+    if (p->state == PROXY_CHANGE_NEW) {
+       char *formatted_cmd;
+
+       formatted_cmd = format_telnet_command(p->remote_addr, p->remote_port,
+                                             p->conf);
+
+       sk_write(p->sub_socket, formatted_cmd, strlen(formatted_cmd));
+       sfree(formatted_cmd);
+
+       p->state = 1;
+       return 0;
+    }
+
+    if (change == PROXY_CHANGE_CLOSING) {
+       /* if our proxy negotiation process involves closing and opening
+        * new sockets, then we would want to intercept this closing
+        * callback when we were expecting it. if we aren't anticipating
+        * a socket close, then some error must have occurred. we'll
+        * just pass those errors up to the backend.
+        */
+       return plug_closing(p->plug, p->closing_error_msg,
+                           p->closing_error_code,
+                           p->closing_calling_back);
+    }
+
+    if (change == PROXY_CHANGE_SENT) {
+       /* some (or all) of what we wrote to the proxy was sent.
+        * we don't do anything new, however, until we receive the
+        * proxy's response. we might want to set a timer so we can
+        * timeout the proxy negotiation after a while...
+        */
+       return 0;
+    }
+
+    if (change == PROXY_CHANGE_ACCEPTING) {
+       /* we should _never_ see this, as we are using our socket to
+        * connect to a proxy, not accepting inbound connections.
+        * what should we do? close the socket with an appropriate
+        * error message?
+        */
+       return plug_accepting(p->plug, p->accepting_sock);
+    }
+
+    if (change == PROXY_CHANGE_RECEIVE) {
+       /* we have received data from the underlying socket, which
+        * we'll need to parse, process, and respond to appropriately.
+        */
+
+       /* we're done */
+       proxy_activate(p);
+       /* proxy activate will have dealt with
+        * whatever is left of the buffer */
+       return 1;
+    }
+
+    plug_closing(p->plug, "Proxy error: Unexpected proxy error",
+                PROXY_ERROR_UNEXPECTED, 0);
+    return 1;
 }