diff options
author | Subv <subv2112@gmail.com> | 2018-01-07 05:19:42 +0100 |
---|---|---|
committer | bunnei <bunneidev@gmail.com> | 2018-01-07 23:11:45 +0100 |
commit | b0ceb4df70c8a387e12e2df0d1031421493ad744 (patch) | |
tree | fbe37a5849d34de75250fd12bc7403cc3c1ea4ad /src/core/hle | |
parent | IPC: Use the correct size when pushing raw data to the command buffer and fixed pushing domain objects. (diff) | |
download | yuzu-b0ceb4df70c8a387e12e2df0d1031421493ad744.tar yuzu-b0ceb4df70c8a387e12e2df0d1031421493ad744.tar.gz yuzu-b0ceb4df70c8a387e12e2df0d1031421493ad744.tar.bz2 yuzu-b0ceb4df70c8a387e12e2df0d1031421493ad744.tar.lz yuzu-b0ceb4df70c8a387e12e2df0d1031421493ad744.tar.xz yuzu-b0ceb4df70c8a387e12e2df0d1031421493ad744.tar.zst yuzu-b0ceb4df70c8a387e12e2df0d1031421493ad744.zip |
Diffstat (limited to 'src/core/hle')
-rw-r--r-- | src/core/hle/kernel/hle_ipc.cpp | 3 | ||||
-rw-r--r-- | src/core/hle/service/sm/sm.cpp | 17 |
2 files changed, 5 insertions, 15 deletions
diff --git a/src/core/hle/kernel/hle_ipc.cpp b/src/core/hle/kernel/hle_ipc.cpp index e784d59cc..ac81dbf3f 100644 --- a/src/core/hle/kernel/hle_ipc.cpp +++ b/src/core/hle/kernel/hle_ipc.cpp @@ -107,8 +107,9 @@ void HLERequestContext::ParseCommandBuffer(u32_le* src_cmdbuf, bool incoming) { ASSERT(data_payload_header->magic == Common::MakeMagic('S', 'F', 'C', 'O')); } - data_payload_offset = rp.GetCurrentOffset(); command = rp.Pop<u32_le>(); + rp.Skip(1, false); // The command is actually an u64, but we don't use the high part. + data_payload_offset = rp.GetCurrentOffset(); } ResultCode HLERequestContext::PopulateFromIncomingCommandBuffer(u32_le* src_cmdbuf, diff --git a/src/core/hle/service/sm/sm.cpp b/src/core/hle/service/sm/sm.cpp index a976385ac..279908cae 100644 --- a/src/core/hle/service/sm/sm.cpp +++ b/src/core/hle/service/sm/sm.cpp @@ -100,8 +100,6 @@ void SM::Initialize(Kernel::HLERequestContext& ctx) { */ void SM::GetService(Kernel::HLERequestContext& ctx) { IPC::RequestParser rp{ctx}; - u32 unk1 = rp.Pop<u32>(); - u32 unk2 = rp.Pop<u32>(); auto name_buf = rp.PopRaw<std::array<char, 9>>(); std::string name(name_buf.data()); @@ -117,22 +115,13 @@ void SM::GetService(Kernel::HLERequestContext& ctx) { } auto session = client_port.Unwrap()->Connect(); + ASSERT(session.Succeeded()); if (session.Succeeded()) { LOG_DEBUG(Service_SM, "called service=%s -> session=%u", name.c_str(), (*session)->GetObjectId()); - IPC::RequestBuilder rb = rp.MakeBuilder(1, 0, 1); - rb.Push(session.Code()); + IPC::RequestBuilder rb = rp.MakeBuilder(2, 0, 1); + rb.Push<u64>(0); rb.PushObjects(std::move(session).Unwrap()); - } else if (session.Code() == Kernel::ERR_MAX_CONNECTIONS_REACHED /*&& return_port_on_failure*/) { - LOG_WARNING(Service_SM, "called service=%s -> ERR_MAX_CONNECTIONS_REACHED, *port*=%u", - name.c_str(), (*client_port)->GetObjectId()); - IPC::RequestBuilder rb = rp.MakeBuilder(1, 0, 1); - rb.Push(ERR_MAX_CONNECTIONS_REACHED); - rb.PushObjects(std::move(client_port).Unwrap()); - } else { - LOG_ERROR(Service_SM, "called service=%s -> error 0x%08X", name.c_str(), session.Code()); - IPC::RequestBuilder rb = rp.MakeBuilder(1, 0, 0); - rb.Push(session.Code()); } } |