]> code.citadel.org Git - citadel.git/blobdiff - citadel/msgbase.c
* Reworked some of the data structures to handle multiple recipients
[citadel.git] / citadel / msgbase.c
index 5296502a522555db995d0ff46743691d4a7ec212..6615cffc0107183e26f0d7a80f53b98d9de16f8d 100644 (file)
@@ -8,9 +8,6 @@
 #include <ctype.h>
 #include <string.h>
 #include <syslog.h>
-#ifdef HAVE_PTHREAD_H
-#include <pthread.h>
-#endif
 #include <limits.h>
 #include "citadel.h"
 #include "server.h"
@@ -59,7 +56,7 @@ char *msgkeys[] = {
        "path",
        "",
        "rcpt",
-       "",
+       "spec",
        "time",
        "subj",
        "",
@@ -630,7 +627,6 @@ struct CtdlMessage *CtdlFetchMessage(long msgnum)
        CIT_UBYTE field_header;
        size_t field_length;
 
-
        dmsgtext = cdb_fetch(CDB_MSGMAIN, &msgnum, sizeof(long));
        if (dmsgtext == NULL) {
                return NULL;
@@ -674,6 +670,10 @@ struct CtdlMessage *CtdlFetchMessage(long msgnum)
 
        cdb_free(dmsgtext);
 
+       /* Always make sure there's something in the msg text field */
+       if (ret->cm_fields['M'] == NULL)
+               ret->cm_fields['M'] = strdoop("<no text>\n");
+
        /* Perform "before read" hooks (aborting if any return nonzero) */
        if (PerformMessageHooks(ret, EVT_BEFOREREAD) > 0) {
                CtdlFreeMessage(ret);
@@ -745,6 +745,7 @@ void output_message(char *msgid, int mode, int headers_only)
        /*                                       */
 
        msg_num = atol(msgid);
+       safestrncpy(mid, msgid, sizeof mid);
 
        if ((!(CC->logged_in)) && (!(CC->internal_pgm))) {
                cprintf("%d Not logged in.\n", ERROR + NOT_LOGGED_IN);
@@ -775,7 +776,7 @@ void output_message(char *msgid, int mode, int headers_only)
 
        /* Are we downloading a MIME component? */
        if (mode == MT_DOWNLOAD) {
-               if (TheMessage->cm_format_type != 4) {
+               if (TheMessage->cm_format_type != FMT_RFC822) {
                        cprintf("%d This is not a MIME message.\n",
                                ERROR);
                } else if (CC->download_fp != NULL) {
@@ -805,7 +806,7 @@ void output_message(char *msgid, int mode, int headers_only)
         * MIME message, *lie* about it and tell the user it's fixed-format.
         */
        if (mode == MT_CITADEL) {
-               if (TheMessage->cm_format_type == 4)
+               if (TheMessage->cm_format_type == FMT_RFC822)
                        cprintf("type=1\n");
                else
                        cprintf("type=%d\n", TheMessage->cm_format_type);
@@ -917,7 +918,7 @@ void output_message(char *msgid, int mode, int headers_only)
        mptr = TheMessage->cm_fields['M'];
 
        /* Tell the client about the MIME parts in this message */
-       if (TheMessage->cm_format_type == 4) {  /* legacy textual dump */
+       if (TheMessage->cm_format_type == FMT_RFC822) { /* legacy text dump */
                if (mode == MT_CITADEL) {
                        mime_parser(mptr, NULL, *list_this_part);
                }
@@ -938,14 +939,14 @@ void output_message(char *msgid, int mode, int headers_only)
        /* signify start of msg text */
        if (mode == MT_CITADEL)
                cprintf("text\n");
-       if ((mode == MT_RFC822) && (TheMessage->cm_format_type != 4))
+       if ((mode == MT_RFC822) && (TheMessage->cm_format_type != FMT_RFC822))
                cprintf("\n");
 
        /* If the format type on disk is 1 (fixed-format), then we want
         * everything to be output completely literally ... regardless of
         * what message transfer format is in use.
         */
-       if (TheMessage->cm_format_type == 1) {
+       if (TheMessage->cm_format_type == FMT_FIXED) {
                strcpy(buf, "");
                while (ch = *mptr++, ch > 0) {
                        if (ch == 13)
@@ -969,7 +970,7 @@ void output_message(char *msgid, int mode, int headers_only)
         * for new paragraphs is correct and the client will reformat the
         * message to the reader's screen width.
         */
-       if (TheMessage->cm_format_type == 0) {
+       if (TheMessage->cm_format_type == FMT_CITADEL) {
                memfmout(80, mptr, 0);
        }
 
@@ -978,7 +979,7 @@ void output_message(char *msgid, int mode, int headers_only)
         * this message is format 1 (fixed format), so the callback function
         * we use will display those parts as-is.
         */
-       if (TheMessage->cm_format_type == 4) {
+       if (TheMessage->cm_format_type == FMT_RFC822) {
                CtdlAllocUserData(SYM_MA_INFO, sizeof(struct ma_info));
                memset(ma, 0, sizeof(struct ma_info));
                mime_parser(mptr, NULL, *fixed_output);
@@ -1392,7 +1393,7 @@ int ReplicationChecks(struct CtdlMessage *msg) {
 /*
  * Save a message to disk
  */
-void CtdlSaveMsg(struct CtdlMessage *msg,      /* message to save */
+long CtdlSaveMsg(struct CtdlMessage *msg,      /* message to save */
                char *rec,                      /* Recipient (mail) */
                char *force,                    /* force a particular room? */
                int mailtype,                   /* local or remote type */
@@ -1413,12 +1414,13 @@ void CtdlSaveMsg(struct CtdlMessage *msg,       /* message to save */
        static int seqnum = 1;
 
        lprintf(9, "CtdlSaveMsg() called\n");
-       if (is_valid_message(msg) == 0) return;         /* self check */
+       if (is_valid_message(msg) == 0) return(-1);     /* self check */
 
        /* If this message has no timestamp, we take the liberty of
         * giving it one, right now.
         */
        if (msg->cm_fields['T'] == NULL) {
+               lprintf(9, "Generating timestamp\n");
                sprintf(aaa, "%ld", time(NULL));
                msg->cm_fields['T'] = strdoop(aaa);
        }
@@ -1426,12 +1428,18 @@ void CtdlSaveMsg(struct CtdlMessage *msg,       /* message to save */
        /* If this message has no path, we generate one.
         */
        if (msg->cm_fields['P'] == NULL) {
-               msg->cm_fields['P'] = strdoop(msg->cm_fields['A']);
-               for (a=0; a<strlen(msg->cm_fields['P']); ++a) {
-                       if (isspace(msg->cm_fields['P'][a])) {
-                               msg->cm_fields['P'][a] = ' ';
+               lprintf(9, "Generating path\n");
+               if (msg->cm_fields['A'] != NULL) {
+                       msg->cm_fields['P'] = strdoop(msg->cm_fields['A']);
+                       for (a=0; a<strlen(msg->cm_fields['P']); ++a) {
+                               if (isspace(msg->cm_fields['P'][a])) {
+                                       msg->cm_fields['P'][a] = ' ';
+                               }
                        }
                }
+               else {
+                       msg->cm_fields['P'] = strdoop("unknown");
+               }
        }
 
        strcpy(force_room, force);
@@ -1443,6 +1451,7 @@ void CtdlSaveMsg(struct CtdlMessage *msg, /* message to save */
                        strcpy(&recipient[a], &recipient[a + 1]);
 
        /* Learn about what's inside, because it's what's inside that counts */
+       lprintf(9, "Learning what's inside\n");
 
        switch (msg->cm_format_type) {
        case 0:
@@ -1474,10 +1483,12 @@ void CtdlSaveMsg(struct CtdlMessage *msg,       /* message to save */
        }
 
        /* Goto the correct room */
+       lprintf(9, "Switching rooms\n");
        strcpy(hold_rm, CC->quickroom.QRname);
        strcpy(actual_rm, CC->quickroom.QRname);
 
        /* If the user is a twit, move to the twit room for posting */
+       lprintf(9, "Handling twit stuff\n");
        if (TWITDETECT) {
                if (CC->usersupp.axlevel == 2) {
                        strcpy(hold_rm, actual_rm);
@@ -1490,17 +1501,21 @@ void CtdlSaveMsg(struct CtdlMessage *msg,       /* message to save */
                strcpy(actual_rm, force_room);
        }
 
+       lprintf(9, "Possibly relocating\n");
        if (strcasecmp(actual_rm, CC->quickroom.QRname))
                getroom(&CC->quickroom, actual_rm);
 
        /* Perform "before save" hooks (aborting if any return nonzero) */
-       if (PerformMessageHooks(msg, EVT_BEFORESAVE) > 0) return;
+       lprintf(9, "Performing before-save hooks\n");
+       if (PerformMessageHooks(msg, EVT_BEFORESAVE) > 0) return(-1);
 
        /* If this message has an Extended ID, perform replication checks */
-       if (ReplicationChecks(msg) > 0) return;
+       lprintf(9, "Performing replication checks\n");
+       if (ReplicationChecks(msg) > 0) return(-1);
 
        /* Network mail - send a copy to the network program. */
-       if ((strlen(recipient) > 0) && (mailtype != MES_LOCAL)) {
+       if ((strlen(recipient) > 0) && (mailtype == MES_BINARY)) {
+               lprintf(9, "Sending network spool\n");
                sprintf(aaa, "./network/spoolin/netmail.%04lx.%04x.%04x",
                        (long) getpid(), CC->cs_pid, ++seqnum);
                lprintf(9, "Saving a copy to %s\n", aaa);
@@ -1510,18 +1525,20 @@ void CtdlSaveMsg(struct CtdlMessage *msg,       /* message to save */
        }
 
        /* Save it to disk */
+       lprintf(9, "Saving to disk\n");
        newmsgid = send_message(msg, generate_id, network_fp);
        if (network_fp != NULL) {
                fclose(network_fp);
                system("exec nohup ./netproc -i >/dev/null 2>&1 &");
        }
 
-       if (newmsgid <= 0L) return;
+       if (newmsgid <= 0L) return(-1);
 
        /* Write a supplemental message info record.  This doesn't have to
         * be a critical section because nobody else knows about this message
         * yet.
         */
+       lprintf(9, "Creating SuppMsgInfo record\n");
        memset(&smi, 0, sizeof(struct SuppMsgInfo));
        smi.smi_msgnum = newmsgid;
        smi.smi_refcount = 0;
@@ -1529,6 +1546,7 @@ void CtdlSaveMsg(struct CtdlMessage *msg, /* message to save */
        PutSuppMsgInfo(&smi);
 
        /* Now figure out where to store the pointers */
+       lprintf(9, "Storing pointers\n");
 
 
        /* If this is being done by the networker delivering a private
@@ -1539,7 +1557,15 @@ void CtdlSaveMsg(struct CtdlMessage *msg,        /* message to save */
                CtdlSaveMsgPointerInRoom(actual_rm, newmsgid, 0);
        }
 
+       /* For internet mail, drop a copy in the outbound queue room */
+       /* FIX  ...  nothing's going to get delivered until we add
+          some delivery instructions!!! */
+       if (mailtype == MES_INTERNET) {
+               CtdlSaveMsgPointerInRoom(SMTP_SPOOLOUT_ROOM, newmsgid, 0);
+       }
+
        /* Bump this user's messages posted counter. */
+       lprintf(9, "Updating user\n");
        lgetuser(&CC->usersupp, CC->curr_user);
        CC->usersupp.posted = CC->usersupp.posted + 1;
        lputuser(&CC->usersupp);
@@ -1549,17 +1575,22 @@ void CtdlSaveMsg(struct CtdlMessage *msg,       /* message to save */
         */
        if ((strlen(recipient) > 0) && (mailtype == MES_LOCAL)) {
                if (getuser(&userbuf, recipient) == 0) {
+                       lprintf(9, "Delivering private mail\n");
                        MailboxName(actual_rm, &userbuf, MAILROOM);
                        CtdlSaveMsgPointerInRoom(actual_rm, newmsgid, 0);
                }
        }
 
        /* Perform "after save" hooks */
+       lprintf(9, "Performing after-save hooks\n");
        PerformMessageHooks(msg, EVT_AFTERSAVE);
 
        /* */
+       lprintf(9, "Returning to original room\n");
        if (strcasecmp(hold_rm, CC->quickroom.QRname))
                getroom(&CC->quickroom, hold_rm);
+
+       return(newmsgid);
 }
 
 
@@ -1589,6 +1620,74 @@ void quickie_message(char *from, char *to, char *room, char *text)
 }
 
 
+
+/*
+ * Back end function used by make_message() and similar functions
+ */
+char *CtdlReadMessageBody(char *terminator,    /* token signalling EOT */
+                       size_t maxlen,          /* maximum message length */
+                       char *exist             /* if non-null, append to it;
+                                                  exist is ALWAYS freed  */
+                       ) {
+       char buf[256];
+       size_t message_len = 0;
+       size_t buffer_len = 0;
+       char *ptr, *append;
+       char *m;
+
+       if (exist == NULL) {
+               m = mallok(4096);
+       }
+       else {
+               m = reallok(exist, strlen(exist) + 4096);
+               if (m == NULL) phree(exist);
+       }
+       if (m == NULL) {
+               while ( (client_gets(buf)>0) && strcmp(buf, terminator) ) ;;
+               return(NULL);
+       } else {
+               buffer_len = 4096;
+               m[0] = 0;
+               message_len = 0;
+       }
+       /* read in the lines of message text one by one */
+       append = NULL;
+       while ( (client_gets(buf)>0) && strcmp(buf, terminator) ) {
+
+               /* augment the buffer if we have to */
+               if ((message_len + strlen(buf) + 2) > buffer_len) {
+                       lprintf(9, "realloking\n");
+                       ptr = reallok(m, (buffer_len * 2) );
+                       if (ptr == NULL) {      /* flush if can't allocate */
+                               while ( (client_gets(buf)>0) &&
+                                       strcmp(buf, terminator)) ;;
+                               return(m);
+                       } else {
+                               buffer_len = (buffer_len * 2);
+                               m = ptr;
+                               append = NULL;
+                               lprintf(9, "buffer_len is %d\n", buffer_len);
+                       }
+               }
+
+               if (append == NULL) append = m;
+               while (strlen(append) > 0) ++append;
+               strcpy(append, buf);
+               strcat(append, "\n");
+               message_len = message_len + strlen(buf) + 1;
+
+               /* if we've hit the max msg length, flush the rest */
+               if (message_len >= maxlen) {
+                       while ( (client_gets(buf)>0) && strcmp(buf, terminator)) ;;
+                       return(m);
+               }
+       }
+       return(m);
+}
+
+
+
+
 /*
  * Build a binary message to be saved on disk.
  */
@@ -1606,9 +1705,6 @@ struct CtdlMessage *make_message(
        int a;
        char dest_node[32];
        char buf[256];
-       size_t message_len = 0;
-       size_t buffer_len = 0;
-       char *ptr;
        struct CtdlMessage *msg;
 
        msg = mallok(sizeof(struct CtdlMessage));
@@ -1663,40 +1759,10 @@ struct CtdlMessage *make_message(
        if (dest_node[0] != 0)
                msg->cm_fields['D'] = strdoop(dest_node);
 
-       msg->cm_fields['M'] = mallok(4096);
-       if (msg->cm_fields['M'] == NULL) {
-               while (client_gets(buf), strcmp(buf, "000")) ;; /* flush */
-               return(msg);
-       } else {
-               buffer_len = 4096;
-               msg->cm_fields['M'][0] = 0;
-               message_len = 0;
-       }
 
-       /* read in the lines of message text one by one */
-       while (client_gets(buf), strcmp(buf, "000")) {
+       msg->cm_fields['M'] = CtdlReadMessageBody("000",
+                                               config.c_maxmsglen, NULL);
 
-               /* augment the buffer if we have to */
-               if ((message_len + strlen(buf) + 2) > buffer_len) {
-                       ptr = reallok(msg->cm_fields['M'], (buffer_len * 2) );
-                       if (ptr == NULL) {      /* flush if can't allocate */
-                               while (client_gets(buf), strcmp(buf, "000")) ;;
-                               return(msg);
-                       } else {
-                               buffer_len = (buffer_len * 2);
-                               msg->cm_fields['M'] = ptr;
-                       }
-               }
-
-               strcat(msg->cm_fields['M'], buf);
-               strcat(msg->cm_fields['M'], "\n");
-
-               /* if we've hit the max msg length, flush the rest */
-               if (message_len >= config.c_maxmsglen) {
-                       while (client_gets(buf), strcmp(buf, "000")) ;;
-                       return(msg);
-               }
-       }
 
        return(msg);
 }
@@ -1862,6 +1928,7 @@ void cmd_ent3(char *entargs)
        char recp[256];
        int a;
        int e = 0;
+       int valid_msg = 1;
        unsigned char ch, which_field;
        struct usersupp tempUS;
        long msglen;
@@ -1935,6 +2002,7 @@ void cmd_ent3(char *entargs)
 
        while (msglen > 0) {
                client_read(&which_field, 1);
+               if (!isalpha(which_field)) valid_msg = 0;
                --msglen;
                tempbuf[0] = 0;
                do {
@@ -1944,11 +2012,12 @@ void cmd_ent3(char *entargs)
                        tempbuf[a+1] = 0;
                        tempbuf[a] = ch;
                } while ( (ch != 0) && (msglen > 0) );
-               msg->cm_fields[which_field] = strdoop(tempbuf);
+               if (valid_msg)
+                       msg->cm_fields[which_field] = strdoop(tempbuf);
        }
 
        msg->cm_flags = CM_SKIP_HOOKS;
-       CtdlSaveMsg(msg, recp, "", e, 0);
+       if (valid_msg) CtdlSaveMsg(msg, recp, "", e, 0);
        CtdlFreeMessage(msg);
        phree(tempbuf);
 }
@@ -2067,13 +2136,14 @@ void cmd_dele(char *delstr)
 void cmd_move(char *args)
 {
        long num;
-       char targ[32];
+       char targ[256];
        struct quickroom qtemp;
        int err;
        int is_copy = 0;
 
        num = extract_long(args, 0);
        extract(targ, args, 1);
+       targ[ROOMNAMELEN - 1] = 0;
        is_copy = extract_int(args, 2);
 
        getuser(&CC->usersupp, CC->curr_user);