diff options
author | Jan Alexander Steffens (heftig) <jan.steffens@gmail.com> | 2018-05-31 17:46:51 +0200 |
---|---|---|
committer | Allan McRae <allan@archlinux.org> | 2018-06-18 05:15:51 +0200 |
commit | 05a3641c76ba1c07dbec6fa1d9e9d3fcf6dec20c (patch) | |
tree | f62dd588636c39ca68669fd5c54af41846cf0d40 /scripts/libmakepkg/tidy | |
parent | ab1e92860bdc5388621e8314b8e1652590fa2b05 (diff) | |
download | pacman-05a3641c76ba1c07dbec6fa1d9e9d3fcf6dec20c.tar.gz pacman-05a3641c76ba1c07dbec6fa1d9e9d3fcf6dec20c.tar.xz |
makepkg: Don't use parameterless return
It's especially dangerous in trap handlers since the return value of the
function becomes the return value of the last command before the trap,
not the last command in the current function. This applies to any
function executed in a trap handler, nested functions included.
In one case, install_packages failed (via return 14), which was inside a
conditional that then ran exit 14, which triggered the EXIT handler,
which called clean_up, which called remove_deps, which had !RMDEPS and
thus returned. The return value of remove_deps became the return value
of install_packages, triggering the ERR handler, which (due to another
problem) was still the user function handler, which then printed a
misleading error message and overrode the exit code with 4.
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'scripts/libmakepkg/tidy')
0 files changed, 0 insertions, 0 deletions