X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=citadel%2Fmodules%2Fsmtp%2Fserv_smtp.c;h=e0253672cae38e79b42e13c4a9d11ca67041f46c;hb=2b6008f54e8b56b79e24617f47308e469fcaca0e;hp=dd336ba2bd625193957ebac895f0f1d2e67c90d3;hpb=18a1c3704ef9e8e27c014f0db67600dd25b3d41a;p=citadel.git diff --git a/citadel/modules/smtp/serv_smtp.c b/citadel/modules/smtp/serv_smtp.c index dd336ba2b..e0253672c 100644 --- a/citadel/modules/smtp/serv_smtp.c +++ b/citadel/modules/smtp/serv_smtp.c @@ -22,6 +22,21 @@ * The VRFY and EXPN commands have been removed from this implementation * because nobody uses these commands anymore, except for spammers. * + * Copyright (c) 1998-2009 by the citadel.org team + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 3 of the License, or + * (at your option) any later version. + * + * 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. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ #include "sysdep.h" @@ -60,7 +75,6 @@ #include "support.h" #include "config.h" #include "control.h" -#include "room_ops.h" #include "user_ops.h" #include "policy.h" #include "database.h" @@ -83,7 +97,7 @@ -struct citsmtp { /* Information about the current session */ +typedef struct _citsmtp { /* Information about the current session */ int command_state; char helo_node[SIZ]; char from[SIZ]; @@ -94,7 +108,7 @@ struct citsmtp { /* Information about the current session */ int is_lmtp; int is_unfiltered; int is_msa; -}; +}citsmtp; enum { /* Command states for login authentication */ smtp_command, @@ -103,11 +117,12 @@ enum { /* Command states for login authentication */ smtp_plain }; -#define SMTP ((struct citsmtp *)CC->session_specific_data) +#define SMTP ((citsmtp *)CC->session_specific_data) int run_queue_now = 0; /* Set to 1 to ignore SMTP send retry times */ +citthread_mutex_t smtp_send_lock; /*****************************************************************************/ @@ -120,21 +135,26 @@ int run_queue_now = 0; /* Set to 1 to ignore SMTP send retry times */ */ void smtp_greeting(int is_msa) { + citsmtp *sSMTP; 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(struct citsmtp)); - memset(SMTP, 0, sizeof(struct citsmtp)); - SMTP->is_msa = is_msa; + CC->session_specific_data = malloc(sizeof(citsmtp)); + memset(SMTP, 0, sizeof(citsmtp)); + sSMTP = SMTP; + sSMTP->is_msa = is_msa; /* If this config option is set, reject connections from problem * addresses immediately instead of after they execute a RCPT */ - if ( (config.c_rbl_at_greeting) && (SMTP->is_msa == 0) ) { + if ( (config.c_rbl_at_greeting) && (sSMTP->is_msa == 0) ) { if (rbl_check(message_to_spammer)) { - cprintf("550 %s\r\n", message_to_spammer); + if (CtdlThreadCheckStop()) + cprintf("421 %s\r\n", message_to_spammer); + else + cprintf("550 %s\r\n", message_to_spammer); CC->kill_me = 1; /* no need to free_recipients(valid), it's not allocated yet */ return; @@ -183,7 +203,10 @@ void smtp_msa_greeting(void) { * LMTP is like SMTP but with some extra bonus footage added. */ void lmtp_greeting(void) { + citsmtp *sSMTP; + smtp_greeting(0); + sSMTP = SMTP; SMTP->is_lmtp = 1; } @@ -200,9 +223,12 @@ void smtp_mta_greeting(void) { * We also have an unfiltered LMTP socket that bypasses spam filters. */ void lmtp_unfiltered_greeting(void) { + citsmtp *sSMTP; + smtp_greeting(0); - SMTP->is_lmtp = 1; - SMTP->is_unfiltered = 1; + sSMTP = SMTP; + sSMTP->is_lmtp = 1; + sSMTP->is_unfiltered = 1; } @@ -211,7 +237,7 @@ void lmtp_unfiltered_greeting(void) { */ void smtp_auth_greeting(void) { cprintf("235 Hello, %s\r\n", CC->user.fullname); - lprintf(CTDL_NOTICE, "SMTP authenticated %s\n", CC->user.fullname); + CtdlLogPrintf(CTDL_NOTICE, "SMTP authenticated %s\n", CC->user.fullname); CC->internal_pgm = 0; CC->cs_flags &= ~CS_STEALTH; } @@ -223,22 +249,23 @@ void smtp_auth_greeting(void) { * which_command: 0=HELO, 1=EHLO, 2=LHLO */ void smtp_hello(char *argbuf, int which_command) { + citsmtp *sSMTP = SMTP; - safestrncpy(SMTP->helo_node, argbuf, sizeof SMTP->helo_node); + safestrncpy(sSMTP->helo_node, argbuf, sizeof sSMTP->helo_node); - if ( (which_command != 2) && (SMTP->is_lmtp) ) { + if ( (which_command != 2) && (sSMTP->is_lmtp) ) { cprintf("500 Only LHLO is allowed when running LMTP\r\n"); return; } - if ( (which_command == 2) && (SMTP->is_lmtp == 0) ) { + if ( (which_command == 2) && (sSMTP->is_lmtp == 0) ) { cprintf("500 LHLO is only allowed when running LMTP\r\n"); return; } if (which_command == 0) { cprintf("250 Hello %s (%s [%s])\r\n", - SMTP->helo_node, + sSMTP->helo_node, CC->cs_host, CC->cs_addr ); @@ -246,7 +273,7 @@ void smtp_hello(char *argbuf, int which_command) { else { if (which_command == 1) { cprintf("250-Hello %s (%s [%s])\r\n", - SMTP->helo_node, + sSMTP->helo_node, CC->cs_host, CC->cs_addr ); @@ -264,7 +291,7 @@ void smtp_hello(char *argbuf, int which_command) { * 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) ) { + if ( (!CC->redirect_ssl) && (sSMTP->is_msa) ) { cprintf("250-STARTTLS\r\n"); } #endif /* HAVE_OPENSSL */ @@ -282,17 +309,7 @@ void smtp_hello(char *argbuf, int which_command) { * Implement HELP command. */ void smtp_help(void) { - cprintf("214-Commands accepted:\r\n"); - cprintf("214- DATA\r\n"); - cprintf("214- EHLO\r\n"); - cprintf("214- HELO\r\n"); - cprintf("214- HELP\r\n"); - cprintf("214- MAIL\r\n"); - cprintf("214- NOOP\r\n"); - cprintf("214- QUIT\r\n"); - cprintf("214- RCPT\r\n"); - cprintf("214- RSET\r\n"); - cprintf("214 \r\n"); + cprintf("214 RTFM http://www.ietf.org/rfc/rfc2821.txt\r\n"); } @@ -302,17 +319,18 @@ void smtp_help(void) { void smtp_get_user(char *argbuf) { char buf[SIZ]; char username[SIZ]; + citsmtp *sSMTP = SMTP; CtdlDecodeBase64(username, argbuf, SIZ); - /* lprintf(CTDL_DEBUG, "Trying <%s>\n", username); */ + /* CtdlLogPrintf(CTDL_DEBUG, "Trying <%s>\n", username); */ if (CtdlLoginExistingUser(NULL, username) == login_ok) { CtdlEncodeBase64(buf, "Password:", 9, 0); cprintf("334 %s\r\n", buf); - SMTP->command_state = smtp_password; + sSMTP->command_state = smtp_password; } else { cprintf("500 No such user.\r\n"); - SMTP->command_state = smtp_command; + sSMTP->command_state = smtp_command; } } @@ -323,8 +341,9 @@ void smtp_get_user(char *argbuf) { void smtp_get_pass(char *argbuf) { char password[SIZ]; + memset(password, 0, sizeof(password)); CtdlDecodeBase64(password, argbuf, SIZ); - /* lprintf(CTDL_DEBUG, "Trying <%s>\n", password); */ + /* CtdlLogPrintf(CTDL_DEBUG, "Trying <%s>\n", password); */ if (CtdlTryPassword(password) == pass_ok) { smtp_auth_greeting(); } @@ -428,16 +447,17 @@ void smtp_auth(char *argbuf) { void smtp_rset(int do_response) { int is_lmtp; int is_unfiltered; + citsmtp *sSMTP = SMTP; /* * 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. */ - is_lmtp = SMTP->is_lmtp; - is_unfiltered = SMTP->is_unfiltered; + is_lmtp = sSMTP->is_lmtp; + is_unfiltered = sSMTP->is_unfiltered; - memset(SMTP, 0, sizeof(struct citsmtp)); + memset(sSMTP, 0, sizeof(citsmtp)); /* * It is somewhat ambiguous whether we want to log out when a RSET @@ -453,8 +473,8 @@ void smtp_rset(int do_response) { /* * Reinstate this little piece of information we saved (see above). */ - SMTP->is_lmtp = is_lmtp; - SMTP->is_unfiltered = is_unfiltered; + sSMTP->is_lmtp = is_lmtp; + sSMTP->is_unfiltered = is_unfiltered; if (do_response) { cprintf("250 Zap!\r\n"); @@ -466,30 +486,34 @@ void smtp_rset(int do_response) { * after the DATA command finishes. */ void smtp_data_clear(void) { - strcpy(SMTP->from, ""); - strcpy(SMTP->recipients, ""); - SMTP->number_of_recipients = 0; - SMTP->delivery_mode = 0; - SMTP->message_originated_locally = 0; + citsmtp *sSMTP = SMTP; + + strcpy(sSMTP->from, ""); + strcpy(sSMTP->recipients, ""); + sSMTP->number_of_recipients = 0; + sSMTP->delivery_mode = 0; + sSMTP->message_originated_locally = 0; } const char *smtp_get_Recipients(void) { - if (SMTP == NULL) - return NULL; - else return SMTP->from; + citsmtp *sSMTP = SMTP; + if (sSMTP == NULL) + return NULL; + else return sSMTP->from; } /* - * Implements the "MAIL From:" command + * Implements the "MAIL FROM:" command */ void smtp_mail(char *argbuf) { char user[SIZ]; char node[SIZ]; char name[SIZ]; + citsmtp *sSMTP = SMTP; - if (!IsEmptyStr(SMTP->from)) { + if (!IsEmptyStr(sSMTP->from)) { cprintf("503 Only one sender permitted\r\n"); return; } @@ -499,10 +523,10 @@ void smtp_mail(char *argbuf) { return; } - strcpy(SMTP->from, &argbuf[5]); - striplt(SMTP->from); - if (haschar(SMTP->from, '<') > 0) { - stripallbut(SMTP->from, '<', '>'); + strcpy(sSMTP->from, &argbuf[5]); + striplt(sSMTP->from); + if (haschar(sSMTP->from, '<') > 0) { + stripallbut(sSMTP->from, '<', '>'); } /* We used to reject empty sender names, until it was brought to our @@ -511,21 +535,21 @@ void smtp_mail(char *argbuf) { * address so we don't have to contend with the empty string causing * other code to fail when it's expecting something there. */ - if (IsEmptyStr(SMTP->from)) { - strcpy(SMTP->from, "someone@somewhere.org"); + if (IsEmptyStr(sSMTP->from)) { + strcpy(sSMTP->from, "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 (CC->logged_in) { - safestrncpy(SMTP->from, CC->cs_inet_email, sizeof SMTP->from); - cprintf("250 Sender ok <%s>\r\n", SMTP->from); - SMTP->message_originated_locally = 1; + safestrncpy(sSMTP->from, CC->cs_inet_email, sizeof sSMTP->from); + cprintf("250 Sender ok <%s>\r\n", sSMTP->from); + sSMTP->message_originated_locally = 1; return; } - else if (SMTP->is_lmtp) { + else if (sSMTP->is_lmtp) { /* Bypass forgery checking for LMTP */ } @@ -533,10 +557,10 @@ void smtp_mail(char *argbuf) { * this system (unless, of course, c_allow_spoofing is enabled) */ else if (config.c_allow_spoofing == 0) { - process_rfc822_addr(SMTP->from, user, node, name); + process_rfc822_addr(sSMTP->from, user, node, name); if (CtdlHostAlias(node) != hostalias_nomatch) { cprintf("550 You must log in to send mail from %s\r\n", node); - strcpy(SMTP->from, ""); + strcpy(sSMTP->from, ""); return; } } @@ -553,8 +577,9 @@ void smtp_rcpt(char *argbuf) { char recp[1024]; char message_to_spammer[SIZ]; struct recptypes *valid = NULL; + citsmtp *sSMTP = SMTP; - if (IsEmptyStr(SMTP->from)) { + if (IsEmptyStr(sSMTP->from)) { cprintf("503 Need MAIL before RCPT\r\n"); return; } @@ -564,9 +589,9 @@ void smtp_rcpt(char *argbuf) { return; } - if ( (SMTP->is_msa) && (!CC->logged_in) ) { + if ( (sSMTP->is_msa) && (!CC->logged_in) ) { cprintf("550 You must log in to send mail on this port.\r\n"); - strcpy(SMTP->from, ""); + strcpy(sSMTP->from, ""); return; } @@ -574,17 +599,20 @@ void smtp_rcpt(char *argbuf) { striplt(recp); stripallbut(recp, '<', '>'); - if ( (strlen(recp) + strlen(SMTP->recipients) + 1 ) >= SIZ) { + if ( (strlen(recp) + strlen(sSMTP->recipients) + 1 ) >= SIZ) { cprintf("452 Too many recipients\r\n"); return; } /* RBL check */ if ( (!CC->logged_in) /* Don't RBL authenticated users */ - && (!SMTP->is_lmtp) ) { /* Don't RBL LMTP clients */ + && (!sSMTP->is_lmtp) ) { /* Don't RBL LMTP clients */ if (config.c_rbl_at_greeting == 0) { /* Don't RBL again if we already did it */ if (rbl_check(message_to_spammer)) { - cprintf("550 %s\r\n", message_to_spammer); + if (CtdlThreadCheckStop()) + 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; } @@ -593,9 +621,11 @@ void smtp_rcpt(char *argbuf) { valid = validate_recipients(recp, smtp_get_Recipients (), - (CC->logged_in)? POST_LOGGED_IN:POST_EXTERNAL); + (sSMTP->is_lmtp)? POST_LMTP: + (CC->logged_in)? POST_LOGGED_IN: + POST_EXTERNAL); if (valid->num_error != 0) { - cprintf("599 Error: %s\r\n", valid->errormsg); + cprintf("550 %s\r\n", valid->errormsg); free_recipients(valid); return; } @@ -611,8 +641,8 @@ void smtp_rcpt(char *argbuf) { } if (valid->num_internet > 0) { - if ( (SMTP->message_originated_locally == 0) - && (SMTP->is_lmtp == 0) ) { + if ( (sSMTP->message_originated_locally == 0) + && (sSMTP->is_lmtp == 0) ) { cprintf("551 <%s> - relaying denied\r\n", recp); free_recipients(valid); return; @@ -620,11 +650,11 @@ void smtp_rcpt(char *argbuf) { } cprintf("250 RCPT ok <%s>\r\n", recp); - if (!IsEmptyStr(SMTP->recipients)) { - strcat(SMTP->recipients, ","); + if (!IsEmptyStr(sSMTP->recipients)) { + strcat(sSMTP->recipients, ","); } - strcat(SMTP->recipients, recp); - SMTP->number_of_recipients += 1; + strcat(sSMTP->recipients, recp); + sSMTP->number_of_recipients += 1; if (valid != NULL) { free_recipients(valid); } @@ -637,7 +667,8 @@ void smtp_rcpt(char *argbuf) { * Implements the DATA command */ void smtp_data(void) { - char *body; + StrBuf *body; + char *defbody; //TODO: remove me struct CtdlMessage *msg = NULL; long msgnum = (-1L); char nowstamp[SIZ]; @@ -645,13 +676,14 @@ void smtp_data(void) { int scan_errors; int i; char result[SIZ]; + citsmtp *sSMTP = SMTP; - if (IsEmptyStr(SMTP->from)) { + if (IsEmptyStr(sSMTP->from)) { cprintf("503 Need MAIL command first.\r\n"); return; } - if (SMTP->number_of_recipients < 1) { + if (sSMTP->number_of_recipients < 1) { cprintf("503 Need RCPT command first.\r\n"); return; } @@ -659,25 +691,37 @@ void smtp_data(void) { cprintf("354 Transmit message now - terminate with '.' by itself\r\n"); datestring(nowstamp, sizeof nowstamp, time(NULL), DATESTRING_RFC822); - body = malloc(4096); - - if (body != NULL) snprintf(body, 4096, - "Received: from %s (%s [%s])\n" - " by %s; %s\n", - SMTP->helo_node, - CC->cs_host, - CC->cs_addr, - config.c_fqdn, - nowstamp); - - body = CtdlReadMessageBody(".", config.c_maxmsglen, body, 1, 0); + defbody = malloc(4096); + + if (defbody != NULL) { + if (sSMTP->is_lmtp && (CC->cs_UDSclientUID != -1)) { + snprintf(defbody, 4096, + "Received: from %s (Citadel from userid %ld)\n" + " by %s; %s\n", + sSMTP->helo_node, + (long int) CC->cs_UDSclientUID, + config.c_fqdn, + nowstamp); + } + else { + snprintf(defbody, 4096, + "Received: from %s (%s [%s])\n" + " by %s; %s\n", + sSMTP->helo_node, + CC->cs_host, + CC->cs_addr, + config.c_fqdn, + nowstamp); + } + } + body = CtdlReadMessageBodyBuf(HKEY("."), config.c_maxmsglen, defbody, 1, NULL); if (body == NULL) { cprintf("550 Unable to save message: internal error.\r\n"); return; } - lprintf(CTDL_DEBUG, "Converting message...\n"); - msg = convert_internet_message(body); + CtdlLogPrintf(CTDL_DEBUG, "Converting message...\n"); + 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, @@ -706,24 +750,26 @@ void smtp_data(void) { if (msg->cm_fields['P'] != NULL) { free(msg->cm_fields['P']); } - msg->cm_fields['P'] = strdup(SMTP->from); + msg->cm_fields['P'] = strdup(sSMTP->from); /* Set the "envelope to" address */ if (msg->cm_fields['V'] != NULL) { free(msg->cm_fields['V']); } - msg->cm_fields['V'] = strdup(SMTP->recipients); + msg->cm_fields['V'] = strdup(sSMTP->recipients); /* Submit the message into the Citadel system. */ - valid = validate_recipients(SMTP->recipients, + valid = validate_recipients(sSMTP->recipients, smtp_get_Recipients (), - (CC->logged_in)? POST_LOGGED_IN:POST_EXTERNAL); + (sSMTP->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 (SMTP->is_unfiltered) { + if (sSMTP->is_unfiltered) { scan_errors = 0; } else { @@ -733,20 +779,19 @@ void smtp_data(void) { if (scan_errors > 0) { /* We don't want this message! */ if (msg->cm_fields['0'] == NULL) { - msg->cm_fields['0'] = strdup( - "5.7.1 Message rejected by filter"); + msg->cm_fields['0'] = strdup("Message rejected by filter"); } sprintf(result, "550 %s\r\n", msg->cm_fields['0']); } else { /* Ok, we'll accept this message. */ - msgnum = CtdlSubmitMsg(msg, valid, ""); + msgnum = CtdlSubmitMsg(msg, valid, "", 0); if (msgnum > 0L) { - sprintf(result, "250 2.0.0 Message accepted.\r\n"); + sprintf(result, "250 Message accepted.\r\n"); } else { - sprintf(result, "550 5.5.0 Internal delivery error\r\n"); + sprintf(result, "550 Internal delivery error\r\n"); } } @@ -756,8 +801,8 @@ void smtp_data(void) { * 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; inumber_of_recipients; ++i) { + if (sSMTP->is_lmtp) { + for (i=0; inumber_of_recipients; ++i) { cprintf("%s", result); } } @@ -772,8 +817,8 @@ void smtp_data(void) { syslog((LOG_MAIL | LOG_INFO), "%ld: from=<%s>, nrcpts=%d, relay=%s [%s], stat=%s", msgnum, - SMTP->from, - SMTP->number_of_recipients, + sSMTP->from, + sSMTP->number_of_recipients, CC->cs_host, CC->cs_addr, result @@ -797,11 +842,11 @@ void smtp_starttls(void) char error_response[SIZ]; sprintf(ok_response, - "220 2.0.0 Begin TLS negotiation now\r\n"); + "220 Begin TLS negotiation now\r\n"); sprintf(nosup_response, - "554 5.7.3 TLS not supported here\r\n"); + "554 TLS not supported here\r\n"); sprintf(error_response, - "554 5.7.3 Internal error\r\n"); + "554 Internal error\r\n"); CtdlModuleStartCryptoMsgs(ok_response, nosup_response, error_response); smtp_rset(0); } @@ -813,26 +858,31 @@ void smtp_starttls(void) */ void smtp_command_loop(void) { char cmdbuf[SIZ]; + citsmtp *sSMTP = SMTP; + + if (sSMTP == NULL) { + CtdlLogPrintf(CTDL_EMERG, "Session SMTP data is null. WTF? We will crash now.\n"); + } time(&CC->lastcmd); memset(cmdbuf, 0, sizeof cmdbuf); /* Clear it, just in case */ if (client_getln(cmdbuf, sizeof cmdbuf) < 1) { - lprintf(CTDL_CRIT, "Client disconnected: ending session.\n"); + CtdlLogPrintf(CTDL_CRIT, "Client disconnected: ending session.\n"); CC->kill_me = 1; return; } - lprintf(CTDL_INFO, "SMTP: %s\n", cmdbuf); + CtdlLogPrintf(CTDL_INFO, "SMTP server: %s\n", cmdbuf); while (strlen(cmdbuf) < 5) strcat(cmdbuf, " "); - if (SMTP->command_state == smtp_user) { + if (sSMTP->command_state == smtp_user) { smtp_get_user(cmdbuf); } - else if (SMTP->command_state == smtp_password) { + else if (sSMTP->command_state == smtp_password) { smtp_get_pass(cmdbuf); } - else if (SMTP->command_state == smtp_plain) { + else if (sSMTP->command_state == smtp_plain) { smtp_try_plain(cmdbuf); } @@ -909,7 +959,7 @@ void smtp_command_loop(void) { * */ void smtp_try(const char *key, const char *addr, int *status, - char *dsn, size_t n, long msgnum) + char *dsn, size_t n, long msgnum, char *envelope_from) { int sock = (-1); char mxhosts[1024]; @@ -925,77 +975,82 @@ void smtp_try(const char *key, const char *addr, int *status, char mx_port[256]; int lp, rp; char *msgtext; - char *ptr; + const char *ptr; size_t msg_size; int scan_done; - + CitContext *CCC=CC; + + /* Parse out the host portion of the recipient address */ process_rfc822_addr(addr, user, node, name); - lprintf(CTDL_DEBUG, "Attempting SMTP delivery to <%s> @ <%s> (%s)\n", + CtdlLogPrintf(CTDL_DEBUG, "SMTP client: Attempting delivery to <%s> @ <%s> (%s)\n", user, node, name); /* Load the message out of the database */ - CC->redirect_buffer = malloc(SIZ); - CC->redirect_len = 0; - CC->redirect_alloc = SIZ; - CtdlOutputMsg(msgnum, MT_RFC822, HEADERS_ALL, 0, 1, NULL); + CCC->redirect_buffer = malloc(SIZ); + CCC->redirect_len = 0; + CCC->redirect_alloc = SIZ; + CtdlOutputMsg(msgnum, MT_RFC822, HEADERS_ALL, 0, 1, NULL, ESC_DOT); msgtext = CC->redirect_buffer; msg_size = CC->redirect_len; - CC->redirect_buffer = NULL; - CC->redirect_len = 0; - CC->redirect_alloc = 0; - - /* Extract something to send later in the 'MAIL From:' command */ - strcpy(mailfrom, ""); - scan_done = 0; - ptr = msgtext; - do { - if (ptr = memreadline(ptr, buf, sizeof buf), *ptr == 0) { - scan_done = 1; - } - if (!strncasecmp(buf, "From:", 5)) { - safestrncpy(mailfrom, &buf[5], sizeof mailfrom); - striplt(mailfrom); - for (i=0; mailfrom[i]; ++i) { - if (!isprint(mailfrom[i])) { - strcpy(&mailfrom[i], &mailfrom[i+1]); - i=0; - } + CCC->redirect_buffer = NULL; + CCC->redirect_len = 0; + CCC->redirect_alloc = 0; + + /* If no envelope_from is supplied, extract one from the message */ + if ( (envelope_from == NULL) || (IsEmptyStr(envelope_from)) ) { + strcpy(mailfrom, ""); + scan_done = 0; + ptr = msgtext; + do { + if (ptr = memreadline(ptr, buf, sizeof buf), *ptr == 0) { + scan_done = 1; } - - /* Strip out parenthesized names */ - lp = (-1); - rp = (-1); - for (i=0; mailfrom[i]; ++i) { - if (mailfrom[i] == '(') lp = i; - if (mailfrom[i] == ')') rp = i; - } - if ((lp>0)&&(rp>lp)) { - strcpy(&mailfrom[lp-1], &mailfrom[rp+1]); - } - - /* Prefer brokketized names */ - lp = (-1); - rp = (-1); - for (i=0; mailfrom[i]; ++i) { - if (mailfrom[i] == '<') lp = i; - if (mailfrom[i] == '>') rp = i; - } - if ( (lp>=0) && (rp>lp) ) { - mailfrom[rp] = 0; - strcpy(mailfrom, &mailfrom[lp]); + if (!strncasecmp(buf, "From:", 5)) { + safestrncpy(mailfrom, &buf[5], sizeof mailfrom); + striplt(mailfrom); + for (i=0; mailfrom[i]; ++i) { + if (!isprint(mailfrom[i])) { + strcpy(&mailfrom[i], &mailfrom[i+1]); + i=0; + } + } + + /* Strip out parenthesized names */ + lp = (-1); + rp = (-1); + for (i=0; mailfrom[i]; ++i) { + if (mailfrom[i] == '(') lp = i; + if (mailfrom[i] == ')') rp = i; + } + if ((lp>0)&&(rp>lp)) { + strcpy(&mailfrom[lp-1], &mailfrom[rp+1]); + } + + /* Prefer brokketized names */ + lp = (-1); + rp = (-1); + for (i=0; mailfrom[i]; ++i) { + if (mailfrom[i] == '<') lp = i; + if (mailfrom[i] == '>') rp = i; + } + if ( (lp>=0) && (rp>lp) ) { + mailfrom[rp] = 0; + strcpy(mailfrom, &mailfrom[lp]); + } + + scan_done = 1; } - - scan_done = 1; - } - } while (scan_done == 0); - if (IsEmptyStr(mailfrom)) strcpy(mailfrom, "someone@somewhere.org"); - stripallbut(mailfrom, '<', '>'); + } while (scan_done == 0); + if (IsEmptyStr(mailfrom)) strcpy(mailfrom, "someone@somewhere.org"); + stripallbut(mailfrom, '<', '>'); + envelope_from = mailfrom; + } /* Figure out what mail exchanger host we have to connect to */ num_mxhosts = getmx(mxhosts, node); - lprintf(CTDL_DEBUG, "Number of MX hosts for <%s> is %d\n", node, num_mxhosts); + CtdlLogPrintf(CTDL_DEBUG, "Number of MX hosts for <%s> is %d [%s]\n", node, num_mxhosts, mxhosts); if (num_mxhosts < 1) { *status = 5; snprintf(dsn, SIZ, "No MX hosts found for <%s>", node); @@ -1029,10 +1084,10 @@ void smtp_try(const char *key, const char *addr, int *status, else { strcpy(mx_port, "25"); } - lprintf(CTDL_DEBUG, "Trying %s : %s ...\n", mx_host, mx_port); + CtdlLogPrintf(CTDL_DEBUG, "SMTP client: connecting to %s : %s ...\n", mx_host, mx_port); sock = sock_connect(mx_host, mx_port, "tcp"); snprintf(dsn, SIZ, "Could not connect: %s", strerror(errno)); - if (sock >= 0) lprintf(CTDL_DEBUG, "Connected!\n"); + if (sock >= 0) CtdlLogPrintf(CTDL_DEBUG, "SMTP client: connected!\n"); if (sock < 0) { if (errno > 0) { snprintf(dsn, SIZ, "%s", strerror(errno)); @@ -1048,13 +1103,17 @@ void smtp_try(const char *key, const char *addr, int *status, return; } + CCC->sReadBuf = NewStrBuf(); + CCC->sMigrateBuf = NewStrBuf(); + CCC->sPos = NULL; + /* Process the SMTP greeting from the server */ - if (ml_sock_gets(sock, buf) < 0) { + if (ml_sock_gets(&sock, buf) < 0) { *status = 4; strcpy(dsn, "Connection broken during SMTP conversation"); goto bail; } - lprintf(CTDL_DEBUG, "<%s\n", buf); + CtdlLogPrintf(CTDL_DEBUG, "<%s\n", buf); if (buf[0] != '2') { if (buf[0] == '4') { *status = 4; @@ -1072,19 +1131,19 @@ void smtp_try(const char *key, const char *addr, int *status, /* Do a EHLO command. If it fails, try the HELO command. */ snprintf(buf, sizeof buf, "EHLO %s\r\n", config.c_fqdn); - lprintf(CTDL_DEBUG, ">%s", buf); - sock_write(sock, buf, strlen(buf)); - if (ml_sock_gets(sock, buf) < 0) { + CtdlLogPrintf(CTDL_DEBUG, ">%s", buf); + sock_write(&sock, buf, strlen(buf)); + if (ml_sock_gets(&sock, buf) < 0) { *status = 4; strcpy(dsn, "Connection broken during SMTP HELO"); goto bail; } - lprintf(CTDL_DEBUG, "<%s\n", buf); + CtdlLogPrintf(CTDL_DEBUG, "<%s\n", buf); if (buf[0] != '2') { snprintf(buf, sizeof buf, "HELO %s\r\n", config.c_fqdn); - lprintf(CTDL_DEBUG, ">%s", buf); - sock_write(sock, buf, strlen(buf)); - if (ml_sock_gets(sock, buf) < 0) { + CtdlLogPrintf(CTDL_DEBUG, ">%s", buf); + sock_write(&sock, buf, strlen(buf)); + if (ml_sock_gets(&sock, buf) < 0) { *status = 4; strcpy(dsn, "Connection broken during SMTP HELO"); goto bail; @@ -1109,14 +1168,14 @@ void smtp_try(const char *key, const char *addr, int *status, sprintf(buf, "%s%c%s%c%s", mx_user, '\0', mx_user, '\0', mx_pass); CtdlEncodeBase64(encoded, buf, strlen(mx_user) + strlen(mx_user) + strlen(mx_pass) + 2, 0); snprintf(buf, sizeof buf, "AUTH PLAIN %s\r\n", encoded); - lprintf(CTDL_DEBUG, ">%s", buf); - sock_write(sock, buf, strlen(buf)); - if (ml_sock_gets(sock, buf) < 0) { + CtdlLogPrintf(CTDL_DEBUG, ">%s", buf); + sock_write(&sock, buf, strlen(buf)); + if (ml_sock_gets(&sock, buf) < 0) { *status = 4; strcpy(dsn, "Connection broken during SMTP AUTH"); goto bail; } - lprintf(CTDL_DEBUG, "<%s\n", buf); + CtdlLogPrintf(CTDL_DEBUG, "<%s\n", buf); if (buf[0] != '2') { if (buf[0] == '4') { *status = 4; @@ -1131,16 +1190,16 @@ void smtp_try(const char *key, const char *addr, int *status, } } - /* previous command succeeded, now try the MAIL From: command */ - snprintf(buf, sizeof buf, "MAIL From: <%s>\r\n", mailfrom); - lprintf(CTDL_DEBUG, ">%s", buf); - sock_write(sock, buf, strlen(buf)); - if (ml_sock_gets(sock, buf) < 0) { + /* previous command succeeded, now try the MAIL FROM: command */ + snprintf(buf, sizeof buf, "MAIL FROM:<%s>\r\n", envelope_from); + CtdlLogPrintf(CTDL_DEBUG, ">%s", buf); + sock_write(&sock, buf, strlen(buf)); + if (ml_sock_gets(&sock, buf) < 0) { *status = 4; strcpy(dsn, "Connection broken during SMTP MAIL"); goto bail; } - lprintf(CTDL_DEBUG, "<%s\n", buf); + CtdlLogPrintf(CTDL_DEBUG, "<%s\n", buf); if (buf[0] != '2') { if (buf[0] == '4') { *status = 4; @@ -1155,15 +1214,15 @@ void smtp_try(const char *key, const char *addr, int *status, } /* MAIL succeeded, now try the RCPT To: command */ - snprintf(buf, sizeof buf, "RCPT To: <%s@%s>\r\n", user, node); - lprintf(CTDL_DEBUG, ">%s", buf); - sock_write(sock, buf, strlen(buf)); - if (ml_sock_gets(sock, buf) < 0) { + snprintf(buf, sizeof buf, "RCPT TO:<%s@%s>\r\n", user, node); + CtdlLogPrintf(CTDL_DEBUG, ">%s", buf); + sock_write(&sock, buf, strlen(buf)); + if (ml_sock_gets(&sock, buf) < 0) { *status = 4; strcpy(dsn, "Connection broken during SMTP RCPT"); goto bail; } - lprintf(CTDL_DEBUG, "<%s\n", buf); + CtdlLogPrintf(CTDL_DEBUG, "<%s\n", buf); if (buf[0] != '2') { if (buf[0] == '4') { *status = 4; @@ -1178,14 +1237,14 @@ void smtp_try(const char *key, const char *addr, int *status, } /* RCPT succeeded, now try the DATA command */ - lprintf(CTDL_DEBUG, ">DATA\n"); - sock_write(sock, "DATA\r\n", 6); - if (ml_sock_gets(sock, buf) < 0) { + CtdlLogPrintf(CTDL_DEBUG, ">DATA\n"); + sock_write(&sock, "DATA\r\n", 6); + if (ml_sock_gets(&sock, buf) < 0) { *status = 4; strcpy(dsn, "Connection broken during SMTP DATA"); goto bail; } - lprintf(CTDL_DEBUG, "<%s\n", buf); + CtdlLogPrintf(CTDL_DEBUG, "<%s\n", buf); if (buf[0] != '3') { if (buf[0] == '4') { *status = 3; @@ -1199,21 +1258,22 @@ void smtp_try(const char *key, const char *addr, int *status, } } - /* If we reach this point, the server is expecting data */ - sock_write(sock, msgtext, msg_size); + /* If we reach this point, the server is expecting data.*/ + sock_write(&sock, msgtext, msg_size); if (msgtext[msg_size-1] != 10) { - lprintf(CTDL_WARNING, "Possible problem: message did not " + CtdlLogPrintf(CTDL_WARNING, "Possible problem: message did not " "correctly terminate. (expecting 0x10, got 0x%02x)\n", buf[msg_size-1]); + sock_write(&sock, "\r\n", 2); } - sock_write(sock, ".\r\n", 3); - if (ml_sock_gets(sock, buf) < 0) { + sock_write(&sock, ".\r\n", 3); + if (ml_sock_gets(&sock, buf) < 0) { *status = 4; strcpy(dsn, "Connection broken during SMTP message transmit"); goto bail; } - lprintf(CTDL_DEBUG, "%s\n", buf); + CtdlLogPrintf(CTDL_DEBUG, "%s\n", buf); if (buf[0] != '2') { if (buf[0] == '4') { *status = 4; @@ -1231,15 +1291,18 @@ void smtp_try(const char *key, const char *addr, int *status, safestrncpy(dsn, &buf[4], 1023); *status = 2; - lprintf(CTDL_DEBUG, ">QUIT\n"); - sock_write(sock, "QUIT\r\n", 6); - ml_sock_gets(sock, buf); - lprintf(CTDL_DEBUG, "<%s\n", buf); - lprintf(CTDL_INFO, "SMTP delivery to <%s> @ <%s> (%s) succeeded\n", + CtdlLogPrintf(CTDL_DEBUG, ">QUIT\n"); + sock_write(&sock, "QUIT\r\n", 6); + ml_sock_gets(&sock, buf); + CtdlLogPrintf(CTDL_DEBUG, "<%s\n", buf); + CtdlLogPrintf(CTDL_INFO, "SMTP client: delivery to <%s> @ <%s> (%s) succeeded\n", user, node, name); bail: free(msgtext); - sock_close(sock); + FreeStrBuf(&CCC->sReadBuf); + FreeStrBuf(&CCC->sMigrateBuf); + if (sock != -1) + sock_close(sock); /* 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). @@ -1287,7 +1350,7 @@ void smtp_do_bounce(char *instr) { size_t omsgsize; long omsgid = (-1); - lprintf(CTDL_DEBUG, "smtp_do_bounce() called\n"); + CtdlLogPrintf(CTDL_DEBUG, "smtp_do_bounce() called\n"); strcpy(bounceto, ""); sprintf(boundary, "=_Citadel_Multipart_%s_%04x%04x", config.c_fqdn, getpid(), ++seq); lines = num_tokens(instr, '\n'); @@ -1354,7 +1417,7 @@ void smtp_do_bounce(char *instr) { extract_token(dsn, buf, 3, '|', sizeof dsn); bounce_this = 0; - lprintf(CTDL_DEBUG, "key=<%s> addr=<%s> status=%d dsn=<%s>\n", + CtdlLogPrintf(CTDL_DEBUG, "key=<%s> addr=<%s> status=%d dsn=<%s>\n", key, addr, status, dsn); if (!strcasecmp(key, "bounceto")) { @@ -1374,7 +1437,7 @@ void smtp_do_bounce(char *instr) { ++num_bounces; if (bmsg->cm_fields['M'] == NULL) { - lprintf(CTDL_ERR, "ERROR ... M field is null " + CtdlLogPrintf(CTDL_ERR, "ERROR ... M field is null " "(%s:%d)\n", __FILE__, __LINE__); } @@ -1404,7 +1467,7 @@ void smtp_do_bounce(char *instr) { CC->redirect_buffer = malloc(SIZ); CC->redirect_len = 0; CC->redirect_alloc = SIZ; - CtdlOutputMsg(omsgid, MT_RFC822, HEADERS_ALL, 0, 1, NULL); + CtdlOutputMsg(omsgid, MT_RFC822, HEADERS_ALL, 0, 1, NULL, 0); omsgtext = CC->redirect_buffer; omsgsize = CC->redirect_len; CC->redirect_buffer = NULL; @@ -1422,13 +1485,13 @@ void smtp_do_bounce(char *instr) { strcat(bmsg->cm_fields['M'], "--\r\n"); /* Deliver the bounce if there's anything worth mentioning */ - lprintf(CTDL_DEBUG, "num_bounces = %d\n", num_bounces); + CtdlLogPrintf(CTDL_DEBUG, "num_bounces = %d\n", num_bounces); if (num_bounces > 0) { /* First try the user who sent the message */ - lprintf(CTDL_DEBUG, "bounce to user? <%s>\n", bounceto); + CtdlLogPrintf(CTDL_DEBUG, "bounce to user? <%s>\n", bounceto); if (IsEmptyStr(bounceto)) { - lprintf(CTDL_ERR, "No bounce address specified\n"); + CtdlLogPrintf(CTDL_ERR, "No bounce address specified\n"); bounce_msgid = (-1L); } @@ -1436,14 +1499,14 @@ void smtp_do_bounce(char *instr) { valid = validate_recipients(bounceto, smtp_get_Recipients (), 0); if (valid != NULL) { if (valid->num_error == 0) { - CtdlSubmitMsg(bmsg, valid, ""); + CtdlSubmitMsg(bmsg, valid, "", QP_EADDR); successful_bounce = 1; } } /* If not, post it in the Aide> room */ if (successful_bounce == 0) { - CtdlSubmitMsg(bmsg, NULL, config.c_aideroom); + CtdlSubmitMsg(bmsg, NULL, config.c_aideroom, QP_EADDR); } /* Free up the memory we used */ @@ -1453,7 +1516,7 @@ void smtp_do_bounce(char *instr) { } CtdlFreeMessage(bmsg); - lprintf(CTDL_DEBUG, "Done processing bounces\n"); + CtdlLogPrintf(CTDL_DEBUG, "Done processing bounces\n"); } @@ -1516,17 +1579,19 @@ void smtp_do_procmsg(long msgnum, void *userdata) { char key[1024]; char addr[1024]; char dsn[1024]; + char envelope_from[1024]; long text_msgid = (-1); int incomplete_deliveries_remaining; time_t attempted = 0L; time_t last_attempted = 0L; time_t retry = SMTP_RETRY_INTERVAL; - lprintf(CTDL_DEBUG, "smtp_do_procmsg(%ld)\n", msgnum); + CtdlLogPrintf(CTDL_DEBUG, "SMTP client: smtp_do_procmsg(%ld)\n", msgnum); + strcpy(envelope_from, ""); msg = CtdlFetchMessage(msgnum, 1); if (msg == NULL) { - lprintf(CTDL_ERR, "SMTP: tried %ld but no such message!\n", msgnum); + CtdlLogPrintf(CTDL_ERR, "SMTP client: tried %ld but no such message!\n", msgnum); return; } @@ -1552,6 +1617,9 @@ void smtp_do_procmsg(long msgnum, void *userdata) { if (!strcasecmp(key, "msgid")) { text_msgid = extract_long(buf, 1); } + if (!strcasecmp(key, "envelope_from")) { + extract_token(envelope_from, buf, 1, '|', sizeof envelope_from); + } if (!strcasecmp(key, "retry")) { /* double the retry interval after each attempt */ retry = extract_long(buf, 1) * 2L; @@ -1571,7 +1639,7 @@ void smtp_do_procmsg(long msgnum, void *userdata) { * Postpone delivery if we've already tried recently. */ if (((time(NULL) - last_attempted) < retry) && (run_queue_now == 0)) { - lprintf(CTDL_DEBUG, "Retry time not yet reached.\n"); + CtdlLogPrintf(CTDL_DEBUG, "SMTP client: Retry time not yet reached.\n"); free(instr); return; } @@ -1581,7 +1649,7 @@ void smtp_do_procmsg(long msgnum, void *userdata) { * Bail out if there's no actual message associated with this */ if (text_msgid < 0L) { - lprintf(CTDL_ERR, "SMTP: no 'msgid' directive found!\n"); + CtdlLogPrintf(CTDL_ERR, "SMTP client: no 'msgid' directive found!\n"); free(instr); return; } @@ -1611,16 +1679,15 @@ void smtp_do_procmsg(long msgnum, void *userdata) { --i; --lines; - lprintf(CTDL_DEBUG, "SMTP: Trying <%s>\n", addr); - smtp_try(key, addr, &status, dsn, sizeof dsn, text_msgid); + CtdlLogPrintf(CTDL_DEBUG, "SMTP client: Trying <%s>\n", addr); + smtp_try(key, addr, &status, dsn, sizeof dsn, text_msgid, envelope_from); if (status != 2) { if (results == NULL) { results = malloc(1024); memset(results, 0, 1024); } else { - results = realloc(results, - strlen(results) + 1024); + results = realloc(results, strlen(results) + 1024); } snprintf(&results[strlen(results)], 1024, "%s|%s|%d|%s\n", @@ -1673,7 +1740,7 @@ void smtp_do_procmsg(long msgnum, void *userdata) { "attempted|%ld\n" "retry|%ld\n", SPOOLMIME, instr, (long)time(NULL), (long)retry ); - CtdlSubmitMsg(msg, NULL, SMTP_SPOOLOUT_ROOM); + CtdlSubmitMsg(msg, NULL, SMTP_SPOOLOUT_ROOM, QP_EADDR); CtdlFreeMessage(msg); } @@ -1682,38 +1749,63 @@ void smtp_do_procmsg(long msgnum, void *userdata) { + /* * smtp_do_queue() * * Run through the queue sending out messages. */ -void smtp_do_queue(void) { - static int doing_queue = 0; +void *smtp_do_queue(void *arg) { + int num_processed = 0; + struct CitContext smtp_queue_CC; - /* - * This is a simple concurrency check to make sure only one queue run - * is done at a time. We could do this with a mutex, but since we - * don't really require extremely fine granularity here, we'll do it - * with a static variable instead. - */ - if (doing_queue) return; - doing_queue = 1; + CtdlLogPrintf(CTDL_INFO, "SMTP client: processing outbound queue\n"); - /* - * Go ahead and run the queue - */ - lprintf(CTDL_INFO, "SMTP: processing outbound queue\n"); + CtdlFillSystemContext(&smtp_queue_CC, "SMTP Send"); + citthread_setspecific(MyConKey, (void *)&smtp_queue_CC ); - if (getroom(&CC->room, SMTP_SPOOLOUT_ROOM) != 0) { - lprintf(CTDL_ERR, "Cannot find room <%s>\n", SMTP_SPOOLOUT_ROOM); - return; + if (CtdlGetRoom(&CC->room, SMTP_SPOOLOUT_ROOM) != 0) { + CtdlLogPrintf(CTDL_ERR, "Cannot find room <%s>\n", SMTP_SPOOLOUT_ROOM); + } + else { + num_processed = CtdlForEachMessage(MSGS_ALL, 0L, NULL, SPOOLMIME, NULL, smtp_do_procmsg, NULL); } - CtdlForEachMessage(MSGS_ALL, 0L, NULL, - SPOOLMIME, NULL, smtp_do_procmsg, NULL); - lprintf(CTDL_INFO, "SMTP: queue run completed\n"); - run_queue_now = 0; - doing_queue = 0; + citthread_mutex_unlock (&smtp_send_lock); + CtdlLogPrintf(CTDL_INFO, "SMTP client: queue run completed; %d messages processed\n", num_processed); + return(NULL); +} + + + +/* + * smtp_queue_thread + * + * Create a thread to run the SMTP queue + * + * This was created as a response to a situation seen on Uncensored where a bad remote was holding + * up SMTP sending for long times. + * Converting to a thread does not fix the problem caused by the bad remote but it does prevent + * the SMTP sending from stopping housekeeping and the EVT_TIMER event system which in turn prevented + * other things from happening. + */ +void smtp_queue_thread (void) +{ + if (citthread_mutex_trylock (&smtp_send_lock)) { + CtdlLogPrintf(CTDL_DEBUG, "SMTP queue run already in progress\n"); + } + else { + CtdlThreadCreate("SMTP Send", CTDLTHREAD_BIGSTACK, smtp_do_queue, NULL); + } +} + + + +void smtp_server_going_down (void) +{ + CtdlLogPrintf(CTDL_DEBUG, "SMTP module clean up for shutdown.\n"); + + citthread_mutex_destroy (&smtp_send_lock); } @@ -1769,15 +1861,15 @@ void smtp_init_spoolout(void) { * Create the room. This will silently fail if the room already * exists, and that's perfectly ok, because we want it to exist. */ - create_room(SMTP_SPOOLOUT_ROOM, 3, "", 0, 1, 0, VIEW_MAILBOX); + CtdlCreateRoom(SMTP_SPOOLOUT_ROOM, 3, "", 0, 1, 0, VIEW_MAILBOX); /* * Make sure it's set to be a "system room" so it doesn't show up * in the nown rooms list for Aides. */ - if (lgetroom(&qrbuf, SMTP_SPOOLOUT_ROOM) == 0) { + if (CtdlGetRoomLock(&qrbuf, SMTP_SPOOLOUT_ROOM) == 0) { qrbuf.QRflags2 |= QR2_SYSTEM; - lputroom(&qrbuf); + CtdlPutRoomLock(&qrbuf); } } @@ -1796,7 +1888,7 @@ 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; - lprintf(CTDL_DEBUG, "Performing SMTP cleanup hook\n"); + CtdlLogPrintf(CTDL_DEBUG, "Performing SMTP cleanup hook\n"); free(SMTP); } @@ -1850,9 +1942,11 @@ CTDL_MODULE_INIT(smtp) CitadelServiceSMTP_LMTP_UNF); smtp_init_spoolout(); - CtdlRegisterSessionHook(smtp_do_queue, EVT_TIMER); + CtdlRegisterSessionHook(smtp_queue_thread, EVT_TIMER); CtdlRegisterSessionHook(smtp_cleanup_function, EVT_STOP); CtdlRegisterProtoHook(cmd_smtp, "SMTP", "SMTP utility commands"); + CtdlRegisterCleanupHook (smtp_server_going_down); + citthread_mutex_init (&smtp_send_lock, NULL); } /* return our Subversion id for the Log */