From b82f376bfb833d192205bdbebfb48f05a96034a4 Mon Sep 17 00:00:00 2001 From: "Michael Cook (mackal)" Date: Sun, 28 Feb 2021 17:50:07 -0500 Subject: [PATCH] Fix stack-buffer-overflow thanks to StopLFP ==1674==ERROR: AddressSanitizer: stack-buffer-overflow on address 0x7fffb47f3068 at pc 0x7f64ee1f531e bp 0x7fffb47f2f50 sp 0x7fffb47f2700 READ of size 432 at 0x7fffb47f3068 thread T0 #0 0x7f64ee1f531d (/lib/x86_64-linux-gnu/libasan.so.5+0x3f31d) #1 0x55c78e8d002e in WorldServer::UpdateLFP(unsigned int, unsigned char, unsigned char, unsigned int, unsigned int, unsigned int, char const*, GroupLFPMemberEntry*) ../zone/worldserver.cpp:3166 #2 0x55c78e8d01d9 in WorldServer::StopLFP(unsigned int) ../zone/worldserver.cpp:3180 --- zone/worldserver.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/zone/worldserver.cpp b/zone/worldserver.cpp index 804a680e2..88e71caf5 100644 --- a/zone/worldserver.cpp +++ b/zone/worldserver.cpp @@ -3176,8 +3176,8 @@ void WorldServer::UpdateLFP(uint32 LeaderID, GroupLFPMemberEntry *LFPMembers) { void WorldServer::StopLFP(uint32 LeaderID) { - GroupLFPMemberEntry LFPMembers; - UpdateLFP(LeaderID, LFPOff, 0, 0, 0, 0, "", &LFPMembers); + GroupLFPMemberEntry LFPMembers[MAX_GROUP_MEMBERS]; + UpdateLFP(LeaderID, LFPOff, 0, 0, 0, 0, "", LFPMembers); } void WorldServer::HandleLFGMatches(ServerPacket *pack) {