Проблема с ReHLDS build > 788

Refresh · 26 Апр 2024

После обновления с ReHLDS билда 788, начались проблемы с именами игроков:

Хронология, примерно, такая:

Код:

L 04/18/2024 - 17:39:32: [ACS_LOG] session: ( 3 ), check_trust: ALL-MS | BOSS < STEAM_1:0:1397272665 > = 58.103118, suspected = 0
L 04/18/2024 - 17:39:33: [ACS_LOG] session: (  3 ), check vpn: 84.54.70.190 [ STEAM_1:0:1397272665 ]
L 04/18/2024 - 17:40:46: [ACS_LOG] session: (  3 ), SetClientUserInfoName(), new_name = владик, punish_processed = 0, punish_flag = 0
L 04/18/2024 - 17:43:04: [ACS_LOG] session: (  3 ), DROPPED!!!, is_drop = 1, msg = Client sent 'drop'
L 04/18/2024 - 17:43:05: [ACS_LOG] session: ( 3 ), new player = владик [ 178.120.X.X:0 ] < STEAM_1:1:7200XXXXXX >

На полном сервере игрок желающий подключиться постоянно жмет кнопочку обновить и как только предыдущий игрок выходит с сервера, тут же подключается новый... НО, под ником предыдущего. Если перезайти (отключиться+подключиться) - то ник обновляется на установленный у нового игрока в настройках клиента. Если же будет просто смена карты - ник все равно останется прежнего игрока, не смотря на настройки клиента.

HLTV вчера у нас тоже зашла с ником игрока и никакие танцы с бубном до полного перезапуска сервера не помогли.

Это какой-то глюк сборки от Арены или баг в ReHLDS, возможно, связанный с ошибкой в SetClientUserInfoName()?

Nordic Warrior · 26 Апр 2024

Refresh написал(а):
Это какой-то глюк сборки от Арены?

В чем проблемы поставить оригинальную версию и проверить?

Refresh · 26 Апр 2024

Nordic Warrior, проблема в наличии 32 игроков... я с ботами не могу повторить подобного сценария.

Nordic Warrior · 26 Апр 2024

Refresh, уверен что дело в рехлдс? У меня была такая проблема, когда я давно ещё плохо разбирался, и написал некачественный плагин блокировки смены ников, и возникали ровно такие же ситуации, что иногда у игрока подхватывался ник другого.
Если есть такой функционал, то стоит его отключить.

Refresh · 26 Апр 2024

Nordic Warrior, это началось с обновления, если бы была проблема ДО - ее уже давно бы заметили. У меня есть код в обработчике SetClientUserInfoName() но он срабатывает только если пользователь меняет ник находясь на сервере (никаких строк отладки этого события в процессе коннекта я никогда не видел). При коннекте ничего что бы могло влиять на "имя пользователя" в любых вариациях отсутствует.

В client_connect() имя пользователя выдается от предыдущего игрока.

Была аналогичная проблема с HLTV имеющая на сервере абсолютный иммунитет, никакие процессы не могут быть запущены для игрока is_user_hltv(), поэтому я уверен, что это проблема на стороне сервера.

Даже если принять во внимание гипотетическую возможность блокировки события SetClientUserInfoName(), например, в процессе работы плагина заблокировалась смена ника предыдущим игроком:

Код:

SetHookChainReturn(ATYPE_BOOL, false);
return HC_BREAK;

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

Nordic Warrior · 26 Апр 2024

Refresh, проверяй на практике. Отключай этот функционал, ставь официальные билды рехлдс, перебором ищи билд, с которого всё пошло не так, если это в рехлдс дело. Гадать сколько угодно можно. Я посмотрел последние коммиты рехлдс, вроде не увидел ничего связанного с name

Refresh · 29 Апр 2024

Nordic Warrior, Я смог устойчиво воспроизвести проблему:

Игрок мертвый. Висит в спектраторах или в своей команде.
Меняет имя: name ZZZTT2S. Имя не изменится сразу до следующего спавна. Я это корректирую, чтобы в TAB-е отображалось корректно:
Код:
```
set_entvar(id, var_netname, new_name)
```
Игрока дропает с сервера или он уходит сам.
В его слот коннектится новый игрок. Причем у него будет тот ник на который предыдущий игрок его изменил и не дождался спавна... В моем примере ZZZTT2S.

Попробую еще установить set_user_info(...) или при дисконнекте var_netname обнулять, но в алгоритме изменения имени игрока очевидный косяк... Если игрок отключится, дальнейшие задачи по нему выполнять бесмысленно. Я попробую поискать ошибку, у вас.

Upd: В дисконнекте set_entvar(id, var_netname, ""); решило проблему... Наверно, нужно сделать аналогично в ReHLDS.

Upd2: Корректно так:

Код:

public client_disconnected(id, bool:drop, message[], maxlen) {
    if (!is_nullent(id))
        set_entvar(id, var_netname, 0);
}

Если не проверять:

Код:

[ReAPI] set_entvar: invalid or uninitialized entity

CSBishop · 29 Апр 2024

Аналогично встречал проблему если менять ник через var_netname, через set_user_info проблема исчезла

Nordic Warrior · 29 Апр 2024

Refresh, ну как я и думал, всё-таки из-за вмешательства плагинами была проблема.

Refresh · 30 Апр 2024

Nordic Warrior, :scratch_one-s_head:

а мне показалось, что если игрок отключился и новый игрок подключается к серверу, то сервер должен брать имя нового игрока, а не предыдущего... в 100 из 100 случаях. Но это не точно... Поскольку логику разрабов понять довольно сложно. Возможно, плагин одной командой сломал сервер.

30 Апр 2024

CSBishop, через set_user_info не меняет сразу ник мертвому игроку или наблюдателю :sad:

Vaqtincha · 30 Апр 2024

Скорее всего это баг в регейме.

Refresh написал(а):
Имя не изменится сразу до следующего спавн

Он хранится в мемберах временно и восстанавливается при след. спавне.

https://github.com/s1lentq/ReGameDLL_CS/blob/master/regamedll/dlls/player.cpp#L176

https://github.com/s1lentq/ReGameDLL_CS/blob/master/regamedll/dlls/player.cpp#L5882

Я вообще не вижу в коде сброс этих 2 мемберов при дисконнекте.
В обычных условиях возможно будет работать нормально. Но при

Refresh написал(а):
наличии 32 игроков

Возможен такой сценарий.

30 Апр 2024

Vaqtincha написал(а):
Скорее всего это баг в регейме.

Хотя может быть это более глубокий баг движка если оно возникнет только при 32 игроках.

Refresh · 30 Апр 2024

Сейчас еще попробую сбросить два мембера:

Код:

Public:
    bool m_bHasChangedName;   
    char m_szNewName[MAX_PLAYER_NAME_LENGTH];

Vaqtincha · 30 Апр 2024

Refresh, А баг стабильно воспроизводится?

Refresh · 30 Апр 2024

Vaqtincha, да, ~~тут можно попробовать 93.85.88.50:27017~~. Пофиксил.

Refresh написал(а):
Сейчас еще попробую сбросить два мембера:

Код:

Public: bool m_bHasChangedName; char m_szNewName[MAX_PLAYER_NAME_LENGTH];

Это не работает, ни в CBasePlayer_SetClientUserInfoName ни в client_disconnected(). Пока рабочий вариант обнуление var_netname при дисконнекте.

30 Апр 2024

Vaqtincha, Да. Количество игроков не имеет значения. Просто когда мало игроков, вероятность такой ситуации - входа в слот где предыдущий игрок "мертвым" изменил имя и вышел до респавна - значительно ниже.

Vaqtincha · 30 Апр 2024

Refresh, вспомнил свой фикс для ботов https://github.com/s1lentq/ReGameDLL_CS/commit/0dd9b97c37784a6130af12ab61f14e917bad2241
Там был такой же баг для ботов.

Refresh · 27 Май 2024

Лечение глюков ReGame:

Код:

public client_disconnected(id, bool:is_drop, message[], maxlen) {
    if (is_drop && !is_nullent(id))
        set_entvar(id, var_netname, 0);
}

CJIaByTu4 · 28 Май 2024

Refresh, не все тут скриптеры, поэтому дополню

Код:

#include <amxmodx>
#include <reapi>

public plugin_init()
{
    register_plugin("[ReAPI] Fix Name After Disconnect (ReGameDLL bug)", "0.1", "Refresh");
}

public client_disconnected(id, bool:is_drop, message[], maxlen)
{
    if (is_drop && !is_nullent(id))
    {
        set_entvar(id, var_netname, 0);
    }
}

Javekson · 30 Май 2024

Проблему воспроизвести не смог, дай свой полный плагин того, что ты нам накодил

Логи

Код:

// Первый заход с ником Javekson
rh_client_connected - user: 32 | username:  | netname:
client_connectex - user: 32 | username:  | netname:  | name: Javekson
client_connect - user: 32 | username: Javekson | netname:
client_authorized - user: 32 | username: Javekson | netname:
client_putinserver - user: 32 | username: Javekson | netname: Javekson

// Меняю ник на Seledka и в хуке SetClientUserInfoName ставлю var_netname
// В табе сразу видно новый ник Seledka
rh_client_change_name - user: 32 | username: Javekson | netname: Javekson | newname: Seledka
rh_client_change_name - user: 32 | username: Javekson | netname: Seledka | newname: Seledka

// Выхожу с сервера
client_disconnected - user: 32 | username: Seledka | netname: Seledka
rh_client_drop - user: 32 | username: Seledka | netname: Seledka
client_remove - user: 32 | username: Seledka | netname: Seledka

// Меняю ник на Vodka и только затем захожу на сервер

// Почему то сработал SetClientUserInfoName при коннекте
rh_client_change_name - user: 32 | username: Seledka | netname: Seledka | newname: Vodka
rh_client_change_name - user: 32 | username: Seledka | netname: Vodka | newname: Vodka

// Сам коннект
rh_client_connected - user: 32 | username: Seledka | netname: Vodka
client_connectex - user: 32 | username: Seledka | netname: Vodka | name: Vodka
client_connect - user: 32 | username: Vodka | netname: Vodka
client_authorized - user: 32 | username: Vodka | netname: Vodka
client_putinserver - user: 32 | username: Vodka | netname: Vodka

// Как видим в client_connect мой новый ник, какой и должен быть, в табе тоже

Плагин

Код:

#include amxmodx
#include reapi

public plugin_init() {
    //RegisterHookChain(RH_SV_ConnectClient, "rh_client_connect", .post = true);
    RegisterHookChain(RH_ClientConnected, "rh_client_connected", .post = true);
    RegisterHookChain(RH_SV_DropClient, "rh_client_drop", .post = true);
    RegisterHookChain(RG_CBasePlayer_SetClientUserInfoName, "rh_client_change_name", .post = true);
}

/*
public rh_client_connect() {
    log_to_file("test_name.log", "rh_client_connect -");
}
*/

public rh_client_connected(const user) {
    if (is_user_bot(user)) return;

    new netname[32];
    get_entvar(user, var_netname, netname, charsmax(netname));

    new username[32];
    get_user_name(user, username, charsmax(username));

    log_to_file("test_name.log", "rh_client_connected - user: %i | username: %s | netname: %s", user, username, netname);
}

public rh_client_drop(const user, const bool:crash, const fmt[]) {
    if (is_user_bot(user)) return;

    new netname[32];
    get_entvar(user, var_netname, netname, charsmax(netname));

    new username[32];
    get_user_name(user, username, charsmax(username));

    log_to_file("test_name.log", "rh_client_drop - user: %i | username: %s | netname: %s", user, username, netname);
}

public rh_client_change_name(const user, const info_buffer[], const new_name[]) {
    if (is_user_bot(user)) return;

    new netname[32];
    get_entvar(user, var_netname, netname, charsmax(netname));

    new username[32];
    get_user_name(user, username, charsmax(username));

    log_to_file("test_name.log", "rh_client_change_name - user: %i | username: %s | netname: %s | newname: %s", user, username, netname, new_name);

    /* ------------------------------ */
    
    set_entvar(user, var_netname, new_name);
    get_entvar(user, var_netname, netname, charsmax(netname));
    get_user_name(user, username, charsmax(username));
    log_to_file("test_name.log", "rh_client_change_name - user: %i | username: %s | netname: %s | newname: %s", user, username, netname, new_name);
}

public client_connect(user) {
    if (is_user_bot(user)) return;

    new netname[32];
    get_entvar(user, var_netname, netname, charsmax(netname));

    new username[32];
    get_user_name(user, username, charsmax(username));

    log_to_file("test_name.log", "client_connect - user: %i | username: %s | netname: %s", user, username, netname);
}

public client_connectex(user, const name[], const ip[], reason[128]) {
    if (is_user_bot(user)) return;

    new netname[32];
    get_entvar(user, var_netname, netname, charsmax(netname));

    new username[32];
    get_user_name(user, username, charsmax(username));

    log_to_file("test_name.log", "client_connectex - user: %i | username: %s | netname: %s | name: %s", user, username, netname, name);
}

public client_authorized(user, const authid[]) {
    if (is_user_bot(user)) return;

    new netname[32];
    get_entvar(user, var_netname, netname, charsmax(netname));

    new username[32];
    get_user_name(user, username, charsmax(username));

    log_to_file("test_name.log", "client_authorized - user: %i | username: %s | netname: %s", user, username, netname);
}

public client_putinserver(user) {
    if (is_user_bot(user)) return;

    new netname[32];
    get_entvar(user, var_netname, netname, charsmax(netname));

    new username[32];
    get_user_name(user, username, charsmax(username));

    log_to_file("test_name.log", "client_putinserver - user: %i | username: %s | netname: %s", user, username, netname);
}

public client_disconnected(user, bool:drop, message[], maxlen) {
    if (is_user_bot(user)) return;

    new netname[32];
    get_entvar(user, var_netname, netname, charsmax(netname));

    new username[32];
    get_user_name(user, username, charsmax(username));

    log_to_file("test_name.log", "client_disconnected - user: %i | username: %s | netname: %s", user, username, netname);
}

public client_remove(user, bool:drop, const message[]) {
    if (is_user_bot(user)) return;

    new netname[32];
    get_entvar(user, var_netname, netname, charsmax(netname));

    new username[32];
    get_user_name(user, username, charsmax(username));

    log_to_file("test_name.log", "client_remove - user: %i | username: %s | netname: %s", user, username, netname);
}

Refresh · 30 Май 2024

Javekson, в плагине хук прерывает смену имени по какому-то критерию.

Код:

RegisterHookChain(RG_CBasePlayer_SetClientUserInfoName, "CBasePlayer_SetClientUserInfoName");

Прерывается хук такой командой:

SetHookChainReturn(ATYPE_BOOL, false);
return HC_BREAK;

У меня стабильно воспроизводился баг вообще без плагинов на пустом сервере с 31 yapb ботом. Я переформатировал тестовый сервер под другой сценарий использования, попозже еще раз проверю.

Javekson · 30 Май 2024

Refresh написал(а):
У меня стабильно воспроизводился баг вообще без плагинов на пустом сервере с 31 yapb ботом

в том и дело, что у меня этого не происходит