summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBen Gamari <bgamari.foss@gmail.com>2010-12-22 13:30:14 -0500
committerMichal Marek <mmarek@suse.cz>2010-12-22 23:23:28 +0100
commita3ba81131aca243bfecfa78c42edec0cd69f72d6 (patch)
treea0123e86341b51ebef2f4e42046e523330cf7004
parent6e5f6856427abe5418f535cb46c454ae8ea7f8e7 (diff)
downloadlinux-a3ba81131aca243bfecfa78c42edec0cd69f72d6.tar.bz2
Make fixdep error handling more explicit
Also add missing error handling to fstat call Signed-off-by: Ben Gamari <bgamari.foss@gmail.com> Signed-off-by: Michal Marek <mmarek@suse.cz>
-rw-r--r--scripts/basic/fixdep.c10
1 files changed, 7 insertions, 3 deletions
diff --git a/scripts/basic/fixdep.c b/scripts/basic/fixdep.c
index ed0584623690..c9a16abacab4 100644
--- a/scripts/basic/fixdep.c
+++ b/scripts/basic/fixdep.c
@@ -286,7 +286,7 @@ static void do_config_file(const char *filename)
fd = open(filename, O_RDONLY);
if (fd < 0) {
- fprintf(stderr, "fixdep: ");
+ fprintf(stderr, "fixdep: error opening config file: ");
perror(filename);
exit(2);
}
@@ -357,11 +357,15 @@ static void print_deps(void)
fd = open(depfile, O_RDONLY);
if (fd < 0) {
- fprintf(stderr, "fixdep: ");
+ fprintf(stderr, "fixdep: error opening depfile: ");
perror(depfile);
exit(2);
}
- fstat(fd, &st);
+ if (fstat(fd, &st) < 0) {
+ fprintf(stderr, "fixdep: error fstat'ing depfile: ");
+ perror(depfile);
+ exit(2);
+ }
if (st.st_size == 0) {
fprintf(stderr,"fixdep: %s is empty\n",depfile);
close(fd);