Core: Fix build and proper deinitialization in HandleTransmogrifyItems (by using vectors instead of new/delete)

1 files changed, 3 insertions, 8 deletions

diff --git a/src/server/game/Handlers/ItemHandler.cpp b/src/server/game/Handlers/ItemHandler.cpp
index 9a8ab497d88..3b2d40f9000 100644
--- a/src/server/game/Handlers/ItemHandler.cpp
+++ b/src/server/game/Handlers/ItemHandler.cpp
@@ -1454,9 +1454,9 @@ void WorldSession::HandleTransmogrifyItems(WorldPacket& recvData)
         return;
     }
 
-    ObjectGuid* itemGuids = new ObjectGuid[count];
-    uint32* newEntries = new uint32[count];
-    uint32* slots = new uint32[count];
+    std::vector<ObjectGuid> itemGuids(count, ObjectGuid(0));
+    std::vector<uint32> newEntries(count, 0);
+    std::vector<uint32> slots(count, 0);
 
     for (uint8 i = 0; i < count; ++i)
     {
@@ -1606,11 +1606,6 @@ void WorldSession::HandleTransmogrifyItems(WorldPacket& recvData)
     // ... unless client was modified
     if (cost) // 0 cost if reverting look
         player->ModifyMoney(-cost);
-
-
-    delete[] itemGuids;
-    delete[] newEntries;
-    delete[] slots;
 }
 
 void WorldSession::SendReforgeResult(bool success)