e6/ea0ba359ac54e7032a57c874bb32cfeaf048f1

   1 Return-Path: <dmitry.kurochkin@gmail.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 1F9FF429E29\r
   6         for <notmuch@notmuchmail.org>; Sun, 19 Jun 2011 02:44:42 -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: -0.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\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 OV2SjjPP0+DA for <notmuch@notmuchmail.org>;\r
  17         Sun, 19 Jun 2011 02:44:41 -0700 (PDT)\r
  18 Received: from mail-bw0-f53.google.com (mail-bw0-f53.google.com\r
  19         [209.85.214.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id E5F04431FB6\r
  22         for <notmuch@notmuchmail.org>; Sun, 19 Jun 2011 02:44:40 -0700 (PDT)\r
  23 Received: by bwg12 with SMTP id 12so2130199bwg.26\r
  24         for <notmuch@notmuchmail.org>; Sun, 19 Jun 2011 02:44:39 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
  26         h=domainkey-signature:from:to:cc:subject:in-reply-to:references\r
  27         :user-agent:date:message-id:mime-version:content-type;\r
  28         bh=G7y+2K7xCs8fEs3mXUrJgHzhdhRFByc5mPsr1Zb7rFQ=;\r
  29         b=qMDBjTNOdBM/rfc5tLxsv/+1ATouN4yvGTZOq4qVXafQC1W6XRxoInqvGgWGehWVeh\r
  30         hamTddkxLipyGP0UFAsj8MboZlBhLexHxIksuKcLMXHeMnGOkcZah2tYerBvo6eLx6Sm\r
  31         v47h3pPG0/HuK4x9oR7UbxSzBzIzjYU1XwE8Q=\r
  32 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma;\r
  33         h=from:to:cc:subject:in-reply-to:references:user-agent:date\r
  34         :message-id:mime-version:content-type;\r
  35         b=YVsxwpdM7mSL06l2QKNQ8LRQNlA87PU3bk6s8sZEaBBSwggG2Q0J4xUwNVuXuL75vm\r
  36         +3RSWcU9FDSvyNo/b8W5PYvzfeqOwVPPuPZsspzgGZrSMxDwTq9wmmMf4LfLUt5f5FLq\r
  37         7nIaGHHW/aFmwcWuidg/qG5zRibwz+J1kiVtA=\r
  38 Received: by 10.204.7.17 with SMTP id b17mr2989201bkb.61.1308476679059;\r
  39         Sun, 19 Jun 2011 02:44:39 -0700 (PDT)\r
  40 Received: from localhost ([91.144.186.21])\r
  41         by mx.google.com with ESMTPS id k16sm3466173bks.1.2011.06.19.02.44.36\r
  42         (version=TLSv1/SSLv3 cipher=OTHER);\r
  43         Sun, 19 Jun 2011 02:44:37 -0700 (PDT)\r
  44 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  45 To: Sebastian Spaeth <Sebastian@SSpaeth.de>,\r
  46         Patrick Totzke <patricktotzke@googlemail.com>\r
  47 Subject: Re: [python] segfaults at Message.get_date\r
  48 In-Reply-To: <87hb7n300m.fsf@SSpaeth.de>\r
  49 References: <20110616215439.GA26997@brick> <87boxxq833.fsf@SSpaeth.de>\r
  50         <20110617161024.GA8154@optimusprime> <87hb7n300m.fsf@SSpaeth.de>\r
  51 User-Agent: Notmuch/0.5-237-gf6d8cdb (http://notmuchmail.org) Emacs/23.3.1\r
  52         (x86_64-pc-linux-gnu)\r
  53 Date: Sun, 19 Jun 2011 13:45:07 +0400\r
  54 Message-ID: <87hb7m5f4s.fsf@gmail.com>\r
  55 MIME-Version: 1.0\r
  56 Content-Type: text/plain; charset=us-ascii\r
  57 Cc: notmuch@notmuchmail.org\r
  58 X-BeenThere: notmuch@notmuchmail.org\r
  59 X-Mailman-Version: 2.1.13\r
  60 Precedence: list\r
  61 List-Id: "Use and development of the notmuch mail system."\r
  62         <notmuch.notmuchmail.org>\r
  63 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  64         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  65 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  66 List-Post: <mailto:notmuch@notmuchmail.org>\r
  67 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  68 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  69         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  70 X-List-Received-Date: Sun, 19 Jun 2011 09:44:42 -0000\r
  71 \r
  72 Hi Sebastian, Patrick.\r
  73 \r
  74 On Sat, 18 Jun 2011 12:30:01 +0200, Sebastian Spaeth <Sebastian@SSpaeth.de> wrote:\r
  75 > On Fri, 17 Jun 2011 17:10:24 +0100, Patrick Totzke :\r
  76 > > #0  0x006eb87d in Xapian::Document::Internal::get_value(unsigned int) const () from /usr/lib/sse2/libxapian.so.22\r
  77 > > #1  0x006eb952 in Xapian::Document::get_value(unsigned int) const () from /usr/lib/sse2/libxapian.so.22\r
  78 > > #2  0x00523963 in notmuch_message_get_date () from /usr/local/lib/libnotmuch.so.1\r
  79 > \r
  80 > One question, what type is libnotmuch really returning here? The code:\r
  81 > \r
  82 > \r
  83 > time_t\r
  84 > notmuch_message_get_date (notmuch_message_t *message)\r
  85 > {   ...\r
  86 >     return Xapian::sortable_unserialise (value);\r
  87 > }\r
  88 > \r
  89 > But Xapian API says that sortable_unserialise() returns floating type "double"\r
  90 > \r
  91 > http://xapian.org/docs/apidoc/html/namespaceXapian.html#326fe2d6b0ee59ac9536f3960e8fd99b\r
  92 > "Convert a string encoded using sortable_serialise back to a floating\r
  93 > point number."\r
  94 > \r
  95 > But time_t is usually a (signed) long and not floating point. Obviously\r
  96 > things have worked just fine so far, but is libnotmuch really returning\r
  97 > the right type here? Sorry, I expose my total lack of basic C++ knowledge\r
  98 > here...\r
  99 > \r
 100 \r
 101 Converting double to time_t does not look good.  Notmuch converts\r
 102 between time_t and double both when setting and getting the date.  I\r
 103 guess it should work good in most cases at least.  Perhaps Carl knows\r
 104 better that it is safe.\r
 105 \r
 106 This is not relevant to the segfault.  The failure happens in:\r
 107 \r
 108   value = message->doc.get_value (NOTMUCH_VALUE_TIMESTAMP);\r
 109 \r
 110 before conversion from double to time_t.\r
 111 \r
 112 Sebastian, are you able to reproduce the issue?  A python script that\r
 113 triggers the bug, perhaps?  I would look at this if I can reproduce the\r
 114 problem.\r
 115 \r
 116 Patrick, can you install debugging symbols for libxapian and get the\r
 117 backtrace again?  On Debian you need to install libxapian22-dbg.\r
 118 \r
 119 Regards,\r
 120   Dmitry\r
 121 \r
 122 > Sebastian\r
 123 > _______________________________________________\r
 124 > notmuch mailing list\r
 125 > notmuch@notmuchmail.org\r
 126 > http://notmuchmail.org/mailman/listinfo/notmuch\r