https://pastebin.com/QsM4Cvdc
I've recently added your SRS statistic plugin to my server and I realized the old memory leak issue has returned. Can't confirm if it's related to the stats plugin. My server running for about 6 hours return 600+ "l4d2_shop.smx IPreparedQuery" values.
I also check error logs etc and did not find anything. Do you think adding the delete handle to error clauses will fix this?
(this code is just a sample!)
Code:
int LoadPoints(int client)
{
Database db = ConnectDB();
if (db != null)
{
char steamId[32];
int db_points = 0;
char error[255];
Handle hSelectStmt = INVALID_HANDLE;
GetClientAuthId(client, AuthId_Steam2, steamId, sizeof(steamId));
///////////////////////// Load special functions
if ((hSelectStmt = SQL_PrepareQuery(db, "SELECT functionNo, isEnabled FROM SpecialFunctions WHERE steamId = ?", error, sizeof(error))) == INVALID_HANDLE)
{
LogError("[ERROR]: SELECT SQL_PrepareQuery: \"%s\"", error);
delete hInsertStmt;
delete hSelectStmt;
return 1;
}
// Disabled all special function first
for(int i =0; i< SF_SIZE; i++)
{
g_bClientSF[client][0][i] = false;
g_bClientSF[client][1][i] = false;
}
SQL_BindParamString(hSelectStmt, 0, steamId, false);
if (SQL_Execute(hSelectStmt))
{
while(SQL_FetchRow(hSelectStmt))
{
char functionNo[32];
SQL_FetchString(hSelectStmt, 0, functionNo, sizeof(functionNo));
int index = GetFunctionIndex(functionNo);
bool isEnabled = SQL_FetchInt(hSelectStmt, 1) == 1;
g_bClientSF[client][0][index] = isEnabled;
g_bClientSF[client][1][index] = true;
}
}
///////////////////////////////////////////////////
////////////////////////////////// Load points
if ((hSelectStmt = SQL_PrepareQuery(db, "SELECT steamId,points FROM Shop WHERE steamId = ?", error, sizeof(error))) == INVALID_HANDLE)
{
LogError("[ERROR]: SELECT SQL_PrepareQuery: \"%s\"", error);
return db_points;
}
SQL_BindParamString(hSelectStmt, 0, steamId, false);
// Find the client in the database
if (SQL_Execute(hSelectStmt))
{
char playerName[64];
GetClientName(client, playerName, sizeof(playerName));
if(SQL_FetchRow(hSelectStmt))
{
SQL_FetchString(hSelectStmt, 0, steamId, sizeof(steamId));
db_points = SQL_FetchInt(hSelectStmt, 1);
LogAction(client, -1, "[LOAD]: \"%L\" loaded the record successfully! Points: \"%d\"", client, db_points);
}
else //INSERT
{
// if the user is not existed in the database, insert new one.
Handle hInsertStmt = INVALID_HANDLE;
if ((hInsertStmt = SQL_PrepareQuery(db, "INSERT INTO Shop(steamId, points) VALUES(?,?)", error, sizeof(error))) == INVALID_HANDLE)
{
LogError("[ERROR]: INSERT SQL_PrepareQuery: \"%s\"", error);
delete hInsertStmt;
delete hSelectStmt;
return 1;
}
else
{
SQL_BindParamString(hInsertStmt, 0, steamId, false);
SQL_BindParamInt(hInsertStmt, 1, 0, false); // Default point is 0
if (!SQL_Execute(hInsertStmt))
{
LogError("[ERROR]: INSERT error Error: \"%s\"", error);
}
int rs = SQL_GetAffectedRows(hInsertStmt);
if(rs>0)
{
LogAction(client, -1, "[INSERT]: \"%L\" is a new user!!", client);
}else
{
LogError("[ERROR]: \"%L\" made insert error \"%s\"", client, error);
delete hInsertStmt;
delete hSelectStmt;
return 1;
}
}
delete hInsertStmt;
}
}
delete hSelectStmt;
return db_points;
}
delete db;
return 0;
}
or can the plugin just clear/delete all the handles on map change? so that it does not accumulate over time?
__________________
Linear Mode
