diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2009-12-09 06:55:19 -0500 |
|---|---|---|
| committer | Michal Marek <mmarek@suse.cz> | 2009-12-12 13:08:17 +0100 |
| commit | 6d87fea4dd7152df4a4605a3846c3bf10f869e0c (patch) | |
| tree | b96803c875770e6e20b1e5035a338c1a61f32684 /usr/gen_init_cpio.c | |
| parent | d9bdcc72ecf055f97ed736a6c0309bd41baf10a7 (diff) | |
| download | lwn-6d87fea4dd7152df4a4605a3846c3bf10f869e0c.tar.gz lwn-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/gen_init_cpio.c')
| -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(); } |