yrtria/mod-guildfunds
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Compare commits

base: yrtria:a45716243943ff1f9fb05c18243d13ba45bad433
Branches Tags
yrtria:main
yrtria:fix/group-zero-guard
..
compare: yrtria:f69bff1cfa4048a61bff9b53ce5054506005052a
Branches Tags
yrtria:main
yrtria:fix/group-zero-guard

No commits in common. "a45716243943ff1f9fb05c18243d13ba45bad433" and "f69bff1cfa4048a61bff9b53ce5054506005052a" have entirely different histories.
a457162439 ... f69bff1cfa

1 changed files with 2 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/mod_guildfunds_loot.cpp
View file

@ -5,9 +5,6 @@
void GuildFunds::OnLootMoney(Player* player, uint32 gold)
{
if (LootMultiplier < 1)
return;
if (Group* group = player->GetGroup())
{
uint32 membersInRange = 0;
@ -20,10 +17,7 @@ void GuildFunds::OnLootMoney(Player* player, uint32 gold)
}
}
// Guard the division. In practice the looter counts themselves, but a group whose
// members all fail the distance check would divide by zero and take worldserver
// down with SIGFPE.
if (!membersInRange)
if (LootMultiplier < 1)
return;
uint32 money = (gold / membersInRange) * LootMultiplier / 100;
@ -55,7 +49,7 @@ void GuildFunds::OnLootMoney(Player* player, uint32 gold)
{
uint32 money = gold * LootMultiplier / 100;
if (money < 1)
if (money < 1 || LootMultiplier < 1)
return;
guild->HandleMemberDepositMoney(player->GetSession(), money);