jeirmeister/Cockatrice
1
0
Fork 0
mirror of https://github.com/Cockatrice/Cockatrice.git synced 2026-04-27 07:48:01 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix client crash when server goes down during registration (#5787)

Browse source
This commit is contained in:
ebbit1q 2025-04-12 05:01:14 +02:00
parent aa3c8c0625
commit feb8191904
2 changed files with 8 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

3
cockatrice/src/client/ui/window_main.cpp
View file

@ -574,6 +574,9 @@ void MainWindow::registerError(Response::ResponseCode r, QString reasonStr, quin
case Response::RespRegistrationFailed:
QMessageBox::critical(this, tr("Error"), tr("Registration failed for a technical problem on the server."));
break;
case Response::RespNotConnected:
QMessageBox::critical(this, tr("Error"), tr("The connection to the server has been lost."));
break;
default:
QMessageBox::critical(this, tr("Error"),
tr("Unknown registration error: %1").arg(static_cast<int>(r)) +

5
cockatrice/src/server/remote/remote_client.cpp
View file

@ -339,6 +339,11 @@ void RemoteClient::registerResponse(const Response &response)
emit registerAcceptedNeedsActivate();
doLogin();
break;
case Response::RespNotConnected:
// this response is created by the client from doDisconnectFromServer, do not call it again!
emit registerError(response.response_code(), QString::fromStdString(resp.denied_reason_str()),
static_cast<quint32>(resp.denied_end_time()));
break;
default:
emit registerError(response.response_code(), QString::fromStdString(resp.denied_reason_str()),
static_cast<quint32>(resp.denied_end_time()));