Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TTSSH パスワード変更ダイアログで、「新パスワード」と「新パスワード確認」が一致しなくてもパスワードが変更されることがある #401

Open
taikoyaP opened this issue Dec 13, 2024 · 3 comments
Milestone

Comments

@taikoyaP
Copy link

taikoyaP commented Dec 13, 2024

TTSSHの「パスワード変更」ダイアログは、「新パスワード」と「新パスワード確認」が一致することをチェックしていますが、
一致しなくても処理が進む場合があります。

例として、
新パスワードに「11111111」を、
新パスワード確認に「11111112」を与えると、
パスワードが一致していませんが処理が進みます。

手元にWindowsの開発環境が無いので確認できないのですが、
ttssh2/ttxssh/ssh.c の 7535行目

if (strcmp(new_passwd, retype_passwd) == 1) {

は、おそらく

if (strcmp(new_passwd, retype_passwd) != 0) {

が正しいのではないでしょうか?

@zmatsuo
Copy link
Member

zmatsuo commented Feb 2, 2025

修正確認が難しく弱っています。
可能でしたら確認をお願いできないでしょうか? @taikoyaP

バイナリは次のページにあります。
https://ci.appveyor.com/project/teraterm/github-snapshot/builds/51437011/artifacts

@taikoyaP
Copy link
Author

taikoyaP commented Feb 3, 2025

ご対応ありがとうございます。
上記バイナリで確認し、「新パスワード」と「新パスワード確認」が以下の場合に、処理が進まない(正しい)動作になることを確認しました。

  • 新パスワード: password1, 新パスワード確認: password2
  • 新パスワード: password2, 新パスワード確認: password1

また、「新パスワード」と「新パスワード確認」が同じ場合に、処理が進むことも確認しました。

@zmatsuo
Copy link
Member

zmatsuo commented Feb 3, 2025

ありがとうございます!
改版履歴ができたらmainへマージしてクローズします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants