Core/Misc: fixed packet structures of CMSG_MESSAGECHAT_ADDON_PARTY, CMSG_MESSAGECHAT_ADDON_OFFICER and CMSG_AUCTION_LIST_ITEMS

src/server/game/Handlers/AuctionHouseHandler.cpp

@@ -759,7 +759,7 @@ void WorldSession::HandleAuctionListItems(WorldPacket& recvData)
     TC_LOG_DEBUG("network", "WORLD: Received CMSG_AUCTION_LIST_ITEMS");
 
     std::string searchedname;
-    uint8 levelmin, levelmax, usable, getAll;
+    uint8 levelmin, levelmax, usable, getAll, sortCount;
     uint32 listfrom, auctionSlotID, auctionMainCategory, auctionSubCategory, quality;
     ObjectGuid guid;
 
@@ -774,9 +774,13 @@ void WorldSession::HandleAuctionListItems(WorldPacket& recvData)
     recvData >> getAll;
     recvData.read_skip<uint8>();                           // unk
 
-    // this block looks like it uses some lame byte packing or similar...
-    for (uint8 i = 0; i < 15; i++)
+    // Sorting
+    recvData >> sortCount;
+    for (uint8 i = 0; i < sortCount; i++)
+    {
         recvData.read_skip<uint8>();
+        recvData.read_skip<uint8>();
+    }
 
     Creature* creature = GetPlayer()->GetNPCIfCanInteractWith(guid, UNIT_NPC_FLAG_AUCTIONEER);
     if (!creature)

src/server/game/Handlers/ChatHandler.cpp

@@ -623,8 +623,8 @@ void WorldSession::HandleAddonMessagechatOpcode(WorldPacket& recvData)
             uint32 msgLen = recvData.ReadBits(9);
             message = recvData.ReadString(msgLen);
             prefix = recvData.ReadString(prefixLen);
+            break;
         }
-
         case CHAT_MSG_GUILD:
         {
             uint32 msgLen = recvData.ReadBits(9);
@@ -638,7 +638,10 @@ void WorldSession::HandleAddonMessagechatOpcode(WorldPacket& recvData)
     }
 
     if (prefix.empty() || prefix.length() > 16)
+    {
+        recvData.rfinish();
         return;
+    }
 
     // Disabled addon channel?
     if (!sWorld->getBoolConfig(CONFIG_ADDON_CHANNEL))
@@ -683,7 +686,6 @@ void WorldSession::HandleAddonMessagechatOpcode(WorldPacket& recvData)
         case CHAT_MSG_PARTY:
         case CHAT_MSG_RAID:
         {
-
             Group* group = sender->GetGroup();
             if (!group || group->isBGGroup())
                 break;