diff options
author | bunnei <bunneidev@gmail.com> | 2018-10-20 04:58:57 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-10-20 04:58:57 +0200 |
commit | 60317e630619ff5942fcb4b16cf1b3a0b2791cc2 (patch) | |
tree | 5fb63e418bc7af69c9d06f7fc4ae3980e7d60aee /src/core/hle/kernel/svc.cpp | |
parent | Merge pull request #1517 from bunnei/dma (diff) | |
parent | svc: Add missing sanitizing checks for MapSharedMemory/UnmapSharedMemory (diff) | |
download | yuzu-60317e630619ff5942fcb4b16cf1b3a0b2791cc2.tar yuzu-60317e630619ff5942fcb4b16cf1b3a0b2791cc2.tar.gz yuzu-60317e630619ff5942fcb4b16cf1b3a0b2791cc2.tar.bz2 yuzu-60317e630619ff5942fcb4b16cf1b3a0b2791cc2.tar.lz yuzu-60317e630619ff5942fcb4b16cf1b3a0b2791cc2.tar.xz yuzu-60317e630619ff5942fcb4b16cf1b3a0b2791cc2.tar.zst yuzu-60317e630619ff5942fcb4b16cf1b3a0b2791cc2.zip |
Diffstat (limited to 'src/core/hle/kernel/svc.cpp')
-rw-r--r-- | src/core/hle/kernel/svc.cpp | 30 |
1 files changed, 27 insertions, 3 deletions
diff --git a/src/core/hle/kernel/svc.cpp b/src/core/hle/kernel/svc.cpp index d3c9d50b5..3b8a2e230 100644 --- a/src/core/hle/kernel/svc.cpp +++ b/src/core/hle/kernel/svc.cpp @@ -584,6 +584,10 @@ static ResultCode MapSharedMemory(Handle shared_memory_handle, VAddr addr, u64 s return ERR_INVALID_SIZE; } + if (!IsValidAddressRange(addr, size)) { + return ERR_INVALID_ADDRESS_STATE; + } + const auto permissions_type = static_cast<MemoryPermission>(permissions); if (permissions_type != MemoryPermission::Read && permissions_type != MemoryPermission::ReadWrite) { @@ -597,8 +601,14 @@ static ResultCode MapSharedMemory(Handle shared_memory_handle, VAddr addr, u64 s return ERR_INVALID_HANDLE; } - return shared_memory->Map(Core::CurrentProcess(), addr, permissions_type, - MemoryPermission::DontCare); + auto* const current_process = Core::CurrentProcess(); + const auto& vm_manager = current_process->VMManager(); + + if (!vm_manager.IsWithinASLRRegion(addr, size)) { + return ERR_INVALID_MEMORY_RANGE; + } + + return shared_memory->Map(current_process, addr, permissions_type, MemoryPermission::DontCare); } static ResultCode UnmapSharedMemory(Handle shared_memory_handle, VAddr addr, u64 size) { @@ -613,10 +623,24 @@ static ResultCode UnmapSharedMemory(Handle shared_memory_handle, VAddr addr, u64 return ERR_INVALID_SIZE; } + if (!IsValidAddressRange(addr, size)) { + return ERR_INVALID_ADDRESS_STATE; + } + auto& kernel = Core::System::GetInstance().Kernel(); auto shared_memory = kernel.HandleTable().Get<SharedMemory>(shared_memory_handle); + if (!shared_memory) { + return ERR_INVALID_HANDLE; + } + + auto* const current_process = Core::CurrentProcess(); + const auto& vm_manager = current_process->VMManager(); + + if (!vm_manager.IsWithinASLRRegion(addr, size)) { + return ERR_INVALID_MEMORY_RANGE; + } - return shared_memory->Unmap(Core::CurrentProcess(), addr); + return shared_memory->Unmap(current_process, addr); } /// Query process memory |