diff options
author | Subv <subv2112@gmail.com> | 2018-01-15 21:31:10 +0100 |
---|---|---|
committer | bunnei <bunneidev@gmail.com> | 2018-01-17 00:42:08 +0100 |
commit | 5bc14e791a8b4260dbf130d2e8724e394db4205c (patch) | |
tree | b5a6e4a6fe2c3d15ce3a7facba12ec62dd16fc38 /src/core | |
parent | Merge pull request #52 from ogniK5377/fsp (diff) | |
download | yuzu-5bc14e791a8b4260dbf130d2e8724e394db4205c.tar yuzu-5bc14e791a8b4260dbf130d2e8724e394db4205c.tar.gz yuzu-5bc14e791a8b4260dbf130d2e8724e394db4205c.tar.bz2 yuzu-5bc14e791a8b4260dbf130d2e8724e394db4205c.tar.lz yuzu-5bc14e791a8b4260dbf130d2e8724e394db4205c.tar.xz yuzu-5bc14e791a8b4260dbf130d2e8724e394db4205c.tar.zst yuzu-5bc14e791a8b4260dbf130d2e8724e394db4205c.zip |
Diffstat (limited to 'src/core')
-rw-r--r-- | src/core/hle/ipc_helpers.h | 22 | ||||
-rw-r--r-- | src/core/hle/kernel/hle_ipc.h | 8 |
2 files changed, 28 insertions, 2 deletions
diff --git a/src/core/hle/ipc_helpers.h b/src/core/hle/ipc_helpers.h index 0f1077d9e..25530a3c8 100644 --- a/src/core/hle/ipc_helpers.h +++ b/src/core/hle/ipc_helpers.h @@ -9,10 +9,13 @@ #include <type_traits> #include <utility> #include "core/hle/ipc.h" +#include "core/hle/kernel/client_port.h" +#include "core/hle/kernel/client_session.h" #include "core/hle/kernel/domain.h" #include "core/hle/kernel/handle_table.h" #include "core/hle/kernel/hle_ipc.h" #include "core/hle/kernel/kernel.h" +#include "core/hle/kernel/server_port.h" namespace IPC { @@ -63,13 +66,20 @@ public: : RequestHelperBase(context) { memset(cmdbuf, 0, sizeof(u32) * IPC::COMMAND_BUFFER_LENGTH); + context.ClearIncomingObjects(); + IPC::CommandHeader header{}; // The entire size of the raw data section in u32 units, including the 16 bytes of mandatory // padding. u32 raw_data_size = sizeof(IPC::DataPayloadHeader) / 4 + 4 + normal_params_size; - if (context.IsDomain()) + if (context.IsDomain()) { raw_data_size += sizeof(DomainMessageHeader) / 4 + num_domain_objects; + } else { + // If we're not in a domain, turn the domain object parameters into move handles. + num_handles_to_move += num_domain_objects; + num_domain_objects = 0; + } header.data_size.Assign(raw_data_size); if (num_handles_to_copy || num_handles_to_move) { @@ -100,7 +110,15 @@ public: template <class T, class... Args> void PushIpcInterface(Args&&... args) { - context->AddDomainObject(std::make_shared<T>(std::forward<Args>(args)...)); + auto iface = std::make_shared<T>(std::forward<Args>(args)...); + if (context->IsDomain()) { + context->AddDomainObject(std::move(iface)); + } else { + auto port = iface->CreatePort(); + auto session = port->Connect(); + ASSERT(session.Succeeded()); + context->AddMoveObject(std::move(session).Unwrap()); + } } // Validate on destruction, as there shouldn't be any case where we don't want it diff --git a/src/core/hle/kernel/hle_ipc.h b/src/core/hle/kernel/hle_ipc.h index 164c6db69..6dceb766d 100644 --- a/src/core/hle/kernel/hle_ipc.h +++ b/src/core/hle/kernel/hle_ipc.h @@ -175,6 +175,14 @@ public: domain_objects.emplace_back(std::move(object)); } + /// Clears the list of objects so that no lingering objects are written accidentally to the + /// response buffer. + void ClearIncomingObjects() { + move_objects.clear(); + copy_objects.clear(); + domain_objects.clear(); + } + private: std::array<u32, IPC::COMMAND_BUFFER_LENGTH> cmd_buf; SharedPtr<Kernel::Domain> domain; |