DescriptorData::string_to_client_encoding (this=0x563a9c2504c0, in_str=0x0, out_str=0x7fffcf516fd0 "") at /home/mud/mud/src/engine/network/descriptor_data.cpp:171 171 for (; *in_str; in_str++, out_str++) {

[bylins]

Program terminated with signal SIGSEGV, Segmentation fault.
#0 0x0000563a772a2519 in DescriptorData::string_to_client_encoding (this=0x563a9c2504c0, in_str=0x0, out_str=0x7fffcf516fd0 "") at /home/mud/mud/src/engine/network/descriptor_data.cpp:171
171 for (; in_str; in_str++, out_str++) {
[Current thread is 1 (Thread 0x785fb5921a40 (LWP 1057832))]
(gdb) bt
#0 0x0000563a772a2519 in DescriptorData::string_to_client_encoding (this=0x563a9c2504c0, in_str=0x0, out_str=0x7fffcf516fd0 "") at /home/mud/mud/src/engine/network/descriptor_data.cpp:171
#1 0x0000563a772bb396 in msdp::GroupReporter::append_char (this=0x563a9f003c80, group=std::shared_ptrmsdp::ArrayValue (use count 1, weak count 0) = {...}, ch=0x563a9c978b10,
character=0x563a9afbd010, leader=false) at /home/mud/mud/src/engine/network/msdp/msdp_reporters.cpp:188
#2 0x0000563a772bc21d in msdp::GroupReporter::get (this=0x563a9f003c80, response=std::shared_ptrmsdp::Variable (empty) = {...})
at /home/mud/mud/src/engine/network/msdp/msdp_reporters.cpp:313
#3 0x0000563a772c4372 in msdp::AbstractSender::build (reporter=std::shared_ptrmsdp::AbstractReporter (use count 1, weak count 0) = {...})
at /home/mud/mud/src/engine/network/msdp/msdp_senders.cpp:10
#4 0x0000563a772c4625 in msdp::ReportSender::send (this=0x7fffcf518180, reporter=std::shared_ptrmsdp::AbstractReporter (use count 1, weak count 0) = {...})
at /home/mud/mud/src/engine/network/msdp/msdp_senders.cpp:45
#5 0x0000563a772a5813 in msdp::Report::operator() (this=0x563a77c2e430 msdp::report, d=0x563a9c2504c0, name="GROUP") at /home/mud/mud/src/engine/network/msdp/msdp.cpp:232
#6 0x0000563a772a22f0 in DescriptorData::msdp_report (this=0x563a9c2504c0, name="GROUP") at /home/mud/mud/src/engine/network/descriptor_data.cpp:143
#7 0x0000563a77513e07 in perform_violence () at /home/mud/mud/src/gameplay/fight/fight.cpp:2131
#8 0x0000563a771487fd in std::__invoke_impl<void, void (&)()> (__f=@0x563a84d5b788: 0x563a775132dd <perform_violence()>) at /usr/include/c++/13/bits/invoke.h:61
#9 0x0000563a771481fa in std::__invoke_r<void, void (&)()> (__fn=@0x563a84d5b788: 0x563a775132dd <perform_violence()>) at /usr/include/c++/13/bits/invoke.h:111
#10 0x0000563a77147bcc in std::_Function_handler<void (), void ()()>::_M_invoke(std::_Any_data const&) (__functor=...) at /usr/include/c++/13/bits/std_function.h:290
#11 0x0000563a7713a826 in std::function<void ()>::operator()() const (this=0x563a84d5b788) at /usr/include/c++/13/bits/std_function.h:591
#12 0x0000563a77128266 in (anonymous namespace)::SimpleCall::perform (this=0x563a84d5b780) at /home/mud/mud/src/engine/core/heartbeat.cpp:93
#13 0x0000563a7712d7a5 in Heartbeat::pulse (this=0x563a77c2d2d0 <(anonymous namespace)::global_objects()::storage+1968>, missed_pulses=0, label=std::unordered_map with 3 elements = {...})
at /home/mud/mud/src/engine/core/heartbeat.cpp:622
#14 0x0000563a7712d0fe in Heartbeat::operator() (this=0x563a77c2d2d0 <(anonymous namespace)::global_objects()::storage+1968>, missed_pulses=0)
at /home/mud/mud/src/engine/core/heartbeat.cpp:553
#15 0x0000563a770f8051 in game_loop (epoll=10, mother_desc=8) at /home/mud/mud/src/engine/core/comm.cpp:1545
#16 0x0000563a770f71b9 in stop_game (port=4000) at /home/mud/mud/src/engine/core/comm.cpp:823
#17 0x0000563a770f6e9b in main_function (argc=2, argv=0x7fffcf518c58) at /home/mud/mud/src/engine/core/comm.cpp:739
#18 0x0000563a770f61ad in main (argc=2, argv=0x7fffcf518c58) at /home/mud/mud/src/main.cpp:4

[kvirund]

Анализ причины

Почему in_str == 0x0

GET_NAME(ch) = ch->get_name().c_str() — у живого std::string c_str() не может вернуть null. Значит, character (0x563a9afbd010) указывает на освобождённую и обнулённую аллокатором память (use-after-free). Когда std::string находится в «long mode» (строка длиннее 15 байт — типично для имён мобов), внутренний heap-pointer обнуляется вместе с остальной памятью, и c_str() возвращает 0x0.

Откуда dangling-указатель в master->followers

В raw_kill() убитый моб добавляется в отложенную очередь:

real_kill(ch, killer);
character_list.AddToExtractedList(ch);  // die_follower() НЕ вызывается!
// ExtractCharFromWorld(ch, true);      // закомментировано

AddToExtractedList только помечает моба purged=true и убирает из комнаты, но не удаляет из master->followers. Удаление происходит в die_follower(), который вызывается из ExtractCharFromWorld(). А ExtractCharFromWorld через PurgeExtractedList() запускается лишь в начале следующего пульса.

MSDP GROUP report при этом генерируется в конце того же пульса, что и убийство:

Heartbeat::pulse()
  -> PurgeExtractedList()    // очищает мобов из прошлого пульса
  -> perform_violence()
       -> моб M убит -> AddToExtractedList(M)
            // M.purged=true, убран из комнаты
            // НО: M всё ещё в master->followers!
       -> MSDP GROUP report
            -> GroupReporter::get() итерирует master->followers
            -> находит M -> GET_NAME(M) -> null -> CRASH

Обычно CharData ещё жив в m_purge_list, но краш всё равно случается — значит моб оказался полностью освобождён до момента MSDP. Наиболее вероятная цепочка: AddToExtractedList вызывает DropEquipment/DropInventory, внутри которых срабатывает DG-скрипт, тот напрямую вызывает ExtractCharFromWorld для другого моба из той же группы (в коде ~20 мест с прямым вызовом: dungeons.cpp, magic.cpp, dg_misc.cpp и др.). Прямой ExtractCharFromWorld вызывает character_list.remove(), что при нулевом счётчике ссылок может освободить CharData. Плюс в условиях вложенных вызовов die_follower / stop_follower порядок освобождения FollowerType-нод может нарушиться, оставив dangling-указатель в списке.

Исправление в PR #2934

Защитная проверка на null в string_to_client_encoding — останавливает краш, не устраняет причину.

Что нужно для полного исправления

Вариант A — вызвать die_follower в AddToExtractedList до DropEquipment/DropInventory (пока DG-скрипты ещё не сработали):

// engine/db/world_characters.cpp: Characters::AddToExtractedList
void Characters::AddToExtractedList(CharData *ch) {
    if (ch->followers || ch->has_master()) {
        die_follower(ch);  // убрать из followers до дропа шмота
    }
    // ... остальное без изменений
}

Вариант B — пропускать purged-персонажей в GroupReporter::append_char:

// engine/network/msdp/msdp_reporters.cpp: GroupReporter::append_char
if (!character || character->purged()) {
    return;
}

Оба варианта нужно проверить на регрессиях с dungeon-группами, туторами и клонами.