diff options
author | Subv <subv2112@gmail.com> | 2018-01-07 15:22:20 +0100 |
---|---|---|
committer | bunnei <bunneidev@gmail.com> | 2018-01-07 23:11:51 +0100 |
commit | 32847d8b860af1210612027680eea1cbf9765b51 (patch) | |
tree | 234cabf0b424e2863540f704752e2838c914d491 | |
parent | IPC: Don't attempt to read the command buffer if it holds a Close request. (diff) | |
download | yuzu-32847d8b860af1210612027680eea1cbf9765b51.tar yuzu-32847d8b860af1210612027680eea1cbf9765b51.tar.gz yuzu-32847d8b860af1210612027680eea1cbf9765b51.tar.bz2 yuzu-32847d8b860af1210612027680eea1cbf9765b51.tar.lz yuzu-32847d8b860af1210612027680eea1cbf9765b51.tar.xz yuzu-32847d8b860af1210612027680eea1cbf9765b51.tar.zst yuzu-32847d8b860af1210612027680eea1cbf9765b51.zip |
-rw-r--r-- | src/core/hle/ipc_helpers.h | 16 | ||||
-rw-r--r-- | src/core/hle/kernel/hle_ipc.cpp | 16 | ||||
-rw-r--r-- | src/core/hle/kernel/hle_ipc.h | 12 |
3 files changed, 42 insertions, 2 deletions
diff --git a/src/core/hle/ipc_helpers.h b/src/core/hle/ipc_helpers.h index 368b50011..e5b296d60 100644 --- a/src/core/hle/ipc_helpers.h +++ b/src/core/hle/ipc_helpers.h @@ -233,6 +233,12 @@ public: */ template <typename T> T PopRaw(); + + template <typename T> + Kernel::SharedPtr<T> GetMoveObject(size_t index); + + template <typename T> + Kernel::SharedPtr<T> GetCopyObject(size_t index); }; /// Pop /// @@ -293,4 +299,14 @@ void RequestParser::Pop(First& first_value, Other&... other_values) { Pop(other_values...); } +template <typename T> +Kernel::SharedPtr<T> RequestParser::GetMoveObject(size_t index) { + return context->GetMoveObject<T>(index); +} + +template <typename T> +Kernel::SharedPtr<T> RequestParser::GetCopyObject(size_t index) { + return context->GetCopyObject<T>(index); +} + } // namespace IPC diff --git a/src/core/hle/kernel/hle_ipc.cpp b/src/core/hle/kernel/hle_ipc.cpp index 0ab28c0a2..1c6adb4a0 100644 --- a/src/core/hle/kernel/hle_ipc.cpp +++ b/src/core/hle/kernel/hle_ipc.cpp @@ -53,8 +53,20 @@ void HLERequestContext::ParseCommandBuffer(u32_le* src_cmdbuf, bool incoming) { if (handle_descriptor_header->send_current_pid) { rp.Skip(2, false); } - rp.Skip(handle_descriptor_header->num_handles_to_copy, false); - rp.Skip(handle_descriptor_header->num_handles_to_move, false); + if (incoming) { + // Populate the object lists with the data in the IPC request. + for (u32 handle = 0; handle < handle_descriptor_header->num_handles_to_copy; ++handle) { + copy_objects.push_back(Kernel::g_handle_table.GetGeneric(rp.Pop<Handle>())); + } + for (u32 handle = 0; handle < handle_descriptor_header->num_handles_to_move; ++handle) { + move_objects.push_back(Kernel::g_handle_table.GetGeneric(rp.Pop<Handle>())); + } + } else { + // For responses we just ignore the handles, they're empty and will be populated when + // translating the response. + rp.Skip(handle_descriptor_header->num_handles_to_copy, false); + rp.Skip(handle_descriptor_header->num_handles_to_move, false); + } } for (unsigned i = 0; i < command_header->num_buf_x_descriptors; ++i) { diff --git a/src/core/hle/kernel/hle_ipc.h b/src/core/hle/kernel/hle_ipc.h index 48730a2b2..266fcf9c1 100644 --- a/src/core/hle/kernel/hle_ipc.h +++ b/src/core/hle/kernel/hle_ipc.h @@ -147,6 +147,18 @@ public: return domain != nullptr; } + template<typename T> + SharedPtr<T> GetCopyObject(size_t index) { + ASSERT(index < copy_objects.size()); + return DynamicObjectCast(copy_objects[index]); + } + + template<typename T> + SharedPtr<T> GetMoveObject(size_t index) { + ASSERT(index < move_objects.size()); + return DynamicObjectCast(move_objects[index]); + } + void AddMoveObject(SharedPtr<Object> object) { move_objects.emplace_back(std::move(object)); } |