removed some debugs
[citadel.git] / citadel / modules / pop3 / serv_pop3.c
index e839f5ec9c6e4c72c0148069ed759206c96211e6..6cdd55cc2ae8b6b4d241816542de7b938dafc222 100644 (file)
@@ -1,20 +1,23 @@
-/*
- * $Id$ 
- *
- * POP3 service for the Citadel system
- * Copyright (C) 1998-2001 by Art Cancro and others.
- * This code is released under the terms of the GNU General Public License.
- *
- * Current status of standards conformance:
- *
- * -> All required POP3 commands described in RFC1939 are implemented.
- * -> All optional POP3 commands described in RFC1939 are also implemented.
- * -> The deprecated "LAST" command is included in this implementation, because
- *    there exist mail clients which insist on using it (such as Bynari
- *    TradeMail, and certain versions of Eudora).
- * -> Capability detection via the method described in RFC2449 is implemented.
- * 
- */
+// POP3 service for the Citadel system
+//
+// 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.
+//
+// Current status of standards conformance:
+//
+// -> All required POP3 commands described in RFC1939 are implemented.
+// -> All optional POP3 commands described in RFC1939 are also implemented.
+// -> The deprecated "LAST" command is included in this implementation, because
+//    there exist mail clients which insist on using it (such as Bynari
+//    TradeMail, and certain versions of Eudora).
+// -> Capability detection via the method described in RFC2449 is implemented.
 
 #include "sysdep.h"
 #include <stdlib.h>
 #include <pwd.h>
 #include <errno.h>
 #include <sys/types.h>
-
-#if TIME_WITH_SYS_TIME
-# include <sys/time.h>
-# include <time.h>
-#else
-# if HAVE_SYS_TIME_H
-#  include <sys/time.h>
-# else
-#  include <time.h>
-# endif
-#endif
-
+#include <time.h>
 #include <sys/wait.h>
 #include <string.h>
 #include <limits.h>
 #include "citserver.h"
 #include "support.h"
 #include "config.h"
-#include "room_ops.h"
 #include "user_ops.h"
-#include "policy.h"
 #include "database.h"
 #include "msgbase.h"
 #include "internet_addressing.h"
 #include "serv_pop3.h"
-#include "md5.h"
-
-
-
 #include "ctdl_module.h"
 
 
-
-/*
- * This cleanup function blows away the temporary memory and files used by
- * the POP3 server.
- */
+// This cleanup function blows away the temporary memory and files used by
+// the POP3 server.
 void pop3_cleanup_function(void) {
-
        /* Don't do this stuff if this is not a POP3 session! */
        if (CC->h_command_function != pop3_command_loop) return;
 
-       CtdlLogPrintf(CTDL_DEBUG, "Performing POP3 cleanup hook\n");
-       if (POP3->msgs != NULL) free(POP3->msgs);
+       struct citpop3 *pop3 = ((struct citpop3 *)CC->session_specific_data);
+       syslog(LOG_DEBUG, "pop3: performing cleanup hook");
+       if (pop3->msgs != NULL) {
+               free(pop3->msgs);
+       }
 
-       free(POP3);
+       free(pop3);
 }
 
 
-
-/*
- * Here's where our POP3 session begins its happy day.
- */
+// Here's where our POP3 session begins its happy day.
 void pop3_greeting(void) {
        strcpy(CC->cs_clientname, "POP3 session");
        CC->internal_pgm = 1;
        CC->session_specific_data = malloc(sizeof(struct citpop3));
        memset(POP3, 0, sizeof(struct citpop3));
 
-       cprintf("+OK Citadel POP3 server %s\r\n",
-               CC->cs_nonce);
+       cprintf("+OK Citadel POP3 server ready.\r\n");
 }
 
 
-/*
- * POP3S is just like POP3, except it goes crypto right away.
- */
+// POP3S is just like POP3, except it goes crypto right away.
 void pop3s_greeting(void) {
        CtdlModuleStartCryptoMsgs(NULL, NULL, NULL);
 
 /* kill session if no crypto */
 #ifdef HAVE_OPENSSL
-       if (!CC->redirect_ssl) CC->kill_me = 1;
+       if (!CC->redirect_ssl) CC->kill_me = KILLME_NO_CRYPTO;
 #else
-       CC->kill_me = 1;
+       CC->kill_me = KILLME_NO_CRYPTO;
 #endif
 
        pop3_greeting();
 }
 
 
-
-/*
- * Specify user name (implements POP3 "USER" command)
- */
+// Specify user name (implements POP3 "USER" command)
 void pop3_user(char *argbuf) {
        char username[SIZ];
 
@@ -125,8 +101,7 @@ void pop3_user(char *argbuf) {
        strcpy(username, argbuf);
        striplt(username);
 
-       /* CtdlLogPrintf(CTDL_DEBUG, "Trying <%s>\n", username); */
-       if (CtdlLoginExistingUser(NULL, username) == login_ok) {
+       if (CtdlLoginExistingUser(username) == login_ok) {
                cprintf("+OK Password required for %s\r\n", username);
        }
        else {
@@ -135,64 +110,53 @@ void pop3_user(char *argbuf) {
 }
 
 
-
-/*
- * Back end for pop3_grab_mailbox()
- */
+// Back end for pop3_grab_mailbox()
 void pop3_add_message(long msgnum, void *userdata) {
        struct MetaData smi;
 
        ++POP3->num_msgs;
-       if (POP3->num_msgs < 2) POP3->msgs = malloc(sizeof(struct pop3msg));
-       else POP3->msgs = realloc(POP3->msgs, 
-               (POP3->num_msgs * sizeof(struct pop3msg)) ) ;
+       if (POP3->num_msgs < 2) {
+               POP3->msgs = malloc(sizeof(struct pop3msg));
+       }
+       else {
+               POP3->msgs = realloc(POP3->msgs, (POP3->num_msgs * sizeof(struct pop3msg)) ) ;
+       }
        POP3->msgs[POP3->num_msgs-1].msgnum = msgnum;
        POP3->msgs[POP3->num_msgs-1].deleted = 0;
 
-       /* We need to know the length of this message when it is printed in
-        * RFC822 format.  Perhaps we have cached this length in the message's
-        * metadata record.  If so, great; if not, measure it and then cache
-        * it for next time.
-        */
+       // We need to know the length of this message when it is printed in
+       // RFC822 format.  Perhaps we have cached this length in the message's
+       // metadata record.  If so, great; if not, measure it and then cache
+       // it for next time.
        GetMetaData(&smi, msgnum);
        if (smi.meta_rfc822_length <= 0L) {
-               CC->redirect_buffer = malloc(SIZ);
-               CC->redirect_len = 0;
-               CC->redirect_alloc = SIZ;
-               CtdlOutputMsg(msgnum, MT_RFC822, HEADERS_ALL, 0, 1, NULL);
-               smi.meta_rfc822_length = CC->redirect_len;
-               free(CC->redirect_buffer);
-               CC->redirect_buffer = NULL;
-               CC->redirect_len = 0;
-               CC->redirect_alloc = 0;
+               CC->redirect_buffer = NewStrBufPlain(NULL, SIZ);
+               CtdlOutputMsg(msgnum, MT_RFC822, HEADERS_ALL, 0, 1, NULL, SUPPRESS_ENV_TO, NULL, NULL, NULL);
+               smi.meta_rfc822_length = StrLength(CC->redirect_buffer);
+               FreeStrBuf(&CC->redirect_buffer);
                PutMetaData(&smi);
        }
        POP3->msgs[POP3->num_msgs-1].rfc822_length = smi.meta_rfc822_length;
 }
 
 
-
-/*
- * Open the inbox and read its contents.
- * (This should be called only once, by pop3_pass(), and returns the number
- * of messages in the inbox, or -1 for error)
- */
+// Open the inbox and read its contents.
+// (This should be called only once, by pop3_pass(), and returns the number
+// of messages in the inbox, or -1 for error)
 int pop3_grab_mailbox(void) {
-        struct visit vbuf;
+        visit vbuf;
        int i;
 
-       if (getroom(&CC->room, MAILROOM) != 0) return(-1);
+       if (CtdlGetRoom(&CC->room, MAILROOM) != 0) return(-1);
 
        /* Load up the messages */
-       CtdlForEachMessage(MSGS_ALL, 0L, NULL, NULL, NULL,
-               pop3_add_message, NULL);
+       CtdlForEachMessage(MSGS_ALL, 0L, NULL, NULL, NULL, pop3_add_message, NULL);
 
        /* Figure out which are old and which are new */
         CtdlGetRelationship(&vbuf, &CC->user, &CC->room);
        POP3->lastseen = (-1);
        if (POP3->num_msgs) for (i=0; i<POP3->num_msgs; ++i) {
-               if (is_msg_in_sequence_set(vbuf.v_seen,
-                  (POP3->msgs[POP3->num_msgs-1].msgnum) )) {
+               if (is_msg_in_sequence_set(vbuf.v_seen, (POP3->msgs[POP3->num_msgs-1].msgnum) )) {
                        POP3->lastseen = i;
                }
        }
@@ -200,15 +164,15 @@ int pop3_grab_mailbox(void) {
        return(POP3->num_msgs);
 }
 
-void pop3_login(void)
-{
+
+void pop3_login(void) {
        int msgs;
        
        msgs = pop3_grab_mailbox();
        if (msgs >= 0) {
                cprintf("+OK %s is logged in (%d messages)\r\n",
                        CC->user.fullname, msgs);
-               CtdlLogPrintf(CTDL_NOTICE, "POP3 authenticated %s\n", CC->user.fullname);
+               syslog(LOG_DEBUG, "pop3: authenticated %s", CC->user.fullname);
        }
        else {
                cprintf("-ERR Can't open your mailbox\r\n");
@@ -216,72 +180,15 @@ void pop3_login(void)
        
 }
 
-void pop3_apop(char *argbuf)
-{
-   char username[SIZ];
-   char userdigest[MD5_HEXSTRING_SIZE];
-   char realdigest[MD5_HEXSTRING_SIZE];
-   char *sptr;
-   
-   if (CC->logged_in)
-   {
-       cprintf("-ERR You are already logged in; not in the AUTHORIZATION phase.\r\n");
-       return;
-   }
-   
-   if ((sptr = strchr(argbuf, ' ')) == NULL)
-   {
-       cprintf("-ERR Invalid APOP line.\r\n");
-       return;
-   }
-   
-   *sptr++ = '\0';
-   
-   while ((*sptr) && isspace(*sptr))
-      sptr++;
-   
-   strncpy(username, argbuf, sizeof(username)-1);
-   username[sizeof(username)-1] = '\0';
-   
-   memset(userdigest, MD5_HEXSTRING_SIZE, 0);
-   strncpy(userdigest, sptr, MD5_HEXSTRING_SIZE-1);
-   
-   if (CtdlLoginExistingUser(NULL, username) != login_ok)
-   {
-       cprintf("-ERR No such user.\r\n");
-       return;
-   }
-   
-   if (getuser(&CC->user, CC->curr_user))
-   {
-       cprintf("-ERR No such user.\r\n");
-       return;
-   }
-   
-   make_apop_string(CC->user.password, CC->cs_nonce, realdigest, sizeof realdigest);
-   if (!strncasecmp(realdigest, userdigest, MD5_HEXSTRING_SIZE-1))
-   {
-       do_login();
-       pop3_login();
-   }
-   else
-   {
-       cprintf("-ERR That is NOT the password.\r\n");
-   }
-}
 
-
-/*
- * Authorize with password (implements POP3 "PASS" command)
- */
+// Authorize with password (implements POP3 "PASS" command)
 void pop3_pass(char *argbuf) {
        char password[SIZ];
 
-       strcpy(password, argbuf);
+       safestrncpy(password, argbuf, sizeof password);
        striplt(password);
 
-       /* CtdlLogPrintf(CTDL_DEBUG, "Trying <%s>\n", password); */
-       if (CtdlTryPassword(password) == pass_ok) {
+       if (CtdlTryPassword(password, strlen(password)) == pass_ok) {
                pop3_login();
        }
        else {
@@ -290,21 +197,17 @@ void pop3_pass(char *argbuf) {
 }
 
 
-
-/*
- * list available msgs
- */
+// list available msgs
 void pop3_list(char *argbuf) {
        int i;
        int which_one;
 
        which_one = atoi(argbuf);
 
-       /* "list one" mode */
+       // "list one" mode
        if (which_one > 0) {
                if (which_one > POP3->num_msgs) {
-                       cprintf("-ERR no such message, only %d are here\r\n",
-                               POP3->num_msgs);
+                       cprintf("-ERR no such message, only %d are here\r\n", POP3->num_msgs);
                        return;
                }
                else if (POP3->msgs[which_one-1].deleted) {
@@ -312,22 +215,17 @@ void pop3_list(char *argbuf) {
                        return;
                }
                else {
-                       cprintf("+OK %d %ld\r\n",
-                               which_one,
-                               (long)POP3->msgs[which_one-1].rfc822_length
-                               );
+                       cprintf("+OK %d %ld\r\n", which_one, (long)POP3->msgs[which_one-1].rfc822_length);
                        return;
                }
        }
 
-       /* "list all" (scan listing) mode */
+       // "list all" (scan listing) mode
        else {
                cprintf("+OK Here's your mail:\r\n");
                if (POP3->num_msgs > 0) for (i=0; i<POP3->num_msgs; ++i) {
                        if (! POP3->msgs[i].deleted) {
-                               cprintf("%d %ld\r\n",
-                                       i+1,
-                                       (long)POP3->msgs[i].rfc822_length);
+                               cprintf("%d %ld\r\n", i+1, (long)POP3->msgs[i].rfc822_length);
                        }
                }
                cprintf(".\r\n");
@@ -335,9 +233,7 @@ void pop3_list(char *argbuf) {
 }
 
 
-/*
- * STAT (tally up the total message count and byte count) command
- */
+// STAT (tally up the total message count and byte count) command
 void pop3_stat(char *argbuf) {
        int total_msgs = 0;
        size_t total_octets = 0;
@@ -354,16 +250,9 @@ void pop3_stat(char *argbuf) {
 }
 
 
-
-/*
- * RETR command (fetch a message)
- */
+// RETR command (fetch a message)
 void pop3_retr(char *argbuf) {
        int which_one;
-       char *msgtext;
-       char *nextline;
-       char *chunk_to_send;
-       char prev_char;
 
        which_one = atoi(argbuf);
        if ( (which_one < 1) || (which_one > POP3->num_msgs) ) {
@@ -377,55 +266,22 @@ void pop3_retr(char *argbuf) {
        }
 
        cprintf("+OK Message %d:\r\n", which_one);
-       CC->redirect_buffer = malloc(SIZ);
-       CC->redirect_len = 0;
-       CC->redirect_alloc = SIZ;
-       CtdlOutputMsg(POP3->msgs[which_one - 1].msgnum,
-                       MT_RFC822, HEADERS_ALL, 0, 1, NULL);
-       msgtext = CC->redirect_buffer;
-       CC->redirect_buffer = NULL;
-       CC->redirect_len = 0;
-       CC->redirect_alloc = 0;
-
-       /* If we reach this point, the client is expecting data.
-        * Need to parse each line of the message here since someone may have sent
-        * a message containing a single dot on a line of its own. In that case we
-        * need to escape it in accordance with RFC821.
-        * We could do this with the tokenizer functions but num_tokens returns an
-        * int and the message may contain more lines than that, also copying each
-        * line would be slow.
-        */
-       nextline = msgtext;
-       while (*nextline)
-       {
-               chunk_to_send = nextline;
-               while (*nextline != '\n')
-                       nextline++;
-               nextline++;
-               prev_char = *nextline;
-               *nextline = '\0';
-               if (!strcmp(chunk_to_send, ".\r\n")) {
-                       client_write("..\r\n", 4);
-               }
-               else {
-                       client_write(chunk_to_send, (size_t)(nextline-chunk_to_send));
-               }
-               *nextline = prev_char;
-       }
+       CtdlOutputMsg(POP3->msgs[which_one - 1].msgnum, MT_RFC822,
+               HEADERS_ALL, 0, 1, NULL,
+               (ESC_DOT|SUPPRESS_ENV_TO), NULL, NULL, NULL
+       );
        cprintf(".\r\n");
 }
 
 
-/*
- * TOP command (dumb way of fetching a partial message or headers-only)
- */
+// TOP command (dumb way of fetching a partial message or headers-only)
 void pop3_top(char *argbuf) {
        int which_one;
        int lines_requested = 0;
        int lines_dumped = 0;
        char buf[1024];
-       char *msgtext;
-       char *ptr;
+       StrBuf *msgtext;
+       const char *ptr;
        int in_body = 0;
        int done = 0;
 
@@ -440,21 +296,22 @@ void pop3_top(char *argbuf) {
                return;
        }
 
-       CC->redirect_buffer = malloc(SIZ);
-       CC->redirect_len = 0;
-       CC->redirect_alloc = SIZ;
+       CC->redirect_buffer = NewStrBufPlain(NULL, SIZ);
+
        CtdlOutputMsg(POP3->msgs[which_one - 1].msgnum,
-                       MT_RFC822, HEADERS_ALL, 0, 1, NULL);
+                     MT_RFC822,
+                     HEADERS_ALL,
+                     0, 1, NULL,
+                     SUPPRESS_ENV_TO,
+                     NULL, NULL, NULL);
+
        msgtext = CC->redirect_buffer;
        CC->redirect_buffer = NULL;
-       CC->redirect_len = 0;
-       CC->redirect_alloc = 0;
 
        cprintf("+OK Message %d:\r\n", which_one);
-
-       ptr = msgtext;
-
-       while (ptr = memreadline(ptr, buf, (sizeof buf - 2)),
+       
+       ptr = ChrPtr(msgtext);
+       while (ptr = cmemreadline(ptr, buf, (sizeof buf - 2)),
              ( (*ptr != 0) && (done == 0))) {
                strcat(buf, "\r\n");
                if (in_body == 1) {
@@ -472,15 +329,13 @@ void pop3_top(char *argbuf) {
        }
 
        if (buf[strlen(buf)-1] != 10) cprintf("\n");
-       free(msgtext);
+       FreeStrBuf(&msgtext);
 
        cprintf(".\r\n");
 }
 
 
-/*
- * DELE (delete message from mailbox)
- */
+// DELE (delete message from mailbox)
 void pop3_dele(char *argbuf) {
        int which_one;
 
@@ -495,23 +350,22 @@ void pop3_dele(char *argbuf) {
                return;
        }
 
-       /* Flag the message as deleted.  Will expunge during QUIT command. */
+       // Flag the message as deleted.  Will expunge during QUIT command.
        POP3->msgs[which_one - 1].deleted = 1;
        cprintf("+OK Message %d deleted.\r\n",
                which_one);
 }
 
 
-/* Perform "UPDATE state" stuff
- */
+// Perform "UPDATE state" stuff
 void pop3_update(void) {
        int i;
-        struct visit vbuf;
+        visit vbuf;
 
        long *deletemsgs = NULL;
        int num_deletemsgs = 0;
 
-       /* Remove messages marked for deletion */
+       // Remove messages marked for deletion
        if (POP3->num_msgs > 0) {
                deletemsgs = malloc(POP3->num_msgs * sizeof(long));
                for (i=0; i<POP3->num_msgs; ++i) {
@@ -525,24 +379,19 @@ void pop3_update(void) {
                free(deletemsgs);
        }
 
-       /* Set last read pointer */
+       // Set last read pointer
        if (POP3->num_msgs > 0) {
-               lgetuser(&CC->user, CC->curr_user);
-
+               CtdlLockGetCurrentUser();
                CtdlGetRelationship(&vbuf, &CC->user, &CC->room);
-               snprintf(vbuf.v_seen, sizeof vbuf.v_seen, "*:%ld",
-                       POP3->msgs[POP3->num_msgs-1].msgnum);
+               snprintf(vbuf.v_seen, sizeof vbuf.v_seen, "*:%ld", POP3->msgs[POP3->num_msgs-1].msgnum);
                CtdlSetRelationship(&vbuf, &CC->user, &CC->room);
-
-               lputuser(&CC->user);
+               CtdlPutCurrentUserLock();
        }
 
 }
 
 
-/* 
- * RSET (reset, i.e. undelete any deleted messages) command
- */
+// RSET (reset, i.e. undelete any deleted messages) command
 void pop3_rset(char *argbuf) {
        int i;
 
@@ -555,19 +404,13 @@ void pop3_rset(char *argbuf) {
 }
 
 
-
-/* 
- * LAST (Determine which message is the last unread message)
- */
+// LAST (Determine which message is the last unread message)
 void pop3_last(char *argbuf) {
        cprintf("+OK %d\r\n", POP3->lastseen + 1);
 }
 
 
-/*
- * CAPA is a command which tells the client which POP3 extensions
- * are supported.
- */
+// CAPA is a command which tells the client which POP3 extensions are supported.
 void pop3_capa(void) {
        cprintf("+OK Capability list follows\r\n"
                "TOP\r\n"
@@ -581,22 +424,18 @@ void pop3_capa(void) {
 }
 
 
-
-/*
- * UIDL (Universal IDentifier Listing) is easy.  Our 'unique' message
- * identifiers are simply the Citadel message numbers in the database.
- */
+// UIDL (Universal IDentifier Listing) is easy.  Our 'unique' message
+// identifiers are simply the Citadel message numbers in the database.
 void pop3_uidl(char *argbuf) {
        int i;
        int which_one;
 
        which_one = atoi(argbuf);
 
-       /* "list one" mode */
+       // "list one" mode
        if (which_one > 0) {
                if (which_one > POP3->num_msgs) {
-                       cprintf("-ERR no such message, only %d are here\r\n",
-                               POP3->num_msgs);
+                       cprintf("-ERR no such message, only %d are here\r\n", POP3->num_msgs);
                        return;
                }
                else if (POP3->msgs[which_one-1].deleted) {
@@ -604,22 +443,17 @@ void pop3_uidl(char *argbuf) {
                        return;
                }
                else {
-                       cprintf("+OK %d %ld\r\n",
-                               which_one,
-                               POP3->msgs[which_one-1].msgnum
-                               );
+                       cprintf("+OK %d %ld\r\n", which_one, POP3->msgs[which_one-1].msgnum);
                        return;
                }
        }
 
-       /* "list all" (scan listing) mode */
+       // "list all" (scan listing) mode
        else {
                cprintf("+OK Here's your mail:\r\n");
                if (POP3->num_msgs > 0) for (i=0; i<POP3->num_msgs; ++i) {
                        if (! POP3->msgs[i].deleted) {
-                               cprintf("%d %ld\r\n",
-                                       i+1,
-                                       POP3->msgs[i].msgnum);
+                               cprintf("%d %ld\r\n", i+1, POP3->msgs[i].msgnum);
                        }
                }
                cprintf(".\r\n");
@@ -627,48 +461,35 @@ void pop3_uidl(char *argbuf) {
 }
 
 
-/*
- * implements the STLS command (Citadel API version)
- */
-void pop3_stls(void)
-{
+// implements the STLS command (Citadel API version)
+void pop3_stls(void) {
        char ok_response[SIZ];
        char nosup_response[SIZ];
        char error_response[SIZ];
 
-       sprintf(ok_response,
-               "+OK Begin TLS negotiation now\r\n");
-       sprintf(nosup_response,
-               "-ERR TLS not supported here\r\n");
-       sprintf(error_response,
-               "-ERR Internal error\r\n");
+       sprintf(ok_response,    "+OK Begin TLS negotiation now\r\n");
+       sprintf(nosup_response, "-ERR TLS not supported here\r\n");
+       sprintf(error_response, "-ERR Internal error\r\n");
        CtdlModuleStartCryptoMsgs(ok_response, nosup_response, error_response);
 }
 
 
-
-
-
-
-
-/* 
- * Main command loop for POP3 sessions.
- */
+// Main command loop for POP3 sessions.
 void pop3_command_loop(void) {
        char cmdbuf[SIZ];
 
        time(&CC->lastcmd);
-       memset(cmdbuf, 0, sizeof cmdbuf); /* Clear it, just in case */
+       memset(cmdbuf, 0, sizeof cmdbuf);                       // Clear it, just in case
        if (client_getln(cmdbuf, sizeof cmdbuf) < 1) {
-               CtdlLogPrintf(CTDL_ERR, "Client disconnected: ending session.\r\n");
-               CC->kill_me = 1;
+               syslog(LOG_INFO, "pop3: client disconnected; ending session.");
+               CC->kill_me = KILLME_CLIENT_DISCONNECTED;
                return;
        }
        if (!strncasecmp(cmdbuf, "PASS", 4)) {
-               CtdlLogPrintf(CTDL_INFO, "POP3: PASS...\r\n");
+               syslog(LOG_DEBUG, "pop3: PASS...");
        }
        else {
-               CtdlLogPrintf(CTDL_INFO, "POP3: %s\r\n", cmdbuf);
+               syslog(LOG_DEBUG, "pop3: %s", cmdbuf);
        }
        while (strlen(cmdbuf) < 5) strcat(cmdbuf, " ");
 
@@ -683,7 +504,7 @@ void pop3_command_loop(void) {
        else if (!strncasecmp(cmdbuf, "QUIT", 4)) {
                cprintf("+OK Goodbye...\r\n");
                pop3_update();
-               CC->kill_me = 1;
+               CC->kill_me = KILLME_CLIENT_LOGGED_OUT;
                return;
        }
 
@@ -695,11 +516,6 @@ void pop3_command_loop(void) {
                pop3_pass(&cmdbuf[5]);
        }
 
-       else if (!strncasecmp(cmdbuf, "APOP", 4))
-       {
-               pop3_apop(&cmdbuf[5]);
-       }
-
 #ifdef HAVE_OPENSSL
        else if (!strncasecmp(cmdbuf, "STLS", 4)) {
                pop3_stls();
@@ -709,6 +525,11 @@ void pop3_command_loop(void) {
        else if (!CC->logged_in) {
                cprintf("-ERR Not logged in.\r\n");
        }
+       
+       else if (CC->nologin) {
+               cprintf("-ERR System busy, try later.\r\n");
+               CC->kill_me = KILLME_NOLOGIN;
+       }
 
        else if (!strncasecmp(cmdbuf, "LIST", 4)) {
                pop3_list(&cmdbuf[5]);
@@ -756,23 +577,23 @@ CTDL_MODULE_INIT(pop3)
 {
        if(!threading)
        {
-               CtdlRegisterServiceHook(config.c_pop3_port,
+               CtdlRegisterServiceHook(CtdlGetConfigInt("c_pop3_port"),
                                        NULL,
                                        pop3_greeting,
                                        pop3_command_loop,
                                        NULL,
                                        CitadelServicePop3);
 #ifdef HAVE_OPENSSL
-               CtdlRegisterServiceHook(config.c_pop3s_port,
+               CtdlRegisterServiceHook(CtdlGetConfigInt("c_pop3s_port"),
                                        NULL,
                                        pop3s_greeting,
                                        pop3_command_loop,
                                        NULL,
                                        CitadelServicePop3S);
 #endif
-               CtdlRegisterSessionHook(pop3_cleanup_function, EVT_STOP);
+               CtdlRegisterSessionHook(pop3_cleanup_function, EVT_STOP, PRIO_STOP + 30);
        }
        
-       /* return our Subversion id for the Log */
-       return "$Id$";
+       /* return our module name for the log */
+       return "pop3";
 }