Core/AuctionHouse: Fixed use after free

Closes #30128

(cherry picked from commit b83de81a70b580045540ed4db5627aff5edc682c)

1 files changed, 4 insertions, 4 deletions

diff --git a/src/server/game/AuctionHouseBot/AuctionHouseBotBuyer.cpp b/src/server/game/AuctionHouseBot/AuctionHouseBotBuyer.cpp
index c96281ca7b9..e88994eba3a 100644
--- a/src/server/game/AuctionHouseBot/AuctionHouseBotBuyer.cpp
+++ b/src/server/game/AuctionHouseBot/AuctionHouseBotBuyer.cpp
@@ -415,12 +415,12 @@ void AuctionBotBuyer::BuyEntry(AuctionPosting* auction, AuctionHouseObject* auct
     // Copy data before freeing AuctionPosting in auctionHouse->RemoveAuction
     // Because auctionHouse->SendAuctionWon can unload items if bidder is offline
     // we need to RemoveAuction before sending mails
-    AuctionPosting copy = *auction;
-    auctionHouse->RemoveAuction(trans, auction);
+    std::map<uint32, AuctionPosting>::node_type removedAuctionNode = auctionHouse->RemoveAuction(trans, auction);
+    auction = &removedAuctionNode.mapped();
 
     // Mails must be under transaction control too to prevent data loss
-    auctionHouse->SendAuctionSold(&copy, nullptr, trans);
-    auctionHouse->SendAuctionWon(&copy, nullptr, trans);
+    auctionHouse->SendAuctionSold(auction, nullptr, trans);
+    auctionHouse->SendAuctionWon(auction, nullptr, trans);
 
     // Run SQLs
     CharacterDatabase.CommitTransaction(trans);