diff options
author | Neil Brown <neilb@suse.de> | 2006-05-29 02:06:32 +0000 |
---|---|---|
committer | Neil Brown <neilb@suse.de> | 2006-05-29 02:06:32 +0000 |
commit | 9fca7d6236e9775d0269b9802f740c08db46f4d7 (patch) | |
tree | aef0c99d844a3eacf037294e6722079c01c44ae9 /super1.c | |
parent | 280a927d3d8da258a70df13df4f780d823fc1da2 (diff) | |
download | mdadm-9fca7d6236e9775d0269b9802f740c08db46f4d7.tar.gz mdadm-9fca7d6236e9775d0269b9802f740c08db46f4d7.tar.xz mdadm-9fca7d6236e9775d0269b9802f740c08db46f4d7.zip |
check return status of all write/fwrite functions as required by glibc 2.4
From: Luca Berra <bluca@vodka.it>
glibc 2.4 is pedantic on ignoring return values from fprintf, fwrite and
write, so now we check the rval and actually do something with it.
in the Grow.c case i only print a warning, since i don't think we can do
anything in case we fail invalidating those superblocks (is should never
happen, but then...)
Signed-off-by: Neil Brown <neilb@suse.de>
Diffstat (limited to 'super1.c')
-rw-r--r-- | super1.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -715,7 +715,8 @@ static int store_super1(struct supertype *st, int fd, void *sbv) (((char*)sb)+1024); if (__le32_to_cpu(bm->magic) == BITMAP_MAGIC) { locate_bitmap1(st, fd, sbv); - write(fd, bm, sizeof(*bm)); + if (write(fd, bm, sizeof(*bm)) != sizeof(*bm)) + return 5; } } fsync(fd); |