Catch plugins setting null Material or BlockData to blocks

This commit is contained in:
md_5 2018-10-13 09:43:05 +11:00
parent 8ab46ff7c6
commit e7ced970d2
2 changed files with 5 additions and 0 deletions

View file

@ -134,6 +134,7 @@ public class CraftBlock implements Block {
@Override
public void setType(Material type, boolean applyPhysics) {
Preconditions.checkArgument(type != null, "Material cannot be null");
setBlockData(type.createBlockData(), applyPhysics);
}
@ -144,6 +145,7 @@ public class CraftBlock implements Block {
@Override
public void setBlockData(BlockData data, boolean applyPhysics) {
Preconditions.checkArgument(data != null, "BlockData cannot be null");
setTypeAndData(((CraftBlockData) data).getState(), applyPhysics);
}

View file

@ -1,5 +1,6 @@
package org.bukkit.craftbukkit.block;
import com.google.common.base.Preconditions;
import net.minecraft.server.BlockPosition;
import org.bukkit.Location;
import org.bukkit.block.Block;
@ -98,6 +99,7 @@ public class CraftBlockState implements BlockState {
@Override
public void setBlockData(BlockData data) {
Preconditions.checkArgument(data != null, "BlockData cannot be null");
this.data = ((CraftBlockData) data).getState();
}
@ -121,6 +123,7 @@ public class CraftBlockState implements BlockState {
}
public void setType(final Material type) {
Preconditions.checkArgument(type != null, "Material cannot be null");
if (this.getType() != type) {
this.data = CraftMagicNumbers.getBlock(type).getBlockData();
}