シェルスクリプトのzipの問題。なぜ間違っているのですか?

シェルスクリプトのzipの問題。なぜ間違っているのですか?

ファイルを圧縮し、ランダムに生成された指定された長さのPINのパスワードハッシュでファイルを保護し、PINを指定されたファイルに保存するシェルスクリプトfile_locker.shを作成します。

#!/bin/bash
file_locker(){

m=$1
for ((i=1;i<=m;i++));
do
num=($(( 0 + RANDOM % 9 )))

echo -n "$num"

done > $3

mypin=$(echo $3)
echo pin $mypin

myhash=$(echo $mypin | sha256sum)
echo hash $myhash

zip -P $myhash $2.zip -r $2 && rm $2

}
file_locker $1 $2 $3

ただし、コマンドラインは次のように伝えます。

zip error: Invalid command arguments (cannot write zip file to terminal)

zipコマンドを改善する方法がわかりません。助けてください!

答え1

パラメータに引用符を使用し、
forループではmの代わりに$ mを使用する必要があります。

#!/bin/bash
file_locker(){

m="$1"
for ((i=1;i<=$m;i++));
do
num=($(( 0 + RANDOM % 9 )))

echo -n "$num"

done > "$3"

mypin=$(echo "$3")
echo pin "$mypin"

myhash=$(echo "$mypin" | sha256sum)
echo hash "$myhash"

zip -P "$myhash" "$2".zip -r "$2" && rm "$2"

}
file_locker "$1" "$2" "$3"

zipエラー:無効なコマンド引数

引数に空白がある場合は、次の結果が表示されます。

関連情報