From b0ceb4df70c8a387e12e2df0d1031421493ad744 Mon Sep 17 00:00:00 2001 From: Subv Date: Sat, 6 Jan 2018 23:19:42 -0500 Subject: [PATCH] IPC: Skip the entire u64 of the command id when receiving an IPC request. Service code now doesn't have to deal with this. --- src/core/hle/kernel/hle_ipc.cpp | 3 ++- 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 e784d59cc6..ac81dbf3fa 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(); + 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 a976385aca..279908cae0 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 unk2 = rp.Pop(); auto name_buf = rp.PopRaw>(); 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(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()); } }