removed some debugs
[citadel.git] / citadel / modules / smtp / serv_smtp.c
index 232f3672421125933d45d19a87c109516e07b160..b567f58e8e26bd7d6175a6d9aad657301a0f02b6 100644 (file)
@@ -1,35 +1,33 @@
-/*
- * This module is an SMTP and ESMTP server for the Citadel system.
- * It is compliant with all of the following:
- *
- * RFC  821 - Simple Mail Transfer Protocol
- * RFC  876 - Survey of SMTP Implementations
- * RFC 1047 - Duplicate messages and SMTP
- * RFC 1652 - 8 bit MIME
- * RFC 1869 - Extended Simple Mail Transfer Protocol
- * RFC 1870 - SMTP Service Extension for Message Size Declaration
- * RFC 2033 - Local Mail Transfer Protocol
- * RFC 2197 - SMTP Service Extension for Command Pipelining
- * RFC 2476 - Message Submission
- * RFC 2487 - SMTP Service Extension for Secure SMTP over TLS
- * RFC 2554 - SMTP Service Extension for Authentication
- * RFC 2821 - Simple Mail Transfer Protocol
- * RFC 2822 - Internet Message Format
- * RFC 2920 - SMTP Service Extension for Command Pipelining
- *  
- * The VRFY and EXPN commands have been removed from this implementation
- * because nobody uses these commands anymore, except for spammers.
- *
- * Copyright (c) 1998-2021 by the citadel.org team
- *
- * This program is open source software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 3.
- *  
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
+// This module is an SMTP and ESMTP server for the Citadel system.
+// It is compliant with all of the following:
+//
+// RFC  821 - Simple Mail Transfer Protocol
+// RFC  876 - Survey of SMTP Implementations
+// RFC 1047 - Duplicate messages and SMTP
+// RFC 1652 - 8 bit MIME
+// RFC 1869 - Extended Simple Mail Transfer Protocol
+// RFC 1870 - SMTP Service Extension for Message Size Declaration
+// RFC 2033 - Local Mail Transfer Protocol
+// RFC 2197 - SMTP Service Extension for Command Pipelining
+// RFC 2476 - Message Submission
+// RFC 2487 - SMTP Service Extension for Secure SMTP over TLS
+// RFC 2554 - SMTP Service Extension for Authentication
+// RFC 2821 - Simple Mail Transfer Protocol
+// RFC 2822 - Internet Message Format
+// RFC 2920 - SMTP Service Extension for Command Pipelining
+//  
+// The VRFY and EXPN commands have been removed from this implementation
+// because nobody uses these commands anymore, except for spammers.
+//
+// Copyright (c) 1998-2022 by the citadel.org team
+//
+// This program is open source software; you can redistribute it and/or modify
+// it under the terms of the GNU General Public License version 3.
+//  
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
 
 #include "sysdep.h"
 #include <stdlib.h>
@@ -71,7 +69,7 @@
 
 #include "smtp_util.h"
 
-enum {                         /* Command states for login authentication */
+enum {                         // Command states for login authentication
        smtp_command,
        smtp_user,
        smtp_password,
@@ -84,43 +82,16 @@ enum SMTP_FLAGS {
        LHLO
 };
 
-typedef void (*smtp_handler)(long offset, long Flags);
-typedef struct _smtp_handler_hook {
-       smtp_handler h;
-       int Flags;
-} smtp_handler_hook;
-
-HashList *SMTPCmds = NULL;
-#define MaxSMTPCmdLen 10
-
-#define RegisterSmtpCMD(First, H, Flags) registerSmtpCMD(HKEY(First), H, Flags)
-void registerSmtpCMD(const char *First, long FLen, smtp_handler H, int Flags) {
-       smtp_handler_hook *h;
-
-       if (FLen >= MaxSMTPCmdLen) {
-               abort();
-       }
-
-       h = (smtp_handler_hook*) malloc(sizeof(smtp_handler_hook));
-       memset(h, 0, sizeof(smtp_handler_hook));
-
-       h->Flags = Flags;
-       h->h = H;
-       Put(SMTPCmds, First, FLen, h, NULL);
-}
-
 
-/*
- * Here's where our SMTP session begins its happy day.
- */
+// Here's where our SMTP session begins its happy day.
 void smtp_greeting(int is_msa) {
        char message_to_spammer[1024];
 
        strcpy(CC->cs_clientname, "SMTP session");
        CC->internal_pgm = 1;
        CC->cs_flags |= CS_STEALTH;
-       CC->session_specific_data = malloc(sizeof(citsmtp));
-       memset(SMTP, 0, sizeof(citsmtp));
+       CC->session_specific_data = malloc(sizeof(struct citsmtp));
+       memset(SMTP, 0, sizeof(struct citsmtp));
        SMTP->is_msa = is_msa;
        SMTP->Cmd = NewStrBufPlain(NULL, SIZ);
        SMTP->helo_node = NewStrBuf();
@@ -130,9 +101,8 @@ void smtp_greeting(int is_msa) {
        SMTP->preferred_sender_email = NULL;
        SMTP->preferred_sender_name = NULL;
 
-       /* If this config option is set, reject connections from problem
-        * addresses immediately instead of after they execute a RCPT
-        */
+       // If this config option is set, reject connections from problem
+       // addresses immediately instead of after they execute a RCPT
        if ( (CtdlGetConfigInt("c_rbl_at_greeting")) && (SMTP->is_msa == 0) ) {
                if (rbl_check(CC->cs_addr, message_to_spammer)) {
                        if (server_shutting_down)
@@ -145,45 +115,38 @@ void smtp_greeting(int is_msa) {
                }
        }
 
-       /* Otherwise we're either clean or we check later. */
+       // Otherwise we're either clean or we check later.
 
        if (CC->nologin==1) {
                cprintf("451 Too many connections are already open; please try again later.\r\n");
                CC->kill_me = KILLME_MAX_SESSIONS_EXCEEDED;
-               /* no need to free_recipients(valid), it's not allocated yet */
+               // no need to free_recipients(valid), it's not allocated yet
                return;
        }
 
-       /* Note: the FQDN *must* appear as the first thing after the 220 code.
-        * Some clients (including citmail.c) depend on it being there.
-        */
+       // Note: the FQDN *must* appear as the first thing after the 220 code.
+       // Some clients (including citmail.c) depend on it being there.
        cprintf("220 %s ESMTP Citadel server ready.\r\n", CtdlGetConfigStr("c_fqdn"));
 }
 
 
-/*
- * SMTPS is just like SMTP, except it goes crypto right away.
- */
+// SMTPS is just like SMTP, except it goes crypto right away.
 void smtps_greeting(void) {
        CtdlModuleStartCryptoMsgs(NULL, NULL, NULL);
 #ifdef HAVE_OPENSSL
-       if (!CC->redirect_ssl) CC->kill_me = KILLME_NO_CRYPTO;          /* kill session if no crypto */
+       if (!CC->redirect_ssl) CC->kill_me = KILLME_NO_CRYPTO;          // kill session if no crypto
 #endif
        smtp_greeting(0);
 }
 
 
-/*
- * SMTP MSA port requires authentication.
- */
+// SMTP MSA port requires authentication.
 void smtp_msa_greeting(void) {
        smtp_greeting(1);
 }
 
 
-/*
- * LMTP is like SMTP but with some extra bonus footage added.
- */
+// LMTP is like SMTP but with some extra bonus footage added.
 void lmtp_greeting(void) {
 
        smtp_greeting(0);
@@ -191,17 +154,13 @@ void lmtp_greeting(void) {
 }
 
 
-/* 
- * Generic SMTP MTA greeting
- */
+// Generic SMTP MTA greeting
 void smtp_mta_greeting(void) {
        smtp_greeting(0);
 }
 
 
-/*
- * We also have an unfiltered LMTP socket that bypasses spam filters.
- */
+// We also have an unfiltered LMTP socket that bypasses spam filters.
 void lmtp_unfiltered_greeting(void) {
        smtp_greeting(0);
        SMTP->is_lmtp = 1;
@@ -209,10 +168,8 @@ void lmtp_unfiltered_greeting(void) {
 }
 
 
-/*
- * Login greeting common to all auth methods
- */
-void smtp_auth_greeting(long offset, long Flags) {
+// Login greeting common to all auth methods
+void smtp_auth_greeting(void) {
        cprintf("235 Hello, %s\r\n", CC->user.fullname);
        syslog(LOG_INFO, "serv_smtp: SMTP authenticated %s", CC->user.fullname);
        CC->internal_pgm = 0;
@@ -220,14 +177,14 @@ void smtp_auth_greeting(long offset, long Flags) {
 }
 
 
-/*
- * Implement HELO and EHLO commands.
- *
- * which_command:  0=HELO, 1=EHLO, 2=LHLO
- */
-void smtp_hello(long offset, long which_command) {
+// Implement HELO and EHLO commands.
+// which_command:  0=HELO, 1=EHLO, 2=LHLO
+void smtp_hello(int which_command) {
 
-       StrBufAppendBuf (SMTP->helo_node, SMTP->Cmd, offset);
+       if (StrLength(SMTP->Cmd) >= 6) {
+               FlushStrBuf(SMTP->helo_node);
+               StrBufAppendBuf(SMTP->helo_node, SMTP->Cmd, 5);
+       }
 
        if ( (which_command != LHLO) && (SMTP->is_lmtp) ) {
                cprintf("500 Only LHLO is allowed when running LMTP\r\n");
@@ -261,16 +218,14 @@ void smtp_hello(long offset, long which_command) {
                cprintf("250-SIZE %ld\r\n", CtdlGetConfigLong("c_maxmsglen"));
 
 #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.
-                */
+               // 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.
                if ( (!CC->redirect_ssl) && (SMTP->is_msa) ) {
                        cprintf("250-STARTTLS\r\n");
                }
-#endif /* HAVE_OPENSSL */
+#endif
 
                cprintf("250-AUTH LOGIN PLAIN\r\n"
                        "250-AUTH=LOGIN PLAIN\r\n"
@@ -280,16 +235,14 @@ void smtp_hello(long offset, long which_command) {
 }
 
 
-/*
- * Backend function for smtp_webcit_preferences_hack().
- * Look at a message and determine if it's the preferences file.
- */
+// Backend function for smtp_webcit_preferences_hack().
+// Look at a message and determine if it's the preferences file.
 void smtp_webcit_preferences_hack_backend(long msgnum, void *userdata) {
        struct CtdlMessage *msg;
        char **webcit_conf = (char **) userdata;
 
        if (*webcit_conf) {
-               return; // already got it
+               return;         // already got it
        }
 
        msg = CtdlFetchMessage(msgnum, 1);
@@ -298,7 +251,7 @@ void smtp_webcit_preferences_hack_backend(long msgnum, void *userdata) {
        }
 
        if ( !CM_IsEmpty(msg, eMsgSubject) && (!strcasecmp(msg->cm_fields[eMsgSubject], "__ WebCit Preferences __"))) {
-               /* This is it!  Change ownership of the message text so it doesn't get freed. */
+               // This is it!  Change ownership of the message text so it doesn't get freed.
                *webcit_conf = (char *)msg->cm_fields[eMesageText];
                msg->cm_fields[eMesageText] = NULL;
        }
@@ -306,10 +259,8 @@ void smtp_webcit_preferences_hack_backend(long msgnum, void *userdata) {
 }
 
 
-/*
- * The configuration item for the user's preferred display name for outgoing email is, unfortunately,
- * stored in the account's WebCit configuration.  We have to fetch it now.
- */
+// The configuration item for the user's preferred display name for outgoing email is, unfortunately,
+// stored in the account's WebCit configuration.  We have to fetch it now.
 void smtp_webcit_preferences_hack(void) {
        char config_roomname[ROOMNAMELEN];
        char *webcit_conf = NULL;
@@ -319,17 +270,14 @@ void smtp_webcit_preferences_hack(void) {
                return;
        }
 
-       /*
-        * Find the WebCit configuration message
-        */
-
+       // Find the WebCit configuration message
        CtdlForEachMessage(MSGS_ALL, 1, NULL, NULL, NULL, smtp_webcit_preferences_hack_backend, (void *)&webcit_conf);
 
        if (!webcit_conf) {
                return;
        }
 
-       /* Parse the webcit configuration and attempt to do something useful with it */
+       // Parse the webcit configuration and attempt to do something useful with it
        char *str = webcit_conf;
        char *saveptr = str;
        char *this_line = NULL;
@@ -350,23 +298,20 @@ void smtp_webcit_preferences_hack(void) {
 }
 
 
-/*
- * Implement HELP command.
- */
-void smtp_help(long offset, long Flags) {
+// Implement HELP command.
+void smtp_help(void) {
        cprintf("214 RTFM http://www.ietf.org/rfc/rfc2821.txt\r\n");
 }
 
 
-/*
- *
- */
-void smtp_get_user(long offset) {
+void smtp_get_user(int offset) {
        char buf[SIZ];
 
-       StrBufDecodeBase64(SMTP->Cmd);
+       StrBuf *UserName = NewStrBufDup(SMTP->Cmd);
+       StrBufCutLeft(UserName, offset);
+       StrBufDecodeBase64(UserName);
 
-       if (CtdlLoginExistingUser(ChrPtr(SMTP->Cmd)) == login_ok) {
+       if (CtdlLoginExistingUser(ChrPtr(UserName)) == login_ok) {
                size_t len = CtdlEncodeBase64(buf, "Password:", 9, 0);
 
                if (buf[len - 1] == '\n') {
@@ -379,21 +324,18 @@ void smtp_get_user(long offset) {
                cprintf("500 No such user.\r\n");
                SMTP->command_state = smtp_command;
        }
+       FreeStrBuf(&UserName);
 }
 
 
-/*
- *
- */
-void smtp_get_pass(long offset, long Flags)
-{
+void smtp_get_pass(void) {
        char password[SIZ];
 
        memset(password, 0, sizeof(password));
        StrBufDecodeBase64(SMTP->Cmd);
        syslog(LOG_DEBUG, "serv_smtp: trying <%s>", password);
        if (CtdlTryPassword(SKEY(SMTP->Cmd)) == pass_ok) {
-               smtp_auth_greeting(offset, Flags);
+               smtp_auth_greeting();
        }
        else {
                cprintf("535 Authentication failed.\r\n");
@@ -402,10 +344,8 @@ void smtp_get_pass(long offset, long Flags)
 }
 
 
-/*
- * Back end for PLAIN auth method (either inline or multistate)
- */
-void smtp_try_plain(long offset, long Flags) {
+// Back end for PLAIN auth method (either inline or multistate)
+void smtp_try_plain(void) {
        const char*decoded_authstring;
        char ident[256] = "";
        char user[256] = "";
@@ -419,8 +359,7 @@ void smtp_try_plain(long offset, long Flags) {
        memset(pass, 0, sizeof(pass));
        decoded_len = StrBufDecodeBase64(SMTP->Cmd);
 
-       if (decoded_len > 0)
-       {
+       if (decoded_len > 0) {
                decoded_authstring = ChrPtr(SMTP->Cmd);
 
                len = safestrncpy(ident, decoded_authstring, sizeof ident);
@@ -428,16 +367,14 @@ void smtp_try_plain(long offset, long Flags) {
                decoded_len -= len - 1;
                decoded_authstring += len + 1;
 
-               if (decoded_len > 0)
-               {
+               if (decoded_len > 0) {
                        len = safestrncpy(user, decoded_authstring, sizeof user);
 
                        decoded_authstring += len + 1;
                        decoded_len -= len - 1;
                }
 
-               if (decoded_len > 0)
-               {
+               if (decoded_len > 0) {
                        plen = safestrncpy(pass, decoded_authstring, sizeof pass);
 
                        if (plen < 0)
@@ -457,7 +394,7 @@ void smtp_try_plain(long offset, long Flags) {
        if (result == login_ok) {
                if (CtdlTryPassword(pass, plen) == pass_ok) {
                        smtp_webcit_preferences_hack();
-                       smtp_auth_greeting(offset, Flags);
+                       smtp_auth_greeting();
                        return;
                }
        }
@@ -465,10 +402,8 @@ void smtp_try_plain(long offset, long Flags) {
 }
 
 
-/*
- * Attempt to perform authenticated SMTP
- */
-void smtp_auth(long offset, long Flags) {
+// Attempt to perform authenticated SMTP
+void smtp_auth(void) {
        char username_prompt[64];
        char method[64];
        char encoded_authstring[1024];
@@ -478,11 +413,17 @@ void smtp_auth(long offset, long Flags) {
                return;
        }
 
-       extract_token(method, ChrPtr(SMTP->Cmd) + offset, 0, ' ', sizeof method);
+       if (StrLength(SMTP->Cmd) < 6) {
+               cprintf("501 Syntax error\r\n");
+               return;
+       }
+
+       extract_token(method, ChrPtr(SMTP->Cmd) + 5, 0, ' ', sizeof method);
 
        if (!strncasecmp(method, "login", 5) ) {
-               if (StrLength(SMTP->Cmd) - offset >= 7) {
-                       smtp_get_user(6);
+               if (StrLength(SMTP->Cmd) >= 12) {
+                       syslog(LOG_DEBUG, "serv_smtp: username <%s> supplied inline", ChrPtr(SMTP->Cmd)+11);
+                       smtp_get_user(11);
                }
                else {
                        size_t len = CtdlEncodeBase64(username_prompt, "Username:", 9, 0);
@@ -497,44 +438,33 @@ void smtp_auth(long offset, long Flags) {
 
        if (!strncasecmp(method, "plain", 5) ) {
                long len;
-               if (num_tokens(ChrPtr(SMTP->Cmd) + offset, ' ') < 2) {
+               if (num_tokens(ChrPtr(SMTP->Cmd) + 5, ' ') < 2) {
                        cprintf("334 \r\n");
                        SMTP->command_state = smtp_plain;
                        return;
                }
 
                len = extract_token(encoded_authstring, 
-                                   ChrPtr(SMTP->Cmd) + offset,
+                                   ChrPtr(SMTP->Cmd) + 5,
                                    1, ' ',
                                    sizeof encoded_authstring);
                StrBufPlain(SMTP->Cmd, encoded_authstring, len);
-               smtp_try_plain(0, Flags);
-               return;
-       }
-
-       if (strncasecmp(method, "login", 5) ) {
-               cprintf("504 Unknown authentication method.\r\n");
+               smtp_try_plain();
                return;
        }
 
+       cprintf("504 Unknown authentication method.\r\n");
+       return;
 }
 
 
-/*
- * Implements the RSET (reset state) command.
- * Currently this just zeroes out the state buffer.  If pointers to data
- * allocated with malloc() are ever placed in the state buffer, we have to
- * be sure to free() them first!
- *
- * Set do_response to nonzero to output the SMTP RSET response code.
- */
-void smtp_rset(long offset, long do_response) {
-       /*
-        * Our entire SMTP state is discarded when a RSET command is issued,
-        * but we need to preserve this one little piece of information, so
-        * we save it for later.
-        */
-
+// Implements the RSET (reset state) command.
+// Currently this just zeroes out the state buffer.  If pointers to data
+// allocated with malloc() are ever placed in the state buffer, we have to
+// be sure to free() them first!
+//
+// Set do_response to nonzero to output the SMTP RSET response code.
+void smtp_rset(int do_response) {
        FlushStrBuf(SMTP->Cmd);
        FlushStrBuf(SMTP->helo_node);
        FlushStrBuf(SMTP->from);
@@ -546,20 +476,7 @@ void smtp_rset(long offset, long do_response) {
        SMTP->delivery_mode = 0;
        SMTP->message_originated_locally = 0;
        SMTP->is_msa = 0;
-       /*
-        * we must remember is_lmtp & is_unfiltered.
-        */
-
-       /*
-        * It is somewhat ambiguous whether we want to log out when a RSET
-        * command is issued.  Here's the code to do it.  It is commented out
-        * because some clients (such as Pine) issue RSET commands before
-        * each message, but still expect to be logged in.
-        *
-        * if (CC->logged_in) {
-        *      logout(CC);
-        * }
-        */
+       // is_lmtp and is_unfiltered should not be cleared. 
 
        if (do_response) {
                cprintf("250 Zap!\r\n");
@@ -567,11 +484,9 @@ void smtp_rset(long offset, long do_response) {
 }
 
 
-/*
- * Clear out the portions of the state buffer that need to be cleared out
- * after the DATA command finishes.
- */
-void smtp_data_clear(long offset, long flags) {
+// Clear out the portions of the state buffer that need to be cleared out
+// after the DATA command finishes.
+void smtp_data_clear(void) {
        FlushStrBuf(SMTP->from);
        FlushStrBuf(SMTP->recipients);
        FlushStrBuf(SMTP->OneRcpt);
@@ -581,10 +496,8 @@ void smtp_data_clear(long offset, long flags) {
 }
 
 
-/*
- * Implements the "MAIL FROM:" command
- */
-void smtp_mail(long offset, long flags) {
+// Implements the "MAIL FROM:" command
+void smtp_mail(void) {
        char user[SIZ];
        char node[SIZ];
        char name[SIZ];
@@ -594,30 +507,33 @@ void smtp_mail(long offset, long flags) {
                return;
        }
 
-       if (strncasecmp(ChrPtr(SMTP->Cmd) + offset, "From:", 5)) {
+       if (StrLength(SMTP->Cmd) < 6) {
+               cprintf("501 Syntax error\r\n");
+               return;
+       }
+
+       if (strncasecmp(ChrPtr(SMTP->Cmd) + 5, "From:", 5)) {
                cprintf("501 Syntax error\r\n");
                return;
        }
 
-       StrBufAppendBuf(SMTP->from, SMTP->Cmd, offset);
+       StrBufAppendBuf(SMTP->from, SMTP->Cmd, 5);
        StrBufTrim(SMTP->from);
        if (strchr(ChrPtr(SMTP->from), '<') != NULL) {
                StrBufStripAllBut(SMTP->from, '<', '>');
        }
 
-       /* We used to reject empty sender names, until it was brought to our
-        * attention that RFC1123 5.2.9 requires that this be allowed.  So now
-        * we allow it, but replace the empty string with a fake
-        * address so we don't have to contend with the empty string causing
-        * other code to fail when it's expecting something there.
-        */
+       // We used to reject empty sender names, until it was brought to our
+       // attention that RFC1123 5.2.9 requires that this be allowed.  So now
+       // we allow it, but replace the empty string with a fake
+       // address so we don't have to contend with the empty string causing
+       // other code to fail when it's expecting something there.
        if (StrLength(SMTP->from) == 0) {
                StrBufPlain(SMTP->from, HKEY("someone@example.com"));
        }
 
-       /* If this SMTP connection is from a logged-in user, force the 'from'
-        * to be the user's Internet e-mail address as Citadel knows it.
-        */
+       // If this SMTP connection is from a logged-in user, force the 'from'
+       // to be the user's Internet e-mail address as Citadel knows it.
        if (CC->logged_in) {
                StrBufPlain(SMTP->from, CC->cs_inet_email, -1);
                cprintf("250 Sender ok <%s>\r\n", ChrPtr(SMTP->from));
@@ -626,12 +542,11 @@ void smtp_mail(long offset, long flags) {
        }
 
        else if (SMTP->is_lmtp) {
-               /* Bypass forgery checking for LMTP */
+               // Bypass forgery checking for LMTP
        }
 
-       /* Otherwise, make sure outsiders aren't trying to forge mail from
-        * this system (unless, of course, c_allow_spoofing is enabled)
-        */
+       // Otherwise, make sure outsiders aren't trying to forge mail from
+       // this system (unless, of course, c_allow_spoofing is enabled)
        else if (CtdlGetConfigInt("c_allow_spoofing") == 0) {
                process_rfc822_addr(ChrPtr(SMTP->from), user, node, name);
                syslog(LOG_DEBUG, "serv_smtp: claimed envelope sender is '%s' == '%s' @ '%s' ('%s')",
@@ -649,10 +564,8 @@ void smtp_mail(long offset, long flags) {
 }
 
 
-/*
- * Implements the "RCPT To:" command
- */
-void smtp_rcpt(long offset, long flags) {
+// Implements the "RCPT To:" command
+void smtp_rcpt(void) {
        char message_to_spammer[SIZ];
        struct recptypes *valid = NULL;
 
@@ -661,7 +574,12 @@ void smtp_rcpt(long offset, long flags) {
                return;
        }
        
-       if (strncasecmp(ChrPtr(SMTP->Cmd) + offset, "To:", 3)) {
+       if (StrLength(SMTP->Cmd) < 6) {
+               cprintf("501 Syntax error\r\n");
+               return;
+       }
+
+       if (strncasecmp(ChrPtr(SMTP->Cmd) + 5, "To:", 3)) {
                cprintf("501 Syntax error\r\n");
                return;
        }
@@ -671,8 +589,9 @@ void smtp_rcpt(long offset, long flags) {
                FlushStrBuf(SMTP->from);
                return;
        }
+
        FlushStrBuf(SMTP->OneRcpt);
-       StrBufAppendBuf(SMTP->OneRcpt, SMTP->Cmd, offset + 3);
+       StrBufAppendBuf(SMTP->OneRcpt, SMTP->Cmd, 8);
        StrBufTrim(SMTP->OneRcpt);
        StrBufStripAllBut(SMTP->OneRcpt, '<', '>');
 
@@ -681,50 +600,59 @@ void smtp_rcpt(long offset, long flags) {
                return;
        }
 
-       /* RBL check */
-       if ( (!CC->logged_in)   /* Don't RBL authenticated users */
-          && (!SMTP->is_lmtp) ) {      /* Don't RBL LMTP clients */
-               if (CtdlGetConfigInt("c_rbl_at_greeting") == 0) {       /* Don't RBL again if we already did it */
-                       if (rbl_check(CC->cs_addr, message_to_spammer)) {
-                               if (server_shutting_down)
-                                       cprintf("421 %s\r\n", message_to_spammer);
-                               else
-                                       cprintf("550 %s\r\n", message_to_spammer);
-                               /* no need to free_recipients(valid), it's not allocated yet */
-                               return;
-                       }
-               }
-       }
-
+       // RBL check
+       if (
+               (!CC->logged_in)                                                // Don't RBL authenticated users
+               && (!SMTP->is_lmtp)                                             // Don't RBL LMTP clients
+               && (CtdlGetConfigInt("c_rbl_at_greeting") == 0)                 // Don't RBL if we did it at connection time
+               && (rbl_check(CC->cs_addr, message_to_spammer))
+       ) {
+               cprintf("550 %s\r\n", message_to_spammer);
+               return;                                                         // no need to free_recipients(valid)
+       }                                                                       // because it hasn't been allocated yet
+
+       // This is a *preliminary* call to validate_recipients() to evaluate one recipient.
        valid = validate_recipients(
-               ChrPtr(SMTP->OneRcpt), 
+               (char *)ChrPtr(SMTP->OneRcpt), 
                smtp_get_Recipients(),
                (SMTP->is_lmtp)? POST_LMTP: (CC->logged_in)? POST_LOGGED_IN: POST_EXTERNAL
        );
+
+       // Any type of error thrown by validate_recipients() will make the SMTP transaction fail at this point.
        if (valid->num_error != 0) {
                cprintf("550 %s\r\n", valid->errormsg);
                free_recipients(valid);
                return;
        }
 
-       if (valid->num_internet > 0) {
-               if (CC->logged_in) {
-                        if (CtdlCheckInternetMailPermission(&CC->user)==0) {
-                               cprintf("551 <%s> - you do not have permission to send Internet mail\r\n", 
-                                       ChrPtr(SMTP->OneRcpt));
-                                free_recipients(valid);
-                                return;
-                        }
-                }
+       if (
+               (valid->num_internet > 0)                                       // If it's outbound Internet mail...
+               && (CC->logged_in)                                              // ...and we're a logged-in user...
+               && (CtdlCheckInternetMailPermission(&CC->user)==0)              // ...who does not have Internet mail rights...
+       ) {
+               cprintf("551 <%s> - you do not have permission to send Internet mail\r\n", ChrPtr(SMTP->OneRcpt));
+               free_recipients(valid);
+               return;
        }
 
-       if (valid->num_internet > 0) {
-               if ( (SMTP->message_originated_locally == 0)
-                  && (SMTP->is_lmtp == 0) ) {
-                       cprintf("551 <%s> - relaying denied\r\n", ChrPtr(SMTP->OneRcpt));
-                       free_recipients(valid);
-                       return;
-               }
+       if (
+               (valid->num_internet > 0)                                       // If it's outbound Internet mail...
+               && (SMTP->message_originated_locally == 0)                      // ...and also inbound Internet mail...
+               && (SMTP->is_lmtp == 0)                                         /// ...and didn't arrive via LMTP...
+       ) {
+               cprintf("551 <%s> - relaying denied\r\n", ChrPtr(SMTP->OneRcpt));
+               free_recipients(valid);
+               return;
+       }
+
+       if (
+               (valid->num_room > 0)                                           // If it's mail to a room (mailing list)...
+               && (SMTP->message_originated_locally == 0)                      // ...and also inbound Internet mail...
+               && (is_email_subscribed_to_list((char *)ChrPtr(SMTP->from), valid->recp_room) == 0)     // ...and not a subscriber
+       ) {
+               cprintf("551 <%s> - The message is not from a list member\r\n", ChrPtr(SMTP->OneRcpt));
+               free_recipients(valid);
+               return;
        }
 
        cprintf("250 RCPT ok <%s>\r\n", ChrPtr(SMTP->OneRcpt));
@@ -739,10 +667,8 @@ void smtp_rcpt(long offset, long flags) {
 }
 
 
-/*
- * Implements the DATA command
- */
-void smtp_data(long offset, long flags) {
+// Implements the DATA command
+void smtp_data(void) {
        StrBuf *body;
        StrBuf *defbody; 
        struct CtdlMessage *msg = NULL;
@@ -800,16 +726,16 @@ void smtp_data(long offset, long flags) {
        syslog(LOG_DEBUG, "serv_smtp: converting message...");
        msg = convert_internet_message_buf(&body);
 
-       /* If the user is locally authenticated, FORCE the From: header to
-        * show up as the real sender.  Yes, this violates the RFC standard,
-        * but IT MAKES SENSE.  If you prefer strict RFC adherence over
-        * common sense, you can disable this in the configuration.
-        *
-        * We also set the "message room name" ('O' field) to MAILROOM
-        * (which is Mail> on most systems) to prevent it from getting set
-        * to something ugly like "0000058008.Sent Items>" when the message
-        * is read with a Citadel client.
-        */
+       // If the user is locally authenticated, FORCE the From: header to
+       // show up as the real sender.  Yes, this violates the RFC standard,
+       // but IT MAKES SENSE.  If you prefer strict RFC adherence over
+       // common sense, you can disable this in the configuration.
+       //
+       // We also set the "message room name" ('O' field) to MAILROOM
+       // (which is Mail> on most systems) to prevent it from getting set
+       // to something ugly like "0000058008.Sent Items>" when the message
+       // is read with a Citadel client.
+
        if ( (CC->logged_in) && (CtdlGetConfigInt("c_rfc822_strict_from") != CFG_SMTP_FROM_NOFILTER) ) {
                int validemail = 0;
                
@@ -855,23 +781,22 @@ void smtp_data(long offset, long flags) {
                }
        }
 
-       /* Set the "envelope from" address */
+       // Set the "envelope from" address
        CM_SetField(msg, eMessagePath, SKEY(SMTP->from));
 
-       /* Set the "envelope to" address */
+       // Set the "envelope to" address
        CM_SetField(msg, eenVelopeTo, SKEY(SMTP->recipients));
 
-       /* Submit the message into the Citadel system. */
+       // Submit the message into the Citadel system.
        valid = validate_recipients(
-               ChrPtr(SMTP->recipients),
+               (char *)ChrPtr(SMTP->recipients),
                smtp_get_Recipients(),
                (SMTP->is_lmtp)? POST_LMTP: (CC->logged_in)? POST_LOGGED_IN: POST_EXTERNAL
        );
 
-       /* If there are modules that want to scan this message before final
-        * submission (such as virus checkers or spam filters), call them now
-        * and give them an opportunity to reject the message.
-        */
+       // If there are modules that want to scan this message before final
+       // submission (such as virus checkers or spam filters), call them now
+       // and give them an opportunity to reject the message.
        if (SMTP->is_unfiltered) {
                scan_errors = 0;
        }
@@ -879,7 +804,7 @@ void smtp_data(long offset, long flags) {
                scan_errors = PerformMessageHooks(msg, valid, EVT_SMTPSCAN);
        }
 
-       if (scan_errors > 0) {  /* We don't want this message! */
+       if (scan_errors > 0) {  // We don't want this message!
 
                if (CM_IsEmpty(msg, eErrorMsg)) {
                        CM_SetField(msg, eErrorMsg, HKEY("Message rejected by filter"));
@@ -888,7 +813,7 @@ void smtp_data(long offset, long flags) {
                StrBufPrintf(SMTP->OneRcpt, "550 %s\r\n", msg->cm_fields[eErrorMsg]);
        }
        
-       else {                  /* Ok, we'll accept this message. */
+       else {                  // Ok, we'll accept this message.
                msgnum = CtdlSubmitMsg(msg, valid, "");
                if (msgnum > 0L) {
                        StrBufPrintf(SMTP->OneRcpt, "250 Message accepted.\r\n");
@@ -898,12 +823,11 @@ void smtp_data(long offset, long flags) {
                }
        }
 
-       /* For SMTP and ESMTP, just print the result message.  For LMTP, we
-        * have to print one result message for each recipient.  Since there
-        * is nothing in Citadel which would cause different recipients to
-        * have different results, we can get away with just spitting out the
-        * same message once for each recipient.
-        */
+       // For SMTP and ESMTP, just print the result message.  For LMTP, we
+       // have to print one result message for each recipient.  Since there
+       // is nothing in Citadel which would cause different recipients to
+       // have different results, we can get away with just spitting out the
+       // same message once for each recipient.
        if (SMTP->is_lmtp) {
                for (i=0; i<SMTP->number_of_recipients; ++i) {
                        cputbuf(SMTP->OneRcpt);
@@ -913,9 +837,8 @@ void smtp_data(long offset, long flags) {
                cputbuf(SMTP->OneRcpt);
        }
 
-       /* Write something to the syslog(which may or may not be where the
-        * rest of the Citadel logs are going; some sysadmins want LOG_MAIL).
-        */
+       // Write something to the syslog(which may or may not be where the
+       // rest of the Citadel logs are going; some sysadmins want LOG_MAIL).
        syslog((LOG_MAIL | LOG_INFO),
                    "%ld: from=<%s>, nrcpts=%d, relay=%s [%s], stat=%s",
                    msgnum,
@@ -926,17 +849,15 @@ void smtp_data(long offset, long flags) {
                    ChrPtr(SMTP->OneRcpt)
        );
 
-       /* Clean up */
+       // Clean up
        CM_Free(msg);
        free_recipients(valid);
-       smtp_data_clear(0, 0);  /* clear out the buffers now */
+       smtp_data_clear();      // clear out the buffers now
 }
 
 
-/*
- * implements the STARTTLS command
- */
-void smtp_starttls(long offset, long flags) {
+// Implements the STARTTLS command
+void smtp_starttls(void) {
        char ok_response[SIZ];
        char nosup_response[SIZ];
        char error_response[SIZ];
@@ -945,18 +866,26 @@ void smtp_starttls(long offset, long flags) {
        sprintf(nosup_response, "554 TLS not supported here\r\n");
        sprintf(error_response, "554 Internal error\r\n");
        CtdlModuleStartCryptoMsgs(ok_response, nosup_response, error_response);
-       smtp_rset(0, 0);
+       smtp_rset(0);
 }
 
 
-/* 
- * Main command loop for SMTP server sessions.
- */
+// Implements the NOOP (NO OPeration) command
+void smtp_noop(void) {
+       cprintf("250 NOOP\r\n");
+}
+
+
+// Implements the QUIT command
+void smtp_quit(void) {
+       cprintf("221 Goodbye...\r\n");
+       CC->kill_me = KILLME_CLIENT_LOGGED_OUT;
+}
+
+
+// Main command loop for SMTP server sessions.
 void smtp_command_loop(void) {
        static const ConstStr AuthPlainStr = {HKEY("AUTH PLAIN")};
-       const char *pch, *pchs;
-       long i;
-       char CMD[MaxSMTPCmdLen + 1];
 
        if (SMTP == NULL) {
                syslog(LOG_ERR, "serv_smtp: Session SMTP data is null.  WTF?  We will crash now.");
@@ -969,11 +898,10 @@ void smtp_command_loop(void) {
                CC->kill_me = KILLME_CLIENT_DISCONNECTED;
                return;
        }
-       syslog(LOG_DEBUG, "serv_smtp: %s", ChrPtr(SMTP->Cmd));
 
        if (SMTP->command_state == smtp_user) {
                if (!strncmp(ChrPtr(SMTP->Cmd), AuthPlainStr.Key, AuthPlainStr.len)) {
-                       smtp_try_plain(0, 0);
+                       smtp_try_plain();
                }
                else {
                        smtp_get_user(0);
@@ -982,52 +910,79 @@ void smtp_command_loop(void) {
        }
 
        else if (SMTP->command_state == smtp_password) {
-               smtp_get_pass(0, 0);
+               smtp_get_pass();
                return;
        }
 
        else if (SMTP->command_state == smtp_plain) {
-               smtp_try_plain(0, 0);
+               smtp_try_plain();
                return;
        }
 
-       pchs = pch = ChrPtr(SMTP->Cmd);
-       i = 0;
-       while ((*pch != '\0') && (!isblank(*pch)) && (pch - pchs <= MaxSMTPCmdLen)) {
-               CMD[i] = toupper(*pch);
-               pch ++;
-               i++;
+       syslog(LOG_DEBUG, "serv_smtp: client sent command <%s>", ChrPtr(SMTP->Cmd));
+
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "NOOP", 4)) {
+               smtp_noop();
+               return;
        }
-       CMD[i] = '\0';
 
-       if ((*pch == '\0') || (isblank(*pch))) {
-               void *v;
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "QUIT", 4)) {
+               smtp_quit();
+               return;
+       }
 
-               if (GetHash(SMTPCmds, CMD, i, &v) &&
-                   (v != NULL))
-               {
-                       smtp_handler_hook *h = (smtp_handler_hook*) v;
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "HELO", 4)) {
+               smtp_hello(HELO);
+               return;
+       }
 
-                       if (isblank(pchs[i]))
-                               i++;
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "EHLO", 4)) {
+               smtp_hello(EHLO);
+               return;
+       }
 
-                       h->h(i, h->Flags);
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "LHLO", 4)) {
+               smtp_hello(LHLO);
+               return;
+       }
 
-                       return;
-               }
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "RSET", 4)) {
+               smtp_rset(1);
+               return;
        }
-       cprintf("502 I'm afraid I can't do that.\r\n");
-}
 
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "AUTH", 4)) {
+               smtp_auth();
+               return;
+       }
 
-void smtp_noop(long offest, long Flags) {
-       cprintf("250 NOOP\r\n");
-}
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "DATA", 4)) {
+               smtp_data();
+               return;
+       }
+
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "HELP", 4)) {
+               smtp_help();
+               return;
+       }
 
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "MAIL", 4)) {
+               smtp_mail();
+               return;
+       }
+       
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "RCPT", 4)) {
+               smtp_rcpt();
+               return;
+       }
+#ifdef HAVE_OPENSSL
+       if (!strncasecmp(ChrPtr(SMTP->Cmd), "STARTTLS", 8)) {
+               smtp_starttls();
+               return;
+       }
+#endif
 
-void smtp_quit(long offest, long Flags) {
-       cprintf("221 Goodbye...\r\n");
-       CC->kill_me = KILLME_CLIENT_LOGGED_OUT;
+       cprintf("502 I'm afraid I can't do that.\r\n");
 }
 
 
@@ -1066,24 +1021,6 @@ const char *CitadelServiceSMTP_LMTP_UNF="LMTP-UnF";
 CTDL_MODULE_INIT(smtp)
 {
        if (!threading) {
-               SMTPCmds = NewHash(1, NULL);
-               
-               RegisterSmtpCMD("AUTH", smtp_auth, 0);
-               RegisterSmtpCMD("DATA", smtp_data, 0);
-               RegisterSmtpCMD("HELO", smtp_hello, HELO);
-               RegisterSmtpCMD("EHLO", smtp_hello, EHLO);
-               RegisterSmtpCMD("LHLO", smtp_hello, LHLO);
-               RegisterSmtpCMD("HELP", smtp_help, 0);
-               RegisterSmtpCMD("MAIL", smtp_mail, 0);
-               RegisterSmtpCMD("NOOP", smtp_noop, 0);
-               RegisterSmtpCMD("QUIT", smtp_quit, 0);
-               RegisterSmtpCMD("RCPT", smtp_rcpt, 0);
-               RegisterSmtpCMD("RSET", smtp_rset, 1);
-#ifdef HAVE_OPENSSL
-               RegisterSmtpCMD("STARTTLS", smtp_starttls, 0);
-#endif
-
-
                CtdlRegisterServiceHook(CtdlGetConfigInt("c_smtp_port"),        /* SMTP MTA */
                                        NULL,
                                        smtp_mta_greeting,