43/1785e34c4c0d445994d5b21119aed833f9dd20

   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 514DE431FC0\r
   6         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 10:01:15 -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 E2Nvsrwx8n2L for <notmuch@notmuchmail.org>;\r
  16         Thu, 19 Jan 2012 10:01:13 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-7.mit.edu (DMZ-MAILSEC-SCANNER-7.MIT.EDU\r
  18         [18.7.68.36])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 2A1F8431FC2\r
  20         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 10:01:09 -0800 (PST)\r
  21 X-AuditID: 12074424-b7fae6d000000906-1e-4f185a648e6b\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-7.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 21.E1.02310.46A581F4; Thu, 19 Jan 2012 13:01:08 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id q0JI17rL013123; \r
  27         Thu, 19 Jan 2012 13:01:07 -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 q0JI16l4016831\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Thu, 19 Jan 2012 13:01:07 -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 1RnwIM-0007gm-Oc; Thu, 19 Jan 2012 13:01:06 -0500\r
  37 From: Austin Clements <amdragon@MIT.EDU>\r
  38 To: notmuch@notmuchmail.org\r
  39 Subject: [PATCH v2 1/2] show: Handle read and write errors\r
  40 Date: Thu, 19 Jan 2012 13:01:04 -0500\r
  41 Message-Id: <1326996065-29467-1-git-send-email-amdragon@mit.edu>\r
  42 X-Mailer: git-send-email 1.7.7.3\r
  43 In-Reply-To: <1326933611-16898-1-git-send-email-amdragon@mit.edu>\r
  44 References: <1326933611-16898-1-git-send-email-amdragon@mit.edu>\r
  45 X-Brightmail-Tracker:\r
  46  H4sIAAAAAAAAA+NgFtrPIsWRmVeSWpSXmKPExsUixCmqrZsSJeFvMPGxpsX1mzOZHRg9nq26\r
  47         xRzAGMVlk5Kak1mWWqRvl8CVcX3JVPaCQ5wVXZsmMDcwbmbvYuTkkBAwkfi4uYURwhaTuHBv\r
  48         PVsXIxeHkMA+RomPf5qZIZwNjBJ/jh+Cyhxnkpg2uZcdwpnPKLF8zkywfjYBDYlt+5eD2SIC\r
  49         0hI7785m7WLk4GAWUJP406UCEhYWsJLYve8IE4jNIqAqcfniNTCbV8BB4trBNVAnKUicW30O\r
  50         zOYUcJS4+uUoWI0QUM2GSytYJjDyL2BkWMUom5JbpZubmJlTnJqsW5ycmJeXWqRrrpebWaKX\r
  51         mlK6iREUNuwuKjsYmw8pHWIU4GBU4uHlcpXwF2JNLCuuzD3EKMnBpCTK6xkJFOJLyk+pzEgs\r
  52         zogvKs1JLT7EKMHBrCTCe1wMKMebklhZlVqUD5OS5mBREufV0HrnJySQnliSmp2aWpBaBJOV\r
  53         4eBQkuCdADJUsCg1PbUiLTOnBCHNxMEJMpwHaHg0SA1vcUFibnFmOkT+FKMux5ffbecZhVjy\r
  54         8vNSpcR560GKBECKMkrz4ObA4v0VozjQW8K8HSBVPMBUATfpFdASJqAlHk1iIEtKEhFSUg2M\r
  55         rarcOmrzepuFWFZL2iULXUm6L/6r5moQy69JC3i2b2yU3bt9y8nbApvfKZf0rizJPPxP1d7G\r
  56         eJp66lK3/RJvP/U8f3DXYPX8TVwMavseFxg/nHNfNfpA3UEz97yP+/mcdVquG+pu3VE330qI\r
  57         iXmroy3/yULNgJw6Z7UGEZb412HzGf7VbrFTYinOSDTUYi4qTgQAj4x1h9ICAAA=\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: Thu, 19 Jan 2012 18:01:15 -0000\r
  71 \r
  72 For showing a message in raw format, rather than silently succeeding\r
  73 when a read or a write fails (or, probably, looping if a read fails),\r
  74 try to print an error message and exit with a non-zero status.\r
  75 \r
  76 This silences one of the buildbot warnings about unused resuls.  While\r
  77 my libc lacks the declarations that trigger these warnings, this can\r
  78 be tested by adding the following to notmuch.h:\r
  79 \r
  80 __attribute__((warn_unused_result))\r
  81 size_t fwrite(const void *ptr, size_t size, size_t nmemb, FILE *stream);\r
  82 ---\r
  83  notmuch-show.c |   12 +++++++++++-\r
  84  1 files changed, 11 insertions(+), 1 deletions(-)\r
  85 \r
  86 diff --git a/notmuch-show.c b/notmuch-show.c\r
  87 index d14dac9..c674e25 100644\r
  88 --- a/notmuch-show.c\r
  89 +++ b/notmuch-show.c\r
  90 @@ -883,7 +883,17 @@ do_show_single (void *ctx,\r
  91  \r
  92         while (!feof (file)) {\r
  93             size = fread (buf, 1, sizeof (buf), file);\r
  94 -           (void) fwrite (buf, size, 1, stdout);\r
  95 +           if (ferror (file)) {\r
  96 +               fprintf (stderr, "Error: Read failed from %s\n", filename);\r
  97 +               fclose (file);\r
  98 +               return 1;\r
  99 +           }\r
 100 +\r
 101 +           if (fwrite (buf, size, 1, stdout) != 1) {\r
 102 +               fprintf (stderr, "Error: Write failed\n");\r
 103 +               fclose (file);\r
 104 +               return 1;\r
 105 +           }\r
 106         }\r
 107  \r
 108         fclose (file);\r
 109 -- \r
 110 1.7.7.3\r
 111 \r