From 8ac62321ee912b7ad75616da469201bb01d5eb2d Mon Sep 17 00:00:00 2001 From: Mark Wooding Date: Tue, 15 Dec 2015 19:15:23 +0000 Subject: [PATCH] hush.in: Dereference variables explicitly inside $((...)). These were somehow missed by f9f9787a... --- hush.in | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hush.in b/hush.in index f10f507..114e6b0 100755 --- a/hush.in +++ b/hush.in @@ -75,7 +75,7 @@ EOF *) echo >&2 "$usage"; exit 1 ;; esac done -shift $(( OPTIND - 1 )) +shift $(( $OPTIND - 1 )) ## Check the arguments. case $# in 0 | 1) echo >&2 "$usage"; exit 1 ;; esac @@ -103,7 +103,7 @@ date=$(date +%Y-%m-%d) seq=1 for i in "$logdir/$tag.$date#"*; do tail=${i##*#} case "$tail" in [!1-9]* | *[!0-9]*) continue ;; esac - if [ -f "$i" -a $tail -ge $seq ]; then seq=$(( tail + 1 )); fi + if [ -f "$i" -a $tail -ge $seq ]; then seq=$(( $tail + 1 )); fi done log="$logdir/$tag.$date#$seq" @@ -112,7 +112,7 @@ log="$logdir/$tag.$date#$seq" ## example) end up giving the wrong group write permission to the file for a ## little bit. umask=$(umask) -case ${mode+t} in t) ;; *) mode=$(printf %o $(( 0666 & ~umask ))) ;; esac +case ${mode+t} in t) ;; *) mode=$(printf %o $(( 0666 & ~$umask ))) ;; esac umask 077; exec 3>"$log"; umask $umask case ${owner+t} in t) chown "$owner" "$log" ;; esac chmod $mode "$log" -- 2.11.0