diff options
author | asomers <asomers@FreeBSD.org> | 2015-11-06 19:18:20 +0000 |
---|---|---|
committer | asomers <asomers@FreeBSD.org> | 2015-11-06 19:18:20 +0000 |
commit | d527e5dce9338b04b72c7e909cfda6e7180b393d (patch) | |
tree | 921d8184ea24feb126acc7ce166c5c4be6c581ea /sbin/savecore/savecore.c | |
parent | bb5898dcfd422af78f1385b70c6581d7e1d7ca69 (diff) | |
download | FreeBSD-src-d527e5dce9338b04b72c7e909cfda6e7180b393d.zip FreeBSD-src-d527e5dce9338b04b72c7e909cfda6e7180b393d.tar.gz |
Always check the return value of lseek.
This is a follow-up to r289845, which only fixed one occurence of CID
1009429.
Coverity CID: 1009429
Reviewed by: markj
MFC after: 2 weeks
X-MFC-With: r289845
Sponsored by: Spectra Logic
Differential Revision: https://reviews.freebsd.org/D4096
Diffstat (limited to 'sbin/savecore/savecore.c')
-rw-r--r-- | sbin/savecore/savecore.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/sbin/savecore/savecore.c b/sbin/savecore/savecore.c index b87b95b..0c1e806 100644 --- a/sbin/savecore/savecore.c +++ b/sbin/savecore/savecore.c @@ -491,9 +491,8 @@ DoFile(const char *savedir, const char *device) } lasthd = mediasize - sectorsize; - lseek(fd, lasthd, SEEK_SET); - error = read(fd, &kdhl, sizeof kdhl); - if (error != sizeof kdhl) { + if (lseek(fd, lasthd, SEEK_SET) != lasthd || + read(fd, &kdhl, sizeof(kdhl)) != sizeof(kdhl)) { syslog(LOG_ERR, "error reading last dump header at offset %lld in %s: %m", (long long)lasthd, device); @@ -569,9 +568,8 @@ DoFile(const char *savedir, const char *device) } dumpsize = dtoh64(kdhl.dumplength); firsthd = lasthd - dumpsize - sizeof kdhf; - lseek(fd, firsthd, SEEK_SET); - error = read(fd, &kdhf, sizeof kdhf); - if (error != sizeof kdhf) { + if (lseek(fd, firsthd, SEEK_SET) != firsthd || + read(fd, &kdhf, sizeof(kdhf)) != sizeof(kdhf)) { syslog(LOG_ERR, "error reading first dump header at offset %lld in %s: %m", (long long)firsthd, device); |