pa/eqemu-server
2
0
Fork 0
mirror of https://github.com/EQEmu/Server.git synced 2025-12-12 09:31:30 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[Fix] Prevent VerifyGroup from setting OOZ membername to Null character. (#3168)

Browse Source
This commit is contained in:
Aeadoin 2023-04-01 12:40:55 -04:00 committed by GitHub
parent ca2072e7bf
commit 4c2271ff69
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 17 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
zone/groups.cpp
View File

@ -1158,17 +1158,26 @@ bool Group::LearnMembers() {
"Error getting group members for group [{}]",
GetID()
);
return false;
}
int memberIndex = 0;
for (const auto& member : rows) {
if (member.name.empty()) {
continue;
if (memberIndex >= MAX_GROUP_MEMBERS) {
LogError(
"Too many members in group [{}]",
GetID()
);
break;
}
if (member.name.empty()) {
members[memberIndex] = nullptr;
membername[memberIndex][0] = '\0';
} else {
members[memberIndex] = nullptr;
strn0cpy(membername[memberIndex], member.name.c_str(), 64);
memberIndex++;
}
++memberIndex;
}
VerifyGroup();
@ -1189,13 +1198,12 @@ void Group::VerifyGroup() {
}
Mob *them = entity_list.GetMob(membername[i]);
if(them == nullptr && members[i] != nullptr) { //they aren't in zone
membername[i][0] = '\0';
if (!them && members[i]) { //they aren't in zone
members[i] = nullptr;
continue;
}
if(them != nullptr && members[i] != them) { //our pointer is out of date... not so good.
if (them != nullptr && members[i] != them) { //our pointer is out of date... not so good.
members[i] = them;
continue;
}