08/1d9b27b610776efcf8d628cba9a6b631511c56

   1 Return-Path: <amthrax@drake.mit.edu>\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 0DB9C431FBF\r
   6         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 14:29:27 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 PeNKLJBp8WAk for <notmuch@notmuchmail.org>;\r
  16         Thu, 19 Jan 2012 14:29:26 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-3.mit.edu (DMZ-MAILSEC-SCANNER-3.MIT.EDU\r
  18         [18.9.25.14])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 75715431FAE\r
  20         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 14:29:26 -0800 (PST)\r
  21 X-AuditID: 1209190e-b7f7c6d0000008c3-57-4f1899454232\r
  22 Received: from mailhub-auth-1.mit.edu ( [18.9.21.35])\r
  23         by dmz-mailsec-scanner-3.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 1F.5B.02243.549981F4; Thu, 19 Jan 2012 17:29:25 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-1.mit.edu (8.13.8/8.9.2) with ESMTP id q0JMTPwm010018; \r
  27         Thu, 19 Jan 2012 17:29:25 -0500\r
  28 Received: from drake.mit.edu (26-4-166.dynamic.csail.mit.edu [18.26.4.166])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q0JMTOcc010686\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Thu, 19 Jan 2012 17:29:25 -0500 (EST)\r
  34 Received: from amthrax by drake.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amthrax@drake.mit.edu>)\r
  36         id 1Ro0U0-000526-CZ; Thu, 19 Jan 2012 17:29:24 -0500\r
  37 From: Austin Clements <amdragon@MIT.EDU>\r
  38 To: notmuch@notmuchmail.org\r
  39 Subject: [PATCH v3 1/2] show: Handle read and write errors\r
  40 Date: Thu, 19 Jan 2012 17:29:18 -0500\r
  41 Message-Id: <1327012159-19242-2-git-send-email-amdragon@mit.edu>\r
  42 X-Mailer: git-send-email 1.7.7.3\r
  43 In-Reply-To: <1327012159-19242-1-git-send-email-amdragon@mit.edu>\r
  44 References: <1326996065-29467-1-git-send-email-amdragon@mit.edu>\r
  45         <1327012159-19242-1-git-send-email-amdragon@mit.edu>\r
  46 X-Brightmail-Tracker:\r
  47  H4sIAAAAAAAAA+NgFjrBIsWRmVeSWpSXmKPExsUixCmqrOs6U8LfYPplG4t9d7YwWVy/OZPZ\r
  48         gclj1/O/TB7PVt1iDmCK4rJJSc3JLEst0rdL4Mq4vmQqe8EhzoquTROYGxg3s3cxcnJICJhI\r
  49         3FrzkRHCFpO4cG89WxcjF4eQwD5GiU1nWlhBEkICGxglbnXWQCSOM0mc//GBCcKZzyjR8uIU\r
  50         WDubgIbEtv3LwWwRAWmJnXdng3UzCxhL/J0+H8jm4BAWsJKY+qYAJMwioCqxon0CG4jNK+Ag\r
  51         8X7aGTaIKxQkzq0+B3Ydp4CjxKwJe9kgjiiXuD79D8sERv4FjAyrGGVTcqt0cxMzc4pTk3WL\r
  52         kxPz8lKLdI31cjNL9FJTSjcxggNJkm8H49eDSocYBTgYlXh4OV0l/IVYE8uKK3MPMUpyMCmJ\r
  53         8m6bDhTiS8pPqcxILM6ILyrNSS0+xCjBwawkwtvQB5TjTUmsrEotyodJSXOwKInzqmm98xMS\r
  54         SE8sSc1OTS1ILYLJynBwKEnwLp8B1ChYlJqeWpGWmVOCkGbi4AQZzgM0fCFIDW9xQWJucWY6\r
  55         RP4Uoy7Hl99t5xmFWPLy81KlxHmrQIoEQIoySvPg5sASwCtGcaC3hHkXgVTxAJMH3KRXQEuY\r
  56         gJZ4NImBLClJREhJNTB2H6tbpyGws3a7UdXE2/rLLJ82ZQgvOSOm3zVXZoVoz+73AqfrWlMq\r
  57         eG5PuLNf/NTOgl2Z+tJWE5yUuc8nfJh/bKPEVe0rBbKWP6LeNl6catdjqVQlIOm1jqcuP1Kn\r
  58         ITrmRNjRHd8nznJ+KmcoYSbJd7/5dEJ3aEug6oHOZSW/PqSnztgb6aXEUpyRaKjFXFScCACx\r
  59         pvne2wIAAA==\r
  60 X-BeenThere: notmuch@notmuchmail.org\r
  61 X-Mailman-Version: 2.1.13\r
  62 Precedence: list\r
  63 List-Id: "Use and development of the notmuch mail system."\r
  64         <notmuch.notmuchmail.org>\r
  65 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  67 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  68 List-Post: <mailto:notmuch@notmuchmail.org>\r
  69 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  70 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  71         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  72 X-List-Received-Date: Thu, 19 Jan 2012 22:29:27 -0000\r
  73 \r
  74 For showing a message in raw format, rather than silently succeeding\r
  75 when a read or a write fails (or, probably, looping if a read fails),\r
  76 try to print an error message and exit with a non-zero status.\r
  77 \r
  78 This silences one of the buildbot warnings about unused resuls.  While\r
  79 my libc lacks the declarations that trigger these warnings, this can\r
  80 be tested by adding the following to notmuch.h:\r
  81 \r
  82 __attribute__((warn_unused_result))\r
  83 size_t fwrite(const void *ptr, size_t size, size_t nmemb, FILE *stream);\r
  84 ---\r
  85  notmuch-show.c |   12 +++++++++++-\r
  86  1 files changed, 11 insertions(+), 1 deletions(-)\r
  87 \r
  88 diff --git a/notmuch-show.c b/notmuch-show.c\r
  89 index d14dac9..c674e25 100644\r
  90 --- a/notmuch-show.c\r
  91 +++ b/notmuch-show.c\r
  92 @@ -883,7 +883,17 @@ do_show_single (void *ctx,\r
  93  \r
  94         while (!feof (file)) {\r
  95             size = fread (buf, 1, sizeof (buf), file);\r
  96 -           (void) fwrite (buf, size, 1, stdout);\r
  97 +           if (ferror (file)) {\r
  98 +               fprintf (stderr, "Error: Read failed from %s\n", filename);\r
  99 +               fclose (file);\r
 100 +               return 1;\r
 101 +           }\r
 102 +\r
 103 +           if (fwrite (buf, size, 1, stdout) != 1) {\r
 104 +               fprintf (stderr, "Error: Write failed\n");\r
 105 +               fclose (file);\r
 106 +               return 1;\r
 107 +           }\r
 108         }\r
 109  \r
 110         fclose (file);\r
 111 -- \r
 112 1.7.7.3\r
 113 \r