Утечка памяти

[diller110]

Помогите разобраться, от чего может происходить утечка:

[SM] Plugin encountered error 17: Stack memory leaked by native
L 02/26/2015 - 00:48:14: [SM] Displaying call stack trace for plugin "ac-core.smx":
L 02/26/2015 - 00:48:14: [SM]   [0]  Line 169, ac-core.sp::Event_PlayerTeam()

Вот код:
Оффтоп

public Event_PlayerTeam(Handle:event, const String:name[], bool:dontBroadcast)
{
new client = GetClientOfUserId(GetEventInt(event, "userid"));

if (GetClientDeaths(client) == 0 && GetClientFrags(client) == 0 && client>0) {
new Adms[33],count = 0;
for (new i = 1; i <= MaxClients; i++)
if (IsClientInGame(i) && (GetUserFlagBits(i) & ADMFLAG_GENERIC)) Adms[count++] = i;
new String:Admins[count] [];
if (count > 0)
{
for (new i = 0; i < count; i++)
{
if (GetUserFlagBits(Adms)) GetClientName(Adms, Admins[count], 30);
}
}
WelcomeTimers[client][0] = CreateTimer(3.0, Timer1, client);
new Handle:pack = CreateDataPack();
WritePackCell(pack, client);
WritePackCell(pack, count);
WritePackString(pack, Admins[0]);
WritePackString(pack, Admins[1]);

if(IsClientInGame(client)) WelcomeTimers[client][1] = CreateTimer(6.0, Timer2, client);
if(IsClientInGame(client)) WelcomeTimers[client][2] = CreateTimer(10.0, Timer3, pack);
if(IsClientInGame(client)) WelcomeTimers[client][3] = CreateTimer(14.0, Timer4, client);
}
}

public Action:Timer3(Handle:timer, any:pack) {
new client = ReadPackCell(pack);
new count = ReadPackCell(pack);
decl String:admin1[30];
decl String:admin2[30];
ReadPackString(pack, admin1, 30);
ReadPackString(pack, admin2, 30);
CloseHandle(pack);
if (count > 0)
{
if(IsClientInGame(client)) PrintHintText(client, "<font size='26' color='#6699FF'> Админы онлайн: </font><br><font color='#0066FF' size='20'> %s<br>%s</font>", admin1, admin2);
}
}

Посмотреть вложение hintmesseger.sp

Строка указанная в ошибке:
new String:Admins[count] [];

 

[FrozDark]

ac-core.smx вот это сюда

 

[diller110]

Посмотреть вложение ac-core.sp
Плагин для экспериментов, если есть какие-то грубые ошибки, прошу указать.

 

[diller110]

Так и не смог понять. Возможно, забываю закрыть какой-то handle. Таймеры все разовые, вроде должны удалять по завершению. Единственный Datapack закрываю в таймере, не уверен что произойдет если закрыть его сразу после посылки в таймер на 10 секунд, скопируется ли он или уничтожится не дождавшись выполнения.

Вот обособленный плагин: Посмотреть вложение ac-hintmessenger.sp. Сама ошибка: Оффтоп

[SM] Plugin encountered error 17: Stack memory leaked by native
[SM] Displaying call stack trace for plugin "ac-hintmessenger.smx":
[SM] [0] Line 33, ac-hintmessenger.sp::Event_PlayerTeam()

И еще теоретический вопрос: Допустим надо вывести в чат сообщение через 5 секунд после присоединения к команде. В player_team ивенте создаем таймер. Будет ли он создан конкретно для этого игрока, или при заходе другого игрока этот же таймер пойдет заного? Если 2-е, то как эффективнее всего создавать подобное?
new Handle:WelcomeTimers[33][4]; - по ощущениям тяжеловато. И возможно утечка памяти именно в этом.

 

[diller110]

время таймера заведомо мало

Нет, последний таймер длится 14 секунд, почти вечность, и без handle не обойтись, чтобы убить таймер. Добавил уничтожение при выходе.

Но где-же утечка?

 

[diller110]

Да намного проще и эффективнее) Спасибо!

 

[diller110]

AlmazON, Все хорошо работает, кроме вывода админов. И снова мозгодробительные ошибки.
Таймеры срабатывают успешно, добавлял в 3 таймер вывод: Timer 3 finished, без проблем выводило, но в хинты не хотел. Написал себе понятную версию вместо:
(Оба варианта в файле Посмотреть вложение hintmesseger.sp)
Оффтоп

public Action:Timer3(Handle:timer, any:userid) {
new client = GetClientOfUserId(userid);
if (client)
{
CreateTimer(4.0, Timer4, userid, TIMER_FLAG_NO_MAPCHANGE);
new admin1, admin2;
for (userid = 1; userid <= MaxClients; userid++) if (IsClientInGame(userid) && (GetUserFlagBits(userid) & ADMFLAG_GENERIC))
{
if(admin1)
{
admin2 = userid;
break;
}admin1 = userid;
}
if (admin2) PrintHintText(client, "<font size='26' color='#6699FF'> Админы онлайн: </font><br><font color='#0066FF' size='20'> %N<br>%N</font>", admin1, admin2);
else if (admin1) PrintHintText(client, "<font size='26' color='#6699FF'> Админ онлайн: </font><br><font color='#0066FF' size='20'> %N<br></font>", admin1);
//else PrintHintText(client, "Админов онлайн нет!");
}
return Plugin_Stop;
}

Вот такую (опять с массивом и лишним счетчиком):
Оффтоп

public Action:Timer3(Handle:timer, any:userid) {
new client = GetClientOfUserId(userid);
if(client)
{
CreateTimer(3.0, Timer4, userid, TIMER_FLAG_NO_MAPCHANGE);
new String:Admins[3][30], number;
for (new i = 1; i <= MaxClients; i++) if (IsClientInGame(i) && ((GetUserFlagBits(i) & ADMFLAG_SLAY) || (GetUserFlagBits(i) & ADMFLAG_ROOT)) && number<=2)
{
GetClientName(GetClientOfUserId(i), Admins[number], 30);
number++;
}
PrintHintText(client, "<font size='26' color='#6699FF'>Админы в онлайне:</font><br><font color='#0066FF' size='20'> %N %N</font>", Admins[0], Admins[1]);
}
return Plugin_Stop;
}

Ошибка заключалась в том, что PrintHintText() - не мог найти client с номером 167*****(видимо макс целочисленное). Как вообще такое возможно не понимаю. Client от момента получения никак не юзается. Userid еще интереснее, пишет ошибку например что userid=209 не найден. В то время как я именно 209 id.

В остальном все работает хорошо! Можешь еще подсказать, что & считает в этой операции: GetUserFlagBits(i) & ADMFLAG_ROOT)?

И еще: Можно в if в скобки вводить условия? && ((GetUserFlagBits(i) & ADMFLAG_SLAY) || (GetUserFlagBits(i) & ADMFLAG_ROOT)) &&

 

[The End Is Near...]

diller110
Ты используешь userid О_О ты же клиента получаешь, вот его и юзай
а если админов больше 2?

 

[AlmazON]

Hint Messenger v1.0 (отвязано от ядра)

Всё работает, за ваши собственные "багоисправления" я не отвечаю! :)
Отвязал плагин от ядра, добавил проверку на ботов (дабы зря система дальше от них не работала), поменял доступ на флаг ROOT, убрал все цвета - все сообщения были отображены (и да, меня, как админа отобразило).
Даже ошибок при компиляции нет, которые могли быть по невнимательности, а ещё жалуются! :-D

 

[diller110]

Спасибо за старания, добавил цвета (вставил из прошлого, поэтому не в них было дело). Все работает отлично!

Интересно все-таки: GetUserFlagBits(userid) & ADMFLAG_ROOT, эквивалентно ==, или что он высчитывает?

а если админов больше 2?

В хинт строки ограничены, поэтому и 2 хватит.

 

[AlmazON]

что он высчитывает

Количество битов. & - побитовый И или побитовое сравнение. Биты смотри в константах include функций.

вставил из прошлого, поэтому не в них было дело

Во флаге было дело - нет такого флага ни у кого (отдельного), вот и сообщения нет.