c1/0e9427167abc6481d22bad984118a03132ac9f

   1 Return-Path: <olly@survex.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 44E51431FB6\r
   6         for <notmuch@notmuchmail.org>; Fri,  6 Sep 2013 16:35:30 -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.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id fypeHTEBJ8-i for <notmuch@notmuchmail.org>;\r
  16         Fri,  6 Sep 2013 16:35:25 -0700 (PDT)\r
  17 X-Greylist: delayed 7525 seconds by postgrey-1.32 at olra;\r
  18         Fri, 06 Sep 2013 16:35:25 PDT\r
  19 Received: from atreus.tartarus.org (atreus.tartarus.org [80.252.125.10])\r
  20         (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
  21         (No client certificate requested)\r
  22         by olra.theworths.org (Postfix) with ESMTPS id 617F4431FAF\r
  23         for <notmuch@notmuchmail.org>; Fri,  6 Sep 2013 16:35:25 -0700 (PDT)\r
  24 Received: from olly by atreus.tartarus.org with local (Exim 4.69)\r
  25         (envelope-from <olly@survex.com>)\r
  26         id 1VI3bG-0006LV-Sv; Fri, 06 Sep 2013 22:29:54 +0100\r
  27 Date: Fri, 6 Sep 2013 22:29:54 +0100\r
  28 From: Olly Betts <olly@survex.com>\r
  29 To: David Bremner <david@tethera.net>\r
  30 Subject: Re: segfault with xapian 1.3.1\r
  31 Message-ID: <20130906212954.GB31164@survex.com>\r
  32 References:\r
  33  <CAHmME9q3H2KKBRb_VU4Rr-pc+2qR=uPxiX8YPBFE4N0DaiidAA@mail.gmail.com>\r
  34         <87txhypzvz.fsf@zancas.localnet> <87r4d2pwv1.fsf@zancas.localnet>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain; charset=us-ascii\r
  37 Content-Disposition: inline\r
  38 In-Reply-To: <87r4d2pwv1.fsf@zancas.localnet>\r
  39 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  40 Cc: notmuch ML <notmuch@notmuchmail.org>\r
  41 X-BeenThere: notmuch@notmuchmail.org\r
  42 X-Mailman-Version: 2.1.13\r
  43 Precedence: list\r
  44 List-Id: "Use and development of the notmuch mail system."\r
  45         <notmuch.notmuchmail.org>\r
  46 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  48 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  49 List-Post: <mailto:notmuch@notmuchmail.org>\r
  50 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  51 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  53 X-List-Received-Date: Fri, 06 Sep 2013 23:35:30 -0000\r
  54 \r
  55 Olly wrote on  IRC:\r
  56 > bremner: ok, 1.2 explicitly no-oped skip_to() on an iterator at_end on\r
  57 > trunk that code has been rewritten without that explicit check, and\r
  58 > the iterator internals are NULL then i think restoring the check is\r
  59 > reasonable, though I'm not sure if we actually promise that's defined\r
  60 > behaviour if you want to work with 1.3.1, then checking against the\r
  61 > end iterator before calling skip_to() will work for any version\r
  62 \r
  63 I've put a NULL check in there, and also for the other iterator classes\r
  64 which use NULL internals to signify being at the end - this only worked\r
  65 for TermIterator in 1.2.x, but it seems reasonable to make it work in\r
  66 general and a NULL pointer check isn't a big overhead.\r
  67 \r
  68 On Thu, Sep 05, 2013 at 10:22:42PM -0300, David Bremner wrote:\r
  69 > So, now we know what to fix.\r
  70 \r
  71 This should work again in 1.3.2 (once it is out), but if you want to\r
  72 support 1.3.1 then you need the check.\r
  73 \r
  74 Cheers,\r
  75     Olly\r