From f4323538501d58298f4df73a034a51375a477cfc Mon Sep 17 00:00:00 2001 From: Sungbo Eo Date: Sun, 1 Aug 2021 22:17:04 +0900 Subject: [PATCH] bcm4908: fix calculation of new cferam index 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: a6a0b252baa6 ("bcm4908: add sysupgrade support") Signed-off-by: Sungbo Eo --- target/linux/bcm4908/base-files/lib/upgrade/platform.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/target/linux/bcm4908/base-files/lib/upgrade/platform.sh b/target/linux/bcm4908/base-files/lib/upgrade/platform.sh index 8aca7119e4..ee43e3a713 100644 --- a/target/linux/bcm4908/base-files/lib/upgrade/platform.sh +++ b/target/linux/bcm4908/base-files/lib/upgrade/platform.sh @@ -1,6 +1,6 @@ # SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause -RAMFS_COPY_BIN="bcm4908img" +RAMFS_COPY_BIN="bcm4908img expr" PART_NAME=firmware @@ -129,7 +129,7 @@ platform_calc_new_cferam() { umount $dir rm -fr $dir - idx=$(((idx + inc) % 1000)) + idx=$(($(expr $idx + $inc) % 1000)) echo $(printf "cferam.%03d" $idx) }