Sebastian Kuschel reports that pfd_closing can be called for a socket
[u/mdw/putty] / config.c
index d3d76f5..02da07d 100644 (file)
--- a/config.c
+++ b/config.c
 
 #define PRINTER_DISABLED_STRING "None (printing disabled)"
 
-static void protocolbuttons_handler(union control *ctrl, void *dlg,
+#define HOST_BOX_TITLE "Host Name (or IP address)"
+#define PORT_BOX_TITLE "Port"
+
+void conf_radiobutton_handler(union control *ctrl, void *dlg,
+                             void *data, int event)
+{
+    int button;
+    Conf *conf = (Conf *)data;
+
+    /*
+     * For a standard radio button set, the context parameter gives
+     * the primary key (CONF_foo), and the extra data per button
+     * gives the value the target field should take if that button
+     * is the one selected.
+     */
+    if (event == EVENT_REFRESH) {
+       int val = conf_get_int(conf, ctrl->radio.context.i);
+       for (button = 0; button < ctrl->radio.nbuttons; button++)
+           if (val == ctrl->radio.buttondata[button].i)
+               break;
+       /* We expected that `break' to happen, in all circumstances. */
+       assert(button < ctrl->radio.nbuttons);
+       dlg_radiobutton_set(ctrl, dlg, button);
+    } else if (event == EVENT_VALCHANGE) {
+       button = dlg_radiobutton_get(ctrl, dlg);
+       assert(button >= 0 && button < ctrl->radio.nbuttons);
+       conf_set_int(conf, ctrl->radio.context.i,
+                    ctrl->radio.buttondata[button].i);
+    }
+}
+
+#define CHECKBOX_INVERT (1<<30)
+void conf_checkbox_handler(union control *ctrl, void *dlg,
+                          void *data, int event)
+{
+    int key, invert;
+    Conf *conf = (Conf *)data;
+
+    /*
+     * For a standard checkbox, the context parameter gives the
+     * primary key (CONF_foo), optionally ORed with CHECKBOX_INVERT.
+     */
+    key = ctrl->checkbox.context.i;
+    if (key & CHECKBOX_INVERT) {
+       key &= ~CHECKBOX_INVERT;
+       invert = 1;
+    } else
+       invert = 0;
+
+    /*
+     * C lacks a logical XOR, so the following code uses the idiom
+     * (!a ^ !b) to obtain the logical XOR of a and b. (That is, 1
+     * iff exactly one of a and b is nonzero, otherwise 0.)
+     */
+
+    if (event == EVENT_REFRESH) {
+       int val = conf_get_int(conf, key);
+       dlg_checkbox_set(ctrl, dlg, (!val ^ !invert));
+    } else if (event == EVENT_VALCHANGE) {
+       conf_set_int(conf, key, !dlg_checkbox_get(ctrl,dlg) ^ !invert);
+    }
+}
+
+void conf_editbox_handler(union control *ctrl, void *dlg,
+                         void *data, int event)
+{
+    /*
+     * The standard edit-box handler expects the main `context'
+     * field to contain the primary key. The secondary `context2'
+     * field indicates the type of this field:
+     *
+     *  - if context2 > 0, the field is a string.
+     *  - if context2 == -1, the field is an int and the edit box
+     *    is numeric.
+     *  - if context2 < -1, the field is an int and the edit box is
+     *    _floating_, and (-context2) gives the scale. (E.g. if
+     *    context2 == -1000, then typing 1.2 into the box will set
+     *    the field to 1200.)
+     */
+    int key = ctrl->editbox.context.i;
+    int length = ctrl->editbox.context2.i;
+    Conf *conf = (Conf *)data;
+
+    if (length > 0) {
+       if (event == EVENT_REFRESH) {
+           char *field = conf_get_str(conf, key);
+           dlg_editbox_set(ctrl, dlg, field);
+       } else if (event == EVENT_VALCHANGE) {
+           char *field = dlg_editbox_get(ctrl, dlg);
+           conf_set_str(conf, key, field);
+           sfree(field);
+       }
+    } else if (length < 0) {
+       if (event == EVENT_REFRESH) {
+           char str[80];
+           int value = conf_get_int(conf, key);
+           if (length == -1)
+               sprintf(str, "%d", value);
+           else
+               sprintf(str, "%g", (double)value / (double)(-length));
+           dlg_editbox_set(ctrl, dlg, str);
+       } else if (event == EVENT_VALCHANGE) {
+           char *str = dlg_editbox_get(ctrl, dlg);
+           if (length == -1)
+               conf_set_int(conf, key, atoi(str));
+           else
+               conf_set_int(conf, key, (int)((-length) * atof(str)));
+           sfree(str);
+       }
+    }
+}
+
+void conf_filesel_handler(union control *ctrl, void *dlg,
+                         void *data, int event)
+{
+    int key = ctrl->fileselect.context.i;
+    Conf *conf = (Conf *)data;
+
+    if (event == EVENT_REFRESH) {
+       dlg_filesel_set(ctrl, dlg, conf_get_filename(conf, key));
+    } else if (event == EVENT_VALCHANGE) {
+       Filename *filename = dlg_filesel_get(ctrl, dlg);
+       conf_set_filename(conf, key, filename);
+        filename_free(filename);
+    }
+}
+
+void conf_fontsel_handler(union control *ctrl, void *dlg,
+                         void *data, int event)
+{
+    int key = ctrl->fontselect.context.i;
+    Conf *conf = (Conf *)data;
+
+    if (event == EVENT_REFRESH) {
+       dlg_fontsel_set(ctrl, dlg, conf_get_fontspec(conf, key));
+    } else if (event == EVENT_VALCHANGE) {
+       FontSpec *fontspec = dlg_fontsel_get(ctrl, dlg);
+       conf_set_fontspec(conf, key, fontspec);
+        fontspec_free(fontspec);
+    }
+}
+
+static void config_host_handler(union control *ctrl, void *dlg,
+                               void *data, int event)
+{
+    Conf *conf = (Conf *)data;
+
+    /*
+     * This function works just like the standard edit box handler,
+     * only it has to choose the control's label and text from two
+     * different places depending on the protocol.
+     */
+    if (event == EVENT_REFRESH) {
+       if (conf_get_int(conf, CONF_protocol) == PROT_SERIAL) {
+           /*
+            * This label text is carefully chosen to contain an n,
+            * since that's the shortcut for the host name control.
+            */
+           dlg_label_change(ctrl, dlg, "Serial line");
+           dlg_editbox_set(ctrl, dlg, conf_get_str(conf, CONF_serline));
+       } else {
+           dlg_label_change(ctrl, dlg, HOST_BOX_TITLE);
+           dlg_editbox_set(ctrl, dlg, conf_get_str(conf, CONF_host));
+       }
+    } else if (event == EVENT_VALCHANGE) {
+       char *s = dlg_editbox_get(ctrl, dlg);
+       if (conf_get_int(conf, CONF_protocol) == PROT_SERIAL)
+           conf_set_str(conf, CONF_serline, s);
+       else
+           conf_set_str(conf, CONF_host, s);
+       sfree(s);
+    }
+}
+
+static void config_port_handler(union control *ctrl, void *dlg,
+                               void *data, int event)
+{
+    Conf *conf = (Conf *)data;
+    char buf[80];
+
+    /*
+     * This function works similarly to the standard edit box handler,
+     * only it has to choose the control's label and text from two
+     * different places depending on the protocol.
+     */
+    if (event == EVENT_REFRESH) {
+       if (conf_get_int(conf, CONF_protocol) == PROT_SERIAL) {
+           /*
+            * This label text is carefully chosen to contain a p,
+            * since that's the shortcut for the port control.
+            */
+           dlg_label_change(ctrl, dlg, "Speed");
+           sprintf(buf, "%d", conf_get_int(conf, CONF_serspeed));
+       } else {
+           dlg_label_change(ctrl, dlg, PORT_BOX_TITLE);
+           if (conf_get_int(conf, CONF_port) != 0)
+               sprintf(buf, "%d", conf_get_int(conf, CONF_port));
+           else
+               /* Display an (invalid) port of 0 as blank */
+               buf[0] = '\0';
+       }
+       dlg_editbox_set(ctrl, dlg, buf);
+    } else if (event == EVENT_VALCHANGE) {
+       char *s = dlg_editbox_get(ctrl, dlg);
+       int i = atoi(s);
+       sfree(s);
+
+       if (conf_get_int(conf, CONF_protocol) == PROT_SERIAL)
+           conf_set_int(conf, CONF_serspeed, i);
+       else
+           conf_set_int(conf, CONF_port, i);
+    }
+}
+
+struct hostport {
+    union control *host, *port;
+};
+
+/*
+ * We export this function so that platform-specific config
+ * routines can use it to conveniently identify the protocol radio
+ * buttons in order to add to them.
+ */
+void config_protocolbuttons_handler(union control *ctrl, void *dlg,
                                    void *data, int event)
 {
-    int button, defport;
-    Config *cfg = (Config *)data;
+    int button;
+    Conf *conf = (Conf *)data;
+    struct hostport *hp = (struct hostport *)ctrl->radio.context.p;
+
     /*
      * This function works just like the standard radio-button
      * handler, except that it also has to change the setting of
-     * the port box. We expect the context parameter to point at
-     * the `union control' structure for the port box.
+     * the port box, and refresh both host and port boxes when. We
+     * expect the context parameter to point at a hostport
+     * structure giving the `union control's for both.
      */
     if (event == EVENT_REFRESH) {
+       int protocol = conf_get_int(conf, CONF_protocol);
        for (button = 0; button < ctrl->radio.nbuttons; button++)
-           if (cfg->protocol == ctrl->radio.buttondata[button].i)
+           if (protocol == ctrl->radio.buttondata[button].i)
                break;
        /* We expected that `break' to happen, in all circumstances. */
        assert(button < ctrl->radio.nbuttons);
        dlg_radiobutton_set(ctrl, dlg, button);
     } else if (event == EVENT_VALCHANGE) {
-       int oldproto = cfg->protocol;
+       int oldproto = conf_get_int(conf, CONF_protocol);
+       int newproto, port;
+
        button = dlg_radiobutton_get(ctrl, dlg);
        assert(button >= 0 && button < ctrl->radio.nbuttons);
-       cfg->protocol = ctrl->radio.buttondata[button].i;
-       if (oldproto != cfg->protocol) {
-           defport = -1;
-           switch (cfg->protocol) {
-             case PROT_SSH: defport = 22; break;
-             case PROT_TELNET: defport = 23; break;
-             case PROT_RLOGIN: defport = 513; break;
-           }
-           if (defport > 0 && cfg->port != defport) {
-               cfg->port = defport;
-               dlg_refresh((union control *)ctrl->radio.context.p, dlg);
-           }
+       newproto = ctrl->radio.buttondata[button].i;
+       conf_set_int(conf, CONF_protocol, newproto);
+
+       if (oldproto != newproto) {
+           Backend *ob = backend_from_proto(oldproto);
+           Backend *nb = backend_from_proto(newproto);
+           assert(ob);
+           assert(nb);
+           /* Iff the user hasn't changed the port from the old protocol's
+            * default, update it with the new protocol's default.
+            * (This includes a "default" of 0, implying that there is no
+            * sensible default for that protocol; in this case it's
+            * displayed as a blank.)
+            * This helps with the common case of tabbing through the
+            * controls in order and setting a non-default port before
+            * getting to the protocol; we want that non-default port
+            * to be preserved. */
+           port = conf_get_int(conf, CONF_port);
+           if (port == ob->default_port)
+               conf_set_int(conf, CONF_port, nb->default_port);
        }
+       dlg_refresh(hp->host, dlg);
+       dlg_refresh(hp->port, dlg);
+    }
+}
+
+static void loggingbuttons_handler(union control *ctrl, void *dlg,
+                                  void *data, int event)
+{
+    int button;
+    Conf *conf = (Conf *)data;
+    /* This function works just like the standard radio-button handler,
+     * but it has to fall back to "no logging" in situations where the
+     * configured logging type isn't applicable.
+     */
+    if (event == EVENT_REFRESH) {
+       int logtype = conf_get_int(conf, CONF_logtype);
+
+        for (button = 0; button < ctrl->radio.nbuttons; button++)
+            if (logtype == ctrl->radio.buttondata[button].i)
+               break;
+
+       /* We fell off the end, so we lack the configured logging type */
+       if (button == ctrl->radio.nbuttons) {
+           button = 0;
+           conf_set_int(conf, CONF_logtype, LGTYP_NONE);
+       }
+       dlg_radiobutton_set(ctrl, dlg, button);
+    } else if (event == EVENT_VALCHANGE) {
+        button = dlg_radiobutton_get(ctrl, dlg);
+        assert(button >= 0 && button < ctrl->radio.nbuttons);
+        conf_set_int(conf, CONF_logtype, ctrl->radio.buttondata[button].i);
     }
 }
 
@@ -54,15 +322,15 @@ static void numeric_keypad_handler(union control *ctrl, void *dlg,
                                   void *data, int event)
 {
     int button;
-    Config *cfg = (Config *)data;
+    Conf *conf = (Conf *)data;
     /*
      * This function works much like the standard radio button
-     * handler, but it has to handle two fields in Config.
+     * handler, but it has to handle two fields in Conf.
      */
     if (event == EVENT_REFRESH) {
-       if (cfg->nethack_keypad)
+       if (conf_get_int(conf, CONF_nethack_keypad))
            button = 2;
-       else if (cfg->app_keypad)
+       else if (conf_get_int(conf, CONF_app_keypad))
            button = 1;
        else
            button = 0;
@@ -72,11 +340,11 @@ static void numeric_keypad_handler(union control *ctrl, void *dlg,
        button = dlg_radiobutton_get(ctrl, dlg);
        assert(button >= 0 && button < ctrl->radio.nbuttons);
        if (button == 2) {
-           cfg->app_keypad = FALSE;
-           cfg->nethack_keypad = TRUE;
+           conf_set_int(conf, CONF_app_keypad, FALSE);
+           conf_set_int(conf, CONF_nethack_keypad, TRUE);
        } else {
-           cfg->app_keypad = (button != 0);
-           cfg->nethack_keypad = FALSE;
+           conf_set_int(conf, CONF_app_keypad, (button != 0));
+           conf_set_int(conf, CONF_nethack_keypad, FALSE);
        }
     }
 }
@@ -84,7 +352,7 @@ static void numeric_keypad_handler(union control *ctrl, void *dlg,
 static void cipherlist_handler(union control *ctrl, void *dlg,
                               void *data, int event)
 {
-    Config *cfg = (Config *)data;
+    Conf *conf = (Conf *)data;
     if (event == EVENT_REFRESH) {
        int i;
 
@@ -92,7 +360,8 @@ static void cipherlist_handler(union control *ctrl, void *dlg,
            { "3DES",                   CIPHER_3DES },
            { "Blowfish",               CIPHER_BLOWFISH },
            { "DES",                    CIPHER_DES },
-           { "AES (SSH 2 only)",       CIPHER_AES },
+           { "AES (SSH-2 only)",       CIPHER_AES },
+           { "Arcfour (SSH-2 only)",   CIPHER_ARCFOUR },
            { "-- warn below here --",  CIPHER_WARN }
        };
 
@@ -101,7 +370,7 @@ static void cipherlist_handler(union control *ctrl, void *dlg,
        dlg_update_start(ctrl, dlg);
        dlg_listbox_clear(ctrl, dlg);
        for (i = 0; i < CIPHER_MAX; i++) {
-           int c = cfg->ssh_cipherlist[i];
+           int c = conf_get_int_int(conf, CONF_ssh_cipherlist, i);
            int j;
            char *cstr = NULL;
            for (j = 0; j < (sizeof ciphers) / (sizeof ciphers[0]); j++) {
@@ -119,18 +388,90 @@ static void cipherlist_handler(union control *ctrl, void *dlg,
 
        /* Update array to match the list box. */
        for (i=0; i < CIPHER_MAX; i++)
-           cfg->ssh_cipherlist[i] = dlg_listbox_getid(ctrl, dlg, i);
+           conf_set_int_int(conf, CONF_ssh_cipherlist, i,
+                            dlg_listbox_getid(ctrl, dlg, i));
+    }
+}
+
+#ifndef NO_GSSAPI
+static void gsslist_handler(union control *ctrl, void *dlg,
+                           void *data, int event)
+{
+    Conf *conf = (Conf *)data;
+    if (event == EVENT_REFRESH) {
+       int i;
+
+       dlg_update_start(ctrl, dlg);
+       dlg_listbox_clear(ctrl, dlg);
+       for (i = 0; i < ngsslibs; i++) {
+           int id = conf_get_int_int(conf, CONF_ssh_gsslist, i);
+           assert(id >= 0 && id < ngsslibs);
+           dlg_listbox_addwithid(ctrl, dlg, gsslibnames[id], id);
+       }
+       dlg_update_done(ctrl, dlg);
+
+    } else if (event == EVENT_VALCHANGE) {
+       int i;
 
+       /* Update array to match the list box. */
+       for (i=0; i < ngsslibs; i++)
+           conf_set_int_int(conf, CONF_ssh_gsslist, i,
+                            dlg_listbox_getid(ctrl, dlg, i));
+    }
+}
+#endif
+
+static void kexlist_handler(union control *ctrl, void *dlg,
+                           void *data, int event)
+{
+    Conf *conf = (Conf *)data;
+    if (event == EVENT_REFRESH) {
+       int i;
+
+       static const struct { char *s; int k; } kexes[] = {
+           { "Diffie-Hellman group 1",         KEX_DHGROUP1 },
+           { "Diffie-Hellman group 14",        KEX_DHGROUP14 },
+           { "Diffie-Hellman group exchange",  KEX_DHGEX },
+           { "RSA-based key exchange",         KEX_RSA },
+           { "-- warn below here --",          KEX_WARN }
+       };
+
+       /* Set up the "kex preference" box. */
+       /* (kexlist assumed to contain all algorithms) */
+       dlg_update_start(ctrl, dlg);
+       dlg_listbox_clear(ctrl, dlg);
+       for (i = 0; i < KEX_MAX; i++) {
+           int k = conf_get_int_int(conf, CONF_ssh_kexlist, i);
+           int j;
+           char *kstr = NULL;
+           for (j = 0; j < (sizeof kexes) / (sizeof kexes[0]); j++) {
+               if (kexes[j].k == k) {
+                   kstr = kexes[j].s;
+                   break;
+               }
+           }
+           dlg_listbox_addwithid(ctrl, dlg, kstr, k);
+       }
+       dlg_update_done(ctrl, dlg);
+
+    } else if (event == EVENT_VALCHANGE) {
+       int i;
+
+       /* Update array to match the list box. */
+       for (i=0; i < KEX_MAX; i++)
+           conf_set_int_int(conf, CONF_ssh_kexlist, i,
+                            dlg_listbox_getid(ctrl, dlg, i));
     }
 }
 
 static void printerbox_handler(union control *ctrl, void *dlg,
                               void *data, int event)
 {
-    Config *cfg = (Config *)data;
+    Conf *conf = (Conf *)data;
     if (event == EVENT_REFRESH) {
        int nprinters, i;
        printer_enum *pe;
+       char *printer;
 
        dlg_update_start(ctrl, dlg);
        /*
@@ -145,50 +486,62 @@ static void printerbox_handler(union control *ctrl, void *dlg,
                dlg_listbox_add(ctrl, dlg, printer_get_name(pe, i));
            printer_finish_enum(pe);
        }
-       dlg_editbox_set(ctrl, dlg,
-                       (*cfg->printer ? cfg->printer :
-                        PRINTER_DISABLED_STRING));
+       printer = conf_get_str(conf, CONF_printer);
+       if (!printer)
+           printer = PRINTER_DISABLED_STRING;
+       dlg_editbox_set(ctrl, dlg, printer);
        dlg_update_done(ctrl, dlg);
     } else if (event == EVENT_VALCHANGE) {
-       dlg_editbox_get(ctrl, dlg, cfg->printer, sizeof(cfg->printer));
-       if (!strcmp(cfg->printer, PRINTER_DISABLED_STRING))
-           *cfg->printer = '\0';
+       char *printer = dlg_editbox_get(ctrl, dlg);
+       if (!strcmp(printer, PRINTER_DISABLED_STRING))
+           printer[0] = '\0';
+       conf_set_str(conf, CONF_printer, printer);
+       sfree(printer);
     }
 }
 
 static void codepage_handler(union control *ctrl, void *dlg,
                             void *data, int event)
 {
-    Config *cfg = (Config *)data;
+    Conf *conf = (Conf *)data;
     if (event == EVENT_REFRESH) {
        int i;
-       const char *cp;
+       const char *cp, *thiscp;
        dlg_update_start(ctrl, dlg);
-       strcpy(cfg->line_codepage,
-              cp_name(decode_codepage(cfg->line_codepage)));
+       thiscp = cp_name(decode_codepage(conf_get_str(conf,
+                                                     CONF_line_codepage)));
        dlg_listbox_clear(ctrl, dlg);
        for (i = 0; (cp = cp_enumerate(i)) != NULL; i++)
            dlg_listbox_add(ctrl, dlg, cp);
-       dlg_editbox_set(ctrl, dlg, cfg->line_codepage);
+       dlg_editbox_set(ctrl, dlg, thiscp);
+       conf_set_str(conf, CONF_line_codepage, thiscp);
        dlg_update_done(ctrl, dlg);
     } else if (event == EVENT_VALCHANGE) {
-       dlg_editbox_get(ctrl, dlg, cfg->line_codepage,
-                       sizeof(cfg->line_codepage));
-       strcpy(cfg->line_codepage,
-              cp_name(decode_codepage(cfg->line_codepage)));
+       char *codepage = dlg_editbox_get(ctrl, dlg);
+       conf_set_str(conf, CONF_line_codepage,
+                    cp_name(decode_codepage(codepage)));
+       sfree(codepage);
     }
 }
 
 static void sshbug_handler(union control *ctrl, void *dlg,
                           void *data, int event)
 {
+    Conf *conf = (Conf *)data;
     if (event == EVENT_REFRESH) {
+        /*
+         * We must fetch the previously configured value from the Conf
+         * before we start modifying the drop-down list, otherwise the
+         * spurious SELCHANGE we trigger in the process will overwrite
+         * the value we wanted to keep.
+         */
+        int oldconf = conf_get_int(conf, ctrl->listbox.context.i);
        dlg_update_start(ctrl, dlg);
        dlg_listbox_clear(ctrl, dlg);
        dlg_listbox_addwithid(ctrl, dlg, "Auto", AUTO);
        dlg_listbox_addwithid(ctrl, dlg, "Off", FORCE_OFF);
        dlg_listbox_addwithid(ctrl, dlg, "On", FORCE_ON);
-       switch (*(int *)ATOFFSET(data, ctrl->listbox.context.i)) {
+       switch (oldconf) {
          case AUTO:      dlg_listbox_select(ctrl, dlg, 0); break;
          case FORCE_OFF: dlg_listbox_select(ctrl, dlg, 1); break;
          case FORCE_ON:  dlg_listbox_select(ctrl, dlg, 2); break;
@@ -200,26 +553,32 @@ static void sshbug_handler(union control *ctrl, void *dlg,
            i = AUTO;
        else
            i = dlg_listbox_getid(ctrl, dlg, i);
-       *(int *)ATOFFSET(data, ctrl->listbox.context.i) = i;
+       conf_set_int(conf, ctrl->listbox.context.i, i);
     }
 }
 
-#define SAVEDSESSION_LEN 2048
-
 struct sessionsaver_data {
     union control *editbox, *listbox, *loadbutton, *savebutton, *delbutton;
     union control *okbutton, *cancelbutton;
-    struct sesslist *sesslist;
+    struct sesslist sesslist;
+    int midsession;
+    char *savedsession;     /* the current contents of ssd->editbox */
 };
 
+static void sessionsaver_data_free(void *ssdv)
+{
+    struct sessionsaver_data *ssd = (struct sessionsaver_data *)ssdv;
+    sfree(ssd->savedsession);
+    sfree(ssd);
+}
+
 /* 
  * Helper function to load the session selected in the list box, if
  * any, as this is done in more than one place below. Returns 0 for
  * failure.
  */
 static int load_selected_session(struct sessionsaver_data *ssd,
-                                char *savedsession,
-                                void *dlg, Config *cfg)
+                                void *dlg, Conf *conf, int *maybe_launch)
 {
     int i = dlg_listbox_index(ssd->listbox, dlg);
     int isdef;
@@ -227,15 +586,12 @@ static int load_selected_session(struct sessionsaver_data *ssd,
        dlg_beep(dlg);
        return 0;
     }
-    isdef = !strcmp(ssd->sesslist->sessions[i], "Default Settings");
-    load_settings(ssd->sesslist->sessions[i], !isdef, cfg);
-    if (!isdef) {
-       strncpy(savedsession, ssd->sesslist->sessions[i],
-               SAVEDSESSION_LEN);
-       savedsession[SAVEDSESSION_LEN-1] = '\0';
-    } else {
-       savedsession[0] = '\0';
-    }
+    isdef = !strcmp(ssd->sesslist.sessions[i], "Default Settings");
+    load_settings(ssd->sesslist.sessions[i], conf);
+    sfree(ssd->savedsession);
+    ssd->savedsession = dupstr(isdef ? "" : ssd->sesslist.sessions[i]);
+    if (maybe_launch)
+        *maybe_launch = !isdef;
     dlg_refresh(NULL, dlg);
     /* Restore the selection, which might have been clobbered by
      * changing the value of the edit box. */
@@ -246,47 +602,47 @@ static int load_selected_session(struct sessionsaver_data *ssd,
 static void sessionsaver_handler(union control *ctrl, void *dlg,
                                 void *data, int event)
 {
-    Config *cfg = (Config *)data;
+    Conf *conf = (Conf *)data;
     struct sessionsaver_data *ssd =
        (struct sessionsaver_data *)ctrl->generic.context.p;
-    char *savedsession;
-
-    /*
-     * The first time we're called in a new dialog, we must
-     * allocate space to store the current contents of the saved
-     * session edit box (since it must persist even when we switch
-     * panels, but is not part of the Config).
-     * 
-     * Of course, this doesn't need to be done mid-session.
-     */
-    if (!ssd->editbox) {
-        savedsession = NULL;
-    } else if (!dlg_get_privdata(ssd->editbox, dlg)) {
-       savedsession = (char *)
-           dlg_alloc_privdata(ssd->editbox, dlg, SAVEDSESSION_LEN);
-       savedsession[0] = '\0';
-    } else {
-       savedsession = dlg_get_privdata(ssd->editbox, dlg);
-    }
 
     if (event == EVENT_REFRESH) {
        if (ctrl == ssd->editbox) {
-           dlg_editbox_set(ctrl, dlg, savedsession);
+           dlg_editbox_set(ctrl, dlg, ssd->savedsession);
        } else if (ctrl == ssd->listbox) {
            int i;
            dlg_update_start(ctrl, dlg);
            dlg_listbox_clear(ctrl, dlg);
-           for (i = 0; i < ssd->sesslist->nsessions; i++)
-               dlg_listbox_add(ctrl, dlg, ssd->sesslist->sessions[i]);
+           for (i = 0; i < ssd->sesslist.nsessions; i++)
+               dlg_listbox_add(ctrl, dlg, ssd->sesslist.sessions[i]);
            dlg_update_done(ctrl, dlg);
        }
     } else if (event == EVENT_VALCHANGE) {
+        int top, bottom, halfway, i;
        if (ctrl == ssd->editbox) {
-           dlg_editbox_get(ctrl, dlg, savedsession,
-                           SAVEDSESSION_LEN);
+            sfree(ssd->savedsession);
+            ssd->savedsession = dlg_editbox_get(ctrl, dlg);
+           top = ssd->sesslist.nsessions;
+           bottom = -1;
+           while (top-bottom > 1) {
+               halfway = (top+bottom)/2;
+               i = strcmp(ssd->savedsession, ssd->sesslist.sessions[halfway]);
+               if (i <= 0 ) {
+                   top = halfway;
+               } else {
+                   bottom = halfway;
+               }
+           }
+           if (top == ssd->sesslist.nsessions) {
+               top -= 1;
+           }
+           dlg_listbox_select(ssd->listbox, dlg, top);
        }
     } else if (event == EVENT_ACTION) {
-       if (ctrl == ssd->listbox || ctrl == ssd->loadbutton) {
+       int mbl = FALSE;
+       if (!ssd->midsession &&
+           (ctrl == ssd->listbox ||
+            (ssd->loadbutton && ctrl == ssd->loadbutton))) {
            /*
             * The user has double-clicked a session, or hit Load.
             * We must load the selected session, and then
@@ -294,50 +650,47 @@ static void sessionsaver_handler(union control *ctrl, void *dlg,
             * double-click on the list box _and_ that session
             * contains a hostname.
             */
-           if (load_selected_session(ssd, savedsession, dlg, cfg) &&
-               (ctrl == ssd->listbox && cfg->host[0])) {
+           if (load_selected_session(ssd, dlg, conf, &mbl) &&
+               (mbl && ctrl == ssd->listbox && conf_launchable(conf))) {
                dlg_end(dlg, 1);       /* it's all over, and succeeded */
            }
        } else if (ctrl == ssd->savebutton) {
-           int isdef = !strcmp(savedsession, "Default Settings");
-           if (!savedsession[0]) {
+           int isdef = !strcmp(ssd->savedsession, "Default Settings");
+           if (!ssd->savedsession[0]) {
                int i = dlg_listbox_index(ssd->listbox, dlg);
                if (i < 0) {
                    dlg_beep(dlg);
                    return;
                }
-               isdef = !strcmp(ssd->sesslist->sessions[i], "Default Settings");
-               if (!isdef) {
-                   strncpy(savedsession, ssd->sesslist->sessions[i],
-                           SAVEDSESSION_LEN);
-                   savedsession[SAVEDSESSION_LEN-1] = '\0';
-               } else {
-                   savedsession[0] = '\0';
-               }
+               isdef = !strcmp(ssd->sesslist.sessions[i], "Default Settings");
+                sfree(ssd->savedsession);
+                ssd->savedsession = dupstr(isdef ? "" :
+                                           ssd->sesslist.sessions[i]);
            }
             {
-                char *errmsg = save_settings(savedsession, !isdef, cfg);
+                char *errmsg = save_settings(ssd->savedsession, conf);
                 if (errmsg) {
                     dlg_error_msg(dlg, errmsg);
                     sfree(errmsg);
                 }
             }
-           get_sesslist(ssd->sesslist, FALSE);
-           get_sesslist(ssd->sesslist, TRUE);
+           get_sesslist(&ssd->sesslist, FALSE);
+           get_sesslist(&ssd->sesslist, TRUE);
            dlg_refresh(ssd->editbox, dlg);
            dlg_refresh(ssd->listbox, dlg);
-       } else if (ctrl == ssd->delbutton) {
+       } else if (!ssd->midsession &&
+                  ssd->delbutton && ctrl == ssd->delbutton) {
            int i = dlg_listbox_index(ssd->listbox, dlg);
            if (i <= 0) {
                dlg_beep(dlg);
            } else {
-               del_settings(ssd->sesslist->sessions[i]);
-               get_sesslist(ssd->sesslist, FALSE);
-               get_sesslist(ssd->sesslist, TRUE);
+               del_settings(ssd->sesslist.sessions[i]);
+               get_sesslist(&ssd->sesslist, FALSE);
+               get_sesslist(&ssd->sesslist, TRUE);
                dlg_refresh(ssd->listbox, dlg);
            }
        } else if (ctrl == ssd->okbutton) {
-            if (!savedsession) {
+            if (ssd->midsession) {
                 /* In a mid-session Change Settings, Apply is always OK. */
                dlg_end(dlg, 1);
                 return;
@@ -349,26 +702,31 @@ static void sessionsaver_handler(union control *ctrl, void *dlg,
             * there was a session selected in that which had a
             * valid host name in it, then load it and go.
             */
-           if (dlg_last_focused(ctrl, dlg) == ssd->listbox && !*cfg->host) {
-               Config cfg2;
-               if (!load_selected_session(ssd, savedsession, dlg, &cfg2)) {
+           if (dlg_last_focused(ctrl, dlg) == ssd->listbox &&
+               !conf_launchable(conf)) {
+               Conf *conf2 = conf_new();
+               int mbl = FALSE;
+               if (!load_selected_session(ssd, dlg, conf2, &mbl)) {
                    dlg_beep(dlg);
+                   conf_free(conf2);
                    return;
                }
                /* If at this point we have a valid session, go! */
-               if (*cfg2.host) {
-                   *cfg = cfg2;       /* structure copy */
-                   cfg->remote_cmd_ptr = cfg->remote_cmd; /* nasty */
+               if (mbl && conf_launchable(conf2)) {
+                   conf_copy_into(conf, conf2);
                    dlg_end(dlg, 1);
                } else
                    dlg_beep(dlg);
+
+               conf_free(conf2);
+                return;
            }
 
            /*
             * Otherwise, do the normal thing: if we have a valid
             * session, get going.
             */
-           if (*cfg->host) {
+           if (conf_launchable(conf)) {
                dlg_end(dlg, 1);
            } else
                dlg_beep(dlg);
@@ -385,7 +743,7 @@ struct charclass_data {
 static void charclass_handler(union control *ctrl, void *dlg,
                              void *data, int event)
 {
-    Config *cfg = (Config *)data;
+    Conf *conf = (Conf *)data;
     struct charclass_data *ccd =
        (struct charclass_data *)ctrl->generic.context.p;
 
@@ -397,20 +755,22 @@ static void charclass_handler(union control *ctrl, void *dlg,
            for (i = 0; i < 128; i++) {
                char str[100];
                sprintf(str, "%d\t(0x%02X)\t%c\t%d", i, i,
-                       (i >= 0x21 && i != 0x7F) ? i : ' ', cfg->wordness[i]);
+                       (i >= 0x21 && i != 0x7F) ? i : ' ',
+                       conf_get_int_int(conf, CONF_wordness, i));
                dlg_listbox_add(ctrl, dlg, str);
            }
            dlg_update_done(ctrl, dlg);
        }
     } else if (event == EVENT_ACTION) {
        if (ctrl == ccd->button) {
-           char str[100];
+           char *str;
            int i, n;
-           dlg_editbox_get(ccd->editbox, dlg, str, sizeof(str));
+           str = dlg_editbox_get(ccd->editbox, dlg);
            n = atoi(str);
+           sfree(str);
            for (i = 0; i < 128; i++) {
                if (dlg_listbox_issel(ccd->listbox, dlg, i))
-                   cfg->wordness[i] = n;
+                   conf_set_int_int(conf, CONF_wordness, i, n);
            }
            dlg_refresh(ccd->listbox, dlg);
        }
@@ -438,10 +798,10 @@ static const char *const colours[] = {
 static void colour_handler(union control *ctrl, void *dlg,
                            void *data, int event)
 {
-    Config *cfg = (Config *)data;
+    Conf *conf = (Conf *)data;
     struct colour_data *cd =
        (struct colour_data *)ctrl->generic.context.p;
-    int update = FALSE, r, g, b;
+    int update = FALSE, clear = FALSE, r, g, b;
 
     if (event == EVENT_REFRESH) {
        if (ctrl == cd->listbox) {
@@ -451,40 +811,43 @@ static void colour_handler(union control *ctrl, void *dlg,
            for (i = 0; i < lenof(colours); i++)
                dlg_listbox_add(ctrl, dlg, colours[i]);
            dlg_update_done(ctrl, dlg);
-           dlg_editbox_set(cd->redit, dlg, "");
-           dlg_editbox_set(cd->gedit, dlg, "");
-           dlg_editbox_set(cd->bedit, dlg, "");
+           clear = TRUE;
+           update = TRUE;
        }
     } else if (event == EVENT_SELCHANGE) {
        if (ctrl == cd->listbox) {
            /* The user has selected a colour. Update the RGB text. */
            int i = dlg_listbox_index(ctrl, dlg);
            if (i < 0) {
-               dlg_beep(dlg);
-               return;
+               clear = TRUE;
+           } else {
+               clear = FALSE;
+               r = conf_get_int_int(conf, CONF_colours, i*3+0);
+               g = conf_get_int_int(conf, CONF_colours, i*3+1);
+               b = conf_get_int_int(conf, CONF_colours, i*3+2);
            }
-           r = cfg->colours[i][0];
-           g = cfg->colours[i][1];
-           b = cfg->colours[i][2];
            update = TRUE;
        }
     } else if (event == EVENT_VALCHANGE) {
        if (ctrl == cd->redit || ctrl == cd->gedit || ctrl == cd->bedit) {
            /* The user has changed the colour using the edit boxes. */
-           char buf[80];
+           char *str;
            int i, cval;
 
-           dlg_editbox_get(ctrl, dlg, buf, lenof(buf));
-           cval = atoi(buf) & 255;
+           str = dlg_editbox_get(ctrl, dlg);
+           cval = atoi(str);
+           sfree(str);
+           if (cval > 255) cval = 255;
+           if (cval < 0)   cval = 0;
 
            i = dlg_listbox_index(cd->listbox, dlg);
            if (i >= 0) {
                if (ctrl == cd->redit)
-                   cfg->colours[i][0] = cval;
+                   conf_set_int_int(conf, CONF_colours, i*3+0, cval);
                else if (ctrl == cd->gedit)
-                   cfg->colours[i][1] = cval;
+                   conf_set_int_int(conf, CONF_colours, i*3+1, cval);
                else if (ctrl == cd->bedit)
-                   cfg->colours[i][2] = cval;
+                   conf_set_int_int(conf, CONF_colours, i*3+2, cval);
            }
        }
     } else if (event == EVENT_ACTION) {
@@ -500,9 +863,9 @@ static void colour_handler(union control *ctrl, void *dlg,
             * pick up the results.
             */
            dlg_coloursel_start(ctrl, dlg,
-                               cfg->colours[i][0],
-                               cfg->colours[i][1],
-                               cfg->colours[i][2]);
+                               conf_get_int_int(conf, CONF_colours, i*3+0),
+                               conf_get_int_int(conf, CONF_colours, i*3+1),
+                               conf_get_int_int(conf, CONF_colours, i*3+2));
        }
     } else if (event == EVENT_CALLBACK) {
        if (ctrl == cd->button) {
@@ -513,19 +876,114 @@ static void colour_handler(union control *ctrl, void *dlg,
             * selector did nothing (user hit Cancel, for example).
             */
            if (dlg_coloursel_results(ctrl, dlg, &r, &g, &b)) {
-               cfg->colours[i][0] = r;
-               cfg->colours[i][1] = g;
-               cfg->colours[i][2] = b;
+               conf_set_int_int(conf, CONF_colours, i*3+0, r);
+               conf_set_int_int(conf, CONF_colours, i*3+1, g);
+               conf_set_int_int(conf, CONF_colours, i*3+2, b);
+               clear = FALSE;
                update = TRUE;
            }
        }
     }
 
     if (update) {
-       char buf[40];
-       sprintf(buf, "%d", r); dlg_editbox_set(cd->redit, dlg, buf);
-       sprintf(buf, "%d", g); dlg_editbox_set(cd->gedit, dlg, buf);
-       sprintf(buf, "%d", b); dlg_editbox_set(cd->bedit, dlg, buf);
+       if (clear) {
+           dlg_editbox_set(cd->redit, dlg, "");
+           dlg_editbox_set(cd->gedit, dlg, "");
+           dlg_editbox_set(cd->bedit, dlg, "");
+       } else {
+           char buf[40];
+           sprintf(buf, "%d", r); dlg_editbox_set(cd->redit, dlg, buf);
+           sprintf(buf, "%d", g); dlg_editbox_set(cd->gedit, dlg, buf);
+           sprintf(buf, "%d", b); dlg_editbox_set(cd->bedit, dlg, buf);
+       }
+    }
+}
+
+struct ttymodes_data {
+    union control *modelist, *valradio, *valbox;
+    union control *addbutton, *rembutton, *listbox;
+};
+
+static void ttymodes_handler(union control *ctrl, void *dlg,
+                            void *data, int event)
+{
+    Conf *conf = (Conf *)data;
+    struct ttymodes_data *td =
+       (struct ttymodes_data *)ctrl->generic.context.p;
+
+    if (event == EVENT_REFRESH) {
+       if (ctrl == td->listbox) {
+           char *key, *val;
+           dlg_update_start(ctrl, dlg);
+           dlg_listbox_clear(ctrl, dlg);
+           for (val = conf_get_str_strs(conf, CONF_ttymodes, NULL, &key);
+                val != NULL;
+                val = conf_get_str_strs(conf, CONF_ttymodes, key, &key)) {
+               char *disp = dupprintf("%s\t%s", key,
+                                      (val[0] == 'A') ? "(auto)" : val+1);
+               dlg_listbox_add(ctrl, dlg, disp);
+               sfree(disp);
+           }
+           dlg_update_done(ctrl, dlg);
+       } else if (ctrl == td->modelist) {
+           int i;
+           dlg_update_start(ctrl, dlg);
+           dlg_listbox_clear(ctrl, dlg);
+           for (i = 0; ttymodes[i]; i++)
+               dlg_listbox_add(ctrl, dlg, ttymodes[i]);
+           dlg_listbox_select(ctrl, dlg, 0); /* *shrug* */
+           dlg_update_done(ctrl, dlg);
+       } else if (ctrl == td->valradio) {
+           dlg_radiobutton_set(ctrl, dlg, 0);
+       }
+    } else if (event == EVENT_ACTION) {
+       if (ctrl == td->addbutton) {
+           int ind = dlg_listbox_index(td->modelist, dlg);
+           if (ind >= 0) {
+               char type = dlg_radiobutton_get(td->valradio, dlg) ? 'V' : 'A';
+               const char *key;
+               char *str, *val;
+               /* Construct new entry */
+               key = ttymodes[ind];
+               str = dlg_editbox_get(td->valbox, dlg);
+               val = dupprintf("%c%s", type, str);
+               sfree(str);
+               conf_set_str_str(conf, CONF_ttymodes, key, val);
+               sfree(val);
+               dlg_refresh(td->listbox, dlg);
+           } else
+               dlg_beep(dlg);
+       } else if (ctrl == td->rembutton) {
+           int i = 0;
+           char *key, *val;
+           int multisel = dlg_listbox_index(td->listbox, dlg) < 0;
+           for (val = conf_get_str_strs(conf, CONF_ttymodes, NULL, &key);
+                val != NULL;
+                val = conf_get_str_strs(conf, CONF_ttymodes, key, &key)) {
+               if (dlg_listbox_issel(td->listbox, dlg, i)) {
+                   if (!multisel) {
+                       /* Populate controls with entry we're about to
+                        * delete, for ease of editing.
+                        * (If multiple entries were selected, don't
+                        * touch the controls.) */
+                       int ind = 0;
+                       val++;
+                       while (ttymodes[ind]) {
+                           if (!strcmp(ttymodes[ind], key))
+                               break;
+                           ind++;
+                       }
+                       dlg_listbox_select(td->modelist, dlg, ind);
+                       dlg_radiobutton_set(td->valradio, dlg,
+                                           (*val == 'V'));
+                       dlg_editbox_set(td->valbox, dlg, val+1);
+                   }
+                   conf_del_str_str(conf, CONF_ttymodes, key);
+               }
+               i++;
+           }
+           dlg_refresh(td->listbox, dlg);
+       }
     }
 }
 
@@ -536,84 +994,67 @@ struct environ_data {
 static void environ_handler(union control *ctrl, void *dlg,
                            void *data, int event)
 {
-    Config *cfg = (Config *)data;
+    Conf *conf = (Conf *)data;
     struct environ_data *ed =
        (struct environ_data *)ctrl->generic.context.p;
 
     if (event == EVENT_REFRESH) {
        if (ctrl == ed->listbox) {
-           char *p = cfg->environmt;
+           char *key, *val;
            dlg_update_start(ctrl, dlg);
            dlg_listbox_clear(ctrl, dlg);
-           while (*p) {
+           for (val = conf_get_str_strs(conf, CONF_environmt, NULL, &key);
+                val != NULL;
+                val = conf_get_str_strs(conf, CONF_environmt, key, &key)) {
+               char *p = dupprintf("%s\t%s", key, val);
                dlg_listbox_add(ctrl, dlg, p);
-               p += strlen(p) + 1;
+               sfree(p);
            }
            dlg_update_done(ctrl, dlg);
        }
     } else if (event == EVENT_ACTION) {
        if (ctrl == ed->addbutton) {
-           char str[sizeof(cfg->environmt)];
-           char *p;
-           dlg_editbox_get(ed->varbox, dlg, str, sizeof(str)-1);
-           if (!*str) {
+           char *key, *val, *str;
+           key = dlg_editbox_get(ed->varbox, dlg);
+           if (!*key) {
+               sfree(key);
                dlg_beep(dlg);
                return;
            }
-           p = str + strlen(str);
-           *p++ = '\t';
-           dlg_editbox_get(ed->valbox, dlg, p, sizeof(str)-1 - (p - str));
-           if (!*p) {
+           val = dlg_editbox_get(ed->valbox, dlg);
+           if (!*val) {
+               sfree(key);
+               sfree(val);
                dlg_beep(dlg);
                return;
            }
-           p = cfg->environmt;
-           while (*p) {
-               while (*p)
-                   p++;
-               p++;
-           }
-           if ((p - cfg->environmt) + strlen(str) + 2 <
-               sizeof(cfg->environmt)) {
-               strcpy(p, str);
-               p[strlen(str) + 1] = '\0';
-               dlg_listbox_add(ed->listbox, dlg, str);
-               dlg_editbox_set(ed->varbox, dlg, "");
-               dlg_editbox_set(ed->valbox, dlg, "");
-           } else {
-               dlg_error_msg(dlg, "Environment too big");
-           }
+           conf_set_str_str(conf, CONF_environmt, key, val);
+           str = dupcat(key, "\t", val, NULL);
+           dlg_editbox_set(ed->varbox, dlg, "");
+           dlg_editbox_set(ed->valbox, dlg, "");
+           sfree(str);
+           sfree(key);
+           sfree(val);
+           dlg_refresh(ed->listbox, dlg);
        } else if (ctrl == ed->rembutton) {
            int i = dlg_listbox_index(ed->listbox, dlg);
            if (i < 0) {
                dlg_beep(dlg);
            } else {
-               char *p, *q;
-
-               dlg_listbox_del(ed->listbox, dlg, i);
-               p = cfg->environmt;
-               while (i > 0) {
-                   if (!*p)
-                       goto disaster;
-                   while (*p)
-                       p++;
-                   p++;
-                   i--;
+               char *key, *val;
+
+               key = conf_get_str_nthstrkey(conf, CONF_environmt, i);
+               if (key) {
+                   /* Populate controls with the entry we're about to delete
+                    * for ease of editing */
+                   val = conf_get_str_str(conf, CONF_environmt, key);
+                   dlg_editbox_set(ed->varbox, dlg, key);
+                   dlg_editbox_set(ed->valbox, dlg, val);
+                   /* And delete it */
+                   conf_del_str_str(conf, CONF_environmt, key);
                }
-               q = p;
-               if (!*p)
-                   goto disaster;
-               while (*p)
-                   p++;
-               p++;
-               while (*p) {
-                   while (*p)
-                       *q++ = *p++;
-                   *q++ = *p++;
-               }
-               *q = '\0';
-               disaster:;
            }
+           dlg_refresh(ed->listbox, dlg);
        }
     }
 }
@@ -621,23 +1062,33 @@ static void environ_handler(union control *ctrl, void *dlg,
 struct portfwd_data {
     union control *addbutton, *rembutton, *listbox;
     union control *sourcebox, *destbox, *direction;
+#ifndef NO_IPV6
+    union control *addressfamily;
+#endif
 };
 
 static void portfwd_handler(union control *ctrl, void *dlg,
                            void *data, int event)
 {
-    Config *cfg = (Config *)data;
+    Conf *conf = (Conf *)data;
     struct portfwd_data *pfd =
        (struct portfwd_data *)ctrl->generic.context.p;
 
     if (event == EVENT_REFRESH) {
        if (ctrl == pfd->listbox) {
-           char *p = cfg->portfwd;
+           char *key, *val;
            dlg_update_start(ctrl, dlg);
            dlg_listbox_clear(ctrl, dlg);
-           while (*p) {
+           for (val = conf_get_str_strs(conf, CONF_portfwd, NULL, &key);
+                val != NULL;
+                val = conf_get_str_strs(conf, CONF_portfwd, key, &key)) {
+               char *p;
+                if (!strcmp(val, "D"))
+                    p = dupprintf("D%s\t", key+1);
+                else
+                    p = dupprintf("%s\t%s", key, val);
                dlg_listbox_add(ctrl, dlg, p);
-               p += strlen(p) + 1;
+               sfree(p);
            }
            dlg_update_done(ctrl, dlg);
        } else if (ctrl == pfd->direction) {
@@ -645,103 +1096,140 @@ static void portfwd_handler(union control *ctrl, void *dlg,
             * Default is Local.
             */
            dlg_radiobutton_set(ctrl, dlg, 0);
+#ifndef NO_IPV6
+       } else if (ctrl == pfd->addressfamily) {
+           dlg_radiobutton_set(ctrl, dlg, 0);
+#endif
        }
     } else if (event == EVENT_ACTION) {
        if (ctrl == pfd->addbutton) {
-           char str[sizeof(cfg->portfwd)];
-           char *p;
-           int whichbutton = dlg_radiobutton_get(pfd->direction, dlg);
+           char *family, *type, *src, *key, *val;
+           int whichbutton;
+
+#ifndef NO_IPV6
+           whichbutton = dlg_radiobutton_get(pfd->addressfamily, dlg);
+           if (whichbutton == 1)
+               family = "4";
+           else if (whichbutton == 2)
+               family = "6";
+           else
+               family = "";
+#endif
+
+           whichbutton = dlg_radiobutton_get(pfd->direction, dlg);
            if (whichbutton == 0)
-               str[0] = 'L';
+               type = "L";
            else if (whichbutton == 1)
-               str[0] = 'R';
+               type = "R";
            else
-               str[0] = 'D';
-           dlg_editbox_get(pfd->sourcebox, dlg, str+1, sizeof(str) - 2);
-           if (!str[1]) {
+               type = "D";
+
+           src = dlg_editbox_get(pfd->sourcebox, dlg);
+           if (!*src) {
                dlg_error_msg(dlg, "You need to specify a source port number");
+               sfree(src);
                return;
            }
-           p = str + strlen(str);
-           if (str[0] != 'D') {
-               *p++ = '\t';
-               dlg_editbox_get(pfd->destbox, dlg, p,
-                               sizeof(str)-1 - (p - str));
-               if (!*p || !strchr(p, ':')) {
+           if (*type != 'D') {
+               val = dlg_editbox_get(pfd->destbox, dlg);
+               if (!*val || !strchr(val, ':')) {
                    dlg_error_msg(dlg,
                                  "You need to specify a destination address\n"
                                  "in the form \"host.name:port\"");
+                   sfree(src);
+                   sfree(val);
                    return;
                }
-           } else
-               *p = '\0';
-           p = cfg->portfwd;
-           while (*p) {
-               while (*p)
-                   p++;
-               p++;
-           }
-           if ((p - cfg->portfwd) + strlen(str) + 2 <
-               sizeof(cfg->portfwd)) {
-               strcpy(p, str);
-               p[strlen(str) + 1] = '\0';
-               dlg_listbox_add(pfd->listbox, dlg, str);
-               dlg_editbox_set(pfd->sourcebox, dlg, "");
-               dlg_editbox_set(pfd->destbox, dlg, "");
            } else {
-               dlg_error_msg(dlg, "Too many forwardings");
+                type = "L";
+               val = dupstr("D");     /* special case */
+            }
+
+           key = dupcat(family, type, src, NULL);
+           sfree(src);
+
+           if (conf_get_str_str_opt(conf, CONF_portfwd, key)) {
+               dlg_error_msg(dlg, "Specified forwarding already exists");
+           } else {
+               conf_set_str_str(conf, CONF_portfwd, key, val);
            }
+
+           sfree(key);
+           sfree(val);
+           dlg_refresh(pfd->listbox, dlg);
        } else if (ctrl == pfd->rembutton) {
            int i = dlg_listbox_index(pfd->listbox, dlg);
-           if (i < 0)
+           if (i < 0) {
                dlg_beep(dlg);
-           else {
-               char *p, *q;
-
-               dlg_listbox_del(pfd->listbox, dlg, i);
-               p = cfg->portfwd;
-               while (i > 0) {
-                   if (!*p)
-                       goto disaster2;
-                   while (*p)
+           } else {
+               char *key, *val, *p;
+
+               key = conf_get_str_nthstrkey(conf, CONF_portfwd, i);
+               if (key) {
+                   static const char *const afs = "A46";
+                   static const char *const dirs = "LRD";
+                   char *afp;
+                   int dir;
+#ifndef NO_IPV6
+                   int idx;
+#endif
+
+                   /* Populate controls with the entry we're about to delete
+                    * for ease of editing */
+                   p = key;
+
+                   afp = strchr(afs, *p);
+#ifndef NO_IPV6
+                   idx = afp ? afp-afs : 0;
+#endif
+                   if (afp)
                        p++;
+#ifndef NO_IPV6
+                   dlg_radiobutton_set(pfd->addressfamily, dlg, idx);
+#endif
+
+                   dir = *p;
+
+                    val = conf_get_str_str(conf, CONF_portfwd, key);
+                   if (!strcmp(val, "D")) {
+                        dir = 'D';
+                       val = "";
+                   }
+
+                   dlg_radiobutton_set(pfd->direction, dlg,
+                                       strchr(dirs, dir) - dirs);
                    p++;
-                   i--;
-               }
-               q = p;
-               if (!*p)
-                   goto disaster2;
-               while (*p)
-                   p++;
-               p++;
-               while (*p) {
-                   while (*p)
-                       *q++ = *p++;
-                   *q++ = *p++;
+
+                   dlg_editbox_set(pfd->sourcebox, dlg, p);
+                   dlg_editbox_set(pfd->destbox, dlg, val);
+                   /* And delete it */
+                   conf_del_str_str(conf, CONF_portfwd, key);
                }
-               *q = '\0';
-               disaster2:;
            }
+           dlg_refresh(pfd->listbox, dlg);
        }
     }
 }
 
-void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
-                     int midsession, int protocol)
+void setup_config_box(struct controlbox *b, int midsession,
+                     int protocol, int protcfginfo)
 {
     struct controlset *s;
     struct sessionsaver_data *ssd;
     struct charclass_data *ccd;
     struct colour_data *cd;
+    struct ttymodes_data *td;
     struct environ_data *ed;
     struct portfwd_data *pfd;
     union control *c;
     char *str;
 
     ssd = (struct sessionsaver_data *)
-       ctrl_alloc(b, sizeof(struct sessionsaver_data));
+       ctrl_alloc_with_free(b, sizeof(struct sessionsaver_data),
+                             sessionsaver_data_free);
     memset(ssd, 0, sizeof(*ssd));
-    ssd->sesslist = (midsession ? NULL : sesslist);
+    ssd->savedsession = dupstr("");
+    ssd->midsession = midsession;
 
     /*
      * The standard panel that appears at the bottom of all panels:
@@ -771,78 +1259,101 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
     sfree(str);
 
     if (!midsession) {
+       struct hostport *hp = (struct hostport *)
+           ctrl_alloc(b, sizeof(struct hostport));
+
        s = ctrl_getset(b, "Session", "hostport",
-                       "Specify your connection by host name or IP address");
+                       "Specify the destination you want to connect to");
        ctrl_columns(s, 2, 75, 25);
-       c = ctrl_editbox(s, "Host Name (or IP address)", 'n', 100,
+       c = ctrl_editbox(s, HOST_BOX_TITLE, 'n', 100,
                         HELPCTX(session_hostname),
-                        dlg_stdeditbox_handler, I(offsetof(Config,host)),
-                        I(sizeof(((Config *)0)->host)));
+                        config_host_handler, I(0), I(0));
        c->generic.column = 0;
-       c = ctrl_editbox(s, "Port", 'p', 100, HELPCTX(session_hostname),
-                        dlg_stdeditbox_handler,
-                        I(offsetof(Config,port)), I(-1));
+       hp->host = c;
+       c = ctrl_editbox(s, PORT_BOX_TITLE, 'p', 100,
+                        HELPCTX(session_hostname),
+                        config_port_handler, I(0), I(0));
        c->generic.column = 1;
+       hp->port = c;
        ctrl_columns(s, 1, 100);
-       if (backends[3].name == NULL) {
-           ctrl_radiobuttons(s, "Protocol:", NO_SHORTCUT, 3,
+
+       if (!backend_from_proto(PROT_SSH)) {
+           ctrl_radiobuttons(s, "Connection type:", NO_SHORTCUT, 3,
                              HELPCTX(session_hostname),
-                             protocolbuttons_handler, P(c),
-                             "Raw", 'r', I(PROT_RAW),
+                             config_protocolbuttons_handler, P(hp),
+                             "Raw", 'w', I(PROT_RAW),
                              "Telnet", 't', I(PROT_TELNET),
                              "Rlogin", 'i', I(PROT_RLOGIN),
                              NULL);
        } else {
-           ctrl_radiobuttons(s, "Protocol:", NO_SHORTCUT, 4,
+           ctrl_radiobuttons(s, "Connection type:", NO_SHORTCUT, 4,
                              HELPCTX(session_hostname),
-                             protocolbuttons_handler, P(c),
-                             "Raw", 'r', I(PROT_RAW),
+                             config_protocolbuttons_handler, P(hp),
+                             "Raw", 'w', I(PROT_RAW),
                              "Telnet", 't', I(PROT_TELNET),
                              "Rlogin", 'i', I(PROT_RLOGIN),
                              "SSH", 's', I(PROT_SSH),
                              NULL);
        }
+    }
 
-       s = ctrl_getset(b, "Session", "savedsessions",
-                       "Load, save or delete a stored session");
-       ctrl_columns(s, 2, 75, 25);
-       ssd->sesslist = sesslist;
-       ssd->editbox = ctrl_editbox(s, "Saved Sessions", 'e', 100,
-                                   HELPCTX(session_saved),
-                                   sessionsaver_handler, P(ssd), P(NULL));
-       ssd->editbox->generic.column = 0;
-       /* Reset columns so that the buttons are alongside the list, rather
-        * than alongside that edit box. */
-       ctrl_columns(s, 1, 100);
-       ctrl_columns(s, 2, 75, 25);
-       ssd->listbox = ctrl_listbox(s, NULL, NO_SHORTCUT,
-                                   HELPCTX(session_saved),
-                                   sessionsaver_handler, P(ssd));
-       ssd->listbox->generic.column = 0;
-       ssd->listbox->listbox.height = 7;
+    /*
+     * The Load/Save panel is available even in mid-session.
+     */
+    s = ctrl_getset(b, "Session", "savedsessions",
+                   midsession ? "Save the current session settings" :
+                   "Load, save or delete a stored session");
+    ctrl_columns(s, 2, 75, 25);
+    get_sesslist(&ssd->sesslist, TRUE);
+    ssd->editbox = ctrl_editbox(s, "Saved Sessions", 'e', 100,
+                               HELPCTX(session_saved),
+                               sessionsaver_handler, P(ssd), P(NULL));
+    ssd->editbox->generic.column = 0;
+    /* Reset columns so that the buttons are alongside the list, rather
+     * than alongside that edit box. */
+    ctrl_columns(s, 1, 100);
+    ctrl_columns(s, 2, 75, 25);
+    ssd->listbox = ctrl_listbox(s, NULL, NO_SHORTCUT,
+                               HELPCTX(session_saved),
+                               sessionsaver_handler, P(ssd));
+    ssd->listbox->generic.column = 0;
+    ssd->listbox->listbox.height = 7;
+    if (!midsession) {
        ssd->loadbutton = ctrl_pushbutton(s, "Load", 'l',
                                          HELPCTX(session_saved),
                                          sessionsaver_handler, P(ssd));
        ssd->loadbutton->generic.column = 1;
-       ssd->savebutton = ctrl_pushbutton(s, "Save", 'v',
-                                         HELPCTX(session_saved),
-                                         sessionsaver_handler, P(ssd));
-       ssd->savebutton->generic.column = 1;
+    } else {
+       /* We can't offer the Load button mid-session, as it would allow the
+        * user to load and subsequently save settings they can't see. (And
+        * also change otherwise immutable settings underfoot; that probably
+        * shouldn't be a problem, but.) */
+       ssd->loadbutton = NULL;
+    }
+    /* "Save" button is permitted mid-session. */
+    ssd->savebutton = ctrl_pushbutton(s, "Save", 'v',
+                                     HELPCTX(session_saved),
+                                     sessionsaver_handler, P(ssd));
+    ssd->savebutton->generic.column = 1;
+    if (!midsession) {
        ssd->delbutton = ctrl_pushbutton(s, "Delete", 'd',
                                         HELPCTX(session_saved),
                                         sessionsaver_handler, P(ssd));
        ssd->delbutton->generic.column = 1;
-       ctrl_columns(s, 1, 100);
+    } else {
+       /* Disable the Delete button mid-session too, for UI consistency. */
+       ssd->delbutton = NULL;
     }
+    ctrl_columns(s, 1, 100);
 
     s = ctrl_getset(b, "Session", "otheropts", NULL);
-    c = ctrl_radiobuttons(s, "Close window on exit:", 'w', 4,
-                         HELPCTX(session_coe),
-                         dlg_stdradiobutton_handler,
-                         I(offsetof(Config, close_on_exit)),
-                         "Always", I(FORCE_ON),
-                         "Never", I(FORCE_OFF),
-                         "Only on clean exit", I(AUTO), NULL);
+    ctrl_radiobuttons(s, "Close window on exit:", 'x', 4,
+                      HELPCTX(session_coe),
+                      conf_radiobutton_handler,
+                      I(CONF_close_on_exit),
+                      "Always", I(FORCE_ON),
+                      "Never", I(FORCE_OFF),
+                      "Only on clean exit", I(AUTO), NULL);
 
     /*
      * The Session/Logging panel.
@@ -855,35 +1366,54 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
      * logging can sensibly be available.
      */
     {
-       char *sshlogname;
+       char *sshlogname, *sshrawlogname;
        if ((midsession && protocol == PROT_SSH) ||
-           (!midsession && backends[3].name != NULL))
-           sshlogname = "Log SSH packet data";
-       else
-           sshlogname = NULL;         /* this will disable the button */
-       ctrl_radiobuttons(s, "Session logging:", NO_SHORTCUT, 1,
+           (!midsession && backend_from_proto(PROT_SSH))) {
+           sshlogname = "SSH packets";
+           sshrawlogname = "SSH packets and raw data";
+        } else {
+           sshlogname = NULL;         /* this will disable both buttons */
+           sshrawlogname = NULL;      /* this will just placate optimisers */
+        }
+       ctrl_radiobuttons(s, "Session logging:", NO_SHORTCUT, 2,
                          HELPCTX(logging_main),
-                         dlg_stdradiobutton_handler,
-                         I(offsetof(Config, logtype)),
-                         "Logging turned off completely", 't', I(LGTYP_NONE),
-                         "Log printable output only", 'p', I(LGTYP_ASCII),
-                         "Log all session output", 'l', I(LGTYP_DEBUG),
+                         loggingbuttons_handler,
+                         I(CONF_logtype),
+                         "None", 't', I(LGTYP_NONE),
+                         "Printable output", 'p', I(LGTYP_ASCII),
+                         "All session output", 'l', I(LGTYP_DEBUG),
                          sshlogname, 's', I(LGTYP_PACKETS),
+                         sshrawlogname, 'r', I(LGTYP_SSHRAW),
                          NULL);
     }
     ctrl_filesel(s, "Log file name:", 'f',
                 NULL, TRUE, "Select session log file name",
                 HELPCTX(logging_filename),
-                dlg_stdfilesel_handler, I(offsetof(Config, logfilename)));
+                conf_filesel_handler, I(CONF_logfilename));
     ctrl_text(s, "(Log file name can contain &Y, &M, &D for date,"
              " &T for time, and &H for host name)",
              HELPCTX(logging_filename));
     ctrl_radiobuttons(s, "What to do if the log file already exists:", 'e', 1,
                      HELPCTX(logging_exists),
-                     dlg_stdradiobutton_handler, I(offsetof(Config,logxfovr)),
+                     conf_radiobutton_handler, I(CONF_logxfovr),
                      "Always overwrite it", I(LGXF_OVR),
                      "Always append to the end of it", I(LGXF_APN),
                      "Ask the user every time", I(LGXF_ASK), NULL);
+    ctrl_checkbox(s, "Flush log file frequently", 'u',
+                HELPCTX(logging_flush),
+                conf_checkbox_handler, I(CONF_logflush));
+
+    if ((midsession && protocol == PROT_SSH) ||
+       (!midsession && backend_from_proto(PROT_SSH))) {
+       s = ctrl_getset(b, "Session/Logging", "ssh",
+                       "Options specific to SSH packet logging");
+       ctrl_checkbox(s, "Omit known password fields", 'k',
+                     HELPCTX(logging_ssh_omit_password),
+                     conf_checkbox_handler, I(CONF_logomitpass));
+       ctrl_checkbox(s, "Omit session data", 'd',
+                     HELPCTX(logging_ssh_omit_data),
+                     conf_checkbox_handler, I(CONF_logomitdata));
+    }
 
     /*
      * The Terminal panel.
@@ -893,34 +1423,36 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
     s = ctrl_getset(b, "Terminal", "general", "Set various terminal options");
     ctrl_checkbox(s, "Auto wrap mode initially on", 'w',
                  HELPCTX(terminal_autowrap),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,wrap_mode)));
+                 conf_checkbox_handler, I(CONF_wrap_mode));
     ctrl_checkbox(s, "DEC Origin Mode initially on", 'd',
                  HELPCTX(terminal_decom),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,dec_om)));
+                 conf_checkbox_handler, I(CONF_dec_om));
     ctrl_checkbox(s, "Implicit CR in every LF", 'r',
                  HELPCTX(terminal_lfhascr),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,lfhascr)));
+                 conf_checkbox_handler, I(CONF_lfhascr));
+    ctrl_checkbox(s, "Implicit LF in every CR", 'f',
+                 HELPCTX(terminal_crhaslf),
+                 conf_checkbox_handler, I(CONF_crhaslf));
     ctrl_checkbox(s, "Use background colour to erase screen", 'e',
                  HELPCTX(terminal_bce),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,bce)));
+                 conf_checkbox_handler, I(CONF_bce));
     ctrl_checkbox(s, "Enable blinking text", 'n',
                  HELPCTX(terminal_blink),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,blinktext)));
+                 conf_checkbox_handler, I(CONF_blinktext));
     ctrl_editbox(s, "Answerback to ^E:", 's', 100,
                 HELPCTX(terminal_answerback),
-                dlg_stdeditbox_handler, I(offsetof(Config,answerback)),
-                I(sizeof(((Config *)0)->answerback)));
+                conf_editbox_handler, I(CONF_answerback), I(1));
 
     s = ctrl_getset(b, "Terminal", "ldisc", "Line discipline options");
     ctrl_radiobuttons(s, "Local echo:", 'l', 3,
                      HELPCTX(terminal_localecho),
-                     dlg_stdradiobutton_handler,I(offsetof(Config,localecho)),
+                     conf_radiobutton_handler,I(CONF_localecho),
                      "Auto", I(AUTO),
                      "Force on", I(FORCE_ON),
                      "Force off", I(FORCE_OFF), NULL);
     ctrl_radiobuttons(s, "Local line editing:", 't', 3,
                      HELPCTX(terminal_localedit),
-                     dlg_stdradiobutton_handler,I(offsetof(Config,localedit)),
+                     conf_radiobutton_handler,I(CONF_localedit),
                      "Auto", I(AUTO),
                      "Force on", I(FORCE_ON),
                      "Force off", I(FORCE_OFF), NULL);
@@ -940,18 +1472,18 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                    "Change the sequences sent by:");
     ctrl_radiobuttons(s, "The Backspace key", 'b', 2,
                      HELPCTX(keyboard_backspace),
-                     dlg_stdradiobutton_handler,
-                     I(offsetof(Config, bksp_is_delete)),
+                     conf_radiobutton_handler,
+                     I(CONF_bksp_is_delete),
                      "Control-H", I(0), "Control-? (127)", I(1), NULL);
     ctrl_radiobuttons(s, "The Home and End keys", 'e', 2,
                      HELPCTX(keyboard_homeend),
-                     dlg_stdradiobutton_handler,
-                     I(offsetof(Config, rxvt_homeend)),
+                     conf_radiobutton_handler,
+                     I(CONF_rxvt_homeend),
                      "Standard", I(0), "rxvt", I(1), NULL);
     ctrl_radiobuttons(s, "The Function keys and keypad", 'f', 3,
                      HELPCTX(keyboard_funkeys),
-                     dlg_stdradiobutton_handler,
-                     I(offsetof(Config, funky_type)),
+                     conf_radiobutton_handler,
+                     I(CONF_funky_type),
                      "ESC[n~", I(0), "Linux", I(1), "Xterm R6", I(2),
                      "VT400", I(3), "VT100+", I(4), "SCO", I(5), NULL);
 
@@ -959,8 +1491,8 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                    "Application keypad settings:");
     ctrl_radiobuttons(s, "Initial state of cursor keys:", 'r', 3,
                      HELPCTX(keyboard_appcursor),
-                     dlg_stdradiobutton_handler,
-                     I(offsetof(Config, app_cursor)),
+                     conf_radiobutton_handler,
+                     I(CONF_app_cursor),
                      "Normal", I(0), "Application", I(1), NULL);
     ctrl_radiobuttons(s, "Initial state of numeric keypad:", 'n', 3,
                      HELPCTX(keyboard_appkeypad),
@@ -977,7 +1509,7 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
     s = ctrl_getset(b, "Terminal/Bell", "style", "Set the style of bell");
     ctrl_radiobuttons(s, "Action to happen when a bell occurs:", 'b', 1,
                      HELPCTX(bell_style),
-                     dlg_stdradiobutton_handler, I(offsetof(Config, beep)),
+                     conf_radiobutton_handler, I(CONF_beep),
                      "None (bell disabled)", I(BELL_DISABLED),
                      "Make default system alert sound", I(BELL_DEFAULT),
                      "Visual bell (flash window)", I(BELL_VISUAL), NULL);
@@ -986,19 +1518,19 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                    "Control the bell overload behaviour");
     ctrl_checkbox(s, "Bell is temporarily disabled when over-used", 'd',
                  HELPCTX(bell_overload),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,bellovl)));
+                 conf_checkbox_handler, I(CONF_bellovl));
     ctrl_editbox(s, "Over-use means this many bells...", 'm', 20,
                 HELPCTX(bell_overload),
-                dlg_stdeditbox_handler, I(offsetof(Config,bellovl_n)), I(-1));
+                conf_editbox_handler, I(CONF_bellovl_n), I(-1));
     ctrl_editbox(s, "... in this many seconds", 't', 20,
                 HELPCTX(bell_overload),
-                dlg_stdeditbox_handler, I(offsetof(Config,bellovl_t)),
+                conf_editbox_handler, I(CONF_bellovl_t),
                 I(-TICKSPERSEC));
     ctrl_text(s, "The bell is re-enabled after a few seconds of silence.",
              HELPCTX(bell_overload));
     ctrl_editbox(s, "Seconds of silence required", 's', 20,
                 HELPCTX(bell_overload),
-                dlg_stdeditbox_handler, I(offsetof(Config,bellovl_s)),
+                conf_editbox_handler, I(CONF_bellovl_s),
                 I(-TICKSPERSEC));
 
     /*
@@ -1010,39 +1542,43 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
     s = ctrl_getset(b, "Terminal/Features", "main", NULL);
     ctrl_checkbox(s, "Disable application cursor keys mode", 'u',
                  HELPCTX(features_application),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,no_applic_c)));
+                 conf_checkbox_handler, I(CONF_no_applic_c));
     ctrl_checkbox(s, "Disable application keypad mode", 'k',
                  HELPCTX(features_application),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,no_applic_k)));
+                 conf_checkbox_handler, I(CONF_no_applic_k));
     ctrl_checkbox(s, "Disable xterm-style mouse reporting", 'x',
                  HELPCTX(features_mouse),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,no_mouse_rep)));
+                 conf_checkbox_handler, I(CONF_no_mouse_rep));
     ctrl_checkbox(s, "Disable remote-controlled terminal resizing", 's',
                  HELPCTX(features_resize),
-                 dlg_stdcheckbox_handler,
-                 I(offsetof(Config,no_remote_resize)));
+                 conf_checkbox_handler,
+                 I(CONF_no_remote_resize));
     ctrl_checkbox(s, "Disable switching to alternate terminal screen", 'w',
                  HELPCTX(features_altscreen),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,no_alt_screen)));
+                 conf_checkbox_handler, I(CONF_no_alt_screen));
     ctrl_checkbox(s, "Disable remote-controlled window title changing", 't',
                  HELPCTX(features_retitle),
-                 dlg_stdcheckbox_handler,
-                 I(offsetof(Config,no_remote_wintitle)));
-    ctrl_checkbox(s, "Disable remote window title querying (SECURITY)",
-                 'q', HELPCTX(features_qtitle), dlg_stdcheckbox_handler,
-                 I(offsetof(Config,no_remote_qtitle)));
+                 conf_checkbox_handler,
+                 I(CONF_no_remote_wintitle));
+    ctrl_radiobuttons(s, "Response to remote title query (SECURITY):", 'q', 3,
+                     HELPCTX(features_qtitle),
+                     conf_radiobutton_handler,
+                     I(CONF_remote_qtitle_action),
+                     "None", I(TITLE_NONE),
+                     "Empty string", I(TITLE_EMPTY),
+                     "Window title", I(TITLE_REAL), NULL);
     ctrl_checkbox(s, "Disable destructive backspace on server sending ^?",'b',
                  HELPCTX(features_dbackspace),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,no_dbackspace)));
+                 conf_checkbox_handler, I(CONF_no_dbackspace));
     ctrl_checkbox(s, "Disable remote-controlled character set configuration",
-                 'r', HELPCTX(features_charset), dlg_stdcheckbox_handler,
-                 I(offsetof(Config,no_remote_charset)));
+                 'r', HELPCTX(features_charset), conf_checkbox_handler,
+                 I(CONF_no_remote_charset));
     ctrl_checkbox(s, "Disable Arabic text shaping",
-                 'l', HELPCTX(features_arabicshaping), dlg_stdcheckbox_handler,
-                 I(offsetof(Config, arabicshaping)));
+                 'l', HELPCTX(features_arabicshaping), conf_checkbox_handler,
+                 I(CONF_arabicshaping));
     ctrl_checkbox(s, "Disable bidirectional text display",
-                 'd', HELPCTX(features_bidi), dlg_stdcheckbox_handler,
-                 I(offsetof(Config, bidi)));
+                 'd', HELPCTX(features_bidi), conf_checkbox_handler,
+                 I(CONF_bidi));
 
     /*
      * The Window panel.
@@ -1053,13 +1589,13 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
 
     s = ctrl_getset(b, "Window", "size", "Set the size of the window");
     ctrl_columns(s, 2, 50, 50);
-    c = ctrl_editbox(s, "Rows", 'r', 100,
+    c = ctrl_editbox(s, "Columns", 'm', 100,
                     HELPCTX(window_size),
-                    dlg_stdeditbox_handler, I(offsetof(Config,height)),I(-1));
+                    conf_editbox_handler, I(CONF_width), I(-1));
     c->generic.column = 0;
-    c = ctrl_editbox(s, "Columns", 'm', 100,
+    c = ctrl_editbox(s, "Rows", 'r', 100,
                     HELPCTX(window_size),
-                    dlg_stdeditbox_handler, I(offsetof(Config,width)), I(-1));
+                    conf_editbox_handler, I(CONF_height),I(-1));
     c->generic.column = 1;
     ctrl_columns(s, 1, 100);
 
@@ -1067,20 +1603,20 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                    "Control the scrollback in the window");
     ctrl_editbox(s, "Lines of scrollback", 's', 50,
                 HELPCTX(window_scrollback),
-                dlg_stdeditbox_handler, I(offsetof(Config,savelines)), I(-1));
+                conf_editbox_handler, I(CONF_savelines), I(-1));
     ctrl_checkbox(s, "Display scrollbar", 'd',
                  HELPCTX(window_scrollback),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,scrollbar)));
+                 conf_checkbox_handler, I(CONF_scrollbar));
     ctrl_checkbox(s, "Reset scrollback on keypress", 'k',
                  HELPCTX(window_scrollback),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,scroll_on_key)));
+                 conf_checkbox_handler, I(CONF_scroll_on_key));
     ctrl_checkbox(s, "Reset scrollback on display activity", 'p',
                  HELPCTX(window_scrollback),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,scroll_on_disp)));
+                 conf_checkbox_handler, I(CONF_scroll_on_disp));
     ctrl_checkbox(s, "Push erased text into scrollback", 'e',
                  HELPCTX(window_erased),
-                 dlg_stdcheckbox_handler,
-                 I(offsetof(Config,erase_to_scrollback)));
+                 conf_checkbox_handler,
+                 I(CONF_erase_to_scrollback));
 
     /*
      * The Window/Appearance panel.
@@ -1093,33 +1629,33 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                    "Adjust the use of the cursor");
     ctrl_radiobuttons(s, "Cursor appearance:", NO_SHORTCUT, 3,
                      HELPCTX(appearance_cursor),
-                     dlg_stdradiobutton_handler,
-                     I(offsetof(Config, cursor_type)),
+                     conf_radiobutton_handler,
+                     I(CONF_cursor_type),
                      "Block", 'l', I(0),
                      "Underline", 'u', I(1),
                      "Vertical line", 'v', I(2), NULL);
     ctrl_checkbox(s, "Cursor blinks", 'b',
                  HELPCTX(appearance_cursor),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,blink_cur)));
+                 conf_checkbox_handler, I(CONF_blink_cur));
 
     s = ctrl_getset(b, "Window/Appearance", "font",
                    "Font settings");
     ctrl_fontsel(s, "Font used in the terminal window", 'n',
                 HELPCTX(appearance_font),
-                dlg_stdfontsel_handler, I(offsetof(Config, font)));
+                conf_fontsel_handler, I(CONF_font));
 
     s = ctrl_getset(b, "Window/Appearance", "mouse",
                    "Adjust the use of the mouse pointer");
     ctrl_checkbox(s, "Hide mouse pointer when typing in window", 'p',
                  HELPCTX(appearance_hidemouse),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,hide_mouseptr)));
+                 conf_checkbox_handler, I(CONF_hide_mouseptr));
 
     s = ctrl_getset(b, "Window/Appearance", "border",
                    "Adjust the window border");
-    ctrl_editbox(s, "Gap between text and window edge:", NO_SHORTCUT, 20,
+    ctrl_editbox(s, "Gap between text and window edge:", 'e', 20,
                 HELPCTX(appearance_border),
-                dlg_stdeditbox_handler,
-                I(offsetof(Config,window_border)), I(-1));
+                conf_editbox_handler,
+                I(CONF_window_border), I(-1));
 
     /*
      * The Window/Behaviour panel.
@@ -1132,17 +1668,16 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                    "Adjust the behaviour of the window title");
     ctrl_editbox(s, "Window title:", 't', 100,
                 HELPCTX(appearance_title),
-                dlg_stdeditbox_handler, I(offsetof(Config,wintitle)),
-                I(sizeof(((Config *)0)->wintitle)));
+                conf_editbox_handler, I(CONF_wintitle), I(1));
     ctrl_checkbox(s, "Separate window and icon titles", 'i',
                  HELPCTX(appearance_title),
-                 dlg_stdcheckbox_handler,
-                 I(CHECKBOX_INVERT | offsetof(Config,win_name_always)));
+                 conf_checkbox_handler,
+                 I(CHECKBOX_INVERT | CONF_win_name_always));
 
     s = ctrl_getset(b, "Window/Behaviour", "main", NULL);
     ctrl_checkbox(s, "Warn before closing window", 'w',
                  HELPCTX(behaviour_closewarn),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,warn_on_close)));
+                 conf_checkbox_handler, I(CONF_warn_on_close));
 
     /*
      * The Window/Translation panel.
@@ -1151,24 +1686,29 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                  "Options controlling character set translation");
 
     s = ctrl_getset(b, "Window/Translation", "trans",
-                   "Character set translation on received data");
-    ctrl_combobox(s, "Received data assumed to be in which character set:",
+                   "Character set translation");
+    ctrl_combobox(s, "Remote character set:",
                  'r', 100, HELPCTX(translation_codepage),
                  codepage_handler, P(NULL), P(NULL));
 
+    s = ctrl_getset(b, "Window/Translation", "tweaks", NULL);
+    ctrl_checkbox(s, "Treat CJK ambiguous characters as wide", 'w',
+                 HELPCTX(translation_cjk_ambig_wide),
+                 conf_checkbox_handler, I(CONF_cjk_ambig_wide));
+
     str = dupprintf("Adjust how %s handles line drawing characters", appname);
     s = ctrl_getset(b, "Window/Translation", "linedraw", str);
     sfree(str);
     ctrl_radiobuttons(s, "Handling of line drawing characters:", NO_SHORTCUT,1,
                      HELPCTX(translation_linedraw),
-                     dlg_stdradiobutton_handler,
-                     I(offsetof(Config, vtmode)),
+                     conf_radiobutton_handler,
+                     I(CONF_vtmode),
                      "Use Unicode line drawing code points",'u',I(VT_UNICODE),
                      "Poor man's line drawing (+, - and |)",'p',I(VT_POORMAN),
                      NULL);
     ctrl_checkbox(s, "Copy and paste line drawing characters as lqqqk",'d',
                  HELPCTX(selection_linedraw),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,rawcnp)));
+                 conf_checkbox_handler, I(CONF_rawcnp));
 
     /*
      * The Window/Selection panel.
@@ -1179,13 +1719,13 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                    "Control use of mouse");
     ctrl_checkbox(s, "Shift overrides application's use of mouse", 'p',
                  HELPCTX(selection_shiftdrag),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,mouse_override)));
+                 conf_checkbox_handler, I(CONF_mouse_override));
     ctrl_radiobuttons(s,
                      "Default selection mode (Alt+drag does the other one):",
                      NO_SHORTCUT, 2,
                      HELPCTX(selection_rect),
-                     dlg_stdradiobutton_handler,
-                     I(offsetof(Config, rect_select)),
+                     conf_radiobutton_handler,
+                     I(CONF_rect_select),
                      "Normal", 'n', I(0),
                      "Rectangular block", 'r', I(1), NULL);
 
@@ -1221,9 +1761,19 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
 
     s = ctrl_getset(b, "Window/Colours", "general",
                    "General options for colour usage");
-    ctrl_checkbox(s, "Bolded text is a different colour", 'b',
-                 HELPCTX(colours_bold),
-                 dlg_stdcheckbox_handler, I(offsetof(Config,bold_colour)));
+    ctrl_checkbox(s, "Allow terminal to specify ANSI colours", 'i',
+                 HELPCTX(colours_ansi),
+                 conf_checkbox_handler, I(CONF_ansi_colour));
+    ctrl_checkbox(s, "Allow terminal to use xterm 256-colour mode", '2',
+                 HELPCTX(colours_xterm256), conf_checkbox_handler,
+                 I(CONF_xterm_256_colour));
+    ctrl_radiobuttons(s, "Indicate bolded text by changing:", 'b', 3,
+                      HELPCTX(colours_bold),
+                      conf_radiobutton_handler, I(CONF_bold_style),
+                      "The font", I(1),
+                      "The colour", I(2),
+                      "Both", I(3),
+                      NULL);
 
     str = dupprintf("Adjust the precise colours %s displays", appname);
     s = ctrl_getset(b, "Window/Colours", "adjust", str);
@@ -1261,28 +1811,11 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
     if (protocol >= 0) {
        ctrl_settitle(b, "Connection", "Options controlling the connection");
 
-       if (!midsession) {
-           s = ctrl_getset(b, "Connection", "data",
-                           "Data to send to the server");
-           ctrl_editbox(s, "Terminal-type string", 't', 50,
-                        HELPCTX(connection_termtype),
-                        dlg_stdeditbox_handler, I(offsetof(Config,termtype)),
-                        I(sizeof(((Config *)0)->termtype)));
-           ctrl_editbox(s, "Terminal speeds", 's', 50,
-                        HELPCTX(connection_termspeed),
-                        dlg_stdeditbox_handler, I(offsetof(Config,termspeed)),
-                        I(sizeof(((Config *)0)->termspeed)));
-           ctrl_editbox(s, "Auto-login username", 'u', 50,
-                        HELPCTX(connection_username),
-                        dlg_stdeditbox_handler, I(offsetof(Config,username)),
-                        I(sizeof(((Config *)0)->username)));
-       }
-
        s = ctrl_getset(b, "Connection", "keepalive",
                        "Sending of null packets to keep session active");
        ctrl_editbox(s, "Seconds between keepalives (0 to turn off)", 'k', 20,
                     HELPCTX(connection_keepalive),
-                    dlg_stdeditbox_handler, I(offsetof(Config,ping_interval)),
+                    conf_editbox_handler, I(CONF_ping_interval),
                     I(-1));
 
        if (!midsession) {
@@ -1290,12 +1823,105 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                            "Low-level TCP connection options");
            ctrl_checkbox(s, "Disable Nagle's algorithm (TCP_NODELAY option)",
                          'n', HELPCTX(connection_nodelay),
-                         dlg_stdcheckbox_handler,
-                         I(offsetof(Config,tcp_nodelay)));
+                         conf_checkbox_handler,
+                         I(CONF_tcp_nodelay));
            ctrl_checkbox(s, "Enable TCP keepalives (SO_KEEPALIVE option)",
                          'p', HELPCTX(connection_tcpkeepalive),
-                         dlg_stdcheckbox_handler,
-                         I(offsetof(Config,tcp_keepalives)));
+                         conf_checkbox_handler,
+                         I(CONF_tcp_keepalives));
+#ifndef NO_IPV6
+           s = ctrl_getset(b, "Connection", "ipversion",
+                         "Internet protocol version");
+           ctrl_radiobuttons(s, NULL, NO_SHORTCUT, 3,
+                         HELPCTX(connection_ipversion),
+                         conf_radiobutton_handler,
+                         I(CONF_addressfamily),
+                         "Auto", 'u', I(ADDRTYPE_UNSPEC),
+                         "IPv4", '4', I(ADDRTYPE_IPV4),
+                         "IPv6", '6', I(ADDRTYPE_IPV6),
+                         NULL);
+#endif
+
+           {
+               char *label = backend_from_proto(PROT_SSH) ?
+                   "Logical name of remote host (e.g. for SSH key lookup):" :
+                   "Logical name of remote host:";
+               s = ctrl_getset(b, "Connection", "identity",
+                               "Logical name of remote host");
+               ctrl_editbox(s, label, 'm', 100,
+                            HELPCTX(connection_loghost),
+                            conf_editbox_handler, I(CONF_loghost), I(1));
+           }
+       }
+
+       /*
+        * A sub-panel Connection/Data, containing options that
+        * decide on data to send to the server.
+        */
+       if (!midsession) {
+           ctrl_settitle(b, "Connection/Data", "Data to send to the server");
+
+           s = ctrl_getset(b, "Connection/Data", "login",
+                           "Login details");
+           ctrl_editbox(s, "Auto-login username", 'u', 50,
+                        HELPCTX(connection_username),
+                        conf_editbox_handler, I(CONF_username), I(1));
+           {
+               /* We assume the local username is sufficiently stable
+                * to include on the dialog box. */
+               char *user = get_username();
+               char *userlabel = dupprintf("Use system username (%s)",
+                                           user ? user : "");
+               sfree(user);
+               ctrl_radiobuttons(s, "When username is not specified:", 'n', 4,
+                                 HELPCTX(connection_username_from_env),
+                                 conf_radiobutton_handler,
+                                 I(CONF_username_from_env),
+                                 "Prompt", I(FALSE),
+                                 userlabel, I(TRUE),
+                                 NULL);
+               sfree(userlabel);
+           }
+
+           s = ctrl_getset(b, "Connection/Data", "term",
+                           "Terminal details");
+           ctrl_editbox(s, "Terminal-type string", 't', 50,
+                        HELPCTX(connection_termtype),
+                        conf_editbox_handler, I(CONF_termtype), I(1));
+           ctrl_editbox(s, "Terminal speeds", 's', 50,
+                        HELPCTX(connection_termspeed),
+                        conf_editbox_handler, I(CONF_termspeed), I(1));
+
+           s = ctrl_getset(b, "Connection/Data", "env",
+                           "Environment variables");
+           ctrl_columns(s, 2, 80, 20);
+           ed = (struct environ_data *)
+               ctrl_alloc(b, sizeof(struct environ_data));
+           ed->varbox = ctrl_editbox(s, "Variable", 'v', 60,
+                                     HELPCTX(telnet_environ),
+                                     environ_handler, P(ed), P(NULL));
+           ed->varbox->generic.column = 0;
+           ed->valbox = ctrl_editbox(s, "Value", 'l', 60,
+                                     HELPCTX(telnet_environ),
+                                     environ_handler, P(ed), P(NULL));
+           ed->valbox->generic.column = 0;
+           ed->addbutton = ctrl_pushbutton(s, "Add", 'd',
+                                           HELPCTX(telnet_environ),
+                                           environ_handler, P(ed));
+           ed->addbutton->generic.column = 1;
+           ed->rembutton = ctrl_pushbutton(s, "Remove", 'r',
+                                           HELPCTX(telnet_environ),
+                                           environ_handler, P(ed));
+           ed->rembutton->generic.column = 1;
+           ctrl_columns(s, 1, 100);
+           ed->listbox = ctrl_listbox(s, NULL, NO_SHORTCUT,
+                                      HELPCTX(telnet_environ),
+                                      environ_handler, P(ed));
+           ed->listbox->listbox.height = 3;
+           ed->listbox->listbox.ncols = 2;
+           ed->listbox->listbox.percentages = snewn(2, int);
+           ed->listbox->listbox.percentages[0] = 30;
+           ed->listbox->listbox.percentages[1] = 70;
        }
 
     }
@@ -1310,8 +1936,8 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
        s = ctrl_getset(b, "Connection/Proxy", "basics", NULL);
        ctrl_radiobuttons(s, "Proxy type:", 't', 3,
                          HELPCTX(proxy_type),
-                         dlg_stdradiobutton_handler,
-                         I(offsetof(Config, proxy_type)),
+                         conf_radiobutton_handler,
+                         I(CONF_proxy_type),
                          "None", I(PROXY_NONE),
                          "SOCKS 4", I(PROXY_SOCKS4),
                          "SOCKS 5", I(PROXY_SOCKS5),
@@ -1321,49 +1947,44 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
        ctrl_columns(s, 2, 80, 20);
        c = ctrl_editbox(s, "Proxy hostname", 'y', 100,
                         HELPCTX(proxy_main),
-                        dlg_stdeditbox_handler,
-                        I(offsetof(Config,proxy_host)),
-                        I(sizeof(((Config *)0)->proxy_host)));
+                        conf_editbox_handler,
+                        I(CONF_proxy_host), I(1));
        c->generic.column = 0;
        c = ctrl_editbox(s, "Port", 'p', 100,
                         HELPCTX(proxy_main),
-                        dlg_stdeditbox_handler,
-                        I(offsetof(Config,proxy_port)),
+                        conf_editbox_handler,
+                        I(CONF_proxy_port),
                         I(-1));
        c->generic.column = 1;
        ctrl_columns(s, 1, 100);
        ctrl_editbox(s, "Exclude Hosts/IPs", 'e', 100,
                     HELPCTX(proxy_exclude),
-                    dlg_stdeditbox_handler,
-                    I(offsetof(Config,proxy_exclude_list)),
-                    I(sizeof(((Config *)0)->proxy_exclude_list)));
+                    conf_editbox_handler,
+                    I(CONF_proxy_exclude_list), I(1));
        ctrl_checkbox(s, "Consider proxying local host connections", 'x',
                      HELPCTX(proxy_exclude),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,even_proxy_localhost)));
+                     conf_checkbox_handler,
+                     I(CONF_even_proxy_localhost));
        ctrl_radiobuttons(s, "Do DNS name lookup at proxy end:", 'd', 3,
                          HELPCTX(proxy_dns),
-                         dlg_stdradiobutton_handler,
-                         I(offsetof(Config, proxy_dns)),
+                         conf_radiobutton_handler,
+                         I(CONF_proxy_dns),
                          "No", I(FORCE_OFF),
                          "Auto", I(AUTO),
                          "Yes", I(FORCE_ON), NULL);
        ctrl_editbox(s, "Username", 'u', 60,
                     HELPCTX(proxy_auth),
-                    dlg_stdeditbox_handler,
-                    I(offsetof(Config,proxy_username)),
-                    I(sizeof(((Config *)0)->proxy_username)));
+                    conf_editbox_handler,
+                    I(CONF_proxy_username), I(1));
        c = ctrl_editbox(s, "Password", 'w', 60,
                         HELPCTX(proxy_auth),
-                        dlg_stdeditbox_handler,
-                        I(offsetof(Config,proxy_password)),
-                        I(sizeof(((Config *)0)->proxy_password)));
+                        conf_editbox_handler,
+                        I(CONF_proxy_password), I(1));
        c->editbox.password = 1;
        ctrl_editbox(s, "Telnet command", 'm', 100,
                     HELPCTX(proxy_command),
-                    dlg_stdeditbox_handler,
-                    I(offsetof(Config,proxy_telnet_command)),
-                    I(sizeof(((Config *)0)->proxy_telnet_command)));
+                    conf_editbox_handler,
+                    I(CONF_proxy_telnet_command), I(1));
     }
 
     /*
@@ -1377,40 +1998,6 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
        ctrl_settitle(b, "Connection/Telnet",
                      "Options controlling Telnet connections");
 
-       if (!midsession) {
-           s = ctrl_getset(b, "Connection/Telnet", "data",
-                           "Data to send to the server");
-           ctrl_text(s, "Environment variables:", HELPCTX(telnet_environ));
-           ctrl_columns(s, 2, 80, 20);
-           ed = (struct environ_data *)
-               ctrl_alloc(b, sizeof(struct environ_data));
-           ed->varbox = ctrl_editbox(s, "Variable", 'v', 60,
-                                     HELPCTX(telnet_environ),
-                                     environ_handler, P(ed), P(NULL));
-           ed->varbox->generic.column = 0;
-           ed->valbox = ctrl_editbox(s, "Value", 'l', 60,
-                                     HELPCTX(telnet_environ),
-                                     environ_handler, P(ed), P(NULL));
-           ed->valbox->generic.column = 0;
-           ed->addbutton = ctrl_pushbutton(s, "Add", 'd',
-                                           HELPCTX(telnet_environ),
-                                           environ_handler, P(ed));
-           ed->addbutton->generic.column = 1;
-           ed->rembutton = ctrl_pushbutton(s, "Remove", 'r',
-                                           HELPCTX(telnet_environ),
-                                           environ_handler, P(ed));
-           ed->rembutton->generic.column = 1;
-           ctrl_columns(s, 1, 100);
-           ed->listbox = ctrl_listbox(s, NULL, NO_SHORTCUT,
-                                      HELPCTX(telnet_environ),
-                                      environ_handler, P(ed));
-           ed->listbox->listbox.height = 3;
-           ed->listbox->listbox.ncols = 2;
-           ed->listbox->listbox.percentages = snewn(2, int);
-           ed->listbox->listbox.percentages[0] = 30;
-           ed->listbox->listbox.percentages[1] = 70;
-       }
-
        s = ctrl_getset(b, "Connection/Telnet", "protocol",
                        "Telnet protocol adjustments");
 
@@ -1418,24 +2005,24 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
            ctrl_radiobuttons(s, "Handling of OLD_ENVIRON ambiguity:",
                              NO_SHORTCUT, 2,
                              HELPCTX(telnet_oldenviron),
-                             dlg_stdradiobutton_handler,
-                             I(offsetof(Config, rfc_environ)),
+                             conf_radiobutton_handler,
+                             I(CONF_rfc_environ),
                              "BSD (commonplace)", 'b', I(0),
                              "RFC 1408 (unusual)", 'f', I(1), NULL);
            ctrl_radiobuttons(s, "Telnet negotiation mode:", 't', 2,
                              HELPCTX(telnet_passive),
-                             dlg_stdradiobutton_handler,
-                             I(offsetof(Config, passive_telnet)),
+                             conf_radiobutton_handler,
+                             I(CONF_passive_telnet),
                              "Passive", I(1), "Active", I(0), NULL);
        }
-       ctrl_checkbox(s, "Keyboard sends telnet Backspace and Interrupt", 'k',
+       ctrl_checkbox(s, "Keyboard sends Telnet special commands", 'k',
                      HELPCTX(telnet_specialkeys),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,telnet_keyboard)));
-       ctrl_checkbox(s, "Return key sends telnet New Line instead of ^M",
-                     NO_SHORTCUT, HELPCTX(telnet_newline),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,telnet_newline)));
+                     conf_checkbox_handler,
+                     I(CONF_telnet_keyboard));
+       ctrl_checkbox(s, "Return key sends Telnet New Line instead of ^M",
+                     'm', HELPCTX(telnet_newline),
+                     conf_checkbox_handler,
+                     I(CONF_telnet_newline));
     }
 
     if (!midsession) {
@@ -1450,16 +2037,16 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                        "Data to send to the server");
        ctrl_editbox(s, "Local username:", 'l', 50,
                     HELPCTX(rlogin_localuser),
-                    dlg_stdeditbox_handler, I(offsetof(Config,localusername)),
-                    I(sizeof(((Config *)0)->localusername)));
+                    conf_editbox_handler, I(CONF_localusername), I(1));
 
     }
 
     /*
-     * All the SSH stuff is omitted in PuTTYtel.
+     * All the SSH stuff is omitted in PuTTYtel, or in a reconfig
+     * when we're not doing SSH.
      */
 
-    if (!midsession && backends[3].name != NULL) {
+    if (backend_from_proto(PROT_SSH) && (!midsession || protocol == PROT_SSH)) {
 
        /*
         * The Connection/SSH panel.
@@ -1467,104 +2054,303 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
        ctrl_settitle(b, "Connection/SSH",
                      "Options controlling SSH connections");
 
-       s = ctrl_getset(b, "Connection/SSH", "data",
-                       "Data to send to the server");
-       ctrl_editbox(s, "Remote command:", 'r', 100,
-                    HELPCTX(ssh_command),
-                    dlg_stdeditbox_handler, I(offsetof(Config,remote_cmd)),
-                    I(sizeof(((Config *)0)->remote_cmd)));
-
-       s = ctrl_getset(b, "Connection/SSH", "protocol", "Protocol options");
-       ctrl_checkbox(s, "Don't allocate a pseudo-terminal", 'p',
-                     HELPCTX(ssh_nopty),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,nopty)));
-       ctrl_checkbox(s, "Enable compression", 'e',
-                     HELPCTX(ssh_compress),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,compression)));
-       ctrl_radiobuttons(s, "Preferred SSH protocol version:", NO_SHORTCUT, 4,
-                         HELPCTX(ssh_protocol),
-                         dlg_stdradiobutton_handler,
-                         I(offsetof(Config, sshprot)),
-                         "1 only", 'l', I(0),
-                         "1", '1', I(1),
-                         "2", '2', I(2),
-                         "2 only", 'n', I(3), NULL);
-
-       s = ctrl_getset(b, "Connection/SSH", "encryption", "Encryption options");
-       c = ctrl_draglist(s, "Encryption cipher selection policy:", 's',
+       if (midsession && protcfginfo == 1) {
+           s = ctrl_getset(b, "Connection/SSH", "disclaimer", NULL);
+           ctrl_text(s, "Nothing on this panel may be reconfigured in mid-"
+                     "session; it is only here so that sub-panels of it can "
+                     "exist without looking strange.", HELPCTX(no_help));
+       }
+
+       if (!midsession) {
+
+           s = ctrl_getset(b, "Connection/SSH", "data",
+                           "Data to send to the server");
+           ctrl_editbox(s, "Remote command:", 'r', 100,
+                        HELPCTX(ssh_command),
+                        conf_editbox_handler, I(CONF_remote_cmd), I(1));
+
+           s = ctrl_getset(b, "Connection/SSH", "protocol", "Protocol options");
+           ctrl_checkbox(s, "Don't start a shell or command at all", 'n',
+                         HELPCTX(ssh_noshell),
+                         conf_checkbox_handler,
+                         I(CONF_ssh_no_shell));
+       }
+
+       if (!midsession || protcfginfo != 1) {
+           s = ctrl_getset(b, "Connection/SSH", "protocol", "Protocol options");
+
+           ctrl_checkbox(s, "Enable compression", 'e',
+                         HELPCTX(ssh_compress),
+                         conf_checkbox_handler,
+                         I(CONF_compression));
+       }
+
+       if (!midsession) {
+           s = ctrl_getset(b, "Connection/SSH", "protocol", "Protocol options");
+
+           ctrl_radiobuttons(s, "Preferred SSH protocol version:", NO_SHORTCUT, 4,
+                             HELPCTX(ssh_protocol),
+                             conf_radiobutton_handler,
+                             I(CONF_sshprot),
+                             "1 only", 'l', I(0),
+                             "1", '1', I(1),
+                             "2", '2', I(2),
+                             "2 only", 'y', I(3), NULL);
+       }
+
+       if (!midsession || protcfginfo != 1) {
+           s = ctrl_getset(b, "Connection/SSH", "encryption", "Encryption options");
+           c = ctrl_draglist(s, "Encryption cipher selection policy:", 's',
+                             HELPCTX(ssh_ciphers),
+                             cipherlist_handler, P(NULL));
+           c->listbox.height = 6;
+
+           ctrl_checkbox(s, "Enable legacy use of single-DES in SSH-2", 'i',
                          HELPCTX(ssh_ciphers),
-                         cipherlist_handler, P(NULL));
-       c->listbox.height = 6;
-       
-       ctrl_checkbox(s, "Enable legacy use of single-DES in SSH 2", 'i',
-                     HELPCTX(ssh_ciphers),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,ssh2_des_cbc)));
+                         conf_checkbox_handler,
+                         I(CONF_ssh2_des_cbc));
+       }
 
        /*
-        * The Connection/SSH/Auth panel.
+        * The Connection/SSH/Kex panel. (Owing to repeat key
+        * exchange, this is all meaningful in mid-session _if_
+        * we're using SSH-2 or haven't decided yet.)
         */
-       ctrl_settitle(b, "Connection/SSH/Auth",
-                     "Options controlling SSH authentication");
-
-       s = ctrl_getset(b, "Connection/SSH/Auth", "methods",
-                       "Authentication methods");
-       ctrl_checkbox(s, "Attempt TIS or CryptoCard auth (SSH1)", 'm',
-                     HELPCTX(ssh_auth_tis),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,try_tis_auth)));
-       ctrl_checkbox(s, "Attempt \"keyboard-interactive\" auth (SSH2)",
-                     'i', HELPCTX(ssh_auth_ki),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,try_ki_auth)));
-
-       s = ctrl_getset(b, "Connection/SSH/Auth", "params",
-                       "Authentication parameters");
-       ctrl_checkbox(s, "Allow agent forwarding", 'f',
-                     HELPCTX(ssh_auth_agentfwd),
-                     dlg_stdcheckbox_handler, I(offsetof(Config,agentfwd)));
-       ctrl_checkbox(s, "Allow attempted changes of username in SSH2", 'u',
-                     HELPCTX(ssh_auth_changeuser),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,change_username)));
-       ctrl_filesel(s, "Private key file for authentication:", 'k',
-                    FILTER_KEY_FILES, FALSE, "Select private key file",
-                    HELPCTX(ssh_auth_privkey),
-                    dlg_stdfilesel_handler, I(offsetof(Config, keyfile)));
+       if (protcfginfo != 1) {
+           ctrl_settitle(b, "Connection/SSH/Kex",
+                         "Options controlling SSH key exchange");
+
+           s = ctrl_getset(b, "Connection/SSH/Kex", "main",
+                           "Key exchange algorithm options");
+           c = ctrl_draglist(s, "Algorithm selection policy:", 's',
+                             HELPCTX(ssh_kexlist),
+                             kexlist_handler, P(NULL));
+           c->listbox.height = 5;
+
+           s = ctrl_getset(b, "Connection/SSH/Kex", "repeat",
+                           "Options controlling key re-exchange");
+
+           ctrl_editbox(s, "Max minutes before rekey (0 for no limit)", 't', 20,
+                        HELPCTX(ssh_kex_repeat),
+                        conf_editbox_handler,
+                        I(CONF_ssh_rekey_time),
+                        I(-1));
+           ctrl_editbox(s, "Max data before rekey (0 for no limit)", 'x', 20,
+                        HELPCTX(ssh_kex_repeat),
+                        conf_editbox_handler,
+                        I(CONF_ssh_rekey_data),
+                        I(16));
+           ctrl_text(s, "(Use 1M for 1 megabyte, 1G for 1 gigabyte etc)",
+                     HELPCTX(ssh_kex_repeat));
+       }
+
+       if (!midsession) {
+
+           /*
+            * The Connection/SSH/Auth panel.
+            */
+           ctrl_settitle(b, "Connection/SSH/Auth",
+                         "Options controlling SSH authentication");
+
+           s = ctrl_getset(b, "Connection/SSH/Auth", "main", NULL);
+           ctrl_checkbox(s, "Bypass authentication entirely (SSH-2 only)", 'b',
+                         HELPCTX(ssh_auth_bypass),
+                         conf_checkbox_handler,
+                         I(CONF_ssh_no_userauth));
+           ctrl_checkbox(s, "Display pre-authentication banner (SSH-2 only)",
+                         'd', HELPCTX(ssh_auth_banner),
+                         conf_checkbox_handler,
+                         I(CONF_ssh_show_banner));
+
+           s = ctrl_getset(b, "Connection/SSH/Auth", "methods",
+                           "Authentication methods");
+           ctrl_checkbox(s, "Attempt authentication using Pageant", 'p',
+                         HELPCTX(ssh_auth_pageant),
+                         conf_checkbox_handler,
+                         I(CONF_tryagent));
+           ctrl_checkbox(s, "Attempt TIS or CryptoCard auth (SSH-1)", 'm',
+                         HELPCTX(ssh_auth_tis),
+                         conf_checkbox_handler,
+                         I(CONF_try_tis_auth));
+           ctrl_checkbox(s, "Attempt \"keyboard-interactive\" auth (SSH-2)",
+                         'i', HELPCTX(ssh_auth_ki),
+                         conf_checkbox_handler,
+                         I(CONF_try_ki_auth));
+
+           s = ctrl_getset(b, "Connection/SSH/Auth", "params",
+                           "Authentication parameters");
+           ctrl_checkbox(s, "Allow agent forwarding", 'f',
+                         HELPCTX(ssh_auth_agentfwd),
+                         conf_checkbox_handler, I(CONF_agentfwd));
+           ctrl_checkbox(s, "Allow attempted changes of username in SSH-2", NO_SHORTCUT,
+                         HELPCTX(ssh_auth_changeuser),
+                         conf_checkbox_handler,
+                         I(CONF_change_username));
+           ctrl_filesel(s, "Private key file for authentication:", 'k',
+                        FILTER_KEY_FILES, FALSE, "Select private key file",
+                        HELPCTX(ssh_auth_privkey),
+                        conf_filesel_handler, I(CONF_keyfile));
+
+#ifndef NO_GSSAPI
+           /*
+            * Connection/SSH/Auth/GSSAPI, which sadly won't fit on
+            * the main Auth panel.
+            */
+           ctrl_settitle(b, "Connection/SSH/Auth/GSSAPI",
+                         "Options controlling GSSAPI authentication");
+           s = ctrl_getset(b, "Connection/SSH/Auth/GSSAPI", "gssapi", NULL);
+
+           ctrl_checkbox(s, "Attempt GSSAPI authentication (SSH-2 only)",
+                         't', HELPCTX(ssh_gssapi),
+                         conf_checkbox_handler,
+                         I(CONF_try_gssapi_auth));
+
+           ctrl_checkbox(s, "Allow GSSAPI credential delegation", 'l',
+                         HELPCTX(ssh_gssapi_delegation),
+                         conf_checkbox_handler,
+                         I(CONF_gssapifwd));
+
+           /*
+            * GSSAPI library selection.
+            */
+           if (ngsslibs > 1) {
+               c = ctrl_draglist(s, "Preference order for GSSAPI libraries:",
+                                 'p', HELPCTX(ssh_gssapi_libraries),
+                                 gsslist_handler, P(NULL));
+               c->listbox.height = ngsslibs;
+
+               /*
+                * I currently assume that if more than one GSS
+                * library option is available, then one of them is
+                * 'user-supplied' and so we should present the
+                * following file selector. This is at least half-
+                * reasonable, because if we're using statically
+                * linked GSSAPI then there will only be one option
+                * and no way to load from a user-supplied library,
+                * whereas if we're using dynamic libraries then
+                * there will almost certainly be some default
+                * option in addition to a user-supplied path. If
+                * anyone ever ports PuTTY to a system on which
+                * dynamic-library GSSAPI is available but there is
+                * absolutely no consensus on where to keep the
+                * libraries, there'll need to be a flag alongside
+                * ngsslibs to control whether the file selector is
+                * displayed. 
+                */
+
+               ctrl_filesel(s, "User-supplied GSSAPI library path:", 's',
+                            FILTER_DYNLIB_FILES, FALSE, "Select library file",
+                            HELPCTX(ssh_gssapi_libraries),
+                            conf_filesel_handler,
+                            I(CONF_ssh_gss_custom));
+           }
+#endif
+       }
+
+       if (!midsession) {
+           /*
+            * The Connection/SSH/TTY panel.
+            */
+           ctrl_settitle(b, "Connection/SSH/TTY", "Remote terminal settings");
+
+           s = ctrl_getset(b, "Connection/SSH/TTY", "sshtty", NULL);
+           ctrl_checkbox(s, "Don't allocate a pseudo-terminal", 'p',
+                         HELPCTX(ssh_nopty),
+                         conf_checkbox_handler,
+                         I(CONF_nopty));
+
+           s = ctrl_getset(b, "Connection/SSH/TTY", "ttymodes",
+                           "Terminal modes");
+           td = (struct ttymodes_data *)
+               ctrl_alloc(b, sizeof(struct ttymodes_data));
+           ctrl_columns(s, 2, 75, 25);
+           c = ctrl_text(s, "Terminal modes to send:", HELPCTX(ssh_ttymodes));
+           c->generic.column = 0;
+           td->rembutton = ctrl_pushbutton(s, "Remove", 'r',
+                                           HELPCTX(ssh_ttymodes),
+                                           ttymodes_handler, P(td));
+           td->rembutton->generic.column = 1;
+           td->rembutton->generic.tabdelay = 1;
+           ctrl_columns(s, 1, 100);
+           td->listbox = ctrl_listbox(s, NULL, NO_SHORTCUT,
+                                      HELPCTX(ssh_ttymodes),
+                                      ttymodes_handler, P(td));
+           td->listbox->listbox.multisel = 1;
+           td->listbox->listbox.height = 4;
+           td->listbox->listbox.ncols = 2;
+           td->listbox->listbox.percentages = snewn(2, int);
+           td->listbox->listbox.percentages[0] = 40;
+           td->listbox->listbox.percentages[1] = 60;
+           ctrl_tabdelay(s, td->rembutton);
+           ctrl_columns(s, 2, 75, 25);
+           td->modelist = ctrl_droplist(s, "Mode:", 'm', 67,
+                                        HELPCTX(ssh_ttymodes),
+                                        ttymodes_handler, P(td));
+           td->modelist->generic.column = 0;
+           td->addbutton = ctrl_pushbutton(s, "Add", 'd',
+                                           HELPCTX(ssh_ttymodes),
+                                           ttymodes_handler, P(td));
+           td->addbutton->generic.column = 1;
+           td->addbutton->generic.tabdelay = 1;
+           ctrl_columns(s, 1, 100);        /* column break */
+           /* Bit of a hack to get the value radio buttons and
+            * edit-box on the same row. */
+           ctrl_columns(s, 3, 25, 50, 25);
+           c = ctrl_text(s, "Value:", HELPCTX(ssh_ttymodes));
+           c->generic.column = 0;
+           td->valradio = ctrl_radiobuttons(s, NULL, NO_SHORTCUT, 2,
+                                            HELPCTX(ssh_ttymodes),
+                                            ttymodes_handler, P(td),
+                                            "Auto", NO_SHORTCUT, P(NULL),
+                                            "This:", NO_SHORTCUT, P(NULL),
+                                            NULL);
+           td->valradio->generic.column = 1;
+           td->valbox = ctrl_editbox(s, NULL, NO_SHORTCUT, 100,
+                                     HELPCTX(ssh_ttymodes),
+                                     ttymodes_handler, P(td), P(NULL));
+           td->valbox->generic.column = 2;
+           ctrl_tabdelay(s, td->addbutton);
+
+       }
+
+       if (!midsession) {
+           /*
+            * The Connection/SSH/X11 panel.
+            */
+           ctrl_settitle(b, "Connection/SSH/X11",
+                         "Options controlling SSH X11 forwarding");
+
+           s = ctrl_getset(b, "Connection/SSH/X11", "x11", "X11 forwarding");
+           ctrl_checkbox(s, "Enable X11 forwarding", 'e',
+                         HELPCTX(ssh_tunnels_x11),
+                         conf_checkbox_handler,I(CONF_x11_forward));
+           ctrl_editbox(s, "X display location", 'x', 50,
+                        HELPCTX(ssh_tunnels_x11),
+                        conf_editbox_handler, I(CONF_x11_display), I(1));
+           ctrl_radiobuttons(s, "Remote X11 authentication protocol", 'u', 2,
+                             HELPCTX(ssh_tunnels_x11auth),
+                             conf_radiobutton_handler,
+                             I(CONF_x11_auth),
+                             "MIT-Magic-Cookie-1", I(X11_MIT),
+                             "XDM-Authorization-1", I(X11_XDM), NULL);
+       }
 
        /*
-        * The Connection/SSH/Tunnels panel.
+        * The Tunnels panel _is_ still available in mid-session.
         */
        ctrl_settitle(b, "Connection/SSH/Tunnels",
-                     "Options controlling SSH tunnelling");
-
-       s = ctrl_getset(b, "Connection/SSH/Tunnels", "x11", "X11 forwarding");
-       ctrl_checkbox(s, "Enable X11 forwarding", 'e',
-                     HELPCTX(ssh_tunnels_x11),
-                     dlg_stdcheckbox_handler,I(offsetof(Config,x11_forward)));
-       ctrl_editbox(s, "X display location", 'x', 50,
-                    HELPCTX(ssh_tunnels_x11),
-                    dlg_stdeditbox_handler, I(offsetof(Config,x11_display)),
-                    I(sizeof(((Config *)0)->x11_display)));
-       ctrl_radiobuttons(s, "Remote X11 authentication protocol", 'u', 2,
-                         HELPCTX(ssh_tunnels_x11auth),
-                         dlg_stdradiobutton_handler,
-                         I(offsetof(Config, x11_auth)),
-                         "MIT-Magic-Cookie-1", I(X11_MIT),
-                         "XDM-Authorization-1", I(X11_XDM), NULL);
+                     "Options controlling SSH port forwarding");
 
        s = ctrl_getset(b, "Connection/SSH/Tunnels", "portfwd",
                        "Port forwarding");
        ctrl_checkbox(s, "Local ports accept connections from other hosts",'t',
                      HELPCTX(ssh_tunnels_portfwd_localhost),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,lport_acceptall)));
-       ctrl_checkbox(s, "Remote ports do the same (SSH v2 only)", 'p',
+                     conf_checkbox_handler,
+                     I(CONF_lport_acceptall));
+       ctrl_checkbox(s, "Remote ports do the same (SSH-2 only)", 'p',
                      HELPCTX(ssh_tunnels_portfwd_localhost),
-                     dlg_stdcheckbox_handler,
-                     I(offsetof(Config,rport_acceptall)));
+                     conf_checkbox_handler,
+                     I(CONF_rport_acceptall));
 
        ctrl_columns(s, 3, 55, 20, 25);
        c = ctrl_text(s, "Forwarded ports:", HELPCTX(ssh_tunnels_portfwd));
@@ -1608,40 +2394,61 @@ void setup_config_box(struct controlbox *b, struct sesslist *sesslist,
                                           "Remote", 'm', P(NULL),
                                           "Dynamic", 'y', P(NULL),
                                           NULL);
+#ifndef NO_IPV6
+       pfd->addressfamily =
+           ctrl_radiobuttons(s, NULL, NO_SHORTCUT, 3,
+                             HELPCTX(ssh_tunnels_portfwd_ipversion),
+                             portfwd_handler, P(pfd),
+                             "Auto", 'u', I(ADDRTYPE_UNSPEC),
+                             "IPv4", '4', I(ADDRTYPE_IPV4),
+                             "IPv6", '6', I(ADDRTYPE_IPV6),
+                             NULL);
+#endif
        ctrl_tabdelay(s, pfd->addbutton);
        ctrl_columns(s, 1, 100);
 
-       /*
-        * The Connection/SSH/Bugs panel.
-        */
-       ctrl_settitle(b, "Connection/SSH/Bugs",
-                     "Workarounds for SSH server bugs");
-
-       s = ctrl_getset(b, "Connection/SSH/Bugs", "main",
-                       "Detection of known bugs in SSH servers");
-       ctrl_droplist(s, "Chokes on SSH1 ignore messages", 'i', 20,
-                     HELPCTX(ssh_bugs_ignore1),
-                     sshbug_handler, I(offsetof(Config,sshbug_ignore1)));
-       ctrl_droplist(s, "Refuses all SSH1 password camouflage", 's', 20,
-                     HELPCTX(ssh_bugs_plainpw1),
-                     sshbug_handler, I(offsetof(Config,sshbug_plainpw1)));
-       ctrl_droplist(s, "Chokes on SSH1 RSA authentication", 'r', 20,
-                     HELPCTX(ssh_bugs_rsa1),
-                     sshbug_handler, I(offsetof(Config,sshbug_rsa1)));
-       ctrl_droplist(s, "Miscomputes SSH2 HMAC keys", 'm', 20,
-                     HELPCTX(ssh_bugs_hmac2),
-                     sshbug_handler, I(offsetof(Config,sshbug_hmac2)));
-       ctrl_droplist(s, "Miscomputes SSH2 encryption keys", 'e', 20,
-                     HELPCTX(ssh_bugs_derivekey2),
-                     sshbug_handler, I(offsetof(Config,sshbug_derivekey2)));
-       ctrl_droplist(s, "Requires padding on SSH2 RSA signatures", 'p', 20,
-                     HELPCTX(ssh_bugs_rsapad2),
-                     sshbug_handler, I(offsetof(Config,sshbug_rsapad2)));
-       ctrl_droplist(s, "Chokes on Diffie-Hellman group exchange", 'd', 20,
-                     HELPCTX(ssh_bugs_dhgex2),
-                     sshbug_handler, I(offsetof(Config,sshbug_dhgex2)));
-       ctrl_droplist(s, "Misuses the session ID in PK auth", 'n', 20,
-                     HELPCTX(ssh_bugs_pksessid2),
-                     sshbug_handler, I(offsetof(Config,sshbug_pksessid2)));
+       if (!midsession) {
+           /*
+            * The Connection/SSH/Bugs panel.
+            */
+           ctrl_settitle(b, "Connection/SSH/Bugs",
+                         "Workarounds for SSH server bugs");
+
+           s = ctrl_getset(b, "Connection/SSH/Bugs", "main",
+                           "Detection of known bugs in SSH servers");
+           ctrl_droplist(s, "Chokes on SSH-1 ignore messages", 'i', 20,
+                         HELPCTX(ssh_bugs_ignore1),
+                         sshbug_handler, I(CONF_sshbug_ignore1));
+           ctrl_droplist(s, "Refuses all SSH-1 password camouflage", 's', 20,
+                         HELPCTX(ssh_bugs_plainpw1),
+                         sshbug_handler, I(CONF_sshbug_plainpw1));
+           ctrl_droplist(s, "Chokes on SSH-1 RSA authentication", 'r', 20,
+                         HELPCTX(ssh_bugs_rsa1),
+                         sshbug_handler, I(CONF_sshbug_rsa1));
+           ctrl_droplist(s, "Chokes on SSH-2 ignore messages", '2', 20,
+                         HELPCTX(ssh_bugs_ignore2),
+                         sshbug_handler, I(CONF_sshbug_ignore2));
+           ctrl_droplist(s, "Chokes on PuTTY's SSH-2 'winadj' requests", 'j',
+                          20, HELPCTX(ssh_bugs_winadj),
+                         sshbug_handler, I(CONF_sshbug_winadj));
+           ctrl_droplist(s, "Miscomputes SSH-2 HMAC keys", 'm', 20,
+                         HELPCTX(ssh_bugs_hmac2),
+                         sshbug_handler, I(CONF_sshbug_hmac2));
+           ctrl_droplist(s, "Miscomputes SSH-2 encryption keys", 'e', 20,
+                         HELPCTX(ssh_bugs_derivekey2),
+                         sshbug_handler, I(CONF_sshbug_derivekey2));
+           ctrl_droplist(s, "Requires padding on SSH-2 RSA signatures", 'p', 20,
+                         HELPCTX(ssh_bugs_rsapad2),
+                         sshbug_handler, I(CONF_sshbug_rsapad2));
+           ctrl_droplist(s, "Misuses the session ID in SSH-2 PK auth", 'n', 20,
+                         HELPCTX(ssh_bugs_pksessid2),
+                         sshbug_handler, I(CONF_sshbug_pksessid2));
+           ctrl_droplist(s, "Handles SSH-2 key re-exchange badly", 'k', 20,
+                         HELPCTX(ssh_bugs_rekey2),
+                         sshbug_handler, I(CONF_sshbug_rekey2));
+           ctrl_droplist(s, "Ignores SSH-2 maximum packet size", 'x', 20,
+                         HELPCTX(ssh_bugs_maxpkt2),
+                         sshbug_handler, I(CONF_sshbug_maxpkt2));
+       }
     }
 }