fix: packet interpretation parseSetOutfit V8-OldProtocol (issues: #3155 )

[kokekanon]

Description

this problem is since August when version was upgraded to 13.40 but no v8 user reported it, until issues: 3155

1) canary expects a packet “isMountRandomized” u16 but is only sent by cipsoft 13.34 and OTCR 13.34

v8 old Protocol no send "isMountRandomized" , last packet is newOutfit.lookMount = msg.get<uint16_t>(); canary read in (L1711)

this is from v8:

https://github.com/opentibiabr/otcv8/blob/main/src/client/protocolgamesend.cpp#L833

(Redemption 11.00 no send isMountRandomized only in 13.34)

2) load this (sendCreatureOutfit and AddCreature) into AddOutfit

(same packets)

Behaviour

Actual

[2024-29-11 14:25:47.050] [error] [NetworkMessage::getByte] Not enough data to read a byte. Current position: 26, Length: 18. Called line 1692:50 in void __cdecl ProtocolGame::parseSetOutfit(class NetworkMessage &)

Expected

Do this and that happens

Fixes

#3155

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested

Test Configuration:

• Server Version: 11.00 / 13.40
• Client: otcr / v8 / cip´soft
• Operating System: win 11

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I checked the PR checks reports
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud