ElderShell/EverWrath
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(Core/Handlers): Properly limit equipment set name length to client maximum. (#23151)

Browse source
This commit is contained in:
Benjamin Jackson 2025-10-09 14:10:43 -04:00 • committed by GitHub
parent 8d2f307483
commit d237ea6552
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 12 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/server/game/Handlers/CharacterHandler.cpp
View file

@ -1754,9 +1754,21 @@ void WorldSession::HandleEquipmentSetSave(WorldPacket& recvData)
std::string name;
recvData >> name;
if (name.length() > 16) // Client limitation
{
LOG_ERROR("entities.player.cheat", "Character GUID {} tried to create equipment set {} with too long a name!", _player->GetGUID().ToString(), setGuid);
return;
}
std::string iconName;
recvData >> iconName;
if (iconName.length() > 100) // DB limitation
{
LOG_ERROR("entities.player.cheat", "Character GUID {} tried to create equipment set {} with too long an icon name!", _player->GetGUID().ToString(), setGuid);
return;
}
EquipmentSet eqSet;
eqSet.Guid = setGuid;