マクロ getpassword で Esc キーを押下したときの挙動がおかしい

• 報告者: imabeppu
• 担当者: nmaya
• チケットの種類: バグ
• 状況: 完了
• コンポーネント: Tera Term Macro
• マイルストーン: Tera Term 4.107 & 5.0
• 優先度: 5 - 中
• 重要度: 5 - 中
• 解決法: 修正済み
• ファイル: なし

詳細 返信

Tera Term Ver.4.106 (SVN# 9298) を使用しています。
マクロの getpassword でダイアログが表示された場合の挙動に問題があるようです。

getpassword でダイアログが表示されたときに Esc キーでダイアログを閉じることができます。 このとき、第3引数で指定した変数の内容が変化しますが、内容が不定なようです。 画面で指定した文字列が入るわけではないようです。空白が1文字入る場合もありますし、文字化けしたような文字列が格納される場合もあります。 想像ですが、Esc キーでダイアログが閉じられてパスワード入力のコントロールが消滅した後に、ダイアログ内の文字列を取得しているのではないでしょうか。 そして、そのままパスワードファイルに書き込んでいるのではないでしょうか。

Esc キーでダイアログを閉じたときは、第3引数で指定した変数の内容は変更しないほうがよいと思います。パスワードファイルにも書き込まないほうが良いと思います。

以下、試した時のマクロ内容です。

 pwFile = 'C:\temp\password.dat'
 :retry
 delpassword pwFile 'test'
 password = ''
 getpassword pwFile 'test' password
 gpRes = result
 strcompare password 'end'
 if result = 0 end
 sprintf2 msg 'result = [%d]\npassword = [%s]' gpRes password
 strspecial msg
 messagebox msg 'TEST'
 goto retry

チケットの履歴 (7 件中 3 件表示)

2023-06-06 12:12 更新者: imabeppu

• 新しいチケット "マクロ getpassword で Esc キーを押下したときの挙動がおかしい" が作成されました

2023-06-06 14:16 更新者: nmaya

• 担当者 が (未割り当て) から nmaya に更新されました
• 解決法 が なし から 受領 に更新されました
• マイルストーン が (未割り当て) から Tera Term 4.107 & 5.0 に更新されました

2023-06-07 01:26 更新者: nmaya

• 解決法 が 受領 から 修正済み に更新されました

コメント 返信

対応しました。

• 4-stable: r10742
• trunk: r10743

2023-06-26 10:45 更新者: imabeppu

コメント 返信

修正内容を見ました。

コンパイルオプションまで確認していませんので、実際は問題ないかもしれませんが、気になる点がありました。 以下のようなことは起きないでしょうか。

---

※ttl_gui.cpp (リビジョン 10743 を確認)

• 220行目で input_string を宣言。初期化を行っていないので内容は不定。

• 222行目で OpenInpDlg() を呼び出し。第1引数に input_string を渡す。

• パスワード入力ダイアログ Esc キーを押下した場合、CInpDlg::OnOK() は呼び出されないので input_string の内容は変更なし。不定のまま。

• 223行目で内容が不定な input_string を WideCharToUTF8() で変換。

• 224行目で「if (Temp2[0]!=0)」という判定を行っているが、内容が不定なので Esc キー押下時も条件が成り立つ可能性がある。

---

できれば、OpenInpDlg() の戻り値で Esc キーを押下したかどうかが判断できるようにし、その値をもとに処理を切り分けたほうがよいと思います。

変更を最小限にするのならば、220行目で input_string を宣言するときに、初期化しておくことでも解決すると思います。

 wchar_t input_string[MaxStrLen] = {};

以上、ご確認ください。

(編集済, 2023-06-27 12:45 更新者: imabeppu)

2023-06-27 23:22 更新者: nmaya

コメント 返信

ありがとうございます。対応してみました。

• 4-stable: r10784
• trunk: r10786

2023-06-28 10:32 更新者: imabeppu

コメント 返信

対応ありがとうございます。次バージョンが楽しみです。

2023-07-02 22:02 更新者: nmaya

• 状況 が オープン から 完了 に更新されました