Unwrapped some old 80-column code
authorArt Cancro <ajc@citadel.org>
Sat, 27 Aug 2022 17:10:29 +0000 (13:10 -0400)
committerArt Cancro <ajc@citadel.org>
Sat, 27 Aug 2022 17:10:29 +0000 (13:10 -0400)
citadel/server/modules/smtp/serv_smtp.c
ctdlsh/main.c
textclient/tuiconfig.c

index b7e269d86b6739749f3db79e1fa6014956d38379..c30df54092da761153ea6b3b64f59cb60b415969 100644 (file)
@@ -48,6 +48,7 @@
 #include <sys/socket.h>
 #include <netinet/in.h>
 #include <arpa/inet.h>
+#include <assert.h>
 #include <libcitadel.h>
 #include "../../citadel.h"
 #include "../../server.h"
@@ -110,7 +111,7 @@ void smtp_greeting(int is_msa) {
                        else
                                cprintf("550 %s\r\n", message_to_spammer);
                        CC->kill_me = KILLME_SPAMMER;
-                       /* no need to free_recipients(valid), it's not allocated yet */
+                       // no need to free_recipients(valid), it's not allocated yet
                        return;
                }
        }
@@ -205,11 +206,7 @@ void smtp_hello(int which_command) {
        }
        else {
                if (which_command == EHLO) {
-                       cprintf("250-Hello %s (%s [%s])\r\n",
-                               ChrPtr(SMTP->helo_node),
-                               CC->cs_host,
-                               CC->cs_addr
-                       );
+                       cprintf("250-Hello %s (%s [%s])\r\n", ChrPtr(SMTP->helo_node), CC->cs_host, CC->cs_addr);
                }
                else {
                        cprintf("250-Greetings and joyous salutations.\r\n");
@@ -219,9 +216,8 @@ void smtp_hello(int which_command) {
 
 #ifdef HAVE_OPENSSL
                // Offer TLS, but only if TLS is not already active.
-               // Furthermore, only offer TLS when running on
-               // the SMTP-MSA port, not on the SMTP-MTA port, due to
-               // questionable reliability of TLS in certain sending MTA's.
+               // Furthermore, only offer TLS when running on the SMTP-MSA port, not on the SMTP-MTA port,
+               // because if our server doesn't have a trusted certificate, some mailers will refuse to talk to it.
                if ( (!CC->redirect_ssl) && (SMTP->is_msa) ) {
                        cprintf("250-STARTTLS\r\n");
                }
@@ -444,10 +440,7 @@ void smtp_auth(void) {
                        return;
                }
 
-               len = extract_token(encoded_authstring, 
-                                   ChrPtr(SMTP->Cmd) + 5,
-                                   1, ' ',
-                                   sizeof encoded_authstring);
+               len = extract_token(encoded_authstring, ChrPtr(SMTP->Cmd) + 5, 1, ' ', sizeof encoded_authstring);
                StrBufPlain(SMTP->Cmd, encoded_authstring, len);
                smtp_try_plain();
                return;
@@ -743,11 +736,10 @@ void smtp_data(void) {
                    ((CtdlGetConfigInt("c_rfc822_strict_from") == CFG_SMTP_FROM_CORRECT) || 
                     (CtdlGetConfigInt("c_rfc822_strict_from") == CFG_SMTP_FROM_REJECT)    )  )
                {
-                       if (!IsEmptyStr(CC->cs_inet_email))
+                       if (!IsEmptyStr(CC->cs_inet_email)) {
                                validemail = strcmp(CC->cs_inet_email, msg->cm_fields[erFc822Addr]) == 0;
-                       if ((!validemail) && 
-                           (!IsEmptyStr(CC->cs_inet_other_emails)))
-                       {
+                       }
+                       if ((!validemail) && (!IsEmptyStr(CC->cs_inet_other_emails))) {
                                int num_secondary_emails = 0;
                                int i;
                                num_secondary_emails = num_tokens(CC->cs_inet_other_emails, '|');
@@ -887,10 +879,7 @@ void smtp_quit(void) {
 void smtp_command_loop(void) {
        static const ConstStr AuthPlainStr = {HKEY("AUTH PLAIN")};
 
-       if (SMTP == NULL) {
-               syslog(LOG_ERR, "serv_smtp: Session SMTP data is null.  WTF?  We will crash now.");
-               abort();
-       }
+       assert(SMTP != NULL);
 
        time(&CC->lastcmd);
        if (CtdlClientGetLine(SMTP->Cmd) < 1) {
@@ -986,16 +975,14 @@ void smtp_command_loop(void) {
 }
 
 
-/*****************************************************************************/
-/*                      MODULE INITIALIZATION STUFF                          */
-/*****************************************************************************/
-/*
- * This cleanup function blows away the temporary memory used by
- * the SMTP server.
- */
-void smtp_cleanup_function(void)
-{
-       /* Don't do this stuff if this is not an SMTP session! */
+// *****************************************************************************
+// *                      MODULE INITIALIZATION STUFF                          *
+// *****************************************************************************
+
+// This cleanup function blows away the temporary memory used by
+// the SMTP server.
+void smtp_cleanup_function(void) {
+       // Don't do this stuff if this is not an SMTP session!
        if (CC->h_command_function != smtp_command_loop) return;
 
        syslog(LOG_DEBUG, "Performing SMTP cleanup hook");
@@ -1021,7 +1008,7 @@ const char *CitadelServiceSMTP_LMTP_UNF="LMTP-UnF";
 // Initialization function, called from modules_init.c
 char *ctdl_module_init_smtp(void) {
        if (!threading) {
-               CtdlRegisterServiceHook(CtdlGetConfigInt("c_smtp_port"),        /* SMTP MTA */
+               CtdlRegisterServiceHook(CtdlGetConfigInt("c_smtp_port"),        // SMTP MTA
                                        NULL,
                                        smtp_mta_greeting,
                                        smtp_command_loop,
@@ -1029,7 +1016,7 @@ char *ctdl_module_init_smtp(void) {
                                        CitadelServiceSMTP_MTA);
 
 #ifdef HAVE_OPENSSL
-               CtdlRegisterServiceHook(CtdlGetConfigInt("c_smtps_port"),       /* SMTPS MTA */
+               CtdlRegisterServiceHook(CtdlGetConfigInt("c_smtps_port"),       // SMTPS MTA
                                        NULL,
                                        smtps_greeting,
                                        smtp_command_loop,
@@ -1037,21 +1024,21 @@ char *ctdl_module_init_smtp(void) {
                                        CitadelServiceSMTPS_MTA);
 #endif
 
-               CtdlRegisterServiceHook(CtdlGetConfigInt("c_msa_port"),         /* SMTP MSA */
+               CtdlRegisterServiceHook(CtdlGetConfigInt("c_msa_port"),         // SMTP MSA
                                        NULL,
                                        smtp_msa_greeting,
                                        smtp_command_loop,
                                        NULL,
                                        CitadelServiceSMTP_MSA);
 
-               CtdlRegisterServiceHook(0,                      /* local LMTP */
+               CtdlRegisterServiceHook(0,                                      // local LMTP
                                        file_lmtp_socket,
                                        lmtp_greeting,
                                        smtp_command_loop,
                                        NULL,
                                        CitadelServiceSMTP_LMTP);
 
-               CtdlRegisterServiceHook(0,                      /* local LMTP */
+               CtdlRegisterServiceHook(0,                                      // local LMTP
                                        file_lmtp_unfiltered_socket,
                                        lmtp_unfiltered_greeting,
                                        smtp_command_loop,
@@ -1061,6 +1048,6 @@ char *ctdl_module_init_smtp(void) {
                CtdlRegisterSessionHook(smtp_cleanup_function, EVT_STOP, PRIO_STOP + 250);
        }
        
-       /* return our module name for the log */
+       // return our module name for the log
        return "smtp";
 }
index 93265244e3bb3590ce6418af3720897bf7b9af19..7e1d93e5baf321da28a67e1cdd1240f0cb1701c5 100644 (file)
@@ -17,17 +17,17 @@ typedef struct {
 } COMMAND;
 
 COMMAND commands[] = {
-       {"?", cmd_help, "Display this message"},
-       {"help", cmd_help, "Display this message"},
-       {"date", cmd_datetime, "Print the server's date and time"},
-       {"config", cmd_config, "Configure the Citadel server"},
-       {"export", cmd_export, "Export all Citadel databases"},
-       {"shutdown", cmd_shutdown, "Shut down the Citadel server"},
-       {"time", cmd_datetime, "Print the server's date and time"},
-       {"passwd", cmd_passwd, "Set or change an account password"},
-       {"who", cmd_who, "Display a list of online users"},
-       {"mailq", cmd_mailq, "Show the outbound email queue"},
-       {NULL, NULL, NULL}
+       { "?", cmd_help, "Display this message" },
+       { "help", cmd_help, "Display this message" },
+       { "date", cmd_datetime, "Print the server's date and time" },
+       { "config", cmd_config, "Configure the Citadel server" },
+       { "export", cmd_export, "Export all Citadel databases" },
+       { "shutdown", cmd_shutdown, "Shut down the Citadel server" },
+       { "time", cmd_datetime, "Print the server's date and time" },
+       { "passwd", cmd_passwd, "Set or change an account password" },
+       { "who", cmd_who, "Display a list of online users" },
+       { "mailq", cmd_mailq, "Show the outbound email queue" },
+       { NULL, NULL, NULL }
 };
 
 
@@ -97,7 +97,8 @@ void do_main_loop(int server_socket)
        sock_getln(server_socket, buf, sizeof buf);
        if (buf[0] == '1') {
                i = 0;
-               while (sock_getln(server_socket, buf, sizeof buf), strcmp(buf, "000")) {
+               while (sock_getln(server_socket, buf, sizeof buf),
+                      strcmp(buf, "000")) {
                        if (i == 1) {
                                sprintf(prompt, "\n%s> ", buf);
                        }
@@ -107,7 +108,7 @@ void do_main_loop(int server_socket)
 
        /* Here we go ... main command loop */
        rl_attempted_completion_function = command_name_completion;
-       while ( (cmd = readline(prompt)) , cmd ) {
+       while ((cmd = readline(prompt)), cmd) {
                if (*cmd) {
                        add_history(cmd);
                        ret = do_one_command(server_socket, cmd);
@@ -143,9 +144,10 @@ int main(int argc, char **argv)
        int is_interactive = ((strlen(cmd) == 0) ? 1 : 0);
 
        if (is_interactive) {
-               printf("\nCitadel administration shell (c) 2009-2020 by citadel.org\n"
-                      "This is open source software made available to you under the terms\n"
-                      "of the GNU General Public License v3.  All other rights reserved.\n");
+               printf
+                   ("\nCitadel administration shell (c) 2009-2020 by citadel.org\n"
+                    "This is open source software made available to you under the terms\n"
+                    "of the GNU General Public License v3.  All other rights reserved.\n");
                printf("Connecting to Citadel server in %s...\n", ctdldir);
        }
 
index 4a5fb97096afbaa41f55027498b37abf7c6dd422..12038ba91276a62c7ba96417fb230640f6b5cedd 100644 (file)
@@ -1,6 +1,6 @@
 // Configuration screens that are part of the text mode client.
 //
-// Copyright (c) 1987-2018 by the citadel.org team
+// Copyright (c) 1987-2022 by the citadel.org team
 //
 // This program is open source software.  Use, duplication, and/or
 // disclosure are subject to the GNU General Purpose License version 3.
@@ -22,9 +22,7 @@ extern int screenwidth;
 char editor_path[PATH_MAX];
 
 
-/* 
- * General system configuration command
- */
+// General system configuration command
 void do_system_configuration(CtdlIPC * ipc) {
        char buf[256];
        char sc[NUM_CONFIGS][256];
@@ -33,13 +31,13 @@ void do_system_configuration(CtdlIPC * ipc) {
        struct ExpirePolicy *mbx_expirepolicy = NULL;
        int a;
        int logpages = 0;
-       int r;                  /* IPC response code */
+       int r;                  // IPC response code
        int server_configs = 0;
 
-       /* Clear out the config buffers */
+       // Clear out the config buffers
        memset(&sc[0][0], 0, sizeof(sc));
 
-       /* Fetch the current config */
+       // Fetch the current config
        r = CtdlIPCGetSystemConfig(ipc, &resp, buf);
        if (r / 100 == 1) {
                server_configs = num_tokens(resp, '\n');
@@ -53,14 +51,12 @@ void do_system_configuration(CtdlIPC * ipc) {
                free(resp);
        }
        resp = NULL;
-       /* Fetch the expire policy (this will silently fail on old servers,
-        * resulting in "default" policy)
-        */
+
+       // Fetch the expire policy (this will silently fail on old servers, resulting in "default" policy)
        r = CtdlIPCGetMessageExpirationPolicy(ipc, 2, &site_expirepolicy, buf);
        r = CtdlIPCGetMessageExpirationPolicy(ipc, 3, &mbx_expirepolicy, buf);
 
-       /* Identification parameters */
-
+       // Identification parameters
        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);
@@ -69,21 +65,16 @@ void do_system_configuration(CtdlIPC * ipc) {
        strprompt("Name of system administrator", &sc[13][0], 25);
        strprompt("Paginator prompt", &sc[10][0], 79);
 
-       /* Security parameters */
-
+       // Security parameters
        snprintf(sc[7], sizeof sc[7], "%d", (boolprompt("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]))));
        strprompt("Initial access level for new users", &sc[6][0], 1);
        strprompt("Access level required to create rooms", &sc[19][0], 1);
        snprintf(sc[67], sizeof sc[67], "%d", (boolprompt("Allow anonymous guest logins", atoi(&sc[67][0]))));
-       snprintf(sc[4], sizeof sc[4], "%d", (boolprompt("Automatically give room admin privs to a user who creates a private room",
-                                                       atoi(&sc[4][0]))));
-
+       snprintf(sc[4], sizeof sc[4], "%d", (boolprompt("Automatically give room admin 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]))));
-
        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]))));
+       snprintf(sc[11], sizeof sc[11], "%d", (boolprompt("Restrict Internet mail to only those with that privilege", atoi(&sc[11][0]))));
        snprintf(sc[26], sizeof sc[26], "%d", (boolprompt("Allow admins to Zap (forget) rooms", atoi(&sc[26][0]))));
 
        if (!IsEmptyStr(&sc[18][0])) {
@@ -100,16 +91,7 @@ void do_system_configuration(CtdlIPC * ipc) {
                sc[18][0] = 0;
        }
 
-       /* Commented out because this setting isn't really appropriate to
-        * change while the server is running.
-        *
-        * snprintf(sc[52], sizeof sc[52], "%d", (boolprompt(
-        *      "Use system authentication",
-        *      atoi(&sc[52][0]))));
-        */
-
-       /* Server tuning */
-
+       // Server tuning
        strprompt("Server connection idle timeout (in seconds)", &sc[5][0], 4);
        strprompt("Maximum concurrent sessions", &sc[14][0], 4);
        strprompt("Maximum message length", &sc[20][0], 20);
@@ -131,9 +113,7 @@ void do_system_configuration(CtdlIPC * ipc) {
        strprompt("XMPP (Jabber) client to server port (-1 to disable)", &sc[62][0], 5);
        // strprompt("XMPP (Jabber) server to server port (-1 to disable)", &sc[63][0], 5);   This is just a placeholder.
 
-       /* This logic flips the question around, because it's one of those
-        * situations where 0=yes and 1=no
-        */
+       // This logic flips the question around, because it's one of those situations where 0=yes and 1=no
        a = atoi(sc[25]);
        a = (a ? 0 : 1);
        a = boolprompt("Correct forged From: lines during authenticated SMTP", a);
@@ -142,24 +122,20 @@ void do_system_configuration(CtdlIPC * ipc) {
 
        snprintf(sc[66], sizeof sc[66], "%d", (boolprompt("Flag messages as spam instead of rejecting", atoi(&sc[66][0]))));
 
-       /* This logic flips the question around, because it's one of those
-        * situations where 0=yes and 1=no
-        */
+       // This logic flips the question around, because it's one of those situations where 0=yes and 1=no
        a = atoi(sc[61]);
        a = (a ? 0 : 1);
        a = boolprompt("Force IMAP posts in public rooms to be from the user who submitted them", a);
        a = (a ? 0 : 1);
        snprintf(sc[61], sizeof sc[61], "%d", a);
 
-       snprintf(sc[45], sizeof sc[45], "%d", (boolprompt("Allow unauthenticated SMTP clients to spoof my domains",
-                                                         atoi(&sc[45][0]))));
-       snprintf(sc[57], sizeof sc[57], "%d", (boolprompt("Perform RBL checks at greeting instead of after RCPT",
-                                                         atoi(&sc[57][0]))));
+       snprintf(sc[45], sizeof sc[45], "%d", (boolprompt("Allow unauthenticated SMTP clients to spoof my domains", atoi(&sc[45][0]))));
+       snprintf(sc[57], sizeof sc[57], "%d", (boolprompt("Perform RBL checks at greeting instead of after RCPT", atoi(&sc[57][0]))));
 
-       /* LDAP settings */
+       // LDAP settings
        if (ipc->ServInfo.supports_ldap) {
                a = strlen(&sc[32][0]);
-               a = (a ? 1 : 0);        /* Set only to 1 or 0 */
+               a = (a ? 1 : 0);        // Set only to 1 or 0
                a = boolprompt("Do you want to configure LDAP authentication?", a);
                if (a) {
                        strprompt("Host name of LDAP server", &sc[32][0], 127);
@@ -167,16 +143,17 @@ void do_system_configuration(CtdlIPC * ipc) {
                        strprompt("Base DN", &sc[34][0], 255);
                        strprompt("Bind DN (or blank for anonymous bind)", &sc[35][0], 255);
                        strprompt("Password for bind DN (or blank for anonymous bind)", &sc[36][0], 255);
-               } else {
+               }
+               else {
                        strcpy(&sc[32][0], "");
                }
        }
 
-       /* Expiry settings */
+       // Expiry settings
        strprompt("Default user purge time (days)", &sc[16][0], 5);
        strprompt("Default room purge time (days)", &sc[17][0], 5);
 
-       /* Angels and demons dancing in my head... */
+       // Angels and demons dancing in my head...
        do {
                snprintf(buf, sizeof buf, "%d", site_expirepolicy->expire_mode);
                strprompt("System default message expire policy (? for list)", buf, 1);
@@ -188,7 +165,7 @@ void do_system_configuration(CtdlIPC * ipc) {
        } while ((buf[0] < '1') || (buf[0] > '3'));
        site_expirepolicy->expire_mode = buf[0] - '0';
 
-       /* ...lunatics and monsters underneath my bed */
+       // ...lunatics and monsters underneath my bed
        if (site_expirepolicy->expire_mode == 2) {
                snprintf(buf, sizeof buf, "%d", site_expirepolicy->expire_value);
                strprompt("Keep how many messages online?", buf, 10);
@@ -200,7 +177,7 @@ void do_system_configuration(CtdlIPC * ipc) {
                site_expirepolicy->expire_value = atol(buf);
        }
 
-       /* Media messiahs preying on my fears... */
+       // Media messiahs preying on my fears...
        do {
                snprintf(buf, sizeof buf, "%d", mbx_expirepolicy->expire_mode);
                strprompt("Mailbox default message expire policy (? for list)", buf, 1);
@@ -213,7 +190,7 @@ void do_system_configuration(CtdlIPC * ipc) {
        } while ((buf[0] < '0') || (buf[0] > '3'));
        mbx_expirepolicy->expire_mode = buf[0] - '0';
 
-       /* ...Pop culture prophets playing in my ears */
+       // ...Pop culture prophets playing in my ears
        if (mbx_expirepolicy->expire_mode == 2) {
                snprintf(buf, sizeof buf, "%d", mbx_expirepolicy->expire_value);
                strprompt("Keep how many messages online?", buf, 10);
@@ -236,13 +213,13 @@ void do_system_configuration(CtdlIPC * ipc) {
                strprompt("Email destination of journalized messages", &sc[48][0], 127);
        }
 
-       /* No more Funambol */
+       // No more Funambol
        sc[53][0] = 0;
        sc[54][0] = 0;
        sc[55][0] = 0;
        sc[56][0] = 0;
 
-       /* External pager stuff */
+       // External pager stuff
        int yes_pager = 0;
        if (strlen(sc[60]) > 0) {
                yes_pager = 1;
@@ -250,11 +227,12 @@ void do_system_configuration(CtdlIPC * ipc) {
        yes_pager = boolprompt("Configure an external pager tool", yes_pager);
        if (yes_pager) {
                strprompt("External pager tool", &sc[60][0], 255);
-       } else {
+       }
+       else {
                sc[60][0] = 0;
        }
 
-       /* Save it */
+       // Save it
        scr_printf("Save this configuration? ");
        if (yesno()) {
                r = 1;
@@ -294,11 +272,8 @@ void do_system_configuration(CtdlIPC * ipc) {
 }
 
 
-/*
- * support function for do_internet_configuration()
- */
-void get_inet_rec_type(CtdlIPC * ipc, char *buf)
-{
+// support function for do_internet_configuration()
+void get_inet_rec_type(CtdlIPC * ipc, char *buf) {
        int sel;
 
        keyopt(" <1> localhost      (Alias for this computer)\n");
@@ -335,11 +310,8 @@ void get_inet_rec_type(CtdlIPC * ipc, char *buf)
 }
 
 
-/*
- * Internet mail configuration
- */
-void do_internet_configuration(CtdlIPC * ipc)
-{
+// Internet mail configuration
+void do_internet_configuration(CtdlIPC * ipc) {
        char buf[256];
        char *resp = NULL;
        int num_recs = 0;
@@ -356,7 +328,7 @@ void do_internet_configuration(CtdlIPC * ipc)
                        extract_token(buf, resp, 0, '\n', sizeof buf);
                        remove_token(resp, 0, '\n');
 
-                       // VILE SLEAZY HACK: replace obsolete "directory" domains with "localhost"
+                       // "directory" is no longer used.  replace it with "localhost"
                        char *d = strstr(buf, "|directory");
                        if (d != NULL) {
                                strcpy(d, "|localhost");
@@ -371,8 +343,9 @@ void do_internet_configuration(CtdlIPC * ipc)
                        strcpy(recs[num_recs - 1], buf);
                }
        }
-       if (resp)
+       if (resp) {
                free(resp);
+       }
 
        do {
                scr_printf("\n");
@@ -401,7 +374,8 @@ void do_internet_configuration(CtdlIPC * ipc)
                                ++num_recs;
                                if (num_recs == 1) {
                                        recs = malloc(sizeof(char *));
-                               } else {
+                               }
+                               else {
                                        recs = realloc(recs, (sizeof(char *)) * num_recs);
                                }
                                strcat(buf, "|");
@@ -421,8 +395,9 @@ void do_internet_configuration(CtdlIPC * ipc)
                        break;
                case 's':
                        r = 1;
-                       for (i = 0; i < num_recs; i++)
+                       for (i = 0; i < num_recs; i++) {
                                r += 1 + strlen(recs[i]);
+                       }
                        resp = (char *) calloc(1, r);
                        if (!resp) {
                                scr_printf("Can't save config - out of memory!\n");
@@ -436,7 +411,8 @@ void do_internet_configuration(CtdlIPC * ipc)
                        r = CtdlIPCSetSystemConfigByType(ipc, INTERNETCFG, resp, buf);
                        if (r / 100 != 4) {
                                scr_printf("%s\n", buf);
-                       } else {
+                       }
+                       else {
                                scr_printf("Wrote %d records.\n", num_recs);
                                modified = 0;
                        }
@@ -458,12 +434,8 @@ void do_internet_configuration(CtdlIPC * ipc)
 }
 
 
-
-/*
- * Edit network configuration for room sharing, mailing lists, etc.
- */
-void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment)
-{
+// Edit network configuration for room sharing, mailing lists, etc.
+void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment) {
        char filename[PATH_MAX];
        char changefile[PATH_MAX];
        int e_ex_code;
@@ -521,7 +493,7 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment)
        }
        fclose(tempfp);
 
-       e_ex_code = 1;          /* start with a failed exit code */
+       e_ex_code = 1;          // start with a failed exit code
        stty_ctdl(SB_RESTORE);
        editor_pid = fork();
        cksum = file_checksum(filename);
@@ -545,10 +517,10 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment)
                e_ex_code = 1;
        }
 
-       if (e_ex_code == 0) {   /* Save changes */
+       if (e_ex_code == 0) {   // Save changes
                changefp = fopen(changefile, "w");
 
-               /* Load all netconfig entries that are *not* of the type we are editing */
+               // Load all netconfig entries that are *not* of the type we are editing
                r = CtdlIPCGetRoomNetworkConfig(ipc, &listing, buf);
                if (r / 100 == 1) {
                        while (listing && !IsEmptyStr(listing)) {
@@ -565,7 +537,7 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment)
                        listing = NULL;
                }
 
-               /* ...and merge that with the data we just edited */
+               // ...and merge that with the data we just edited
                tempfp = fopen(filename, "r");
                while (fgets(buf, sizeof buf, tempfp) != NULL) {
                        for (i = 0; i < strlen(buf); ++i) {
@@ -580,7 +552,7 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment)
                fclose(tempfp);
                fclose(changefp);
 
-               /* now write it to the server... */
+               // now write it to the server...
                changefp = fopen(changefile, "r");
                if (changefp != NULL) {
                        listing = load_message_from_file(changefp);
@@ -593,16 +565,13 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment)
                }
        }
 
-       unlink(filename);       /* Delete the temporary files */
+       unlink(filename);       // Delete the temporary files
        unlink(changefile);
 }
 
 
-/*
- * POP3 aggregation client configuration
- */
-void do_pop3client_configuration(CtdlIPC * ipc)
-{
+// POP3 aggregation client configuration
+void do_pop3client_configuration(CtdlIPC * ipc) {
        char buf[SIZ];
        int num_recs = 0;
        char **recs = NULL;
@@ -708,7 +677,7 @@ void do_pop3client_configuration(CtdlIPC * ipc)
                                        strcat(listing, "\n");
                                }
 
-                       /* Retrieve all the *other* records for merging */
+                       // Retrieve all the *other* records for merging
                        r = CtdlIPCGetRoomNetworkConfig(ipc, &other_listing, buf);
                        if (r / 100 == 1) {
                                for (i = 0; i < num_tokens(other_listing, '\n'); ++i) {
@@ -727,7 +696,8 @@ void do_pop3client_configuration(CtdlIPC * ipc)
 
                        if (r / 100 != 4) {
                                scr_printf("%s\n", buf);
-                       } else {
+                       }
+                       else {
                                scr_printf("Wrote %d records.\n", num_recs);
                                modified = 0;
                        }
@@ -742,18 +712,16 @@ void do_pop3client_configuration(CtdlIPC * ipc)
        } while (!quitting);
 
        if (recs != NULL) {
-               for (i = 0; i < num_recs; ++i)
+               for (i = 0; i < num_recs; ++i) {
                        free(recs[i]);
+               }
                free(recs);
        }
 }
 
 
-/*
- * RSS feed retrieval client configuration
- */
-void do_rssclient_configuration(CtdlIPC * ipc)
-{
+// RSS feed retrieval client configuration
+void do_rssclient_configuration(CtdlIPC * ipc) {
        char buf[SIZ];
        int num_recs = 0;
        char **recs = NULL;
@@ -814,7 +782,8 @@ void do_rssclient_configuration(CtdlIPC * ipc)
                        ++num_recs;
                        if (num_recs == 1) {
                                recs = malloc(sizeof(char *));
-                       } else {
+                       }
+                       else {
                                recs = realloc(recs, (sizeof(char *)) * num_recs);
                        }
                        strcpy(buf, "rssclient|");
@@ -847,7 +816,7 @@ void do_rssclient_configuration(CtdlIPC * ipc)
                                        strcat(listing, "\n");
                                }
 
-                       /* Retrieve all the *other* records for merging */
+                       // Retrieve all the *other* records for merging
                        r = CtdlIPCGetRoomNetworkConfig(ipc, &other_listing, buf);
                        if (r / 100 == 1) {
                                for (i = 0; i < num_tokens(other_listing, '\n'); ++i) {
@@ -866,7 +835,8 @@ void do_rssclient_configuration(CtdlIPC * ipc)
 
                        if (r / 100 != 4) {
                                scr_printf("%s\n", buf);
-                       } else {
+                       }
+                       else {
                                scr_printf("Wrote %d records.\n", num_recs);
                                modified = 0;
                        }