Stopped multiple fingerprint message boxes popping up #4910

This commit is contained in:
Xinyu Hou 2015-07-30 16:51:00 -07:00 committed by Jerry (Xinyu Hou)
parent d70c19b099
commit 0612ba585d
1 changed files with 26 additions and 19 deletions

View File

@ -436,27 +436,34 @@ void MainWindow::checkFingerprint(const QString& line)
return; return;
} }
QMessageBox::StandardButton fingerprintReply = static bool messageBoxAlreadyShown = false;
QMessageBox::information(
this, tr("Security question"),
tr("Do you trust this fingerprint?\n\n"
"%1\n\n"
"This is a server fingerprint. You should compare this "
"fingerprint to the one on your server's screen. If the "
"two don't match exactly, then it's probably not the server "
"you're expecting (it could be a malicious user).\n\n"
"To automatically trust this fingerprint for future "
"connections, click Yes. To reject this fingerprint and "
"disconnect from the server, click No.")
.arg(fingerprint),
QMessageBox::Yes | QMessageBox::No);
stopSynergy(); if (!messageBoxAlreadyShown) {
messageBoxAlreadyShown = true;
QMessageBox::StandardButton fingerprintReply =
QMessageBox::information(
this, tr("Security question"),
tr("Do you trust this fingerprint?\n\n"
"%1\n\n"
"This is a server fingerprint. You should compare this "
"fingerprint to the one on your server's screen. If the "
"two don't match exactly, then it's probably not the server "
"you're expecting (it could be a malicious user).\n\n"
"To automatically trust this fingerprint for future "
"connections, click Yes. To reject this fingerprint and "
"disconnect from the server, click No.")
.arg(fingerprint),
QMessageBox::Yes | QMessageBox::No);
if (fingerprintReply == QMessageBox::Yes) { messageBoxAlreadyShown = false;
// restart core process after trusting fingerprint.
Fingerprint::trustedServers().trust(fingerprint); stopSynergy();
startSynergy();
if (fingerprintReply == QMessageBox::Yes) {
// restart core process after trusting fingerprint.
Fingerprint::trustedServers().trust(fingerprint);
startSynergy();
}
} }
} }