backups/MinecraftConsoles
2
0
Fork 1
mirror of https://github.com/smartcmd/MinecraftConsoles.git synced 2026-08-20 09:57:09 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix memory leak by listing info pointers

Browse source
This commit is contained in:
Siepert123 2026-03-04 22:12:41 +01:00
parent 91a4fc2f6c
commit cbc93765e3
1 changed files with 15 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
Minecraft.Client/Common/Network/PlatformNetworkManagerStub.cpp
View file

@ -697,6 +697,8 @@ void CPlatformNetworkManagerStub::TickSearch()
#endif #endif
} }
std::vector<FriendSessionInfo*>* allocatedServers = NULL;
void CPlatformNetworkManagerStub::SearchForGames() void CPlatformNetworkManagerStub::SearchForGames()
{ {
#ifdef _WINDOWS64 #ifdef _WINDOWS64
@ -732,9 +734,18 @@ void CPlatformNetworkManagerStub::SearchForGames()
friendsSessions[0].push_back(info); friendsSessions[0].push_back(info);
} }
//TODO: add file checking for servers.txt
#ifdef _WINDOWS64 //Should we have this windows64 only? idk i havent tested so... #ifdef _WINDOWS64 //Should we have this windows64 only? idk i havent tested so...
ifstream ServersTxt("servers.txt"); ifstream ServersTxt("servers.txt");
//Memory leak prevention
if (allocatedServers != NULL) {
for (FriendSessionInfo* info : *allocatedServers) {
delete(info);
}
delete(allocatedServers);
}
allocatedServers = new vector<FriendSessionInfo*>;
if (ServersTxt) { if (ServersTxt) {
string line; string line;
@ -774,6 +785,9 @@ void CPlatformNetworkManagerStub::SearchForGames()
info->data.hostPort = stoi(port); info->data.hostPort = stoi(port);
info->sessionId = (SessionID)((unsigned __int64)inet_addr(ip.c_str()) | ((unsigned __int64)stoi(port) << 32)); info->sessionId = (SessionID)((unsigned __int64)inet_addr(ip.c_str()) | ((unsigned __int64)stoi(port) << 32));
friendsSessions[0].push_back(info); friendsSessions[0].push_back(info);
//save for later deletion!
allocatedServers->push_back(info);
} }
} }
ServersTxt.close(); ServersTxt.close();