N-archive/citra
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Kernel: Ensure objects are kept alive during ClientSession disconnection

Browse source 
Fixes #2760
This commit is contained in:
Yuri Kunde Schlesner 2017-06-08 00:33:24 -07:00
parent 8c22334f96
commit d666e01cdf
1 changed files with 13 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
src/core/hle/kernel/client_session.cpp
View file

@ -16,9 +16,13 @@ ClientSession::~ClientSession() {
// This destructor will be called automatically when the last ClientSession handle is closed by // This destructor will be called automatically when the last ClientSession handle is closed by
// the emulated application. // the emulated application.
if (parent->server) { // Local references to ServerSession and SessionRequestHandler are necessary to guarantee they
if (parent->server->hle_handler) // will be kept alive until after ClientDisconnected() returns.
parent->server->hle_handler->ClientDisconnected(parent->server); SharedPtr<ServerSession> server = parent->server;
if (server) {
std::shared_ptr<SessionRequestHandler> hle_handler = server->hle_handler;
if (hle_handler)
hle_handler->ClientDisconnected(server);
// TODO(Subv): Force a wake up of all the ServerSession's waiting threads and set // TODO(Subv): Force a wake up of all the ServerSession's waiting threads and set
// their WaitSynchronization result to 0xC920181A. // their WaitSynchronization result to 0xC920181A.
@ -28,11 +32,13 @@ ClientSession::~ClientSession() {
} }
ResultCode ClientSession::SendSyncRequest() { ResultCode ClientSession::SendSyncRequest() {
// Signal the server session that new data is available // Keep ServerSession alive until we're done working with it.
if (parent->server) SharedPtr<ServerSession> server = parent->server;
return parent->server->HandleSyncRequest(); if (server == nullptr)
return ERR_SESSION_CLOSED_BY_REMOTE;
return ERR_SESSION_CLOSED_BY_REMOTE; // Signal the server session that new data is available
return server->HandleSyncRequest();
} }
} // namespace } // namespace