diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2010-11-24 01:54:02 +0100 |
|---|---|---|
| committer | Michal Marek <mmarek@suse.cz> | 2010-12-14 15:06:04 +0100 |
| commit | 2979076fbf17a0947d6eba367b0cac19c907c160 (patch) | |
| tree | a6491d3867e0b953453dee28ffca79da06deb0df | |
| parent | gen_init_cpio: remove leading `/' from file names (diff) | |
| download | linux-2979076fbf17a0947d6eba367b0cac19c907c160.tar.xz linux-2979076fbf17a0947d6eba367b0cac19c907c160.zip | |
headers_install: check exit status of unifdef
If unifdef fails for any reason (like segfaulting), we should be aborting
the install steps. So check its exit status in this unlikely scenario.
Reported-by: Diego Elio Pettenò <flameeyes@gentoo.org>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Michal Marek <mmarek@suse.cz>
| -rw-r--r-- | scripts/headers_install.pl | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/scripts/headers_install.pl b/scripts/headers_install.pl index 4ca3be3b2e50..efb3be10d428 100644 --- a/scripts/headers_install.pl +++ b/scripts/headers_install.pl @@ -45,6 +45,13 @@ foreach my $file (@files) { close $in; system $unifdef . " $tmpfile > $installdir/$file"; + # unifdef will exit 0 on success, and will exit 1 when the + # file was processed successfully but no changes were made, + # so abort only when it's higher than that. + my $e = $? >> 8; + if ($e > 1) { + die "$tmpfile: $!\n"; + } unlink $tmpfile; } exit 0; |