diff options
| author | Mike Frysinger | 2009-12-09 12:55:19 +0100 |
|---|---|---|
| committer | Michal Marek | 2009-12-12 13:08:17 +0100 |
| commit | 6d87fea4dd7152df4a4605a3846c3bf10f869e0c (patch) | |
| tree | b96803c875770e6e20b1e5035a338c1a61f32684 /usr | |
| parent | kbuild: fix make clean after mismerge (diff) | |
| download | kernel-qcow2-linux-6d87fea4dd7152df4a4605a3846c3bf10f869e0c.tar.gz kernel-qcow2-linux-6d87fea4dd7152df4a4605a3846c3bf10f869e0c.tar.xz kernel-qcow2-linux-6d87fea4dd7152df4a4605a3846c3bf10f869e0c.zip | |
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>
Diffstat (limited to 'usr')
| -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, 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(); } |