57/715d065d3f9f43647d535519a21fd43ae98de9

   1 Return-Path: <jan@ryngle.com>\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 CB30E431FBC\r
   6         for <notmuch@notmuchmail.org>; Fri, 20 Nov 2009 05:20:49 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id ZPXbS1YgfrpS for <notmuch@notmuchmail.org>;\r
  11         Fri, 20 Nov 2009 05:20:47 -0800 (PST)\r
  12 Received: from mail-bw0-f224.google.com (mail-bw0-f224.google.com\r
  13         [209.85.218.224])\r
  14         by olra.theworths.org (Postfix) with ESMTP id 7BA1E431FAE\r
  15         for <notmuch@notmuchmail.org>; Fri, 20 Nov 2009 05:20:47 -0800 (PST)\r
  16 Received: by bwz24 with SMTP id 24so2261612bwz.30\r
  17         for <notmuch@notmuchmail.org>; Fri, 20 Nov 2009 05:20:46 -0800 (PST)\r
  18 MIME-Version: 1.0\r
  19 Received: by 10.204.155.92 with SMTP id r28mr1360381bkw.121.1258723245238; \r
  20         Fri, 20 Nov 2009 05:20:45 -0800 (PST)\r
  21 In-Reply-To: <935ead450911200510ie80d28dv1d82ccbd11803128@mail.gmail.com>\r
  22 References: <935ead450911181000w2cddfe95qe9efd5bea9f9209d@mail.gmail.com>\r
  23         <20091119164543.25e7afe5@hikari>\r
  24         <935ead450911191823s776fda6eyb6f6949ac982bd03@mail.gmail.com>\r
  25         <87lji11leu.fsf@yoom.home.cworth.org>\r
  26         <935ead450911200510ie80d28dv1d82ccbd11803128@mail.gmail.com>\r
  27 Date: Fri, 20 Nov 2009 14:20:45 +0100\r
  28 Message-ID: <f35dbb950911200520s2de6bf12u1dd88ff110eca105@mail.gmail.com>\r
  29 From: Jan Janak <jan@ryngle.com>\r
  30 To: Jeffrey Ollie <jeff@ocjtech.us>\r
  31 Content-Type: text/plain; charset=UTF-8\r
  32 Content-Transfer-Encoding: quoted-printable\r
  33 Cc: Not Much Mail <notmuch@notmuchmail.org>\r
  34 Subject: Re: [notmuch] Segfault searching for tags\r
  35 X-BeenThere: notmuch@notmuchmail.org\r
  36 X-Mailman-Version: 2.1.12\r
  37 Precedence: list\r
  38 List-Id: "Use and development of the notmuch mail system."\r
  39         <notmuch.notmuchmail.org>\r
  40 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  41         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  42 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  43 List-Post: <mailto:notmuch@notmuchmail.org>\r
  44 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  45 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  46         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  47 X-List-Received-Date: Fri, 20 Nov 2009 13:20:50 -0000\r
  48 \r
  49 On Fri, Nov 20, 2009 at 2:10 PM, Jeffrey Ollie <jeff@ocjtech.us> wrote:\r
  50 > On Fri, Nov 20, 2009 at 5:32 AM, Carl Worth <cworth@cworth.org> wrote:\r
  51 >> On Thu, 19 Nov 2009 16:45:43 +0100, Adrian Perez de Castro <aperez@igali=\r
  52 a.com> wrote:\r
  53 >>> The thing is that in notmuch_message_get_in_reply_to(), line 288, a NUL=\r
  54 L\r
  55 >>> instance of Xapian::TermIterator is dereferenced. In my particular case=\r
  56 ,\r
  57 >>> the culpript is a cache file of Claws-Mail, as seen in the following GD=\r
  58 B\r
  59 >>> session:\r
  60 >>\r
  61 >> Not quite NULL, (nor is it quite dereferencing---this is nasty C++\r
  62 >> overloading), but yeah, the idea is the same. We need to protect all of\r
  63 >> our "calls" to this overloaded operator to not call it when the iterator\r
  64 >> is equal to the value returned by termlist_end ().\r
  65 >>\r
  66 >> On Thu, 19 Nov 2009 20:23:15 -0600, Jeffrey Ollie <jeff@ocjtech.us> wrot=\r
  67 e:\r
  68 >>> I straced some of the crashes, and the last file that was read before\r
  69 >>> the crash was a malformed message. =C2=A0I've attached one of the messa=\r
  70 ges.\r
  71 >>\r
  72 >> Thanks for the file. I never like to push code that I haven't tested, so\r
  73 >> this was very helpful.\r
  74 >>\r
  75 >> Below is the patch that I just pushed which seems to do the trick.\r
  76 >\r
  77 > Ah, excellent! =C2=A0This does indeed seem to prevent the crash. =C2=A0No=\r
  78 w I\r
  79 > just need to figure out how to get all my mail out of GMail.\r
  80 \r
  81 I did exactly that with offlineimap. It crashes from time to time, but\r
  82 then you can just restart it and continue.\r
  83 \r
  84 A few days ago I sent a patch which converts mail subdirectories to\r
  85 tags and because Gmail IMAP server converts labels to subdirectories,\r
  86 you can use that to convert gmail's labels to notmuch tags\r
  87 automatically.\r
  88 \r
  89    -- Jan\r