78/06a104714cf842ff32b957399b497b6ade6448

   1 Return-Path: <cworth@cworth.org>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 7C9614196F2\r
   6         for <notmuch@notmuchmail.org>; Tue, 13 Apr 2010 08:55:02 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -2.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id i70CcxPLyl0m; Tue, 13 Apr 2010 08:55:01 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 7EF56431FC1;\r
  19         Tue, 13 Apr 2010 08:55:01 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id 3500A5883A6; Tue, 13 Apr 2010 08:55:01 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: "martin f. krafft" <madduck@madduck.net>, notmuch@notmuchmail.org\r
  24 Subject: Re: [notmuch] [PATCH] Do not segfault on empty mime parts\r
  25 In-Reply-To: <1267543888-18134-1-git-send-email-madduck@madduck.net>\r
  26 References: <1267543888-18134-1-git-send-email-madduck@madduck.net>\r
  27 Date: Tue, 13 Apr 2010 08:55:01 -0700\r
  28 Message-ID: <87ljcre4a2.fsf@yoom.home.cworth.org>\r
  29 MIME-Version: 1.0\r
  30 Content-Type: multipart/signed; boundary="=-=-=";\r
  31         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  32 Cc: "martin f. krafft" <madduck@madduck.net>\r
  33 X-BeenThere: notmuch@notmuchmail.org\r
  34 X-Mailman-Version: 2.1.13\r
  35 Precedence: list\r
  36 List-Id: "Use and development of the notmuch mail system."\r
  37         <notmuch.notmuchmail.org>\r
  38 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  39         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  40 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  41 List-Post: <mailto:notmuch@notmuchmail.org>\r
  42 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  43 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  45 X-List-Received-Date: Tue, 13 Apr 2010 15:55:02 -0000\r
  46 \r
  47 --=-=-=\r
  48 Content-Transfer-Encoding: quoted-printable\r
  49 \r
  50 On Tue,  2 Mar 2010 16:31:28 +0100, "martin f. krafft" <madduck@madduck.net=\r
  51 > wrote:\r
  52 > notmuch previously unconditionally checked mime parts for various\r
  53 > properties, but not for NULL, which is the case if libgmime encounters\r
  54 > an empty mime part.\r
  55 \r
  56 Thanks for the fix.\r
  57 \r
  58 > Upon encounter of an empty mime part, the following is printed to\r
  59 > stderr (the second line due to my patch):\r
  60 >=20\r
  61 >   (process:17197): gmime-CRITICAL **: g_mime_message_get_mime_part: asser=\r
  62 tion `GMIME_IS_MESSAGE (message)' failed\r
  63 >   Warning: Not indexing empty mime part.\r
  64 \r
  65 That message is all quite ugly. I'm perpetually annoyed by libraries\r
  66 that claim to detect major problems, (even "CRITICAL" problems!), and\r
  67 just print messages and march along.\r
  68 \r
  69 Either there *is* a critical problem here, in which case the library\r
  70 should halt after printing the message, (see for example, what notmuch\r
  71 does with its INTERNAL_ERROR macro), or there's no problem at all and\r
  72 the code should operate correctly without these obnoxious messages.\r
  73 \r
  74 Anyway, I've put a note into TODO that we should do the following (in\r
  75 order):\r
  76 \r
  77   1. Add a test case to trigger this message\r
  78 \r
  79   2. File a bug against GMime to eliminate its portion of the message\r
  80 \r
  81   3. Remove the notmuch portion of the message\r
  82 \r
  83 =2DCarl\r
  84 \r
  85 --=-=-=\r
  86 Content-Type: application/pgp-signature\r
  87 \r
  88 -----BEGIN PGP SIGNATURE-----\r
  89 Version: GnuPG v1.4.10 (GNU/Linux)\r
  90 \r
  91 iD8DBQFLxJPV6JDdNq8qSWgRApdxAJ9kLDhwuKnm6PUo4tYQbS20SUiwMQCdHSUA\r
  92 8n/6OWa8t9JPf6mdmiF0fSs=\r
  93 =0ETF\r
  94 -----END PGP SIGNATURE-----\r
  95 --=-=-=--\r