aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0269-Don-t-allow-digging-into-unloaded-chunks.patch
diff options
context:
space:
mode:
authorRiley Park <[email protected]>2024-05-15 17:06:59 -0700
committerGitHub <[email protected]>2024-05-15 17:06:59 -0700
commitf17519338bc589c045e0b32bfc37e048b23544d5 (patch)
treee50182ec698b4a9de8f366f485ee089b1901bbd9 /patches/server/0269-Don-t-allow-digging-into-unloaded-chunks.patch
parent3fc93581bb876e8149b2ca423375a98f5ca12d27 (diff)
downloadPaper-f17519338bc589c045e0b32bfc37e048b23544d5.tar.gz
Paper-f17519338bc589c045e0b32bfc37e048b23544d5.zip
Expose server build information (#10729)
* Expose server build information * squash patches * final tweaks --------- Co-authored-by: Jake Potrebic <[email protected]> Co-authored-by: masmc05 <[email protected]>
Diffstat (limited to 'patches/server/0269-Don-t-allow-digging-into-unloaded-chunks.patch')
-rw-r--r--patches/server/0269-Don-t-allow-digging-into-unloaded-chunks.patch77
1 files changed, 77 insertions, 0 deletions
diff --git a/patches/server/0269-Don-t-allow-digging-into-unloaded-chunks.patch b/patches/server/0269-Don-t-allow-digging-into-unloaded-chunks.patch
new file mode 100644
index 0000000000..9d4d0f1082
--- /dev/null
+++ b/patches/server/0269-Don-t-allow-digging-into-unloaded-chunks.patch
@@ -0,0 +1,77 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Shane Freeder <[email protected]>
+Date: Sun, 11 Nov 2018 21:01:09 +0000
+Subject: [PATCH] Don't allow digging into unloaded chunks
+
+
+diff --git a/src/main/java/net/minecraft/server/level/ServerPlayerGameMode.java b/src/main/java/net/minecraft/server/level/ServerPlayerGameMode.java
+index 5f2dec1917f1c1c3bb69446832321f3fc21dc129..cd6f34ee326228036f8c025e4e6d04e0c15ba06f 100644
+--- a/src/main/java/net/minecraft/server/level/ServerPlayerGameMode.java
++++ b/src/main/java/net/minecraft/server/level/ServerPlayerGameMode.java
+@@ -122,8 +122,8 @@ public class ServerPlayerGameMode {
+ BlockState iblockdata;
+
+ if (this.hasDelayedDestroy) {
+- iblockdata = this.level.getBlockState(this.delayedDestroyPos);
+- if (iblockdata.isAir()) {
++ iblockdata = this.level.getBlockStateIfLoaded(this.delayedDestroyPos); // Paper - Don't allow digging into unloaded chunks
++ if (iblockdata == null || iblockdata.isAir()) { // Paper - Don't allow digging into unloaded chunks
+ this.hasDelayedDestroy = false;
+ } else {
+ float f = this.incrementDestroyProgress(iblockdata, this.delayedDestroyPos, this.delayedTickStart);
+@@ -134,7 +134,13 @@ public class ServerPlayerGameMode {
+ }
+ }
+ } else if (this.isDestroyingBlock) {
+- iblockdata = this.level.getBlockState(this.destroyPos);
++ // Paper start - Don't allow digging into unloaded chunks; don't want to do same logic as above, return instead
++ iblockdata = this.level.getBlockStateIfLoaded(this.destroyPos);
++ if (iblockdata == null) {
++ this.isDestroyingBlock = false;
++ return;
++ }
++ // Paper end - Don't allow digging into unloaded chunks
+ if (iblockdata.isAir()) {
+ this.level.destroyBlockProgress(this.player.getId(), this.destroyPos, -1);
+ this.lastSentState = -1;
+@@ -163,6 +169,7 @@ public class ServerPlayerGameMode {
+
+ public void handleBlockBreakAction(BlockPos pos, ServerboundPlayerActionPacket.Action action, Direction direction, int worldHeight, int sequence) {
+ if (!this.player.canInteractWithBlock(pos, 1.0D)) {
++ if (true) return; // Paper - Don't allow digging into unloaded chunks; Don't notify if unreasonably far away
+ this.debugLogging(pos, false, sequence, "too far");
+ } else if (pos.getY() >= worldHeight) {
+ this.player.connection.send(new ClientboundBlockUpdatePacket(pos, this.level.getBlockState(pos)));
+@@ -302,10 +309,12 @@ public class ServerPlayerGameMode {
+ this.debugLogging(pos, true, sequence, "stopped destroying");
+ } else if (action == ServerboundPlayerActionPacket.Action.ABORT_DESTROY_BLOCK) {
+ this.isDestroyingBlock = false;
+- if (!Objects.equals(this.destroyPos, pos)) {
++ if (!Objects.equals(this.destroyPos, pos) && !BlockPos.ZERO.equals(this.destroyPos)) { // Paper
+ ServerPlayerGameMode.LOGGER.debug("Mismatch in destroy block pos: {} {}", this.destroyPos, pos); // CraftBukkit - SPIGOT-5457 sent by client when interact event cancelled
+- this.level.destroyBlockProgress(this.player.getId(), this.destroyPos, -1);
+- this.debugLogging(pos, true, sequence, "aborted mismatched destroying");
++ BlockState type = this.level.getBlockStateIfLoaded(this.destroyPos); // Paper - don't load unloaded chunks for stale records here
++ if (type != null) this.level.destroyBlockProgress(this.player.getId(), this.destroyPos, -1);
++ if (type != null) this.debugLogging(pos, true, sequence, "aborted mismatched destroying");
++ this.destroyPos = BlockPos.ZERO; // Paper
+ }
+
+ this.level.destroyBlockProgress(this.player.getId(), pos, -1);
+diff --git a/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java b/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java
+index c5a9bd1d5dcf96d0b206de48000ebf63f7a0fcb1..e880b41e1824f986d04a0a32cc735d77f25cf987 100644
+--- a/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java
++++ b/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java
+@@ -1583,6 +1583,12 @@ public class ServerGamePacketListenerImpl extends ServerCommonPacketListenerImpl
+ case START_DESTROY_BLOCK:
+ case ABORT_DESTROY_BLOCK:
+ case STOP_DESTROY_BLOCK:
++ // Paper start - Don't allow digging into unloaded chunks
++ if (this.player.level().getChunkIfLoadedImmediately(blockposition.getX() >> 4, blockposition.getZ() >> 4) == null) {
++ this.player.connection.ackBlockChangesUpTo(packet.getSequence());
++ return;
++ }
++ // Paper end - Don't allow digging into unloaded chunks
+ this.player.gameMode.handleBlockBreakAction(blockposition, packetplayinblockdig_enumplayerdigtype, packet.getDirection(), this.player.level().getMaxBuildHeight(), packet.getSequence());
+ this.player.connection.ackBlockChangesUpTo(packet.getSequence());
+ return;