Core/Commands: Fix heap-use-after-free in ".ticket delete" command

Reported by Address Sanitizer
This commit is contained in:
jackpoz
2019-04-28 18:56:45 +02:00
parent f45ebe71b0
commit 3f0ce98f7c

View File

@@ -291,9 +291,6 @@ public:
std::string msg = ticket->FormatMessageString(*handler, nullptr, nullptr, nullptr, handler->GetSession() ? handler->GetSession()->GetPlayer()->GetName().c_str() : "Console", nullptr);
handler->SendGlobalGMSysMessage(msg.c_str());
sTicketMgr->RemoveTicket(ticket->GetId());
sTicketMgr->UpdateLastChange();
if (Player* player = ticket->GetPlayer())
{
// Force abandon ticket
@@ -302,6 +299,9 @@ public:
player->SendDirectMessage(&data);
}
sTicketMgr->RemoveTicket(ticket->GetId());
sTicketMgr->UpdateLastChange();
return true;
}