summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2009-12-09 06:55:19 -0500
committerMichal Marek <mmarek@suse.cz>2009-12-12 13:08:17 +0100
commit6d87fea4dd7152df4a4605a3846c3bf10f869e0c (patch)
treeb96803c875770e6e20b1e5035a338c1a61f32684
parentd9bdcc72ecf055f97ed736a6c0309bd41baf10a7 (diff)
downloadop-kernel-dev-6d87fea4dd7152df4a4605a3846c3bf10f869e0c.zip
op-kernel-dev-6d87fea4dd7152df4a4605a3846c3bf10f869e0c.tar.gz
gen_init_cpio: fixed fwrite warning
On compilers with security warnings enabled by default, we get: usr/gen_init_cpio.c: In function ‘cpio_mkfile’: usr/gen_init_cpio.c:357: warning: ignoring return value of ‘fwrite’, declared with attribute warn_unused_result So check the return value and handle errors accordingly. Signed-off-by: Mike Frysinger <vapier@gentoo.org> Signed-off-by: Michal Marek <mmarek@suse.cz>
-rw-r--r--usr/gen_init_cpio.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/usr/gen_init_cpio.c b/usr/gen_init_cpio.c
index 83b3dde..b2b3c2d 100644
--- a/usr/gen_init_cpio.c
+++ b/usr/gen_init_cpio.c
@@ -354,7 +354,10 @@ static int cpio_mkfile(const char *name, const char *location,
push_pad();
if (size) {
- fwrite(filebuf, size, 1, stdout);
+ if (fwrite(filebuf, size, 1, stdout) != 1) {
+ fprintf(stderr, "writing filebuf failed\n");
+ goto error;
+ }
offset += size;
push_pad();
}
OpenPOWER on IntegriCloud