* $Id$
*
* Implements the FETCH command in IMAP.
- * This command is way too convoluted. Marc Crispin is a fscking idiot.
+ * This is a good example of the protocol's gratuitous complexity.
*
*/
-struct imap_fetch_part {
- char desired_section[SIZ];
- FILE *output_fp;
-};
-
/*
* Individual field functions for imap_do_fetch_msg() ...
*/
-
-
void imap_fetch_uid(int seq) {
cprintf("UID %ld", IMAP->msgids[seq-1]);
}
cprintf("\\Answered");
++num_flags_printed;
}
+ if (IMAP->flags[seq] & IMAP_RECENT) {
+ if (num_flags_printed > 0) cprintf(" ");
+ cprintf("\\Recent");
+ ++num_flags_printed;
+ }
cprintf(")");
}
* "RFC822.SIZE" size of translated message
* "RFC822.TEXT" body only (without leading blank line)
*/
-void imap_fetch_rfc822(int msgnum, char *whichfmt, struct CtdlMessage *msg) {
- char buf[1024];
+void imap_fetch_rfc822(long msgnum, char *whichfmt) {
+ char buf[SIZ];
char *ptr;
- long headers_size, text_size, total_size;
- long bytes_remaining = 0;
- long blocksize;
- FILE *tmp;
-
- tmp = tmpfile();
- if (tmp == NULL) {
- lprintf(1, "Cannot open temp file: %s\n",
- strerror(errno));
- return;
+ size_t headers_size, text_size, total_size;
+ size_t bytes_to_send = 0;
+ struct MetaData smi;
+ int need_to_rewrite_metadata = 0;
+
+ /* If this is an RFC822.SIZE fetch, first look in the message's
+ * metadata record to see if we've saved that information.
+ */
+ if (!strcasecmp(whichfmt, "RFC822.SIZE")) {
+ GetMetaData(&smi, msgnum);
+ if (smi.meta_rfc822_length > 0L) {
+ cprintf("RFC822.SIZE %ld", smi.meta_rfc822_length);
+ return;
+ }
+ need_to_rewrite_metadata = 1;
+ }
+
+ /* Cache the most recent RFC822 FETCH because some clients like to
+ * fetch in pieces, and we don't want to have to go back to the
+ * message store for each piece.
+ */
+ if ((IMAP->cached_rfc822_data != NULL)
+ && (IMAP->cached_rfc822_msgnum == msgnum)) {
+ /* Good to go! */
+ }
+ else if (IMAP->cached_rfc822_data != NULL) {
+ /* Some other message is cached -- free it */
+ free(IMAP->cached_rfc822_data);
+ IMAP->cached_rfc822_data = NULL;
+ IMAP->cached_rfc822_msgnum = (-1);
+ IMAP->cached_rfc822_len = 0;
}
- /*
- * Load the message into a temp file for translation
- * and measurement
+ /* At this point, we now can fetch and convert the message iff it's not
+ * the one we had cached.
*/
- CtdlRedirectOutput(tmp, -1);
- CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822,
- HEADERS_ALL, 0, 1);
- CtdlRedirectOutput(NULL, -1);
- if (!is_valid_message(msg)) {
- lprintf(1, "WARNING: output clobbered the message!\n");
+ if (IMAP->cached_rfc822_data == NULL) {
+ /*
+ * Load the message into memory for translation & measurement
+ */
+ CC->redirect_buffer = malloc(SIZ);
+ CC->redirect_len = 0;
+ CC->redirect_alloc = SIZ;
+ CtdlOutputMsg(msgnum, MT_RFC822, HEADERS_ALL, 0, 1);
+ IMAP->cached_rfc822_data = CC->redirect_buffer;
+ IMAP->cached_rfc822_len = CC->redirect_len;
+ IMAP->cached_rfc822_msgnum = msgnum;
+ CC->redirect_buffer = NULL;
+ CC->redirect_len = 0;
+ CC->redirect_alloc = 0;
+ if (need_to_rewrite_metadata) {
+ smi.meta_rfc822_length = (long)IMAP->cached_rfc822_len;
+ PutMetaData(&smi);
+ }
}
/*
* Now figure out where the headers/text break is. IMAP considers the
* intervening blank line to be part of the headers, not the text.
*/
- rewind(tmp);
- headers_size = 0L;
+ headers_size = 0;
+ text_size = 0;
+ total_size = 0;
+
+ ptr = IMAP->cached_rfc822_data;
do {
- ptr = fgets(buf, sizeof buf, tmp);
- if (ptr != NULL) {
+ ptr = memreadline(ptr, buf, sizeof buf);
+ if (*ptr != 0) {
striplt(buf);
if (strlen(buf) == 0) {
- headers_size = ftell(tmp);
+ headers_size = ptr - IMAP->cached_rfc822_data;
}
}
- } while ( (headers_size == 0L) && (ptr != NULL) );
- fseek(tmp, 0L, SEEK_END);
- total_size = ftell(tmp);
+ } while ( (headers_size == 0) && (*ptr != 0) );
+
+ total_size = IMAP->cached_rfc822_len;
text_size = total_size - headers_size;
+ lprintf(CTDL_DEBUG, "RFC822: headers=%d, text=%d, total=%d\n",
+ headers_size, text_size, total_size);
+
if (!strcasecmp(whichfmt, "RFC822.SIZE")) {
- cprintf("RFC822.SIZE %ld", total_size);
- fclose(tmp);
+ cprintf("RFC822.SIZE %d", total_size);
return;
}
else if (!strcasecmp(whichfmt, "RFC822")) {
- bytes_remaining = total_size;
- rewind(tmp);
+ ptr = IMAP->cached_rfc822_data;
+ bytes_to_send = total_size;
}
else if (!strcasecmp(whichfmt, "RFC822.HEADER")) {
- bytes_remaining = headers_size;
- rewind(tmp);
+ ptr = IMAP->cached_rfc822_data;
+ bytes_to_send = headers_size;
}
else if (!strcasecmp(whichfmt, "RFC822.TEXT")) {
- bytes_remaining = text_size;
- fseek(tmp, headers_size, SEEK_SET);
+ ptr = &IMAP->cached_rfc822_data[headers_size];
+ bytes_to_send = text_size;
}
- cprintf("%s {%ld}\r\n", whichfmt, bytes_remaining);
- blocksize = sizeof(buf);
- while (bytes_remaining > 0L) {
- if (blocksize > bytes_remaining) blocksize = bytes_remaining;
- fread(buf, blocksize, 1, tmp);
- client_write(buf, blocksize);
- bytes_remaining = bytes_remaining - blocksize;
- }
-
- fclose(tmp);
+ cprintf("%s {%d}\r\n", whichfmt, bytes_to_send);
+ client_write(ptr, bytes_to_send);
}
void *content, char *cbtype, size_t length, char *encoding,
void *cbuserdata)
{
- struct imap_fetch_part *imfp;
- char mbuf2[1024];
+ char mbuf2[SIZ];
+ char *desired_section;
- imfp = (struct imap_fetch_part *)cbuserdata;
+ desired_section = (char *)cbuserdata;
- if (!strcasecmp(partnum, imfp->desired_section)) {
- fwrite(content, length, 1, imfp->output_fp);
+ if (!strcasecmp(partnum, desired_section)) {
+ client_write(content, length);
}
snprintf(mbuf2, sizeof mbuf2, "%s.MIME", partnum);
- if (!strcasecmp(imfp->desired_section, mbuf2)) {
- fprintf(imfp->output_fp, "Content-type: %s", cbtype);
+ if (!strcasecmp(desired_section, mbuf2)) {
+ cprintf("Content-type: %s", cbtype);
if (strlen(name) > 0)
- fprintf(imfp->output_fp, "; name=\"%s\"", name);
- fprintf(imfp->output_fp, "\r\n");
+ cprintf("; name=\"%s\"", name);
+ cprintf("\r\n");
if (strlen(encoding) > 0)
- fprintf(imfp->output_fp,
- "Content-Transfer-Encoding: %s\r\n", encoding);
+ cprintf("Content-Transfer-Encoding: %s\r\n", encoding);
if (strlen(encoding) > 0) {
- fprintf(imfp->output_fp, "Content-Disposition: %s",
- disp);
+ cprintf("Content-Disposition: %s", disp);
if (strlen(filename) > 0) {
- fprintf(imfp->output_fp, "; filename=\"%s\"",
- filename);
+ cprintf("; filename=\"%s\"", filename);
}
- fprintf(imfp->output_fp, "\r\n");
+ cprintf("\r\n");
}
- fprintf(imfp->output_fp, "Content-Length: %ld\r\n", (long)length);
- fprintf(imfp->output_fp, "\r\n");
+ cprintf("Content-Length: %ld\r\n", (long)length);
+ cprintf("\r\n");
}
* really need to make this suck less.
*/
void imap_output_envelope_from(struct CtdlMessage *msg) {
- char user[1024], node[1024], name[1024];
+ char user[SIZ], node[SIZ], name[SIZ];
+ /* For anonymous messages, it's so easy! */
+ if (!is_room_aide() && (msg->cm_anon_type == MES_ANONONLY)) {
+ cprintf("((\"----\" NIL \"x\" \"x.org\")) ");
+ return;
+ }
+ if (!is_room_aide() && (msg->cm_anon_type == MES_ANONOPT)) {
+ cprintf("((\"anonymous\" NIL \"x\" \"x.org\")) ");
+ return;
+ }
+
+ /* For everything else, we do stuff. */
cprintf("(("); /* open double-parens */
imap_strout(msg->cm_fields['A']); /* personal name */
cprintf(" NIL "); /* source route (not used) */
+
if (msg->cm_fields['F'] != NULL) {
process_rfc822_addr(msg->cm_fields['F'], user, node, name);
imap_strout(user); /* mailbox name (user id) */
}
+
+/*
+ * Output an envelope address (or set of addresses) in the official,
+ * convuluted, braindead format. (Note that we can't use this for
+ * the "From" address because its data may come from a number of different
+ * fields. But we can use it for "To" and possibly others.
+ */
+void imap_output_envelope_addr(char *addr) {
+ char individual_addr[256];
+ int num_addrs;
+ int i;
+ char user[256];
+ char node[256];
+ char name[256];
+
+ if (addr == NULL) {
+ cprintf("NIL ");
+ return;
+ }
+
+ if (strlen(addr) == 0) {
+ cprintf("NIL ");
+ return;
+ }
+
+ cprintf("(");
+
+ /* How many addresses are listed here? */
+ num_addrs = num_tokens(addr, ',');
+
+ /* Output them one by one. */
+ for (i=0; i<num_addrs; ++i) {
+ extract_token(individual_addr, addr, i, ',', sizeof individual_addr);
+ striplt(individual_addr);
+ process_rfc822_addr(individual_addr, user, node, name);
+ cprintf("(");
+ imap_strout(name);
+ cprintf(" NIL ");
+ imap_strout(user);
+ cprintf(" ");
+ imap_strout(node);
+ cprintf(")");
+ if (i < (num_addrs-1)) cprintf(" ");
+ }
+
+ cprintf(") ");
+}
+
+
/*
* Implements the ENVELOPE fetch item
*
- * FIXME ... we only output some of the fields right now. Definitely need
- * to do all of them. Accurately, too.
- *
* Note that the imap_strout() function can cleverly output NULL fields as NIL,
* so we don't have to check for that condition like we do elsewhere.
*/
/* From */
imap_output_envelope_from(msg);
- /* Sender */
- if (0) {
- /* FIXME */
+ /* Sender (default to same as 'From' if not present) */
+ fieldptr = rfc822_fetch_field(msg->cm_fields['M'], "Sender");
+ if (fieldptr != NULL) {
+ imap_output_envelope_addr(fieldptr);
+ free(fieldptr);
}
else {
imap_output_envelope_from(msg);
}
/* Reply-to */
- if (0) {
- /* FIXME */
+ fieldptr = rfc822_fetch_field(msg->cm_fields['M'], "Reply-to");
+ if (fieldptr != NULL) {
+ imap_output_envelope_addr(fieldptr);
+ free(fieldptr);
}
else {
imap_output_envelope_from(msg);
}
- cprintf("NIL "); /* to */
+ /* To */
+ imap_output_envelope_addr(msg->cm_fields['R']);
- cprintf("NIL "); /* cc */
+ /* Cc */
+ fieldptr = rfc822_fetch_field(msg->cm_fields['M'], "Cc");
+ imap_output_envelope_addr(fieldptr);
+ if (fieldptr != NULL) free(fieldptr);
- cprintf("NIL "); /* bcc */
+ /* Bcc */
+ fieldptr = rfc822_fetch_field(msg->cm_fields['M'], "Bcc");
+ imap_output_envelope_addr(fieldptr);
+ if (fieldptr != NULL) free(fieldptr);
/* In-reply-to */
fieldptr = rfc822_fetch_field(msg->cm_fields['M'], "In-reply-to");
imap_strout(fieldptr);
cprintf(" ");
- if (fieldptr != NULL) phree(fieldptr);
+ if (fieldptr != NULL) free(fieldptr);
/* message ID */
imap_strout(msg->cm_fields['I']);
/*
- * Strip any non header information out of a chunk of RFC822 data on disk,
- * then boil it down to just the fields we want.
+ * This function is called only when CC->redirect_buffer contains a set of
+ * RFC822 headers with no body attached. Its job is to strip that set of
+ * headers down to *only* the ones we're interested in.
*/
-void imap_strip_headers(FILE *fp, char *section) {
- char buf[1024];
+void imap_strip_headers(char *section) {
+ char buf[SIZ];
char *which_fields = NULL;
int doing_headers = 0;
int headers_not = 0;
char *boiled_headers = NULL;
int ok = 0;
int done_headers = 0;
+ char *ptr = NULL;
- which_fields = strdoop(section);
+ if (CC->redirect_buffer == NULL) return;
+
+ which_fields = strdup(section);
if (!strncasecmp(which_fields, "HEADER.FIELDS", 13))
doing_headers = 1;
}
num_parms = imap_parameterize(parms, which_fields);
- fseek(fp, 0L, SEEK_END);
- boiled_headers = mallok((size_t)(ftell(fp) + 256L));
+ boiled_headers = malloc(CC->redirect_alloc);
strcpy(boiled_headers, "");
- rewind(fp);
+ ptr = CC->redirect_buffer;
ok = 0;
- while ( (done_headers == 0) && (fgets(buf, sizeof buf, fp) != NULL) ) {
+ while ( (done_headers == 0) && (ptr = memreadline(ptr, buf, sizeof buf), *ptr != 0) ) {
if (!isspace(buf[0])) {
ok = 0;
if (doing_headers == 0) ok = 1;
if (ok) {
strcat(boiled_headers, buf);
+ strcat(boiled_headers, "\r\n");
}
if (strlen(buf) == 0) done_headers = 1;
strcat(boiled_headers, "\r\n");
- /* Now write it back */
- rewind(fp);
- fwrite(boiled_headers, strlen(boiled_headers), 1, fp);
- fflush(fp);
- ftruncate(fileno(fp), ftell(fp));
- fflush(fp);
- rewind(fp);
- phree(which_fields);
- phree(boiled_headers);
+ /* Now save it back (it'll always be smaller) */
+ strcpy(CC->redirect_buffer, boiled_headers);
+ CC->redirect_len = strlen(boiled_headers);
+
+ free(which_fields);
+ free(boiled_headers);
}
/*
* Implements the BODY and BODY.PEEK fetch items
*/
-void imap_fetch_body(long msgnum, char *item, int is_peek,
- struct CtdlMessage *msg) {
- char section[1024];
- char partial[1024];
+void imap_fetch_body(long msgnum, char *item, int is_peek) {
+ struct CtdlMessage *msg = NULL;
+ char section[SIZ];
+ char partial[SIZ];
int is_partial = 0;
- char buf[1024];
- int i;
- FILE *tmp;
- long bytes_remaining = 0;
- long blocksize;
- long pstart, pbytes;
- struct imap_fetch_part imfp;
+ size_t pstart, pbytes;
+ int loading_body_now = 0;
/* extract section */
- strcpy(section, item);
- for (i=0; i<strlen(section); ++i) {
- if (section[i]=='[') strcpy(section, §ion[i+1]);
- }
- for (i=0; i<strlen(section); ++i) {
- if (section[i]==']') section[i] = 0;
+ safestrncpy(section, item, sizeof section);
+ if (strchr(section, '[') != NULL) {
+ stripallbut(section, '[', ']');
}
- lprintf(9, "Section is %s\n", section);
+ /* lprintf(CTDL_DEBUG, "Section is: %s%s\n", section, ((strlen(section)==0) ? "(empty)" : "") ); */
- /* extract partial */
- strcpy(partial, item);
- for (i=0; i<strlen(partial); ++i) {
- if (partial[i]=='<') {
- strcpy(partial, &partial[i+1]);
- is_partial = 1;
+ /* Burn the cache if we don't have the same section of the
+ * same message again.
+ */
+ if (IMAP->cached_body != NULL) {
+ if ((IMAP->cached_bodymsgnum != msgnum)
+ || (strcasecmp(IMAP->cached_bodypart, section)) ) {
+ free(IMAP->cached_body);
+ IMAP->cached_body_len = 0;
+ IMAP->cached_body = NULL;
+ IMAP->cached_bodymsgnum = (-1);
+ strcpy(IMAP->cached_bodypart, "");
}
}
- for (i=0; i<strlen(partial); ++i) {
- if (partial[i]=='>') partial[i] = 0;
+
+ /* extract partial */
+ safestrncpy(partial, item, sizeof partial);
+ if (strchr(partial, '<') != NULL) {
+ stripallbut(partial, '<', '>');
+ is_partial = 1;
}
if (is_partial == 0) strcpy(partial, "");
- if (strlen(partial) > 0) lprintf(9, "Partial is %s\n", partial);
+ /* if (strlen(partial) > 0) lprintf(CTDL_DEBUG, "Partial is %s\n", partial); */
- tmp = tmpfile();
- if (tmp == NULL) {
- lprintf(1, "Cannot open temp file: %s\n", strerror(errno));
- return;
+ if (IMAP->cached_body == NULL) {
+ CC->redirect_buffer = malloc(SIZ);
+ CC->redirect_len = 0;
+ CC->redirect_alloc = SIZ;
+ loading_body_now = 1;
+ msg = CtdlFetchMessage(msgnum, 1);
}
/* Now figure out what the client wants, and get it */
- if ( (!strcmp(section, "1")) && (msg->cm_format_type != 4) ) {
- CtdlRedirectOutput(tmp, -1);
- CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822,
- HEADERS_NONE, 0, 1);
- CtdlRedirectOutput(NULL, -1);
+ if (!loading_body_now) {
+ /* What we want is already in memory */
+ }
+
+ else if ( (!strcmp(section, "1")) && (msg->cm_format_type != 4) ) {
+ CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822, HEADERS_NONE, 0, 1);
}
else if (!strcmp(section, "")) {
- CtdlRedirectOutput(tmp, -1);
- CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822,
- HEADERS_ALL, 0, 1);
- CtdlRedirectOutput(NULL, -1);
+ CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822, HEADERS_ALL, 0, 1);
}
/*
* fields, strip it down.
*/
else if (!strncasecmp(section, "HEADER", 6)) {
- CtdlRedirectOutput(tmp, -1);
- CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822,
- HEADERS_ONLY, 0, 1);
- CtdlRedirectOutput(NULL, -1);
- imap_strip_headers(tmp, section);
+ CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822, HEADERS_ONLY, 0, 1);
+ imap_strip_headers(section);
}
/*
* Strip it down if the client asked for everything _except_ headers.
*/
else if (!strncasecmp(section, "TEXT", 4)) {
- CtdlRedirectOutput(tmp, -1);
- CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822,
- HEADERS_NONE, 0, 1);
- CtdlRedirectOutput(NULL, -1);
+ CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822, HEADERS_NONE, 0, 1);
}
/*
* (Note value of 1 passed as 'dont_decode' so client gets it encoded)
*/
else {
- safestrncpy(imfp.desired_section, section,
- sizeof(imfp.desired_section));
- imfp.output_fp = tmp;
-
mime_parser(msg->cm_fields['M'], NULL,
*imap_load_part, NULL, NULL,
- (void *)&imfp,
+ section,
1);
}
-
- fseek(tmp, 0L, SEEK_END);
- bytes_remaining = ftell(tmp);
+ if (loading_body_now) {
+ IMAP->cached_body = CC->redirect_buffer;
+ IMAP->cached_body_len = CC->redirect_len;
+ IMAP->cached_bodymsgnum = msgnum;
+ strcpy(IMAP->cached_bodypart, section);
+ CC->redirect_buffer = NULL;
+ CC->redirect_len = 0;
+ CC->redirect_alloc = 0;
+ }
if (is_partial == 0) {
- rewind(tmp);
- cprintf("BODY[%s] {%ld}\r\n", section, bytes_remaining);
+ cprintf("BODY[%s] {%d}\r\n", section, IMAP->cached_body_len);
+ pstart = 0;
+ pbytes = IMAP->cached_body_len;
}
else {
- sscanf(partial, "%ld.%ld", &pstart, &pbytes);
- if ((bytes_remaining - pstart) < pbytes) {
- pbytes = bytes_remaining - pstart;
+ sscanf(partial, "%d.%d", &pstart, &pbytes);
+ if (pbytes > (IMAP->cached_body_len - pstart)) {
+ pbytes = IMAP->cached_body_len - pstart;
}
- fseek(tmp, pstart, SEEK_SET);
- bytes_remaining = pbytes;
- cprintf("BODY[%s]<%ld> {%ld}\r\n",
- section, pstart, bytes_remaining);
+ cprintf("BODY[%s]<%d> {%d}\r\n", section, pstart, pbytes);
}
- blocksize = sizeof(buf);
- while (bytes_remaining > 0L) {
- if (blocksize > bytes_remaining) blocksize = bytes_remaining;
- fread(buf, blocksize, 1, tmp);
- client_write(buf, blocksize);
- bytes_remaining = bytes_remaining - blocksize;
- }
+ /* Here we go -- output it */
+ client_write(&IMAP->cached_body[pstart], pbytes);
- fclose(tmp);
+ if (msg != NULL) {
+ CtdlFreeMessage(msg);
+ }
/* Mark this message as "seen" *unless* this is a "peek" operation */
if (is_peek == 0) {
void *cbuserdata
) {
- char subtype[SIZ];
+ char subtype[128];
cprintf(" ");
/* disposition */
- extract_token(subtype, cbtype, 1, '/');
+ extract_token(subtype, cbtype, 1, '/', sizeof subtype);
imap_strout(subtype);
/* body language */
int have_encoding = 0;
int lines = 0;
size_t i;
- char cbmaintype[SIZ];
- char cbsubtype[SIZ];
+ char cbmaintype[128];
+ char cbsubtype[128];
if (cbtype != NULL) if (strlen(cbtype)>0) have_cbtype = 1;
if (have_cbtype) {
- extract_token(cbmaintype, cbtype, 0, '/');
- extract_token(cbsubtype, cbtype, 1, '/');
+ extract_token(cbmaintype, cbtype, 0, '/', sizeof cbmaintype);
+ extract_token(cbsubtype, cbtype, 1, '/', sizeof cbsubtype);
}
else {
strcpy(cbmaintype, "TEXT");
cprintf("%ld ", (long)length); /* bytes */
/* The next field is the number of lines in the part, if and only
- * if the part is TEXT. Crispin is a fscking idiot.
+ * if the part is TEXT. More gratuitous complexity.
*/
if (!strcasecmp(cbmaintype, "TEXT")) {
if (length) for (i=0; i<length; ++i) {
cprintf("%d ", lines);
}
- /* More of Crispin being a fscking idiot */
+ /* More gratuitous complexity */
if ((!strcasecmp(cbmaintype, "MESSAGE"))
&& (!strcasecmp(cbsubtype, "RFC822"))) {
/* FIXME
/*
- * Spew the BODYSTRUCTURE data for a message. (Do you need a silencer if
- * you're going to shoot a MIME? Do you need a reason to shoot Mark Crispin?
- * No, and no.)
+ * Spew the BODYSTRUCTURE data for a message.
*
*/
void imap_fetch_bodystructure (long msgnum, char *item,
struct CtdlMessage *msg) {
- FILE *tmp;
- char buf[1024];
- long lines = 0L;
- long start_of_body = 0L;
- long body_bytes = 0L;
+ char *rfc822 = NULL;
+ char *rfc822_body = NULL;
+ size_t rfc822_len;
+ size_t rfc822_headers_len;
+ size_t rfc822_body_len;
+ char *ptr = NULL;
+ char buf[SIZ];
+ int lines = 0;
/* For non-RFC822 (ordinary Citadel) messages, this is short and
* sweet...
if (msg->cm_format_type != FMT_RFC822) {
/* *sigh* We have to RFC822-format the message just to be able
- * to measure it.
+ * to measure it. FIXME use smi cached fields if possible
*/
- tmp = tmpfile();
- if (tmp == NULL) return;
- CtdlRedirectOutput(tmp, -1);
- CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822, 0, 0, 1);
- CtdlRedirectOutput(NULL, -1);
- rewind(tmp);
- while (fgets(buf, sizeof buf, tmp) != NULL) {
+ CC->redirect_buffer = malloc(SIZ);
+ CC->redirect_len = 0;
+ CC->redirect_alloc = SIZ;
+ CtdlOutputPreLoadedMsg(msg, msgnum, MT_RFC822, 0, 0, 1);
+ rfc822 = CC->redirect_buffer;
+ rfc822_len = CC->redirect_len;
+ CC->redirect_buffer = NULL;
+ CC->redirect_len = 0;
+ CC->redirect_alloc = 0;
+
+ ptr = rfc822;
+ while (ptr = memreadline(ptr, buf, sizeof buf), *ptr != 0) {
++lines;
- if ((!strcmp(buf, "\r\n")) && (start_of_body == 0L)) {
- start_of_body = ftell(tmp);
+ if ((strlen(buf) == 0) && (rfc822_body == NULL)) {
+ rfc822_body = ptr;
}
}
- body_bytes = ftell(tmp) - start_of_body;
- fclose(tmp);
+
+ rfc822_headers_len = rfc822_body - rfc822;
+ rfc822_body_len = rfc822_len - rfc822_headers_len;
+ free(rfc822);
cprintf("BODYSTRUCTURE (\"TEXT\" \"PLAIN\" "
"(\"CHARSET\" \"US-ASCII\") NIL NIL "
- "\"7BIT\" %ld %ld)", body_bytes, lines);
+ "\"7BIT\" %d %d)", rfc822_body_len, lines);
return;
}
}
-
-
-
-
/*
* imap_do_fetch() calls imap_do_fetch_msg() to output the data of an
- * individual message, once it has been successfully loaded from disk.
+ * individual message, once it has been selected for output.
*/
-void imap_do_fetch_msg(int seq, struct CtdlMessage *msg,
- int num_items, char **itemlist) {
+void imap_do_fetch_msg(int seq, int num_items, char **itemlist) {
int i;
+ struct CtdlMessage *msg = NULL;
cprintf("* %d FETCH (", seq);
for (i=0; i<num_items; ++i) {
- if (!strncasecmp(itemlist[i], "BODY[", 5)) {
- imap_fetch_body(IMAP->msgids[seq-1], itemlist[i],
- 0, msg);
- }
- else if (!strncasecmp(itemlist[i], "BODY.PEEK[", 10)) {
- imap_fetch_body(IMAP->msgids[seq-1], itemlist[i],
- 1, msg);
- }
- else if (!strcasecmp(itemlist[i], "BODYSTRUCTURE")) {
- imap_fetch_bodystructure(IMAP->msgids[seq-1],
- itemlist[i], msg);
- }
- else if (!strcasecmp(itemlist[i], "ENVELOPE")) {
- imap_fetch_envelope(IMAP->msgids[seq-1], msg);
+ /* Fetchable without going to the message store at all */
+ if (!strcasecmp(itemlist[i], "UID")) {
+ imap_fetch_uid(seq);
}
else if (!strcasecmp(itemlist[i], "FLAGS")) {
imap_fetch_flags(seq-1);
}
- else if (!strcasecmp(itemlist[i], "INTERNALDATE")) {
- imap_fetch_internaldate(msg);
- }
+
+ /* Potentially fetchable from cache, if the client requests
+ * stuff from the same message several times in a row.
+ */
else if (!strcasecmp(itemlist[i], "RFC822")) {
- imap_fetch_rfc822(IMAP->msgids[seq-1], itemlist[i], msg);
+ imap_fetch_rfc822(IMAP->msgids[seq-1], itemlist[i]);
}
else if (!strcasecmp(itemlist[i], "RFC822.HEADER")) {
- imap_fetch_rfc822(IMAP->msgids[seq-1], itemlist[i], msg);
+ imap_fetch_rfc822(IMAP->msgids[seq-1], itemlist[i]);
}
else if (!strcasecmp(itemlist[i], "RFC822.SIZE")) {
- imap_fetch_rfc822(IMAP->msgids[seq-1], itemlist[i], msg);
+ imap_fetch_rfc822(IMAP->msgids[seq-1], itemlist[i]);
}
else if (!strcasecmp(itemlist[i], "RFC822.TEXT")) {
- imap_fetch_rfc822(IMAP->msgids[seq-1], itemlist[i], msg);
+ imap_fetch_rfc822(IMAP->msgids[seq-1], itemlist[i]);
}
- else if (!strcasecmp(itemlist[i], "UID")) {
- imap_fetch_uid(seq);
+
+ /* BODY fetches do their own fetching and caching too. */
+ else if (!strncasecmp(itemlist[i], "BODY[", 5)) {
+ imap_fetch_body(IMAP->msgids[seq-1], itemlist[i], 0);
+ }
+ else if (!strncasecmp(itemlist[i], "BODY.PEEK[", 10)) {
+ imap_fetch_body(IMAP->msgids[seq-1], itemlist[i], 1);
+ }
+
+ /* Otherwise, load the message into memory.
+ */
+ else if (!strcasecmp(itemlist[i], "BODYSTRUCTURE")) {
+ if (msg == NULL) msg = CtdlFetchMessage(IMAP->msgids[seq-1], 1);
+ imap_fetch_bodystructure(IMAP->msgids[seq-1],
+ itemlist[i], msg);
+ }
+ else if (!strcasecmp(itemlist[i], "ENVELOPE")) {
+ if (msg == NULL) msg = CtdlFetchMessage(IMAP->msgids[seq-1], 1);
+ imap_fetch_envelope(IMAP->msgids[seq-1], msg);
+ }
+ else if (!strcasecmp(itemlist[i], "INTERNALDATE")) {
+ if (msg == NULL) msg = CtdlFetchMessage(IMAP->msgids[seq-1], 1);
+ imap_fetch_internaldate(msg);
}
if (i != num_items-1) cprintf(" ");
}
cprintf(")\r\n");
+ if (msg != NULL) {
+ CtdlFreeMessage(msg);
+ }
}
*/
void imap_do_fetch(int num_items, char **itemlist) {
int i;
- struct CtdlMessage *msg;
-
- if (IMAP->num_msgs > 0)
- for (i = 0; i < IMAP->num_msgs; ++i)
- if (IMAP->flags[i] & IMAP_SELECTED) {
- msg = CtdlFetchMessage(IMAP->msgids[i]);
- if (msg != NULL) {
- imap_do_fetch_msg(i+1, msg, num_items, itemlist);
- CtdlFreeMessage(msg);
- }
- else {
- cprintf("* %d FETCH <internal error>\r\n", i+1);
+
+ if (IMAP->num_msgs > 0) {
+ for (i = 0; i < IMAP->num_msgs; ++i) {
+ if (IMAP->flags[i] & IMAP_SELECTED) {
+ imap_do_fetch_msg(i+1, num_items, itemlist);
+ }
}
}
}
* is not a generic search-and-replace function.
*/
void imap_macro_replace(char *str, char *find, char *replace) {
- char holdbuf[1024];
+ char holdbuf[SIZ];
if (!strncasecmp(str, find, strlen(find))) {
if (str[strlen(find)]==' ') {
int i;
int num_sets;
int s;
- char setstr[SIZ], lostr[SIZ], histr[SIZ]; /* was 1024 */
- int lo, hi;
+ char setstr[SIZ], lostr[SIZ], histr[SIZ];
+ long lo, hi;
char actual_range[SIZ];
/*
*/
num_sets = num_tokens(actual_range, ',');
for (s=0; s<num_sets; ++s) {
- extract_token(setstr, actual_range, s, ',');
+ extract_token(setstr, actual_range, s, ',', sizeof setstr);
- extract_token(lostr, setstr, 0, ':');
+ extract_token(lostr, setstr, 0, ':', sizeof lostr);
if (num_tokens(setstr, ':') >= 2) {
- extract_token(histr, setstr, 1, ':');
- if (!strcmp(histr, "*")) snprintf(histr, sizeof histr, "%d", INT_MAX);
+ extract_token(histr, setstr, 1, ':', sizeof histr);
+ if (!strcmp(histr, "*")) snprintf(histr, sizeof histr, "%ld", LONG_MAX);
}
else {
- strcpy(histr, lostr);
+ safestrncpy(histr, lostr, sizeof histr);
}
- lo = atoi(lostr);
- hi = atoi(histr);
+ lo = atol(lostr);
+ hi = atol(histr);
/* Loop through the array, flipping bits where appropriate */
for (i = 1; i <= IMAP->num_msgs; ++i) {
* This function is called by the main command loop.
*/
void imap_fetch(int num_parms, char *parms[]) {
- char items[SIZ]; /* was 1024 */
+ char items[SIZ];
char *itemlist[SIZ];
int num_items;
int i;
* This function is called by the main command loop.
*/
void imap_uidfetch(int num_parms, char *parms[]) {
- char items[SIZ]; /* was 1024 */
+ char items[SIZ];
char *itemlist[SIZ];
int num_items;
int i;