toph/claudebox
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(04): WR-02 add stride-3 guard and safe arithmetic in dry-run ENV_ARGS loop

Browse source
This commit is contained in:
Christopher Mühl 2026-04-10 09:27:39 +00:00
parent adb9dd117d
commit 0922b752a5
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
claudebox.sh
View file

@ -307,10 +307,15 @@ if [[ "$DRY_RUN" == true ]]; then
{ {
echo "bwrap \\" echo "bwrap \\"
echo " --clearenv \\" echo " --clearenv \\"
# Guard: ENV_ARGS must be a multiple of 3 (--setenv NAME VALUE triplets)
if (( ${#ENV_ARGS[@]} % 3 != 0 )); then
echo "BUG: ENV_ARGS length ${#ENV_ARGS[@]} is not a multiple of 3" >&2
exit 1
fi
dry_run_i=0 dry_run_i=0
while (( dry_run_i < ${#ENV_ARGS[@]} )); do while (( dry_run_i < ${#ENV_ARGS[@]} )); do
printf ' %s %s %q \\\n' "${ENV_ARGS[$dry_run_i]}" "${ENV_ARGS[$((dry_run_i+1))]}" "${ENV_ARGS[$((dry_run_i+2))]}" printf ' %s %s %q \\\n' "${ENV_ARGS[$dry_run_i]}" "${ENV_ARGS[$((dry_run_i+1))]}" "${ENV_ARGS[$((dry_run_i+2))]}"
(( dry_run_i += 3 )) dry_run_i=$(( dry_run_i + 3 ))
done done
echo " --tmpfs / \\" echo " --tmpfs / \\"
echo " --proc /proc \\" echo " --proc /proc \\"