diff options
| author | John Crispin <john@openwrt.org> | 2014-06-02 12:43:22 +0000 |
|---|---|---|
| committer | John Crispin <john@openwrt.org> | 2014-06-02 12:43:22 +0000 |
| commit | f933a741a9c44cea58b2b3837dca99fc55d0b957 (patch) | |
| tree | b87ca90d9e7b0432bfda4c69a8fef02d08f8e748 /target/linux/ar71xx/base-files | |
| parent | a866b7eb9d45ecb986c275cbd2033322b54d2606 (diff) | |
| download | mtk-20170518-f933a741a9c44cea58b2b3837dca99fc55d0b957.zip mtk-20170518-f933a741a9c44cea58b2b3837dca99fc55d0b957.tar.gz mtk-20170518-f933a741a9c44cea58b2b3837dca99fc55d0b957.tar.bz2 | |
target/linux/*/base-files/lib/upgrade/platform.sh - wrong check for ARGC
ARGC is a 'C-ism', but not known/valid in shell-syntax - insert the correct
var $# (=number of args) here. under normal conditions this had no impact,
but we should at least correct it. the error was observable like this:
root@box:~ [ -e "/etc/functions.sh" ] && . /etc/functions.sh
root@box:~ [ -e "/lib/functions.sh" ] && . /lib/functions.sh
root@box:~ . /lib/upgrade/platform.sh
root@box:~ . /lib/upgrade/common.sh
root@box:~ platform_check_image /tmp/myfirmware.bin
ash: bad number
root@box:~ echo $?
0
Signed-off-by: Bastian Bittorf <bittorf@bluebottle.com>
SVN-Revision: 40915
Diffstat (limited to 'target/linux/ar71xx/base-files')
| -rwxr-xr-x | target/linux/ar71xx/base-files/lib/upgrade/platform.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/target/linux/ar71xx/base-files/lib/upgrade/platform.sh b/target/linux/ar71xx/base-files/lib/upgrade/platform.sh index 37ceea6..dfec133 100755 --- a/target/linux/ar71xx/base-files/lib/upgrade/platform.sh +++ b/target/linux/ar71xx/base-files/lib/upgrade/platform.sh @@ -111,7 +111,7 @@ platform_check_image() { local magic="$(get_magic_word "$1")" local magic_long="$(get_magic_long "$1")" - [ "$ARGC" -gt 1 ] && return 1 + [ "$#" -gt 1 ] && return 1 case "$board" in all0315n | \ |