diff options
author | Tony Ambardar <itugrok@yahoo.com> | 2018-03-02 20:04:36 -0800 |
---|---|---|
committer | Jo-Philipp Wich <jo@mein.io> | 2018-11-29 11:54:46 +0100 |
commit | e42415723bbc49b1d0bd3659915bb463bae066f0 (patch) | |
tree | ff5a440105c4050b4b98a6d1d3ce3a908894edac /package | |
parent | ecf104c023b82a364e8e41b4e26312a8601cffc6 (diff) | |
download | upstream-e42415723bbc49b1d0bd3659915bb463bae066f0.tar.gz upstream-e42415723bbc49b1d0bd3659915bb463bae066f0.tar.bz2 upstream-e42415723bbc49b1d0bd3659915bb463bae066f0.zip |
base-files: fix prerm return value, align with postinst code
The return value of a package prerm script is discarded and not returned
correctly by default_prerm(). This allows other operations like service
shutdown to "leak" their return value, prompting workarounds like commit
48cfc826 which do not address the root cause.
Preserve a package prerm script return value for use by default_prerm(),
sharing the corresponding code from default_postinst() for consistency.
Also use consistent code for handling of /etc/init.d/ scripts.
Run Tested on: LEDE 17.01.4 running ar71xx.
Signed-off-by: Tony Ambardar <itugrok@yahoo.com>
(cherry picked from commit 8806da86f5da3b1b1e4d24259d168e2219c01a26)
Diffstat (limited to 'package')
-rwxr-xr-x | package/base-files/files/lib/functions.sh | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/package/base-files/files/lib/functions.sh b/package/base-files/files/lib/functions.sh index c69feb33b8..51ca8d5a0c 100755 --- a/package/base-files/files/lib/functions.sh +++ b/package/base-files/files/lib/functions.sh @@ -165,22 +165,27 @@ insert_modules() { default_prerm() { local root="${IPKG_INSTROOT}" - local name + local pkgname="$(basename ${1%.*})" + local ret=0 - name=$(basename ${1%.*}) - [ -f "$root/usr/lib/opkg/info/${name}.prerm-pkg" ] && . "$root/usr/lib/opkg/info/${name}.prerm-pkg" + if [ -f "$root/usr/lib/opkg/info/${pkgname}.prerm-pkg" ]; then + ( . "$root/usr/lib/opkg/info/${pkgname}.prerm-pkg" ) + ret=$? + fi local shell="$(which bash)" - for i in `cat "$root/usr/lib/opkg/info/${name}.list" | grep "^/etc/init.d/"`; do + for i in $(grep -s "^/etc/init.d/" "$root/usr/lib/opkg/info/${pkgname}.list"); do if [ -n "$root" ]; then ${shell:-/bin/sh} "$root/etc/rc.common" "$root$i" disable else if [ "$PKG_UPGRADE" != "1" ]; then "$i" disable fi - "$i" stop || /bin/true + "$i" stop fi done + + return $ret } add_group_and_user() { |