notmuch-index-message: Add explicit support for multipart mime.
authorCarl Worth <cworth@cworth.org>
Wed, 14 Oct 2009 22:33:16 +0000 (15:33 -0700)
committerCarl Worth <cworth@cworth.org>
Wed, 14 Oct 2009 22:36:13 +0000 (15:36 -0700)
Instead of using the recursive "foreach" method, we implement our
own recursive function. This allows us to ignore the signature
component of a multipart/signed message, (which we certainly
don't need to index).

notmuch-index-message.cc

index bb393641af1da745841f108ec6022f28e4904f9a..852f4e16259fba6fee4d5e51918f856a4c6702b4 100644 (file)
@@ -395,16 +395,32 @@ gen_terms_body_str (Xapian::TermGenerator term_gen,
 
 /* Callback to generate terms for each mime part of a message. */
 static void
 
 /* Callback to generate terms for each mime part of a message. */
 static void
-gen_terms_part (GMimeObject *parent,
-               GMimeObject *part,
-               gpointer user_data)
+gen_terms_part (Xapian::TermGenerator term_gen,
+               GMimeObject *part)
 {
 {
-    Xapian::TermGenerator *term_gen = (Xapian::TermGenerator *) user_data;
     GMimeStream *stream;
     GMimeDataWrapper *wrapper;
     GByteArray *byte_array;
     char *body;
 
     GMimeStream *stream;
     GMimeDataWrapper *wrapper;
     GByteArray *byte_array;
     char *body;
 
+    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 mutlipart/signed. Indexing anyway.\n");
+           }
+           gen_terms_part (term_gen,
+                           g_mime_multipart_get_part (multipart, i));
+       }
+       return;
+    }
+
     if (! GMIME_IS_PART (part)) {
        fprintf (stderr, "Warning: Not indexing unknown mime part: %s.\n",
                 g_type_name (G_OBJECT_TYPE (part)));
     if (! GMIME_IS_PART (part)) {
        fprintf (stderr, "Warning: Not indexing unknown mime part: %s.\n",
                 g_type_name (G_OBJECT_TYPE (part)));
@@ -422,7 +438,7 @@ gen_terms_part (GMimeObject *parent,
 
     body = (char *) g_byte_array_free (byte_array, FALSE);
 
 
     body = (char *) g_byte_array_free (byte_array, FALSE);
 
-    gen_terms_body_str (*term_gen, body);
+    gen_terms_body_str (term_gen, body);
 
     free (body);
 }
 
     free (body);
 }
@@ -483,7 +499,7 @@ index_file (Xapian::WritableDatabase db,
     gen_terms (term_gen, "subject", subject);
     gen_terms (term_gen, "body", subject);
 
     gen_terms (term_gen, "subject", subject);
     gen_terms (term_gen, "body", subject);
 
-    g_mime_message_foreach (message, gen_terms_part, &term_gen);
+    gen_terms_part (term_gen, g_mime_message_get_mime_part (message));
 
     parents = g_ptr_array_new ();
 
 
     parents = g_ptr_array_new ();