PreyプロジェクトのBashスクリプトが正しくフォーマットされていませんか？ネストされたバックティックの問題？

Question

私はbashスクリプトではひどい初心者です。誰かがこの問題で私を助けてくれることを願っています。

Preyプロジェクトのスタンドアロンスクリプトに問題があります。電子メールを送信する行があり、正しくフォーマットされていないようです。 MailSenderのはブランドンZehmのPERLのsendEmailスクリプトへのエイリアスがある

response=`mailsender -f "$mail_from" -t "$mail_to" -u "$complete_subject" \ 
      -s $smtp_server -a $file_list -o message-file="$trace_file.msg" \ 
      tls=auto username=$smtp_username \ 
      password=\`decrypt \"$smtp_password\"\`` 

は、$ smtp_passwordはパスワードの無意味なbase64エンコーディングで、復号化され：

decrypt() { 
    echo "$1" | openssl enc -base64 -d 
} 

だから、誰もが持つ間違っているものを私に伝えることができますスクリプト？参考までに、私が暗号解読部分全体を平文のパスワードで置き換えるだけでうまくいきます。

response=$(Documents/Projects/Shell\ Scripting/printargs -f "$mail_from" \ 
    -t "$mail_to" -u "$complete_subject" -s $smtp_server -a $file_list \ 
    -o message-file="$trace_file.msg" tls=auto username=$smtp_username \ 
    password="$(decrypt "$smtp_password")") 

Answer 1

最も簡単な方法は、バッククォートを避け、代わりに$()を使用しています

decrypt() 
{ 
    echo "$1" | tr 'a-z' 'A-Z' 
} 
xxx=`echo xxx=yyy pass=\`decrypt \"xyz abc\"\`` 
echo "$xxx" 

'sh -x xxx'（ 'SHは' 変装で 'bashの' です）を実行します。

$ sh -x xxx 
+++ decrypt '"xyz' 'abc"' 
+++ echo '"xyz' 
+++ tr a-z A-Z 
++ echo xxx=yyy 'pass="XYZ' 
+ xxx='xxx=yyy pass="XYZ' 
+ echo 'xxx=yyy pass="XYZ' 
xxx=yyy pass="XYZ 
$ 

見た目が分かっていれば、どこに問題があるかを確認できます。 decryptコマンドラインには2つの引数があり、目的は1つだけで、引数には最初の二重引用符が含まれ、2番目の引数の最後には二重引用符が含まれます。

あなたのスクリプトでは、decryptに渡された引数には、二重引用符が含まれています。これはおそらくあなたが望むものではありません。

我々ははるかにきれいにネスト「$(...)」表記を使用して、スクリプトを書き換えた場合、我々が得る：私は

$ sh -x xxx 
+++ decrypt 'xyz abc' 
+++ echo 'xyz abc' 
+++ tr a-z A-Z 
++ echo zzz=yyy pass=XYZ ABC 
+ yyy='zzz=yyy pass=XYZ ABC' 
+ echo 'zzz=yyy pass=XYZ ABC' 
zzz=yyy pass=XYZ ABC 
$ 

Answer 2

私は、このスクリプトがあなたと同型だと思う：： - ：つまり、彼らは特別な必要なエスケープで、きれいに巣

response=`mailsender -f "$mail_from" -t "$mail_to" -u "$complete_subject" \ 
      -s $smtp_server -a $file_list -o message-file="$trace_file.msg" \ 
      tls=auto username=$smtp_username password=actual_password` 

Answer 3

：

decrypt() 
{ 
    echo "$1" | tr 'a-z' 'A-Z' 
} 

yyy=$(echo zzz=yyy pass=$(decrypt "xyz abc")) 
echo "$yyy" 

トレースこれからは次のようになりますプレイからの人の一人。この脆弱性は昨日確認され、修正は既にコミットされています。

私は、$()がバックティッキング（特にバックバックティッキング）よりも読みやすく、実際にはそれが私たちが取り組んでいることの一つです（大きなコードリファクタリング）。

最近私はSkullと呼ばれるBashフレームワークに取り組んでいます。このフレームワークはシェルスクリプトの作成にもっと良いインターフェースを提供しています。うまくいけば、Prey 0.6は完全にそれに基づいており、余分なバックティッカーは誰でも簡単に読むために$()に置き換えられます。