diff options
author | Tom Rini <trini@konsulko.com> | 2015-12-07 16:26:08 -0500 |
---|---|---|
committer | Michal Marek <mmarek@suse.com> | 2016-02-17 22:52:04 +0100 |
commit | 46fe94ad18aa7ce6b3dad8c035fb538942020f2b (patch) | |
tree | ef7acb440291f285efdb6dc910aec8c1abf1db70 /scripts/basic | |
parent | a043934207c5eb271deeaed2e9bd019c3be92cad (diff) | |
download | op-kernel-dev-46fe94ad18aa7ce6b3dad8c035fb538942020f2b.zip op-kernel-dev-46fe94ad18aa7ce6b3dad8c035fb538942020f2b.tar.gz |
kbuild: fixdep: Check fstat(2) return value
Coverity has recently added a check that will find when we don't check
the return code from fstat(2). Copy/paste the checking logic that
print_deps() has with an appropriate re-wording of the perror() message.
Signed-off-by: Tom Rini <trini@konsulko.com>
Signed-off-by: Michal Marek <mmarek@suse.com>
Diffstat (limited to 'scripts/basic')
-rw-r--r-- | scripts/basic/fixdep.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/scripts/basic/fixdep.c b/scripts/basic/fixdep.c index 5b327c6..caef815 100644 --- a/scripts/basic/fixdep.c +++ b/scripts/basic/fixdep.c @@ -274,7 +274,11 @@ static void do_config_file(const char *filename) perror(filename); exit(2); } - fstat(fd, &st); + if (fstat(fd, &st) < 0) { + fprintf(stderr, "fixdep: error fstat'ing config file: "); + perror(filename); + exit(2); + } if (st.st_size == 0) { close(fd); return; |