mirror of
https://git.notmuchmail.org/git/notmuch
synced 2024-11-22 19:08:09 +01:00
449a418c65
notmuch previously unconditionally checked mime parts for various properties, but not for NULL, which is the case if libgmime encounters an empty mime part. Upon encounter of an empty mime part, the following is printed to stderr (the second line due to my patch): (process:17197): gmime-CRITICAL **: g_mime_message_get_mime_part: assertion `GMIME_IS_MESSAGE (message)' failed Warning: Not indexing empty mime part. This is probably a bug that should get addressed in libgmime, but for not, my patch is an acceptable workaround. Signed-off-by: martin f. krafft <madduck@madduck.net>
478 lines
13 KiB
C++
478 lines
13 KiB
C++
/*
|
|
* Copyright © 2009 Carl Worth
|
|
*
|
|
* 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, see http://www.gnu.org/licenses/ .
|
|
*
|
|
* Author: Carl Worth <cworth@cworth.org>
|
|
*/
|
|
|
|
#include "notmuch-private.h"
|
|
|
|
#include <gmime/gmime.h>
|
|
#include <gmime/gmime-filter.h>
|
|
|
|
#include <xapian.h>
|
|
|
|
/* Oh, how I wish that gobject didn't require so much noisy boilerplate!
|
|
* (Though I have at least eliminated some of the stock set...) */
|
|
typedef struct _NotmuchFilterDiscardUuencode NotmuchFilterDiscardUuencode;
|
|
typedef struct _NotmuchFilterDiscardUuencodeClass NotmuchFilterDiscardUuencodeClass;
|
|
|
|
/**
|
|
* NotmuchFilterDiscardUuencode:
|
|
*
|
|
* @parent_object: parent #GMimeFilter
|
|
* @encode: encoding vs decoding
|
|
* @state: State of the parser
|
|
*
|
|
* A filter to discard uuencoded portions of an email.
|
|
*
|
|
* A uuencoded portion is identified as beginning with a line
|
|
* matching:
|
|
*
|
|
* begin [0-7][0-7][0-7] .*
|
|
*
|
|
* After that detection, and beginning with the following line,
|
|
* characters will be discarded as long as the first character of each
|
|
* line begins with M and subsequent characters on the line are within
|
|
* the range of ASCII characters from ' ' to '`'.
|
|
*
|
|
* This is not a perfect UUencode filter. It's possible to have a
|
|
* message that will legitimately match that pattern, (so that some
|
|
* legitimate content is discarded). And for most UUencoded files, the
|
|
* final line of encoded data (the line not starting with M) will be
|
|
* indexed.
|
|
**/
|
|
struct _NotmuchFilterDiscardUuencode {
|
|
GMimeFilter parent_object;
|
|
int state;
|
|
};
|
|
|
|
struct _NotmuchFilterDiscardUuencodeClass {
|
|
GMimeFilterClass parent_class;
|
|
};
|
|
|
|
GMimeFilter *notmuch_filter_discard_uuencode_new (void);
|
|
|
|
static void notmuch_filter_discard_uuencode_finalize (GObject *object);
|
|
|
|
static GMimeFilter *filter_copy (GMimeFilter *filter);
|
|
static void filter_filter (GMimeFilter *filter, char *in, size_t len, size_t prespace,
|
|
char **out, size_t *outlen, size_t *outprespace);
|
|
static void filter_complete (GMimeFilter *filter, char *in, size_t len, size_t prespace,
|
|
char **out, size_t *outlen, size_t *outprespace);
|
|
static void filter_reset (GMimeFilter *filter);
|
|
|
|
|
|
static GMimeFilterClass *parent_class = NULL;
|
|
|
|
static void
|
|
notmuch_filter_discard_uuencode_class_init (NotmuchFilterDiscardUuencodeClass *klass)
|
|
{
|
|
GObjectClass *object_class = G_OBJECT_CLASS (klass);
|
|
GMimeFilterClass *filter_class = GMIME_FILTER_CLASS (klass);
|
|
|
|
parent_class = (GMimeFilterClass *) g_type_class_ref (GMIME_TYPE_FILTER);
|
|
|
|
object_class->finalize = notmuch_filter_discard_uuencode_finalize;
|
|
|
|
filter_class->copy = filter_copy;
|
|
filter_class->filter = filter_filter;
|
|
filter_class->complete = filter_complete;
|
|
filter_class->reset = filter_reset;
|
|
}
|
|
|
|
static void
|
|
notmuch_filter_discard_uuencode_finalize (GObject *object)
|
|
{
|
|
G_OBJECT_CLASS (parent_class)->finalize (object);
|
|
}
|
|
|
|
static GMimeFilter *
|
|
filter_copy (GMimeFilter *gmime_filter)
|
|
{
|
|
(void) gmime_filter;
|
|
return notmuch_filter_discard_uuencode_new ();
|
|
}
|
|
|
|
static void
|
|
filter_filter (GMimeFilter *gmime_filter, char *inbuf, size_t inlen, size_t prespace,
|
|
char **outbuf, size_t *outlen, size_t *outprespace)
|
|
{
|
|
NotmuchFilterDiscardUuencode *filter = (NotmuchFilterDiscardUuencode *) gmime_filter;
|
|
register const char *inptr = inbuf;
|
|
const char *inend = inbuf + inlen;
|
|
char *outptr;
|
|
|
|
(void) prespace;
|
|
|
|
/* Simple, linear state-transition diagram for our filter.
|
|
*
|
|
* If the character being processed is within the range of [a, b]
|
|
* for the current state then we transition next_if_match
|
|
* state. If not, we transition to the next_if_not_match state.
|
|
*
|
|
* The final two states are special in that they are the states in
|
|
* which we discard data. */
|
|
static const struct {
|
|
int state;
|
|
int a;
|
|
int b;
|
|
int next_if_match;
|
|
int next_if_not_match;
|
|
} states[] = {
|
|
{0, 'b', 'b', 1, 0},
|
|
{1, 'e', 'e', 2, 0},
|
|
{2, 'g', 'g', 3, 0},
|
|
{3, 'i', 'i', 4, 0},
|
|
{4, 'n', 'n', 5, 0},
|
|
{5, ' ', ' ', 6, 0},
|
|
{6, '0', '7', 7, 0},
|
|
{7, '0', '7', 8, 0},
|
|
{8, '0', '7', 9, 0},
|
|
{9, ' ', ' ', 10, 0},
|
|
{10, '\n', '\n', 11, 10},
|
|
{11, 'M', 'M', 12, 0},
|
|
{12, ' ', '`', 12, 11}
|
|
};
|
|
int next;
|
|
|
|
g_mime_filter_set_size (gmime_filter, inlen, FALSE);
|
|
outptr = gmime_filter->outbuf;
|
|
|
|
while (inptr < inend) {
|
|
if (*inptr >= states[filter->state].a &&
|
|
*inptr <= states[filter->state].b)
|
|
{
|
|
next = states[filter->state].next_if_match;
|
|
}
|
|
else
|
|
{
|
|
next = states[filter->state].next_if_not_match;
|
|
}
|
|
|
|
if (filter->state < 11)
|
|
*outptr++ = *inptr;
|
|
|
|
filter->state = next;
|
|
inptr++;
|
|
}
|
|
|
|
*outlen = outptr - gmime_filter->outbuf;
|
|
*outprespace = gmime_filter->outpre;
|
|
*outbuf = gmime_filter->outbuf;
|
|
}
|
|
|
|
static void
|
|
filter_complete (GMimeFilter *filter, char *inbuf, size_t inlen, size_t prespace,
|
|
char **outbuf, size_t *outlen, size_t *outprespace)
|
|
{
|
|
if (inbuf && inlen)
|
|
filter_filter (filter, inbuf, inlen, prespace, outbuf, outlen, outprespace);
|
|
}
|
|
|
|
static void
|
|
filter_reset (GMimeFilter *gmime_filter)
|
|
{
|
|
NotmuchFilterDiscardUuencode *filter = (NotmuchFilterDiscardUuencode *) gmime_filter;
|
|
|
|
filter->state = 0;
|
|
}
|
|
|
|
/**
|
|
* notmuch_filter_discard_uuencode_new:
|
|
*
|
|
* Returns: a new #NotmuchFilterDiscardUuencode filter.
|
|
**/
|
|
GMimeFilter *
|
|
notmuch_filter_discard_uuencode_new (void)
|
|
{
|
|
static GType type = 0;
|
|
NotmuchFilterDiscardUuencode *filter;
|
|
|
|
if (!type) {
|
|
static const GTypeInfo info = {
|
|
sizeof (NotmuchFilterDiscardUuencodeClass),
|
|
NULL, /* base_class_init */
|
|
NULL, /* base_class_finalize */
|
|
(GClassInitFunc) notmuch_filter_discard_uuencode_class_init,
|
|
NULL, /* class_finalize */
|
|
NULL, /* class_data */
|
|
sizeof (NotmuchFilterDiscardUuencode),
|
|
0, /* n_preallocs */
|
|
NULL, /* instance_init */
|
|
NULL /* value_table */
|
|
};
|
|
|
|
type = g_type_register_static (GMIME_TYPE_FILTER, "NotmuchFilterDiscardUuencode", &info, (GTypeFlags) 0);
|
|
}
|
|
|
|
filter = (NotmuchFilterDiscardUuencode *) g_object_newv (type, 0, NULL);
|
|
filter->state = 0;
|
|
|
|
return (GMimeFilter *) filter;
|
|
}
|
|
|
|
/* We're finally down to a single (NAME + address) email "mailbox". */
|
|
static void
|
|
_index_address_mailbox (notmuch_message_t *message,
|
|
const char *prefix_name,
|
|
InternetAddress *address)
|
|
{
|
|
InternetAddressMailbox *mailbox = INTERNET_ADDRESS_MAILBOX (address);
|
|
const char *name, *addr;
|
|
void *local = talloc_new (message);
|
|
|
|
name = internet_address_get_name (address);
|
|
addr = internet_address_mailbox_get_addr (mailbox);
|
|
|
|
/* In the absence of a name, we'll strip the part before the @
|
|
* from the address. */
|
|
if (! name) {
|
|
const char *at;
|
|
|
|
at = strchr (addr, '@');
|
|
if (at)
|
|
name = talloc_strndup (local, addr, at - addr);
|
|
}
|
|
|
|
if (name)
|
|
_notmuch_message_gen_terms (message, prefix_name, name);
|
|
if (addr)
|
|
_notmuch_message_gen_terms (message, prefix_name, addr);
|
|
|
|
talloc_free (local);
|
|
}
|
|
|
|
static void
|
|
_index_address_list (notmuch_message_t *message,
|
|
const char *prefix_name,
|
|
InternetAddressList *addresses);
|
|
|
|
/* The outer loop over the InternetAddressList wasn't quite enough.
|
|
* There can actually be a tree here where a single member of the list
|
|
* is a "group" containing another list. Recurse please.
|
|
*/
|
|
static void
|
|
_index_address_group (notmuch_message_t *message,
|
|
const char *prefix_name,
|
|
InternetAddress *address)
|
|
{
|
|
InternetAddressGroup *group;
|
|
InternetAddressList *list;
|
|
|
|
group = INTERNET_ADDRESS_GROUP (address);
|
|
list = internet_address_group_get_members (group);
|
|
|
|
if (! list)
|
|
return;
|
|
|
|
_index_address_list (message, prefix_name, list);
|
|
}
|
|
|
|
static void
|
|
_index_address_list (notmuch_message_t *message,
|
|
const char *prefix_name,
|
|
InternetAddressList *addresses)
|
|
{
|
|
int i;
|
|
InternetAddress *address;
|
|
|
|
if (addresses == NULL)
|
|
return;
|
|
|
|
for (i = 0; i < internet_address_list_length (addresses); i++) {
|
|
address = internet_address_list_get_address (addresses, i);
|
|
if (INTERNET_ADDRESS_IS_MAILBOX (address)) {
|
|
_index_address_mailbox (message, prefix_name, address);
|
|
} else if (INTERNET_ADDRESS_IS_GROUP (address)) {
|
|
_index_address_group (message, prefix_name, address);
|
|
} else {
|
|
INTERNAL_ERROR ("GMime InternetAddress is neither a mailbox nor a group.\n");
|
|
}
|
|
}
|
|
}
|
|
|
|
static const char *
|
|
skip_re_in_subject (const char *subject)
|
|
{
|
|
const char *s = subject;
|
|
|
|
if (subject == NULL)
|
|
return NULL;
|
|
|
|
while (*s) {
|
|
while (*s && isspace (*s))
|
|
s++;
|
|
if (strncasecmp (s, "re:", 3) == 0)
|
|
s += 3;
|
|
else
|
|
break;
|
|
}
|
|
|
|
return s;
|
|
}
|
|
|
|
/* Callback to generate terms for each mime part of a message. */
|
|
static void
|
|
_index_mime_part (notmuch_message_t *message,
|
|
GMimeObject *part)
|
|
{
|
|
GMimeStream *stream, *filter;
|
|
GMimeFilter *discard_uuencode_filter;
|
|
GMimeDataWrapper *wrapper;
|
|
GByteArray *byte_array;
|
|
GMimeContentDisposition *disposition;
|
|
char *body;
|
|
|
|
if (! part) {
|
|
fprintf (stderr, "Warning: Not indexing empty mime part.\n");
|
|
return;
|
|
}
|
|
|
|
if (GMIME_IS_MULTIPART (part)) {
|
|
GMimeMultipart *multipart = GMIME_MULTIPART (part);
|
|
int i;
|
|
|
|
for (i = 0; i < g_mime_multipart_get_count (multipart); i++) {
|
|
if (GMIME_IS_MULTIPART_SIGNED (multipart)) {
|
|
/* Don't index the signature. */
|
|
if (i == 1)
|
|
continue;
|
|
if (i > 1)
|
|
fprintf (stderr, "Warning: Unexpected extra parts of multipart/signed. Indexing anyway.\n");
|
|
}
|
|
_index_mime_part (message,
|
|
g_mime_multipart_get_part (multipart, i));
|
|
}
|
|
return;
|
|
}
|
|
|
|
if (GMIME_IS_MESSAGE_PART (part)) {
|
|
GMimeMessage *mime_message;
|
|
|
|
mime_message = g_mime_message_part_get_message (GMIME_MESSAGE_PART (part));
|
|
|
|
_index_mime_part (message, g_mime_message_get_mime_part (mime_message));
|
|
|
|
return;
|
|
}
|
|
|
|
if (! (GMIME_IS_PART (part))) {
|
|
fprintf (stderr, "Warning: Not indexing unknown mime part: %s.\n",
|
|
g_type_name (G_OBJECT_TYPE (part)));
|
|
return;
|
|
}
|
|
|
|
disposition = g_mime_object_get_content_disposition (part);
|
|
if (disposition &&
|
|
strcmp (disposition->disposition, GMIME_DISPOSITION_ATTACHMENT) == 0)
|
|
{
|
|
const char *filename = g_mime_part_get_filename (GMIME_PART (part));
|
|
|
|
_notmuch_message_add_term (message, "tag", "attachment");
|
|
_notmuch_message_gen_terms (message, "attachment", filename);
|
|
|
|
/* XXX: Would be nice to call out to something here to parse
|
|
* the attachment into text and then index that. */
|
|
return;
|
|
}
|
|
|
|
byte_array = g_byte_array_new ();
|
|
|
|
stream = g_mime_stream_mem_new_with_byte_array (byte_array);
|
|
g_mime_stream_mem_set_owner (GMIME_STREAM_MEM (stream), FALSE);
|
|
|
|
filter = g_mime_stream_filter_new (stream);
|
|
discard_uuencode_filter = notmuch_filter_discard_uuencode_new ();
|
|
|
|
g_mime_stream_filter_add (GMIME_STREAM_FILTER (filter),
|
|
discard_uuencode_filter);
|
|
|
|
wrapper = g_mime_part_get_content_object (GMIME_PART (part));
|
|
if (wrapper)
|
|
g_mime_data_wrapper_write_to_stream (wrapper, filter);
|
|
|
|
g_object_unref (stream);
|
|
g_object_unref (filter);
|
|
g_object_unref (discard_uuencode_filter);
|
|
|
|
g_byte_array_append (byte_array, (guint8 *) "\0", 1);
|
|
body = (char *) g_byte_array_free (byte_array, FALSE);
|
|
|
|
if (body) {
|
|
_notmuch_message_gen_terms (message, NULL, body);
|
|
|
|
free (body);
|
|
}
|
|
}
|
|
|
|
notmuch_status_t
|
|
_notmuch_message_index_file (notmuch_message_t *message,
|
|
const char *filename)
|
|
{
|
|
GMimeStream *stream = NULL;
|
|
GMimeParser *parser = NULL;
|
|
GMimeMessage *mime_message = NULL;
|
|
InternetAddressList *addresses;
|
|
FILE *file = NULL;
|
|
const char *from, *subject;
|
|
notmuch_status_t ret = NOTMUCH_STATUS_SUCCESS;
|
|
static int initialized = 0;
|
|
|
|
if (! initialized) {
|
|
g_mime_init (0);
|
|
initialized = 1;
|
|
}
|
|
|
|
file = fopen (filename, "r");
|
|
if (! file) {
|
|
fprintf (stderr, "Error opening %s: %s\n", filename, strerror (errno));
|
|
ret = NOTMUCH_STATUS_FILE_ERROR;
|
|
goto DONE;
|
|
}
|
|
|
|
/* Evil GMime steals my FILE* here so I won't fclose it. */
|
|
stream = g_mime_stream_file_new (file);
|
|
|
|
parser = g_mime_parser_new_with_stream (stream);
|
|
|
|
mime_message = g_mime_parser_construct_message (parser);
|
|
|
|
from = g_mime_message_get_sender (mime_message);
|
|
addresses = internet_address_list_parse_string (from);
|
|
|
|
_index_address_list (message, "from", addresses);
|
|
|
|
addresses = g_mime_message_get_all_recipients (mime_message);
|
|
_index_address_list (message, "to", addresses);
|
|
|
|
subject = g_mime_message_get_subject (mime_message);
|
|
subject = skip_re_in_subject (subject);
|
|
_notmuch_message_gen_terms (message, "subject", subject);
|
|
|
|
_index_mime_part (message, g_mime_message_get_mime_part (mime_message));
|
|
|
|
DONE:
|
|
if (mime_message)
|
|
g_object_unref (mime_message);
|
|
|
|
if (parser)
|
|
g_object_unref (parser);
|
|
|
|
if (stream)
|
|
g_object_unref (stream);
|
|
|
|
return ret;
|
|
}
|