[Addons] Weapon Skin Manager

[thegame]

Использую плагин Мистрика и Реапи.

public CBasePlayer_TakeDamage_Pre(const this, pevInflictor, pevAttacker, Float:flDamage, bitsDamageType)
{   
    if(is_user_connected(pevAttacker) && get_user_flags(pevAttacker) & WM_ACCESS_FLAG && pevAttacker != this)
    {
        new weapon = get_member(pevAttacker, m_pActiveItem);
    
        if(wss_get_weapon_skin_index(weapon) && !(bitsDamageType & (DMG_GRENADE|DMG_BLAST)))
        {
            SetHookChainArg(4, ATYPE_FLOAT, flDamage * 1.4);
        }
    }
}

Получаем ошибку

L 02/21/2018 - 20:52:47: [FAKEMETA] Invalid entity
L 02/21/2018 - 20:52:47: [AMXX] Run time error 10 (plugin "weapon_skin_system.amxx") (native "pev") - debug not enabled!
L 02/21/2018 - 20:52:47: [AMXX] To enable debug mode, add "debug" after the plugin name in plugins.ini (without quotes).
L 02/21/2018 - 20:52:47: Unhandled dynamic native error
L 02/21/2018 - 20:52:47: [AMXX] Displaying debug trace (plugin "power_damage.amxx", version "0.1")
L 02/21/2018 - 20:52:47: [AMXX] Run time error 10: native error (native "wss_get_weapon_skin_index")

//line

 if(wss_get_weapon_skin_index(weapon) && !(bitsDamageType & (DMG_GRENADE|DMG_BLAST)))
//

Суть: new weapon = get_member(pevAttacker, m_pActiveItem); посылает некорректный энтити в плагин мистрика, тот в свою очередь тупо оффает сервер.
Тестил только с этими двумя плагинами, поэтому "конфликт или другие плагины" исключены сразу...

Мистрик призываю Тебя.

 

[Minni]

А где return HC_CONTINUE; в конце функции?

Да и нужен включенный debug на плагине, ибо ошибка не из-за ReAPI.

 

[thegame]

Minni, по поводу return HC_CONTINUE ------->
Если до этого есть какие-то ретурны то определенно нужною. А если оно там просто так, то в принципе нет, по умолчанию функция и так продолжает работать.

По поводу дебага. Дебаг на мистрика плагине не происходит, а там стоит пункт debug. дебажится только сам плагин-аддон. И я неоднократно слышал от неугомона, макарона, что m_pActiveItem возвращает не правильный индекс энтити в игре.

 

[Minni]

Если до этого есть какие-то ретурны то определенно нужною. А если оно там просто так, то в принципе нет, по умолчанию функция и так продолжает работать.

Порядок должен быть в любом случае. Не нужно разводить грязь.

По поводу дебага. Дебаг на мистрика плагине не происходит, а там стоит пункт debug. дебажится только сам плагин-аддон. И я неоднократно слышал от неугомона, макарона, что m_pActiveItem возвращает не правильный индекс энтити в игре.

Да, но прикол в том, что это от FakeMeta, а не от ReAPI.

L 02/21/2018 - 20:52:47: [FAKEMETA] Invalid entity
L 02/21/2018 - 20:52:47: [AMXX] Run time error 10 (plugin "weapon_skin_system.amxx") (native "pev") - debug not enabled!

21 Фев 2018

И по-сути, больше инфы нет. Включи дебаг, предоставь нормальные логи ошибок и тогда будет проще найти ошибку. А пока - это бессмысленно.

 

[thegame]

Minni, а это от плагина использующего его инклюды.
[AMXX] Run time error 10: native error (native "wss_get_weapon_skin_index")

Плагин-аддон ----> запрашивает у плагина мистрика имеет ли этот энтити скин ? Плагин отвечает Да\Нет ( при условии что энтити вообще валиден)
если плагин аддон-посылает невалидный энт ,тот тупо крашится.

Спойлер: Что то для Минни

>Порядок должен быть в любом случае. Не нужно разводить грязь.
- Скажи это ПростоТеме, он покажет как оформлять надо ;)

и так вдобавок о Вашей находчивости.
https://dev-cs.ru/threads/1032/post-10395

там вы тоже говорили, что дело не в плагине, это у меня что-то не так работает...но увы оказалось реально баг ------- вот так поворот)

 

[Sonyx]

thegame, ошибку сыпет все время? Не из-за грены ли весь сыр-бор?

 

[thegame]

и да Minni, за попытки помощи спасибо, у меня нету желания батл тут какой-то устраивать, просто хочу довести до ума работу плагина.

Одним из вариантов было предложено сделать так:

в плагине мистрика

public native_get_weapon_skin_index(plugin, params)
{
    enum { arg_weapon = 1 };
   
    new weapon = get_param(arg_weapon);
   
+   if(!is_valid_pev(weapon)) return 0;
   
    return get_weapon_skin(weapon);
}

но в течении минут 20 поймал выедание ядра на 100%

21 Фев 2018

Sonyx, нет ошибка появляется один раз и сразу офф сервер

21 Фев 2018

та и дамаг от грены я исключил if(wss_get_weapon_skin_index(weapon) && !(bitsDamageType & (DMG_GRENADE|DMG_BLAST)))

 

[Sonyx]

thegame, ну во первых вы дамаг от грены исключили только для множителя, а в натив все летит.
во вторых попробуйте проверять в хуке валидность:

public CBasePlayer_TakeDamage_Pre(const this, pevInflictor, pevAttacker, Float:flDamage, bitsDamageType)
{   
    if(is_user_connected(pevAttacker) && get_user_flags(pevAttacker) & WM_ACCESS_FLAG && pevAttacker != this)
    {
        new weapon = get_member(pevAttacker, m_pActiveItem);
        if(!is_entity(weapon))
        {
            return HC_CONTINUE;
        }
        if(wss_get_weapon_skin_index(weapon) && !(bitsDamageType & (DMG_GRENADE|DMG_BLAST)))
        {
            SetHookChainArg(4, ATYPE_FLOAT, flDamage * 1.4);
        }
    }
}

 

[thegame]

Sonyx, делал примерно так же, но вот так:

public CBasePlayer_TakeDamage_Pre(const this, pevInflictor, pevAttacker, Float:flDamage, bitsDamageType)
{  
    if(is_user_connected(pevAttacker) && get_user_flags(pevAttacker) & WM_ACCESS_FLAG && pevAttacker != this)
    {
        new weapon = get_member(pevAttacker, m_pActiveItem);
       
-       if(wss_get_weapon_skin_index(weapon) && !(bitsDamageType & (DMG_GRENADE|DMG_BLAST)))
+       if(is_entity(weapon) && wss_get_weapon_skin_index(weapon) && !(bitsDamageType & (DMG_GRENADE|DMG_BLAST)))
        {
            SetHookChainArg(4, ATYPE_FLOAT, flDamage * 1.4);
        }
    }
}

 

[Minni]

у меня нету желания батл тут какой-то устраивать

О чем речь? Какой батл? Я чего-то не знаю?

 

[Salvatore]

new weapon = get_member(pevAttacker, m_pActiveItem);

=>

new pActiveItem = get_member(pevAttacker, m_pActiveItem);
new pWeaponID = get_member(pActiveItem, m_iId);

Передавай не ActiveItem, а WeaponID.

 

[thegame]

Salvatore, возьми передай и увидишь, что будет. Ответ: Ничего, ибо там идет отлов индекса энтити в игре

 

[Salvatore]

thegame, а ты пробовал, прежде чем ответить?

 

[Mistrick]

Передавай не ActiveItem, а WeaponID.

Куда вы собрались передавать WeaponID? Мой плагин хранит индекс скина в оружии(энтити), а не в константе.

 

[thegame]

Mistrick, скажите может ли вот эта проверка от ПростоТемы в вашем плагине создать выедание ядра на 100%?

public native_get_weapon_skin_index(plugin, params)
{
    enum { arg_weapon = 1 };
  
    new weapon = get_param(arg_weapon);
  
+   if(!is_valid_pev(weapon)) return 0;
  
    return get_weapon_skin(weapon);

 

[Mistrick]

thegame, не помню чтобы подобное даже в синке могло грузить. Но я бы лучше проверял валидность оружия до вызова wss_get_weapon_skin_index(), чем добавлять проверку в натив.

 

[thegame]

Mistrick, к сожалению в реапи is_entity() пропускает летящий невалидный энт и получаем ошибку + офф.
Как вариант подключить или фейкмету или енджин и через нее делать проверку на валидность в самом плагине, а натив оставить впокое

 

[Salvatore]

thegame, ты пытаешься исключить гранату? Или что?

 

[thegame]

Salvatore, невалидный энт

 

[Sonyx]

thegame, для улучшения понимания картины, прологируй id ентити сразу после получения.