phaneron/Kelsidavis-WoWee
1
0
Fork 0
mirror of https://github.com/Kelsidavis/WoWee.git synced 2026-04-17 17:43:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix Classic chat parser missing second GUID for SAY/YELL/PARTY

Browse source 
CMaNGOS-Classic writes senderGuid twice for SAY/PARTY/YELL messages.
Our parser only read one, causing the second GUID bytes to be consumed
as messageLen, resulting in empty/garbage message text and no sender
name. Also fix MONSTER_SAY/MONSTER_YELL which have a leading
senderGuid before the name that we were skipping.
This commit is contained in:
Kelsi 2026-02-14 21:34:09 -08:00
parent 0d4eff65d0
commit 1a9bdfb64b
1 changed files with 29 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

38
src/game/packet_parsers_classic.cpp
View file

@ -427,10 +427,8 @@ bool ClassicPacketParsers::parseMessageChat(network::Packet& packet, MessageChat
// Classic: NO uint32 unknown field here (WotLK has one) // Classic: NO uint32 unknown field here (WotLK has one)
// Type-specific data // Type-specific data (matches CMaNGOS-Classic BuildChatPacket)
switch (data.type) { switch (data.type) {
case ChatType::MONSTER_SAY:
case ChatType::MONSTER_YELL:
case ChatType::MONSTER_EMOTE: { case ChatType::MONSTER_EMOTE: {
// nameLen(u32) + name + targetGuid(u64) // nameLen(u32) + name + targetGuid(u64)
uint32_t nameLen = packet.readUInt32(); uint32_t nameLen = packet.readUInt32();
@ -439,7 +437,33 @@ bool ClassicPacketParsers::parseMessageChat(network::Packet& packet, MessageChat
for (uint32_t i = 0; i < nameLen; ++i) { for (uint32_t i = 0; i < nameLen; ++i) {
data.senderName[i] = static_cast<char>(packet.readUInt8()); data.senderName[i] = static_cast<char>(packet.readUInt8());
} }
// Remove null terminator if present if (!data.senderName.empty() && data.senderName.back() == '\0') {
data.senderName.pop_back();
}
}
data.receiverGuid = packet.readUInt64();
break;
}
case ChatType::SAY:
case ChatType::PARTY:
case ChatType::YELL: {
// senderGuid(u64) + senderGuid(u64) — written twice by server
data.senderGuid = packet.readUInt64();
/*duplicateGuid*/ packet.readUInt64();
break;
}
case ChatType::MONSTER_SAY:
case ChatType::MONSTER_YELL: {
// senderGuid(u64) + nameLen(u32) + name + targetGuid(u64)
data.senderGuid = packet.readUInt64();
uint32_t nameLen = packet.readUInt32();
if (nameLen > 0 && nameLen < 256) {
data.senderName.resize(nameLen);
for (uint32_t i = 0; i < nameLen; ++i) {
data.senderName[i] = static_cast<char>(packet.readUInt8());
}
if (!data.senderName.empty() && data.senderName.back() == '\0') { if (!data.senderName.empty() && data.senderName.back() == '\0') {
data.senderName.pop_back(); data.senderName.pop_back();
} }
@ -457,16 +481,12 @@ bool ClassicPacketParsers::parseMessageChat(network::Packet& packet, MessageChat
} }
default: { default: {
// Most types: senderGuid(u64) // GUILD, OFFICER, RAID, WHISPER, WHISPER_INFORM, etc: senderGuid(u64)
data.senderGuid = packet.readUInt64(); data.senderGuid = packet.readUInt64();
break; break;
} }
} }
// Classic SAY/YELL/PARTY/GUILD/etc: NO second GUID before message.
// Only WHISPER_INFORM has a receiver GUID (same as sender for echo).
// Do NOT read an extra uint64 here — it would consume messageLen + message bytes.
// Read message length // Read message length
uint32_t messageLen = packet.readUInt32(); uint32_t messageLen = packet.readUInt32();