diff options
author | Mike Frysinger <vapier@gentoo.org> | 2009-12-09 06:55:19 -0500 |
---|---|---|
committer | Michal Marek <mmarek@suse.cz> | 2009-12-12 07:08:17 -0500 |
commit | 6d87fea4dd7152df4a4605a3846c3bf10f869e0c (patch) | |
tree | b96803c875770e6e20b1e5035a338c1a61f32684 | |
parent | d9bdcc72ecf055f97ed736a6c0309bd41baf10a7 (diff) |
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.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/usr/gen_init_cpio.c b/usr/gen_init_cpio.c index 83b3dde1a83b..b2b3c2d1cf8b 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, | |||
354 | push_pad(); | 354 | push_pad(); |
355 | 355 | ||
356 | if (size) { | 356 | if (size) { |
357 | fwrite(filebuf, size, 1, stdout); | 357 | if (fwrite(filebuf, size, 1, stdout) != 1) { |
358 | fprintf(stderr, "writing filebuf failed\n"); | ||
359 | goto error; | ||
360 | } | ||
358 | offset += size; | 361 | offset += size; |
359 | push_pad(); | 362 | push_pad(); |
360 | } | 363 | } |