13/4ce2f1f8c890438ec6cd35f0ec50198416b76f

   1 Return-Path: <bremner@tethera.net>\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 ECC05431FAF\r
   6         for <notmuch@notmuchmail.org>; Sat, 19 Jan 2013 10:26:42 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 G9j2Wa7zgQxL for <notmuch@notmuchmail.org>;\r
  16         Sat, 19 Jan 2013 10:26:42 -0800 (PST)\r
  17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id C3EC4431FD2\r
  21         for <notmuch@notmuchmail.org>; Sat, 19 Jan 2013 10:26:11 -0800 (PST)\r
  22 Received: from fctnnbsc30w-156034082078.dhcp-dynamic.fibreop.nb.bellaliant.net\r
  23         ([156.34.82.78] helo=zancas.localnet)\r
  24         by tesseract.cs.unb.ca with esmtpsa\r
  25         (TLS1.2:DHE_RSA_AES_128_CBC_SHA1:128) (Exim 4.80)\r
  26         (envelope-from <bremner@tethera.net>)\r
  27         id 1Twd7J-0007xN-L1; Sat, 19 Jan 2013 14:26:09 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.80)\r
  29         (envelope-from <bremner@tethera.net>)\r
  30         id 1Twd7E-0005b4-28; Sat, 19 Jan 2013 14:26:04 -0400\r
  31 From: david@tethera.net\r
  32 To: notmuch@notmuchmail.org\r
  33 Subject: [PATCH 7/7] CLI: add simple error handling for talloc logging\r
  34 Date: Sat, 19 Jan 2013 14:25:58 -0400\r
  35 Message-Id: <1358619958-21209-8-git-send-email-david@tethera.net>\r
  36 X-Mailer: git-send-email 1.7.10.4\r
  37 In-Reply-To: <1358619958-21209-1-git-send-email-david@tethera.net>\r
  38 References: <1358619958-21209-1-git-send-email-david@tethera.net>\r
  39 X-Spam_bar: -\r
  40 Cc: David Bremner <bremner@debian.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: Sat, 19 Jan 2013 18:26:43 -0000\r
  54 \r
  55 From: David Bremner <bremner@debian.org>\r
  56 \r
  57 This really should have been there before. I think it's better to do\r
  58 the actual operation and then possibly fail writing the memory log,\r
  59 but it would not be too hard to change it to abort earlier.\r
  60 ---\r
  61  notmuch.c |    7 ++++++-\r
  62  1 file changed, 6 insertions(+), 1 deletion(-)\r
  63 \r
  64 diff --git a/notmuch.c b/notmuch.c\r
  65 index f8d4b35..c3336e8 100644\r
  66 --- a/notmuch.c\r
  67 +++ b/notmuch.c\r
  68 @@ -301,7 +301,12 @@ main (int argc, char *argv[])\r
  69  \r
  70             if (leak_report && (strcmp (leak_report, "") != 0)) {\r
  71                 FILE *report = fopen (leak_report, "w");\r
  72 -               talloc_report_full (NULL, report);\r
  73 +               if (report) {\r
  74 +                   talloc_report_full (NULL, report);\r
  75 +               } else {\r
  76 +                   ret = 1;\r
  77 +                   perror (leak_report);\r
  78 +               }\r
  79             }\r
  80  \r
  81             return ret;\r
  82 -- \r
  83 1.7.10.4\r
  84 \r