#565 All packets that serialize with the serializeCountPacket function need mutex protection

Закриті
4 місяців тому відкрито image · 3 коментарів
image відкоментовано 4 місяців тому

the XOR/other packets get constantly created/deleted, if we call the function twice (we are multithreaded) it will cause a crash. We need to prevent with a unique_lock.

the XOR/other packets get constantly created/deleted, if we call the function twice (we are multithreaded) it will cause a crash. We need to prevent with a unique_lock.
image відкоментовано 4 місяців тому
Співавтор

Client::SendRecipeList

PlayerItemList::serialize

EquipmentItemList::serialize

Player::GetSpellBookUpdatePacket <-- this is what we crashed on in the previous instance, but any are possible

PlayerSkillList::GetSkillPacket

Client::SendRecipeList PlayerItemList::serialize EquipmentItemList::serialize Player::GetSpellBookUpdatePacket <-- this is what we crashed on in the previous instance, but any are possible PlayerSkillList::GetSkillPacket
image відкоментовано 4 місяців тому
Співавтор

char sheet is also involved, pretty much anywhere we have XOR packets or temp buffers that get deleted/newed for packets.

char sheet is also involved, pretty much anywhere we have XOR packets or temp buffers that get deleted/newed for packets.
image відкоментовано 2 місяців тому
Співавтор

EquipmentItemList::serialize - updated to properly protect with MEquipmentItems PlayerItemList::serialize - already protected by MPlayerItems Player::GetSpellBookUpdatePacket - added a mutex to lock this function Client::SendRecipeList - uses nullptr for the orig/xor packet not impacted PlayerSkillList::GetSkillPacket - changed MPlayerSkills from shared to unique lock

EquipmentItemList::serialize - updated to properly protect with MEquipmentItems PlayerItemList::serialize - already protected by MPlayerItems Player::GetSpellBookUpdatePacket - added a mutex to lock this function Client::SendRecipeList - uses nullptr for the orig/xor packet not impacted PlayerSkillList::GetSkillPacket - changed MPlayerSkills from shared to unique lock
Підпишіться щоб приєднатися до обговорення.
Завантажується...
Скасувати
Зберегти
Тут ще немає жодного вмісту.