diff options
author | Bo YU <tsu.yubo@gmail.com> | 2018-10-30 16:21:55 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-05-31 16:47:16 +0300 |
commit | de91d20c46d03c285cb0424a52f01eca31b1e2bc (patch) | |
tree | e3ff5fc85c60f2d05b47fe14b97a4691c6503fe4 /arch | |
parent | 8c1bbe99d2a81530804927cdfbdc780edac8c0a8 (diff) | |
download | linux-de91d20c46d03c285cb0424a52f01eca31b1e2bc.tar.xz |
powerpc/boot: Fix missing check of lseek() return value
[ Upstream commit 5d085ec04a000fefb5182d3b03ee46ca96d8389b ]
This is detected by Coverity scan: CID: 1440481
Signed-off-by: Bo YU <tsu.yubo@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/powerpc/boot/addnote.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/arch/powerpc/boot/addnote.c b/arch/powerpc/boot/addnote.c index 9d9f6f334d3c..3da3e2b1b51b 100644 --- a/arch/powerpc/boot/addnote.c +++ b/arch/powerpc/boot/addnote.c @@ -223,7 +223,11 @@ main(int ac, char **av) PUT_16(E_PHNUM, np + 2); /* write back */ - lseek(fd, (long) 0, SEEK_SET); + i = lseek(fd, (long) 0, SEEK_SET); + if (i < 0) { + perror("lseek"); + exit(1); + } i = write(fd, buf, n); if (i < 0) { perror("write"); |