Implemented configuration settings for journaling.
[citadel.git] / citadel / routines2.c
index c3d4269f9cfb316d7c6de287c7c6297275a95f6f..f20add33f8a18c3f2e24fa83974345c216879f65 100644 (file)
@@ -5,7 +5,6 @@
  *
  */
 
-#include "sysdep.h"
 #include <stdlib.h>
 #include <unistd.h>
 #include <sys/types.h>
@@ -31,6 +30,7 @@
 #include <pwd.h>
 #include <errno.h>
 #include <stdarg.h>
+#include "sysdep.h"
 #include "citadel.h"
 #include "citadel_ipc.h"
 #include "citadel_decls.h"
@@ -90,16 +90,16 @@ void entregis(CtdlIPC *ipc)
 {                              /* register with name and address */
 
        char buf[SIZ];
-       char tmpname[SIZ];
-       char tmpaddr[SIZ];
-       char tmpcity[SIZ];
-       char tmpstate[SIZ];
-       char tmpzip[SIZ];
-       char tmpphone[SIZ];
+       char tmpname[30];
+       char tmpaddr[25];
+       char tmpcity[15];
+       char tmpstate[3];
+       char tmpzip[11];
+       char tmpphone[15];
        char tmpemail[SIZ];
-       char tmpcountry[SIZ];
-       char diruser[SIZ];
-       char dirnode[SIZ];
+       char tmpcountry[32];
+       char diruser[256];
+       char dirnode[256];
        char holdemail[SIZ];
        char *reg = NULL;
        int ok = 0;
@@ -120,25 +120,25 @@ void entregis(CtdlIPC *ipc)
 
                while (reg && strlen(reg) > 0) {
 
-                       extract_token(buf, reg, 0, '\n');
+                       extract_token(buf, reg, 0, '\n', sizeof buf);
                        remove_token(reg, 0, '\n');
 
                        if (a == 2)
-                               strcpy(tmpname, buf);
+                               safestrncpy(tmpname, buf, sizeof tmpname);
                        else if (a == 3)
-                               strcpy(tmpaddr, buf);
+                               safestrncpy(tmpaddr, buf, sizeof tmpaddr);
                        else if (a == 4)
-                               strcpy(tmpcity, buf);
+                               safestrncpy(tmpcity, buf, sizeof tmpcity);
                        else if (a == 5)
-                               strcpy(tmpstate, buf);
+                               safestrncpy(tmpstate, buf, sizeof tmpstate);
                        else if (a == 6)
-                               strcpy(tmpzip, buf);
+                               safestrncpy(tmpzip, buf, sizeof tmpzip);
                        else if (a == 7)
-                               strcpy(tmpphone, buf);
+                               safestrncpy(tmpphone, buf, sizeof tmpphone);
                        else if (a == 9)
-                               strcpy(tmpemail, buf);
+                               safestrncpy(tmpemail, buf, sizeof tmpemail);
                        else if (a == 10)
-                               strcpy(tmpcountry, buf);
+                               safestrncpy(tmpcountry, buf, sizeof tmpcountry);
                        ++a;
                }
        }
@@ -152,16 +152,16 @@ void entregis(CtdlIPC *ipc)
 
        do {
                ok = 1;
-               strcpy(holdemail, tmpemail);
+               safestrncpy(holdemail, tmpemail, sizeof holdemail);
                strprompt("Email address", tmpemail, 31);
                r = CtdlIPCDirectoryLookup(ipc, tmpemail, buf);
                if (r / 100 == 2) {
-                       extract_token(diruser, buf, 0, '@');
-                       extract_token(dirnode, buf, 1, '@');
+                       extract_token(diruser, buf, 0, '@', sizeof diruser);
+                       extract_token(dirnode, buf, 1, '@', sizeof dirnode);
                        striplt(diruser);
                        striplt(dirnode);
                        if ((strcasecmp(diruser, fullname))
-                          || (strcasecmp(dirnode, serv_info.serv_nodename))) {
+                          || (strcasecmp(dirnode, ipc->ServInfo.nodename))) {
                                scr_printf(
                                        "\nYou can't use %s as your address.\n",
                                        tmpemail);
@@ -169,13 +169,13 @@ void entregis(CtdlIPC *ipc)
                                        "It is already in use by %s @ %s.\n",
                                        diruser, dirnode);
                                ok = 0;
-                               strcpy(tmpemail, holdemail);
+                               safestrncpy(tmpemail, holdemail, sizeof tmpemail);
                        }
                }
        } while (ok == 0);
 
        /* now send the registration info back to the server */
-       reg = (char *)realloc(reg, 4096);       /* Overkill? */
+       reg = (char *)realloc(reg, SIZ);
        if (reg) {
                sprintf(reg, "%s\n%s\n%s\n%s\n%s\n%s\n%s\n%s\n",
                        tmpname, tmpaddr, tmpcity, tmpstate,
@@ -259,7 +259,7 @@ void cli_upload(CtdlIPC *ipc)
                if (haschar(tbuf, '/'))
                        extract_token(tbuf, flnm,
                                num_tokens(tbuf, '/') - 1,
-                               '/'
+                               '/', sizeof tbuf
                        );
                /* filename.1, filename.2, etc */
                if (a > 0) {
@@ -345,7 +345,7 @@ void upload(CtdlIPC *ipc, int c)
                chdir(tempdir);
                switch (c) {
                case 0:
-                       sttybbs(0);
+                       stty_ctdl(0);
                        scr_printf("Receiving %s - press Ctrl-D to end.\n", flnm);
                        fp = fopen(flnm, "w");
                        do {
@@ -362,17 +362,17 @@ void upload(CtdlIPC *ipc, int c)
                        exit(0);
                case 1:
                        screen_reset();
-                       sttybbs(3);
+                       stty_ctdl(3);
                        execlp("rx", "rx", flnm, NULL);
                        exit(1);
                case 2:
                        screen_reset();
-                       sttybbs(3);
+                       stty_ctdl(3);
                        execlp("rb", "rb", NULL);
                        exit(1);
                case 3:
                        screen_reset();
-                       sttybbs(3);
+                       stty_ctdl(3);
                        execlp("rz", "rz", NULL);
                        exit(1);
                }
@@ -380,7 +380,7 @@ void upload(CtdlIPC *ipc, int c)
                do {
                        b = ka_wait(&a);
                } while ((b != xfer_pid) && (b != (-1)));
-       sttybbs(0);
+       stty_ctdl(0);
        screen_set();
 
        if (a != 0) {
@@ -425,7 +425,7 @@ int val_user(CtdlIPC *ipc, char *user, int do_validate)
        if (r / 100 == 1) {
                a = 0;
                do {
-                       extract_token(buf, resp, 0, '\n');
+                       extract_token(buf, resp, 0, '\n', sizeof buf);
                        remove_token(resp, 0, '\n');
                        ++a;
                        if (a == 1)
@@ -501,7 +501,7 @@ void validate(CtdlIPC *ipc)
                if (r / 100 == 2)
                        scr_printf("%s\n", cmd);
                if (r / 100 == 3) {
-                       extract(buf, cmd, 0);
+                       extract_token(buf, cmd, 0, '|', sizeof buf);
                        if (val_user(ipc, buf, 1) != 0) finished = 1;
                }
        } while (finished == 0);
@@ -510,10 +510,11 @@ void validate(CtdlIPC *ipc)
 void subshell(void)
 {
        int a, b;
+
+       screen_reset();
+       stty_ctdl(SB_RESTORE);
        a = fork();
        if (a == 0) {
-               screen_reset();
-               sttybbs(SB_RESTORE);
                signal(SIGINT, SIG_DFL);
                signal(SIGQUIT, SIG_DFL);
                execlp(getenv("SHELL"), getenv("SHELL"), NULL);
@@ -523,7 +524,7 @@ void subshell(void)
        do {
                b = ka_wait(NULL);
        } while ((a != b) && (a != (-1)));
-       sttybbs(0);
+       stty_ctdl(0);
        screen_set();
 }
 
@@ -592,7 +593,7 @@ void list_bio(CtdlIPC *ipc)
                return;
        }
        while (strlen(resp)) {
-               extract_token(buf, resp, 0, '\n');
+               extract_token(buf, resp, 0, '\n', sizeof buf);
                remove_token(resp, 0, '\n');
                if ((pos + strlen(buf) + 5) > screenwidth) {
                        pprintf("\n");
@@ -629,7 +630,7 @@ void read_bio(CtdlIPC *ipc)
                return;
        }
        while (strlen(resp)) {
-               extract_token(buf, resp, 0, '\n');
+               extract_token(buf, resp, 0, '\n', sizeof buf);
                remove_token(resp, 0, '\n');
                pprintf("%s\n", buf);
        }
@@ -642,14 +643,18 @@ void read_bio(CtdlIPC *ipc)
  */
 void do_system_configuration(CtdlIPC *ipc)
 {
+
+#define NUM_CONFIGS 49
+
        char buf[SIZ];
-       char sc[32][SIZ];
+       char sc[NUM_CONFIGS][256];
        char *resp = NULL;
        struct ExpirePolicy *site_expirepolicy = NULL;
        struct ExpirePolicy *mbx_expirepolicy = NULL;
        int a;
        int logpages = 0;
        int r;                  /* IPC response code */
+       int server_configs = 0;
 
        /* Clear out the config buffers */
        memset(&sc[0][0], 0, sizeof(sc));
@@ -657,14 +662,11 @@ void do_system_configuration(CtdlIPC *ipc)
        /* Fetch the current config */
        r = CtdlIPCGetSystemConfig(ipc, &resp, buf);
        if (r / 100 == 1) {
-               a = 0;
-               while (strlen(resp)) {
-                       extract_token(buf, resp, 0, '\n');
-                       remove_token(resp, 0, '\n');
-                       if (a < 32) {
-                               strcpy(&sc[a][0], buf);
+               server_configs = num_tokens(resp, '\n');
+               for (a=0; a<server_configs; ++a) {
+                       if (a < NUM_CONFIGS) {
+                               extract_token(&sc[a][0], resp, a, '\n', sizeof sc[a]);
                        }
-                       ++a;
                }
        }
        if (resp) free(resp);
@@ -680,7 +682,7 @@ void do_system_configuration(CtdlIPC *ipc)
        strprompt("Node name", &sc[0][0], 15);
        strprompt("Fully qualified domain name", &sc[1][0], 63);
        strprompt("Human readable node name", &sc[2][0], 20);
-       strprompt("Modem dialup number", &sc[3][0], 15);
+       strprompt("Telephone number", &sc[3][0], 15);
        strprompt("Geographic location of this system", &sc[12][0], 31);
        strprompt("Name of system administrator", &sc[13][0], 25);
        strprompt("Paginator prompt", &sc[10][0], 79);
@@ -688,31 +690,28 @@ void do_system_configuration(CtdlIPC *ipc)
        /* Security parameters */
 
        snprintf(sc[7], sizeof sc[7], "%d", (boolprompt(
-                                   "Require registration for new users",
-                                                   atoi(&sc[7][0]))));
+               "Require registration for new users",
+               atoi(&sc[7][0]))));
        snprintf(sc[29], sizeof sc[29], "%d", (boolprompt(
-             "Disable self-service user account creation",
-                                                    atoi(&sc[29][0]))));
+               "Disable self-service user account creation",
+               atoi(&sc[29][0]))));
        strprompt("Initial access level for new users", &sc[6][0], 1);
        strprompt("Access level required to create rooms", &sc[19][0], 1);
        snprintf(sc[4], sizeof sc[4], "%d", (boolprompt(
-                                                   "Automatically give room aide privs to a user who creates a private room",
-                                                   atoi(&sc[4][0]))));
+               "Automatically give room aide privs to a user who creates a private room",
+               atoi(&sc[4][0]))));
 
        snprintf(sc[8], sizeof sc[8], "%d", (boolprompt(
-                "Automatically move problem user messages to twit room",
-                                                   atoi(&sc[8][0]))));
+               "Automatically move problem user messages to twit room",
+               atoi(&sc[8][0]))));
 
        strprompt("Name of twit room", &sc[9][0], ROOMNAMELEN);
        snprintf(sc[11], sizeof sc[11], "%d", (boolprompt(
-             "Restrict Internet mail to only those with that privilege",
-                                                    atoi(&sc[11][0]))));
+               "Restrict Internet mail to only those with that privilege",
+               atoi(&sc[11][0]))));
        snprintf(sc[26], sizeof sc[26], "%d", (boolprompt(
-             "Allow Aides to Zap (forget) rooms",
-                                                    atoi(&sc[26][0]))));
-       snprintf(sc[30], sizeof sc[30], "%d", (boolprompt(
-             "Allow system Aides access to user mailboxes",
-                                                    atoi(&sc[30][0]))));
+               "Allow Aides to Zap (forget) rooms",
+               atoi(&sc[26][0]))));
 
        if (strlen(&sc[18][0]) > 0) logpages = 1;
        else logpages = 0;
@@ -732,10 +731,18 @@ void do_system_configuration(CtdlIPC *ipc)
        strprompt("Maximum message length", &sc[20][0], 20);
        strprompt("Minimum number of worker threads", &sc[21][0], 3);
        strprompt("Maximum number of worker threads", &sc[22][0], 3);
+       snprintf(sc[43], sizeof sc[43], "%d", (boolprompt(
+               "Automatically delete committed database logs",
+               atoi(&sc[43][0]))));
 
+       strprompt("Server IP address (0.0.0.0 for 'any')", &sc[37][0], 15);
        strprompt("POP3 server port (-1 to disable)", &sc[23][0], 5);
+       strprompt("POP3S server port (-1 to disable)", &sc[40][0], 5);
        strprompt("IMAP server port (-1 to disable)", &sc[27][0], 5);
-       strprompt("SMTP server port (-1 to disable)", &sc[24][0], 5);
+       strprompt("IMAPS server port (-1 to disable)", &sc[39][0], 5);
+       strprompt("SMTP MTA server port (-1 to disable)", &sc[24][0], 5);
+       strprompt("SMTP MSA server port (-1 to disable)", &sc[38][0], 5);
+       strprompt("SMTPS server port (-1 to disable)", &sc[41][0], 5);
 
        /* This logic flips the question around, because it's one of those
         * situations where 0=yes and 1=no
@@ -746,6 +753,31 @@ void do_system_configuration(CtdlIPC *ipc)
                a);
        a = (a ? 0 : 1);
        snprintf(sc[25], sizeof sc[25], "%d", a);
+       snprintf(sc[45], sizeof sc[45], "%d", (boolprompt(
+               "Allow unauthenticated SMTP clients to spoof my domains",
+               atoi(&sc[45][0]))));
+       snprintf(sc[44], sizeof sc[44], "%d", (boolprompt(
+               "Instantly expunge deleted IMAP messages",
+               atoi(&sc[44][0]))));
+
+       /* LDAP settings */
+       if (ipc->ServInfo.supports_ldap) {
+               a = strlen(&sc[32][0]);
+               a = (a ? 1 : 0);        /* Set only to 1 or 0 */
+               a = boolprompt("Connect this Citadel to an LDAP directory", a);
+               if (a) {
+                       strprompt("Host name of LDAP server",
+                               &sc[32][0], 127);
+                       strprompt("Port number of LDAP service",
+                               &sc[33][0], 5);
+                       strprompt("Base DN", &sc[34][0], 255);
+                       strprompt("Bind DN", &sc[35][0], 255);
+                       strprompt("Password for bind DN", &sc[36][0], 255);
+               }
+               else {
+                       strcpy(&sc[32][0], "");
+               }
+       }
 
        /* Expiry settings */
        strprompt("Default user purge time (days)", &sc[16][0], 5);
@@ -806,19 +838,33 @@ void do_system_configuration(CtdlIPC *ipc)
 
        strprompt("How often to run network jobs (in seconds)", &sc[28][0], 5);
        strprompt("Hour to run purges (0-23)", &sc[31][0], 2);
+       snprintf(sc[42], sizeof sc[42], "%d", (boolprompt(
+               "Enable full text search index (warning: resource intensive)",
+               atoi(&sc[42][0]))));
+
+       snprintf(sc[46], sizeof sc[46], "%d", (boolprompt(
+               "Perform journaling of email messages",
+               atoi(&sc[46][0]))));
+       snprintf(sc[47], sizeof sc[47], "%d", (boolprompt(
+               "Perform journaling of non-email messages",
+               atoi(&sc[47][0]))));
+       if ( (atoi(&sc[46][0])) || (atoi(&sc[47][0])) ) {
+               strprompt("Email destination of journalized messages",
+                       &sc[48][0], 127);
+       }
 
        /* Save it */
        scr_printf("Save this configuration? ");
        if (yesno()) {
                r = 1;
-               for (a = 0; a < 32; a++)
+               for (a = 0; a < NUM_CONFIGS; a++)
                        r += 1 + strlen(sc[a]);
                resp = (char *)calloc(1, r);
                if (!resp) {
                        err_printf("Can't save config - out of memory!\n");
                        logoff(ipc, 1);
                }
-               for (a = 0; a < 32; a++) {
+               for (a = 0; a < NUM_CONFIGS; a++) {
                        strcat(resp, sc[a]);
                        strcat(resp, "\n");
                }
@@ -885,12 +931,13 @@ void do_internet_configuration(CtdlIPC *ipc)
        int badkey;
        int i, j;
        int quitting = 0;
+       int modified = 0;
        int r;
        
        r = CtdlIPCGetSystemConfigByType(ipc, INTERNETCFG, &resp, buf);
        if (r / 100 == 1) {
                while (strlen(resp)) {
-                       extract_token(buf, resp, 0, '\n');
+                       extract_token(buf, resp, 0, '\n', sizeof buf);
                        remove_token(resp, 0, '\n');
                        ++num_recs;
                        if (num_recs == 1) recs = malloc(sizeof(char *));
@@ -910,10 +957,10 @@ void do_internet_configuration(CtdlIPC *ipc)
                for (i=0; i<num_recs; ++i) {
                color(DIM_WHITE);
                scr_printf("%3d ", i+1);
-               extract(buf, recs[i], 0);
+               extract_token(buf, recs[i], 0, '|', sizeof buf);
                color(BRIGHT_CYAN);
                scr_printf("%-50s ", buf);
-               extract(buf, recs[i], 1);
+               extract_token(buf, recs[i], 1, '|', sizeof buf);
                color(BRIGHT_MAGENTA);
                scr_printf("%-20s\n", buf);
                color(DIM_WHITE);
@@ -936,6 +983,7 @@ void do_internet_configuration(CtdlIPC *ipc)
                                                        &buf[strlen(buf)]);
                                        recs[num_recs-1] = strdup(buf);
                                }
+                               modified = 1;
                                break;
                        case 'd':
                                i = intprompt("Delete which one",
@@ -944,6 +992,7 @@ void do_internet_configuration(CtdlIPC *ipc)
                                --num_recs;
                                for (j=i; j<num_recs; ++j)
                                        recs[j] = recs[j+1];
+                               modified = 1;
                                break;
                        case 's':
                                r = 1;
@@ -961,17 +1010,19 @@ void do_internet_configuration(CtdlIPC *ipc)
                                r = CtdlIPCSetSystemConfigByType(ipc, INTERNETCFG, resp, buf);
                                if (r / 100 != 4) {
                                        err_printf("%s\n", buf);
+                               } else {
+                                       scr_printf("Wrote %d records.\n", num_recs);
+                                       modified = 0;
                                }
-                               quitting = 1;
                                break;
                        case 'q':
-                               quitting = boolprompt(
+                               quitting = !modified || boolprompt(
                                        "Quit without saving", 0);
                                break;
                        default:
                                badkey = 1;
                }
-       } while (quitting == 0);
+       } while (!quitting);
 
        if (recs != NULL) {
                for (i=0; i<num_recs; ++i) free(recs[i]);
@@ -991,12 +1042,14 @@ void network_config_management(CtdlIPC *ipc, char *entrytype, char *comment)
        int e_ex_code;
        pid_t editor_pid;
        int cksum;
-       int b, i;
+       int b, i, tokens;
        char buf[SIZ];
        char instr[SIZ];
        char addr[SIZ];
        FILE *tempfp;
        FILE *changefp;
+       char *listing = NULL;
+       int r;
 
        if (strlen(editor_paths[0]) == 0) {
                scr_printf("You must have an external editor configured in"
@@ -1004,8 +1057,8 @@ void network_config_management(CtdlIPC *ipc, char *entrytype, char *comment)
                return;
        }
 
-       snprintf(filename, sizeof filename, "%s.listedit", tmpnam(NULL));
-       snprintf(changefile, sizeof changefile, "%s.listedit", tmpnam(NULL));
+       CtdlMakeTempFileName(filename, sizeof filename);
+       CtdlMakeTempFileName(changefile, sizeof changefile);
 
        tempfp = fopen(filename, "w");
        if (tempfp == NULL) {
@@ -1018,22 +1071,34 @@ void network_config_management(CtdlIPC *ipc, char *entrytype, char *comment)
        fprintf(tempfp, "# Specify one per line.\n"
                        "\n\n");
 
-       CtdlIPC_putline(ipc, "GNET");
-       CtdlIPC_getline(ipc, buf);
-       if (buf[0] == '1') {
-               while(CtdlIPC_getline(ipc, buf), strcmp(buf, "000")) {
-                       extract(instr, buf, 0);
+       r = CtdlIPCGetRoomNetworkConfig(ipc, &listing, buf);
+       if (r / 100 == 1) {
+               while(listing && strlen(listing)) {
+                       extract_token(buf, listing, 0, '\n', sizeof buf);
+                       remove_token(listing, 0, '\n');
+                       extract_token(instr, buf, 0, '|', sizeof instr);
                        if (!strcasecmp(instr, entrytype)) {
-                               extract(addr, buf, 1);
-                               fprintf(tempfp, "%s\n", addr);
+                               tokens = num_tokens(buf, '|');
+                               for (i=1; i<tokens; ++i) {
+                                       extract_token(addr, buf, i, '|', sizeof addr);
+                                       fprintf(tempfp, "%s", addr);
+                                       if (i < (tokens-1)) {
+                                               fprintf(tempfp, "|");
+                                       }
+                               }
+                               fprintf(tempfp, "\n");
                        }
                }
        }
+       if (listing) {
+               free(listing);
+               listing = NULL;
+       }
        fclose(tempfp);
 
        e_ex_code = 1;  /* start with a failed exit code */
        screen_reset();
-       sttybbs(SB_RESTORE);
+       stty_ctdl(SB_RESTORE);
        editor_pid = fork();
        cksum = file_checksum(filename);
        if (editor_pid == 0) {
@@ -1048,7 +1113,7 @@ void network_config_management(CtdlIPC *ipc, char *entrytype, char *comment)
                        b = ka_wait(&e_ex_code);
                } while ((b != editor_pid) && (b >= 0));
        editor_pid = (-1);
-       sttybbs(0);
+       stty_ctdl(0);
        screen_set();
        }
 
@@ -1059,16 +1124,21 @@ void network_config_management(CtdlIPC *ipc, char *entrytype, char *comment)
 
        if (e_ex_code == 0) {           /* Save changes */
                changefp = fopen(changefile, "w");
-               CtdlIPC_putline(ipc, "GNET");
-               CtdlIPC_getline(ipc, buf);
-               if (buf[0] == '1') {
-                       while(CtdlIPC_getline(ipc, buf), strcmp(buf, "000")) {
-                               extract(instr, buf, 0);
+               r = CtdlIPCGetRoomNetworkConfig(ipc, &listing, buf);
+               if (r / 100 == 1) {
+                       while(listing && strlen(listing)) {
+                               extract_token(buf, listing, 0, '\n', sizeof buf);
+                               remove_token(listing, 0, '\n');
+                               extract_token(instr, buf, 0, '|', sizeof instr);
                                if (strcasecmp(instr, entrytype)) {
                                        fprintf(changefp, "%s\n", buf);
                                }
                        }
                }
+               if (listing) {
+                       free(listing);
+                       listing = NULL;
+               }
                tempfp = fopen(filename, "r");
                while (fgets(buf, sizeof buf, tempfp) != NULL) {
                        for (i=0; i<strlen(buf); ++i) {
@@ -1083,19 +1153,15 @@ void network_config_management(CtdlIPC *ipc, char *entrytype, char *comment)
                fclose(changefp);
 
                /* now write it to the server... */
-               CtdlIPC_putline(ipc, "SNET");
-               CtdlIPC_getline(ipc, buf);
-               if (buf[0] == '4') {
-                       changefp = fopen(changefile, "r");
-                       if (changefp != NULL) {
-                               while (fgets(buf, sizeof buf,
-                                      changefp) != NULL) {
-                                       buf[strlen(buf) - 1] = 0;
-                                       CtdlIPC_putline(ipc, buf);
-                               }
-                               fclose(changefp);
+               changefp = fopen(changefile, "r");
+               if (changefp != NULL) {
+                       listing = load_message_from_file(changefp);
+                       if (listing) {
+                               r = CtdlIPCSetRoomNetworkConfig(ipc, listing, buf);
+                               free(listing);
+                               listing = NULL;
                        }
-                       CtdlIPC_putline(ipc, "000");
+                       fclose(changefp);
                }
        }
 
@@ -1115,18 +1181,22 @@ void do_ignet_configuration(CtdlIPC *ipc) {
        int badkey;
        int i, j;
        int quitting = 0;
-       
+       int modified = 0;
+       char *listing = NULL;
+       int r;
+
+       r = CtdlIPCGetSystemConfigByType(ipc, IGNETCFG, &listing, buf);
+       if (r / 100 == 1) while (*listing && strlen(listing)) {
+               extract_token(buf, listing, 0, '\n', sizeof buf);
+               remove_token(listing, 0, '\n');
 
-       snprintf(buf, sizeof buf, "CONF getsys|%s", IGNETCFG);
-       CtdlIPC_putline(ipc, buf);
-       CtdlIPC_getline(ipc, buf);
-       if (buf[0] == '1') while (CtdlIPC_getline(ipc, buf), strcmp(buf, "000")) {
                ++num_recs;
                if (num_recs == 1) recs = malloc(sizeof(char *));
                else recs = realloc(recs, (sizeof(char *)) * num_recs);
                recs[num_recs-1] = malloc(SIZ);
                strcpy(recs[num_recs-1], buf);
        }
+       if (listing) free(listing);
 
        do {
                scr_printf("\n");
@@ -1145,20 +1215,21 @@ void do_ignet_configuration(CtdlIPC *ipc) {
                for (i=0; i<num_recs; ++i) {
                color(DIM_WHITE);
                scr_printf("%3d ", i+1);
-               extract(buf, recs[i], 0);
+               extract_token(buf, recs[i], 0, '|', sizeof buf);
                color(BRIGHT_CYAN);
                scr_printf("%-16s ", buf);
-               extract(buf, recs[i], 1);
+               extract_token(buf, recs[i], 1, '|', sizeof buf);
                color(BRIGHT_MAGENTA);
                scr_printf("%-18s ", buf);
-               extract(buf, recs[i], 2);
+               extract_token(buf, recs[i], 2, '|', sizeof buf);
                color(BRIGHT_CYAN);
                scr_printf("%-32s ", buf);
-               extract(buf, recs[i], 3);
+               extract_token(buf, recs[i], 3, '|', sizeof buf);
                color(BRIGHT_MAGENTA);
                scr_printf("%-3s\n", buf);
                color(DIM_WHITE);
                }
+               scr_printf("\n");
 
                ch = keymenu("", "<A>dd|<D>elete|<S>ave|<Q>uit");
                switch(ch) {
@@ -1179,6 +1250,7 @@ void do_ignet_configuration(CtdlIPC *ipc) {
                                strprompt("Enter port number  : ",
                                        &buf[strlen(buf)-3], 5);
                                recs[num_recs-1] = strdup(buf);
+                               modified = 1;
                                break;
                        case 'd':
                                i = intprompt("Delete which one",
@@ -1187,30 +1259,37 @@ void do_ignet_configuration(CtdlIPC *ipc) {
                                --num_recs;
                                for (j=i; j<num_recs; ++j)
                                        recs[j] = recs[j+1];
+                               modified = 1;
                                break;
                        case 's':
-                               snprintf(buf, sizeof buf, "CONF putsys|%s", IGNETCFG);
-                               CtdlIPC_putline(ipc, buf);
-                               CtdlIPC_getline(ipc, buf);
-                               if (buf[0] == '4') {
-                                       for (i=0; i<num_recs; ++i) {
-                                               CtdlIPC_putline(ipc, recs[i]);
-                                       }
-                                       CtdlIPC_putline(ipc, "000");
+                               r = 1;
+                               for (i = 0; i < num_recs; ++i)
+                                       r += 1 + strlen(recs[i]);
+                               listing = (char*) calloc(1, r);
+                               if (!listing) {
+                                       err_printf("Can't save config - out of memory!\n");
+                                       logoff(ipc, 1);
                                }
-                               else {
-                                       scr_printf("%s\n", &buf[4]);
+                               if (num_recs) for (i = 0; i < num_recs; ++i) {
+                                       strcat(listing, recs[i]);
+                                       strcat(listing, "\n");
+                               }
+                               r = CtdlIPCSetSystemConfigByType(ipc, IGNETCFG, listing, buf);
+                               if (r / 100 != 4) {
+                                       scr_printf("%s\n", buf);
+                               } else {
+                                       scr_printf("Wrote %d records.\n", num_recs);
+                                       modified = 0;
                                }
-                               quitting = 1;
                                break;
                        case 'q':
-                               quitting = boolprompt(
+                               quitting = !modified || boolprompt(
                                        "Quit without saving", 0);
                                break;
                        default:
                                badkey = 1;
                }
-       } while (quitting == 0);
+       } while (!quitting);
 
        if (recs != NULL) {
                for (i=0; i<num_recs; ++i) free(recs[i]);
@@ -1230,18 +1309,22 @@ void do_filterlist_configuration(CtdlIPC *ipc)
        int badkey;
        int i, j;
        int quitting = 0;
-       
+       int modified = 0;
+       char *listing = NULL;
+       int r;
+
+       r = CtdlIPCGetSystemConfigByType(ipc, FILTERLIST, &listing, buf);
+       if (r / 100 == 1) while (*listing && strlen(listing)) {
+               extract_token(buf, listing, 0, '\n', sizeof buf);
+               remove_token(listing, 0, '\n');
 
-       snprintf(buf, sizeof buf, "CONF getsys|%s", FILTERLIST);
-       CtdlIPC_putline(ipc, buf);
-       CtdlIPC_getline(ipc, buf);
-       if (buf[0] == '1') while (CtdlIPC_getline(ipc, buf), strcmp(buf, "000")) {
                ++num_recs;
                if (num_recs == 1) recs = malloc(sizeof(char *));
                else recs = realloc(recs, (sizeof(char *)) * num_recs);
                recs[num_recs-1] = malloc(SIZ);
                strcpy(recs[num_recs-1], buf);
        }
+       if (listing) free(listing);
 
        do {
                scr_printf("\n");
@@ -1260,16 +1343,16 @@ void do_filterlist_configuration(CtdlIPC *ipc)
                for (i=0; i<num_recs; ++i) {
                color(DIM_WHITE);
                scr_printf("%3d ", i+1);
-               extract(buf, recs[i], 0);
+               extract_token(buf, recs[i], 0, '|', sizeof buf);
                color(BRIGHT_CYAN);
                scr_printf("%-28s ", buf);
-               extract(buf, recs[i], 1);
+               extract_token(buf, recs[i], 1, '|', sizeof buf);
                color(BRIGHT_MAGENTA);
                scr_printf("%-28s ", buf);
-               extract(buf, recs[i], 2);
+               extract_token(buf, recs[i], 2, '|', sizeof buf);
                color(BRIGHT_CYAN);
                scr_printf("%-16s\n", buf);
-               extract(buf, recs[i], 3);
+               extract_token(buf, recs[i], 3, '|', sizeof buf);
                color(DIM_WHITE);
                }
 
@@ -1290,6 +1373,7 @@ void do_filterlist_configuration(CtdlIPC *ipc)
                                        &buf[strlen(buf)], 16);
                                strcat(buf, "|");
                                recs[num_recs-1] = strdup(buf);
+                               modified = 1;
                                break;
                        case 'd':
                                i = intprompt("Delete which one",
@@ -1298,30 +1382,37 @@ void do_filterlist_configuration(CtdlIPC *ipc)
                                --num_recs;
                                for (j=i; j<num_recs; ++j)
                                        recs[j] = recs[j+1];
+                               modified = 1;
                                break;
                        case 's':
-                               snprintf(buf, sizeof buf, "CONF putsys|%s", FILTERLIST);
-                               CtdlIPC_putline(ipc, buf);
-                               CtdlIPC_getline(ipc, buf);
-                               if (buf[0] == '4') {
-                                       for (i=0; i<num_recs; ++i) {
-                                               CtdlIPC_putline(ipc, recs[i]);
-                                       }
-                                       CtdlIPC_putline(ipc, "000");
+                               r = 1;
+                               for (i = 0; i < num_recs; ++i)
+                                       r += 1 + strlen(recs[i]);
+                               listing = (char*) calloc(1, r);
+                               if (!listing) {
+                                       err_printf("Can't save config - out of memory!\n");
+                                       logoff(ipc, 1);
                                }
-                               else {
-                                       scr_printf("%s\n", &buf[4]);
+                               if (num_recs) for (i = 0; i < num_recs; ++i) {
+                                       strcat(listing, recs[i]);
+                                       strcat(listing, "\n");
+                               }
+                               r = CtdlIPCSetSystemConfigByType(ipc, FILTERLIST, listing, buf);
+                               if (r / 100 != 4) {
+                                       scr_printf("%s\n", buf);
+                               } else {
+                                       scr_printf("Wrote %d records.\n", num_recs);
+                                       modified = 0;
                                }
-                               quitting = 1;
                                break;
                        case 'q':
-                               quitting = boolprompt(
+                               quitting = !modified || boolprompt(
                                        "Quit without saving", 0);
                                break;
                        default:
                                badkey = 1;
                }
-       } while (quitting == 0);
+       } while (!quitting);
 
        if (recs != NULL) {
                for (i=0; i<num_recs; ++i) free(recs[i]);