53/a76e90ca965043427c104c4ab4bc473124b844

   1 Return-Path: <bremner@unb.ca>\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 0F8CB429E26\r
   6         for <notmuch@notmuchmail.org>; Tue, 27 Sep 2011 09:59:09 -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 yYhkVkB6-Onn for <notmuch@notmuchmail.org>;\r
  16         Tue, 27 Sep 2011 09:59:08 -0700 (PDT)\r
  17 Received: from tempo.its.unb.ca (tempo.its.unb.ca [131.202.1.21])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 54345431FD0\r
  21         for <notmuch@notmuchmail.org>; Tue, 27 Sep 2011 09:59:08 -0700 (PDT)\r
  22 Received: from convex-new.cs.unb.ca ([131.202.13.154])\r
  23         by tempo.its.unb.ca (8.13.8/8.13.8) with ESMTP id p8RGx2If006103;\r
  24         Tue, 27 Sep 2011 13:59:02 -0300\r
  25 Received: from bremner by convex-new.cs.unb.ca with local (Exim 4.72)\r
  26         (envelope-from <bremner@unb.ca>)\r
  27         id 1R8azm-0001G5-Cx; Tue, 27 Sep 2011 13:59:02 -0300\r
  28 From: David Bremner <david@tethera.net>\r
  29 To: Ali Polatel <polatel@gmail.com>, notmuch@notmuchmail.org\r
  30 Subject: Re: Concerns regarding some library functions\r
  31 In-Reply-To: <871uv2unfd.fsf@gmail.com>\r
  32 References: <871uv2unfd.fsf@gmail.com>\r
  33 User-Agent: Notmuch/0.6db1 (http://notmuchmail.org) Emacs/23.3.1\r
  34         (x86_64-pc-linux-gnu)\r
  35 Date: Tue, 27 Sep 2011 13:59:02 -0300\r
  36 Message-ID: <87fwjhx6p5.fsf@convex-new.cs.unb.ca>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain; charset=us-ascii\r
  39 Cc: Austin Clements <amdragon@mit.edu>\r
  40 X-BeenThere: notmuch@notmuchmail.org\r
  41 X-Mailman-Version: 2.1.13\r
  42 Precedence: list\r
  43 List-Id: "Use and development of the notmuch mail system."\r
  44         <notmuch.notmuchmail.org>\r
  45 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  46         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  47 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  48 List-Post: <mailto:notmuch@notmuchmail.org>\r
  49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  50 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  51         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  52 X-List-Received-Date: Tue, 27 Sep 2011 16:59:09 -0000\r
  53 \r
  54 On Tue, 27 Sep 2011 16:25:58 +0300, Ali Polatel <polatel@gmail.com> wrote:\r
  55 \r
  56 > The problem with their design is NULL return may both mean an error\r
  57 > condition and "message not found". However, we already have a similar\r
  58 > function which does not have such a flaw, namely notmuch_database_add_message().\r
  59 \r
  60 So, I take there is no way to distinguish those two outcomes? That does\r
  61 sound bad. Looking at the code for notmuch-new, it looks like the return\r
  62 value of notmuch_database_find_message_by_filename is used without\r
  63 checking it for NULL.  Austin, can you comment on that at all?\r
  64 \r
  65 > I am not providing a patch here considering the simplicity of the\r
  66 > problem but if anyone needs elaboration, I will be happy to submit a\r
  67 > patch.\r
  68 \r
  69 Well, also all the places that call these functions in the library and\r
  70 command line client would need to be modified, as well as the go and\r
  71 python bindings.  So it isn't completely trivial. Nor is is terribly\r
  72 difficult of course.\r
  73 \r
  74 d\r
  75 \r
  76 \r