Files
AnyPS5/core/libs/tests/GuestRudpStatus.cpp
DavidandClaude Fable 5.1 4029b5d2c9 fix(libSceRudp): swap sceRudpEnd and sceRudpTerminate
sceRudpEnd takes no argument and shuts the library down; sceRudpTerminate
takes a context id and releases that context. main had the two swapped,
so a title calling sceRudpEnd() passed a stray register as a context id
and the library stayed initialised.

Evidence:
- PS5 console measurement by hugodearaujo in #3073 (firmware 13.60):
  "After `sceRudpEnd()` all three return `0x80770001` again"
  (sceRudpGetMaxSegmentSize, sceRudpProcessEvents, sceRudpNetFlush).
- AlexAltea/orbital cc408367da6a6ebbf5ac1ac559fefeadc73b2831
  tools/ida/db_types.json:1971,1996: `int sceRudpEnd(void)`,
  `int sceRudpTerminate(int ctxId)`.
- RPCS3 26098fb42d090cd522b4afcf3b754dd987c82978
  rpcs3/Emu/Cell/Modules/cellRudp.cpp:108: `cellRudpEnd()` tears the
  library down.
- AndnixSH/Vita3K-PS-Vita-OS-Fork c86ac732bc811057e690bed280c5076bdf95385e
  vita3k/modules/SceRudp/SceLibRudp.cpp:1032,1614: `sceRudpEnd()` clears
  every context, `sceRudpTerminate(int ctx_id)` erases one.

The bodies are unchanged, only exchanged. guest_offline_imports now ends
the library with sceRudpEnd(), and guest_rudp_status checks that
sceRudpTerminate releases one context and sceRudpEnd all of them.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
2026-10-13 07:51:20 +00:00

93 lines
3.9 KiB
C++

#include "SceTypes.hpp"
#include <array>
#include <cstddef>
#include <cstdint>
#include <cstdlib>
#include <cstring>
extern "C" {
using GuestRudpEventHandler = void(APS5_VABI*)(int, int, int, void*);
int APS5_VABI sceRudpInit_nid_postfix(void*, int);
int APS5_VABI sceRudpGetStatus(void*, std::size_t);
int APS5_VABI sceRudpCreateContext(GuestRudpEventHandler, void*, int*);
int APS5_VABI sceRudpTerminate(int);
int APS5_VABI sceRudpEnd();
}
static void Require(bool value) { if (!value) std::abort(); }
static void APS5_VABI Handler(int, int, int, void*) {}
int main() {
constexpr int notInitialized = static_cast<int>(0x80770001);
constexpr int alreadyInitialized = static_cast<int>(0x80770002);
constexpr int invalidContextId = static_cast<int>(0x80770003);
constexpr int invalidArgument = static_cast<int>(0x80770004);
constexpr int memory = static_cast<int>(0x80770007);
constexpr std::size_t statusSize = 0xF8;
constexpr std::size_t currentContexts = 0x64;
alignas(16) static unsigned char pool[0x10000];
std::array<unsigned char, 0x200> status;
status.fill(0x5a);
const auto untouched = status;
Require(sceRudpGetStatus(status.data(), statusSize) == notInitialized);
Require(sceRudpGetStatus(nullptr, 0) == notInitialized);
Require(sceRudpGetStatus(status.data(), statusSize + 1) == notInitialized);
Require(status == untouched);
Require(sceRudpEnd() == notInitialized);
Require(sceRudpTerminate(0) == notInitialized);
Require(sceRudpInit_nid_postfix(nullptr, sizeof(pool)) == invalidArgument);
Require(sceRudpInit_nid_postfix(pool, 0) == invalidArgument);
Require(sceRudpInit_nid_postfix(pool, -1) == invalidArgument);
Require(sceRudpInit_nid_postfix(pool, 0xC57) == memory);
for (std::size_t offset : {1u, 2u, 4u}) Require(sceRudpInit_nid_postfix(pool + offset, 0x1000) == memory);
Require(sceRudpInit_nid_postfix(pool + 8, 0xC58) == 0);
Require(sceRudpInit_nid_postfix(nullptr, 0) == alreadyInitialized);
Require(sceRudpGetStatus(nullptr, statusSize) == invalidArgument);
Require(sceRudpGetStatus(nullptr, 0) == invalidArgument);
Require(sceRudpGetStatus(status.data(), 0) == invalidArgument);
Require(sceRudpGetStatus(status.data(), statusSize + 1) == invalidArgument);
Require(status == untouched);
Require(sceRudpGetStatus(status.data(), statusSize) == 0);
for (std::size_t i = 0; i < statusSize; ++i) Require(status[i] == 0);
for (std::size_t i = statusSize; i < status.size(); ++i) Require(status[i] == 0x5a);
int first = -1;
int second = -1;
Require(sceRudpCreateContext(Handler, nullptr, &first) == 0);
Require(sceRudpCreateContext(Handler, nullptr, &second) == 0);
status.fill(0x5a);
Require(sceRudpGetStatus(status.data(), statusSize) == 0);
std::uint32_t contexts = 0;
std::memcpy(&contexts, status.data() + currentContexts, sizeof(contexts));
Require(contexts == 2);
for (std::size_t i = 0; i < statusSize; ++i) {
if (i < currentContexts || i >= currentContexts + sizeof(contexts)) Require(status[i] == 0);
}
status.fill(0x5a);
Require(sceRudpGetStatus(status.data(), 1) == 0);
Require(status[0] == 0);
for (std::size_t i = 1; i < status.size(); ++i) Require(status[i] == 0x5a);
Require(sceRudpTerminate(first) == 0);
Require(sceRudpTerminate(first) == invalidContextId);
Require(sceRudpGetStatus(status.data(), statusSize) == 0);
std::memcpy(&contexts, status.data() + currentContexts, sizeof(contexts));
Require(contexts == 1);
Require(sceRudpEnd() == 0);
Require(sceRudpEnd() == notInitialized);
Require(sceRudpTerminate(second) == notInitialized);
Require(sceRudpGetStatus(status.data(), statusSize) == notInitialized);
Require(sceRudpInit_nid_postfix(pool + 8, 0xC58) == 0);
Require(sceRudpTerminate(second) == invalidContextId);
Require(sceRudpEnd() == 0);
return 0;
}