[reyani-dev]

What

FreeBSD's unlink(2) fails with EPERM when the path is a directory. sceKernelUnlink passed the host error through, so the guest got a different code on each host:

Host	unlink("dir") on main	FreeBSD
Windows (_wunlink)	EACCES (13), measured	EPERM (1)
Linux (unlink)	EISDIR (21), per unlink(2)	EPERM (1)

sceKernelUnlink now checks the path with symlink_status before unlinking and returns EPERM for a directory. A symbolic link to a directory is still unlinked, as on FreeBSD. If the path can't be inspected, the check doesn't match and the native unlink reports its own error as before. unlink (the POSIX wrapper) goes through sceKernelUnlink, so both are covered.

Tested

Windows 11, WinLibs GCC 15.2.0 r7, Release, -DBUILD_TESTING=ON:

• Two checks in guest_filesystem_tests, next to the other unlink checks: unlink(dir) returns -1 with errno 1 and leaves the directory in place, and sceKernelUnlink(dir) returns 0x80020001. On main the first fails with guest errno 13.
• ctest -R "guest_filesystem|guest_kernel_sync|guest_file": 3/3 passed.
• tools/check_conventions.py passes, and tools/pr_overlap.py --branch reports no overlap. It also merges cleanly with fix(kernel): return guest errors from scalar file operations #2144, fix(files): isolate guest descriptors from host resources #2170, fix(libkernel): honour open flags when Windows falls back to a directory descriptor #1628, feat(libkernel): open /dev/random and /dev/urandom for reading #1777, fix(libkernel): give flock BSD semantics on Windows #1755, fix(libkernel): return EWOULDBLOCK from a contended LOCK_NB flock #1764, feat(libs): implement font, libc, libkernel and misc prx stubs #2193 and my fix(libkernel): report POSIX rename errors from sceKernelRename #2260, which touch the same files (git merge-tree). fix(libkernel): read and write guest sockets #1727 already conflicts with main on its own (TechnicalDebt.md).

Checklist

• Based on current main; no other open PR implements the same functions
• One topic per PR; follow-ups go in a new PR
• No comments in code except technical debt
• Unimplemented paths throw (NotImplemented_nid_no_patch); silent stubs are listed in TechnicalDebt
• No notes, investigation .md files or images added to the repository (attach them to this PR)
• New third-party code is a submodule built from source
• Generic behaviour, not specific to one title
• Depends on: none
• AI-assisted: yes

🤖 Generated with Claude Code

[reyani-dev]

Full Linux and Windows builds and tests for this head (27516327), using the project's Build workflow. I can't start the upstream manual run, so I ran the same build.yml in my fork through a mirror PR. Like the upstream run, the jobs check out the PR head merged into current main (d70b8998):

Job	Result
linux	success; full build, ctest: 516/516 passed
windows	success; full build, ctest: 507/507 passed
relinker (Linux, Windows, macOS)	success

Run: https://github.com/AchrafReyani/AnyPS5/actions/runs/37946468349