The-Homebrew-Cloud/skyline
2
0
Fork 0
mirror of https://github.com/skyline-emu/skyline.git synced 2024-11-26 09:44:15 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix SvcClearEvent resetting behaviour

Browse Source 
SvcClearEvent previously set the `signalled` flag directly rather than
calling `ResetSignal`, which skipped the locking necessary to make it
globally visible. Switch it to use `ResetSignal` to fix this.
This commit is contained in:
◱ PixelyIon 2021-02-18 11:51:11 +00:00 committed by ◱ Mark
parent a621408b9c
commit 861d7e9eb2
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
app/src/main/cpp/skyline/kernel/svc.cpp
View File

@ -443,9 +443,16 @@ namespace skyline::kernel::svc {
} }
void ClearEvent(const DeviceState &state) { void ClearEvent(const DeviceState &state) {
auto object{state.process->GetHandle<type::KEvent>(state.ctx->gpr.w0)}; KHandle handle{state.ctx->gpr.w0};
object->signalled = false; try {
state.ctx->gpr.w0 = Result{}; std::static_pointer_cast<type::KEvent>(state.process->GetHandle(handle))->ResetSignal();
state.logger->Debug("svcClearEvent: Clearing event: 0x{:X}", handle);
state.ctx->gpr.w0 = Result{};
} catch (const std::out_of_range &) {
state.logger->Warn("svcClearEvent: 'handle' invalid: 0x{:X}", handle);
state.ctx->gpr.w0 = result::InvalidHandle;
return;
}
} }
void MapSharedMemory(const DeviceState &state) { void MapSharedMemory(const DeviceState &state) {