d5/0dd7a5492270a764c7e0fd1493a67d22f452ba

   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 D049D431FDA\r
   6         for <notmuch@notmuchmail.org>; Tue, 29 Jan 2013 04:14:03 -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 RXLzKFxSVQhq for <notmuch@notmuchmail.org>;\r
  16         Tue, 29 Jan 2013 04:14:03 -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 69884431FD7\r
  21         for <notmuch@notmuchmail.org>; Tue, 29 Jan 2013 04:14:01 -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 1U0A4e-0008Lg-9I; Tue, 29 Jan 2013 08:14:00 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.80)\r
  29         (envelope-from <bremner@tethera.net>)\r
  30         id 1U0A4Y-0000UQ-OF; Tue, 29 Jan 2013 08:13:54 -0400\r
  31 From: david@tethera.net\r
  32 To: notmuch@notmuchmail.org\r
  33 Subject: [PATCH 2/2] CLI: add simple error handling for talloc logging\r
  34 Date: Tue, 29 Jan 2013 08:13:40 -0400\r
  35 Message-Id: <1359461620-1716-3-git-send-email-david@tethera.net>\r
  36 X-Mailer: git-send-email 1.7.10.4\r
  37 In-Reply-To: <1359461620-1716-1-git-send-email-david@tethera.net>\r
  38 References: <1359461620-1716-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: Tue, 29 Jan 2013 12:14:04 -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 |   12 +++++++-----\r
  62  1 file changed, 7 insertions(+), 5 deletions(-)\r
  63 \r
  64 diff --git a/notmuch.c b/notmuch.c\r
  65 index a674481..cfb009b 100644\r
  66 --- a/notmuch.c\r
  67 +++ b/notmuch.c\r
  68 @@ -294,10 +294,6 @@ main (int argc, char *argv[])\r
  69  \r
  70             ret = (command->function)(local, argc - opt_index, argv + opt_index);\r
  71  \r
  72 -           /* in the future support for this environment variable may\r
  73 -            * be supplemented or replaced by command line arguments\r
  74 -            * --leak-report and/or --leak-report-full */\r
  75 -\r
  76             talloc_report = getenv ("NOTMUCH_TALLOC_REPORT");\r
  77  \r
  78             /* this relies on the previous call to\r
  79 @@ -305,7 +301,13 @@ main (int argc, char *argv[])\r
  80  \r
  81             if (talloc_report && strcmp (talloc_report, "") != 0) {\r
  82                 FILE *report = fopen (talloc_report, "w");\r
  83 -               talloc_report_full (NULL, report);\r
  84 +               if (report) {\r
  85 +                   talloc_report_full (NULL, report);\r
  86 +               } else {\r
  87 +                   ret = 1;\r
  88 +                   fprintf (stderr, "ERROR: unable to write talloc log. ");\r
  89 +                   perror (talloc_report);\r
  90 +               }\r
  91             }\r
  92  \r
  93             return ret;\r
  94 -- \r
  95 1.7.10.4\r
  96 \r