Altitude/AlttdGriefPrevention
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add negative value check when accruing claim blocks

Browse Source 
to prevent plugins from doing nasties
This commit is contained in:
RoboMWM 2016-11-20 17:40:26 -08:00
parent 11d6d3cf13
commit 514b0e5949
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/me/ryanhamshire/GriefPrevention/DeliverClaimBlocksTask.java
View File

@ -85,8 +85,9 @@ class DeliverClaimBlocksTask implements Runnable
GriefPrevention.AddLogEntry(player.getName() + " claim block delivery was canceled by another plugin.", CustomLogEntryTypes.Debug, true); GriefPrevention.AddLogEntry(player.getName() + " claim block delivery was canceled by another plugin.", CustomLogEntryTypes.Debug, true);
return; return;
} }
accrualRate = event.getBlocksToAccrue();
playerData.accrueBlocks(event.getBlocksToAccrue()); if(accrualRate < 0) accrualRate = 0;
playerData.accrueBlocks(accrualRate);
GriefPrevention.AddLogEntry("Delivering " + event.getBlocksToAccrue() + " blocks to " + player.getName(), CustomLogEntryTypes.Debug, true); GriefPrevention.AddLogEntry("Delivering " + event.getBlocksToAccrue() + " blocks to " + player.getName(), CustomLogEntryTypes.Debug, true);
//intentionally NOT saving data here to reduce overall secondary storage access frequency //intentionally NOT saving data here to reduce overall secondary storage access frequency

1
src/me/ryanhamshire/GriefPrevention/events/AccrueClaimBlocksEvent.java
View File

@ -29,7 +29,6 @@ public class AccrueClaimBlocksEvent extends Event
public AccrueClaimBlocksEvent(Player player, int blocksToAccrue) public AccrueClaimBlocksEvent(Player player, int blocksToAccrue)
{ {
this.player = player; this.player = player;
if(blocksToAccrue < 0) blocksToAccrue = 1;
this.blocksToAccrue = blocksToAccrue / 6; this.blocksToAccrue = blocksToAccrue / 6;
} }