diff options
| author | Fedja Beader <fedja@protonmail.ch> | 2025-03-30 21:22:25 +0000 |
|---|---|---|
| committer | Fedja Beader <fedja@protonmail.ch> | 2025-03-30 21:22:25 +0000 |
| commit | ca989afcaf5bc81289afd4edf244d81418a90ac8 (patch) | |
| tree | e62fffd198c11532aea96cae140f2d3b7490afa2 | |
| parent | b5ff3318e1fef1e108b6001838d02266b29e01c0 (diff) | |
| download | mv-ca989afcaf5bc81289afd4edf244d81418a90ac8.tar.gz mv-ca989afcaf5bc81289afd4edf244d81418a90ac8.tar.bz2 mv-ca989afcaf5bc81289afd4edf244d81418a90ac8.tar.xz mv-ca989afcaf5bc81289afd4edf244d81418a90ac8.zip | |
processTradeItemAddResponse: fix early exit before full packet is consumed.
2025-03/22/#Debug.log-[19:38:37] Wrong actual or planned inbound packet size!. Packet id: 433(0x1b1), Planned size: 7, Actual size: 4
2025-03/22/#Debug.log:[19:38:52] You picked up 1 [@@760|Overlord's Helmet@@].
****
mana/plus!146
| -rw-r--r-- | src/net/tmwa/traderecv.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/net/tmwa/traderecv.cpp b/src/net/tmwa/traderecv.cpp index 101666820..1f4079e7c 100644 --- a/src/net/tmwa/traderecv.cpp +++ b/src/net/tmwa/traderecv.cpp @@ -98,6 +98,9 @@ void TradeRecv::processTradeItemAddResponse(Net::MessageIn &msg) { // Trade: New Item add response (was 0x00ea, now 01b1) const int index = msg.readInt16("index") - INVENTORY_OFFSET; + const int quantity = msg.readInt16("amount"); + const uint8_t res = msg.readUInt8("status"); + Item *item = nullptr; if (PlayerInfo::getInventory() != nullptr) item = PlayerInfo::getInventory()->getItem(index); @@ -108,9 +111,7 @@ void TradeRecv::processTradeItemAddResponse(Net::MessageIn &msg) tradeWindow->receivedOk(true); return; } - const int quantity = msg.readInt16("amount"); - const uint8_t res = msg.readUInt8("status"); switch (res) { case 0: |