[PR #3976] [MERGED] Net: Delete unused fds in sceNetEpollDestroy and sys_socketclose #3837

New issue

Closed

opened 2026-02-27 22:05:10 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 22:05:10 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/shadps4-emu/shadPS4/pull/3976
Author: @StevenMiller123
Created: 1/29/2026
Status: ✅ Merged
Merged: 1/29/2026
Merged by: @georgemoralis

Base: main ← Head: net-fd-fixes

📝 Commits (1)

7e7574a Delete unused fds in sceNetEpollDestroy and sys_socketclose

📊 Changes

2 files changed (+2 additions, -0 deletions)

View changed files

📝 src/core/libraries/network/net.cpp (+1 -0)
📝 src/core/libraries/network/sys_net.cpp (+1 -0)

📄 Description

sceNetEpollDestroy and sys_socketclose should delete the descriptors associated with their file.
Not doing so led to some games (particularly from EA) blowing up the fd table with thousands of epolls and sockets.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/shadps4-emu/shadPS4/pull/3976 **Author:** [@StevenMiller123](https://github.com/StevenMiller123) **Created:** 1/29/2026 **Status:** ✅ Merged **Merged:** 1/29/2026 **Merged by:** [@georgemoralis](https://github.com/georgemoralis) **Base:** `main` ← **Head:** `net-fd-fixes` --- ### 📝 Commits (1) - [`7e7574a`](https://github.com/shadps4-emu/shadPS4/commit/7e7574ac18445f2c04f1ff1ca8a3df7eb5945cdc) Delete unused fds in sceNetEpollDestroy and sys_socketclose ### 📊 Changes **2 files changed** (+2 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `src/core/libraries/network/net.cpp` (+1 -0) 📝 `src/core/libraries/network/sys_net.cpp` (+1 -0) </details> ### 📄 Description sceNetEpollDestroy and sys_socketclose should delete the descriptors associated with their file. Not doing so led to some games (particularly from EA) blowing up the fd table with thousands of epolls and sockets. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>