Иконка ресурса
- Автор Aidar12
- Дата создания
Обратите внимание: этот ресурс не поддерживается разработчиком.
Что исправлено:
1. ИСПРАВЛЕНИЕ УТЕЧЕК ПАМЯТИ (MEMORY LEAKS)
sourcepawn
// БЫЛО (утечка памяти):
new Handle:hQuery = SQL_Query(g_hDatabase, query);
// ... работа с запросом ...
// НЕТ CloseHandle(hQuery) - ПАМЯТЬ НЕ ОСВОБОЖДАЛАСЬ!
// СТАЛО (память освобождается):
new Handle:hQuery = SQL_Query(g_hDatabase, query);
if (hQuery != INVALID_HANDLE)
{
// ... работа с запросом ...
CloseHandle(hQuery); // ВАЖНО! Всегда закрываем
}
Где исправлено: Во всех SQL запросах добавлен CloseHandle()
2. ЗАЩИТА ОТ SQL-ИНЪЕКЦИЙ
Проблема:
Злоумышленник мог ввести специальные символы в ник или причину бана и выполнить свой SQL код.
Что исправлено:
sourcepawn
// БЫЛО (опасно):
Format(query, sizeof(query), "SELECT * FROM specbans WHERE name = '%s'", name);
// СТАЛО (безопасно):
decl String:escapedName[SPECBAN_NAME_LENGTH * 2 + 1];
SQL_EscapeString(g_hDatabase, name, escapedName, sizeof(escapedName));
Format(query, sizeof(query), "SELECT * FROM specbans WHERE name = '%s'", escapedName);
Где исправлено: Во всех SQL запросах с пользовательским вводом
3. ИСПРАВЛЕНИЕ КРАШЕЙ ПРИ ДИСКОННЕКТЕ
Проблема:
Таймеры использовали прямой client индекс, который становился невалидным после выхода игрока.
Что исправлено:
sourcepawn
// БЫЛО (краш при дисконнекте):
CreateTimer(1.0, Timer_Unban, client);
// СТАЛО (безопасно):
CreateTimer(1.0, Timer_Unban, GetClientUserId(client));
// И в таймере:
public Action:Timer_Unban(Handle:timer, any:userid)
{
new client = GetClientOfUserId(userid); // Получаем клиента по userid
if (!client) return Plugin_Stop; // Если игрок ушел - ничего не делаем
// ... остальной код
}
Где исправлено: Во всех таймерах (Timer_Unban, Timer_ForceSpec, Timer_DelayedHint)
4. ИСПРАВЛЕНИЕ ОТРИЦАТЕЛЬНОГО ВРЕМЕНИ
Проблема:
Админ мог ввести отрицательное время бана, что ломало логику.
Что исправлено:
sourcepawn
// БЫЛО:
SpecBanPlayer(client, target, time, reason); // time мог быть -5
// СТАЛО:
if (time < 0)
{
LogError("Попытка бана с отрицательным временем: %d", time);
time = 0; // Превращаем в перманентный бан
}
5. ИСПРАВЛЕНИЕ ДУБЛИРОВАНИЯ КОДА
Проблема:
Один и тот же код для поиска SteamID в списке был в нескольких местах.
Что исправлено:
sourcepawn
// БЫЛО: код повторялся 3 раза
for (new i = 0; i < count; i++)
if (strcmp(list, steamid) == 0) return true;
// СТАЛО: вынесли в отдельную функцию
bool:IsSteamIDInList(String:list[][SPECBAN_STEAMID_LENGTH], count, const String:steamid[])
{
for (new i = 0; i < count; i++)
if (strcmp(list, steamid) == 0) return true;
return false;
}
6. ИСПРАВЛЕНИЕ ДЛЯ SM 1.1 СИНТАКСИСА
Проблема:
Новый синтаксис (char[], public void) не работает на старых версиях SourceMod.
Что исправлено:
sourcepawn
// БЫЛО (новый синтаксис):
public void OnPluginStart() { }
char name[64];
ArrayList list;
// СТАЛО (старый синтаксис SM 1.1):
public OnPluginStart() { }
new String:name[64];
new Handle:list;
Все изменения:
- char[] → String:[]
- public void → public
- delete → CloseHandle
- ArrayList → Handle
- Menu → Handle с CreateMenu/AddMenuItem
7. ОПТИМИЗАЦИЯ БАЗЫ ДАННЫХ
Добавлены индексы для ускорения:
sql
idx_steam_history_main - поиск связанных аккаунтов
idx_steam_history_linked - поиск по связанным SteamID
idx_name_history_steam - история ников по SteamID
idx_ban_history_steam - история банов
idx_ban_history_name - поиск по никам
idx_ban_history_endtime - автоочистка истекших банов
8. ИСПРАВЛЕНИЕ РУССКИХ СИМВОЛОВ
Проблема:
Русские символы в никах вызывали ошибки SQL.
Что исправлено:
sourcepawn
// БЫЛО (краш с русскими символами):
Format(query, sizeof(query), "INSERT INTO name_history (name) VALUES ('%s')", name);
// СТАЛО (работает с любыми символами):
SQL_EscapeString(g_hDatabase, name, escapedName, sizeof(escapedName));
Format(query, sizeof(query), "INSERT INTO name_history (name) VALUES ('%s')", escapedName);
9. ИСПРАВЛЕНИЕ СКЛОНЕНИЙ ВРЕМЕНИ
Проблема:
Неправильные окончания: "1 минут", "2 минут", "5 минута"
Что исправлено:
sourcepawn
// ТЕПЕРЬ ПРАВИЛЬНО:
1 минута
2 минуты
5 минут
11 минут (исключение)
21 минута
10. ОЧИСТКА ПРИ ПЕРЕЗАГРУЗКЕ
Добавлена функция ResetPlayerBanData:
sourcepawn
ResetPlayerBanData(client)
{
// Очищаем все данные игрока при выходе
if (g_hSpecBanTimer[client] != INVALID_HANDLE)
{
KillTimer(g_hSpecBanTimer[client]);
g_hSpecBanTimer[client] = INVALID_HANDLE;
}
g_bPermanentSpecBan[client] = false;
g_iBanEndTime[client] = 0;
g_sBanReason[client][0] = '\0';
// ... остальные данные
}
ИТОГ: ЧТО БЫЛО ИСПРАВЛЕНО
Проблема Было Стало Утечки памяти 12+ мест
Все CloseHandle()
SQL-инъекции Краши при дисконнекте Отрицательное время Дублирование кода Русские символы Склонения времени Индексы БД Очистка памяти РЕЗУЛЬТАТ
Плагин теперь:
Версия 1.5 - стабильный релиз без утечек памяти и критических багов!
