domenico/openwrt_NSS_ACW
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bcm4908: fix calculation of new cferam index

Browse Source 
The arithmetic expansion fails when idx becomes a two digit number.
Fix this by relying on expr command.

root@OpenWrt:/# echo $(((028 + 0) % 1000))
/bin/ash: arithmetic syntax error
root@OpenWrt:/# echo $(($(expr 028 + 0) % 1000))
28

Fixes: a6a0b252ba ("bcm4908: add sysupgrade support")
Signed-off-by: Sungbo Eo <mans0n@gorani.run>
(cherry picked from commit f432353850)
This commit is contained in:
Sungbo Eo
2021-08-01 22:17:04 +09:00
committed by Rafał Miłecki
parent b4c40a7efc
commit 4607f55439
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
target/linux/bcm4908/base-files/lib/upgrade/platform.sh
View File

@@ -1,6 +1,6 @@
# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause # SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause
RAMFS_COPY_BIN="bcm4908img" RAMFS_COPY_BIN="bcm4908img expr"
PART_NAME=firmware PART_NAME=firmware
@@ -129,7 +129,7 @@ platform_calc_new_cferam() {
umount $dir umount $dir
rm -fr $dir rm -fr $dir
idx=$(((idx + inc) % 1000)) idx=$(($(expr $idx + $inc) % 1000))
echo $(printf "cferam.%03d" $idx) echo $(printf "cferam.%03d" $idx)
} }