Always check return code from iod_Write
authorMarc Dionne <marc.c.dionne@gmail.com>
Sat, 11 Sep 2010 14:46:50 +0000 (10:46 -0400)
committerDerrick Brashear <shadow@dementia.org>
Tue, 14 Sep 2010 08:00:53 +0000 (01:00 -0700)
The return code from iod_Write is checked at every call site
in the file, except this one.  Check it, and return VOLSERDUMPERROR
if appropriate.

Spotted by a set but unused warning from gcc 4.6

Change-Id: I84f38a4b3b1e37c25be9c76702b0d2818058454e
Reviewed-on: http://gerrit.openafs.org/2733
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

src/volser/dumpstuff.c

index f8b7fa1..845d642 100644 (file)
@@ -664,6 +664,8 @@ DumpStandardTag(struct iod *iodp, char tag, afs_uint32 section)
         return VOLSERDUMPERROR;
     }
     code = iod_Write(iodp, &tag, 1);
+    if (code != 1)
+       return VOLSERDUMPERROR;
     return 0;
 }