aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0723-Buffer-OOB-setBlock-calls.patch
diff options
context:
space:
mode:
authormaxcom1 <[email protected]>2024-03-23 22:26:17 +0100
committerGitHub <[email protected]>2024-03-23 17:26:17 -0400
commitb6001403e9703cadaa6e8c8558e732b91c3c6d6e (patch)
treea8c57bbc334a8ad48d4ad2b43db335667b142bee /patches/server/0723-Buffer-OOB-setBlock-calls.patch
parent9ec7dfcbc41c6e625de0050b6997160a75df9f44 (diff)
downloadPaper-b6001403e9703cadaa6e8c8558e732b91c3c6d6e.tar.gz
Paper-b6001403e9703cadaa6e8c8558e732b91c3c6d6e.zip
Add methods to change entity physics (#10334)
Diffstat (limited to 'patches/server/0723-Buffer-OOB-setBlock-calls.patch')
-rw-r--r--patches/server/0723-Buffer-OOB-setBlock-calls.patch42
1 files changed, 42 insertions, 0 deletions
diff --git a/patches/server/0723-Buffer-OOB-setBlock-calls.patch b/patches/server/0723-Buffer-OOB-setBlock-calls.patch
new file mode 100644
index 0000000000..ae47164389
--- /dev/null
+++ b/patches/server/0723-Buffer-OOB-setBlock-calls.patch
@@ -0,0 +1,42 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Shane Freeder <[email protected]>
+Date: Sat, 19 Mar 2022 12:12:22 +0000
+Subject: [PATCH] Buffer OOB setBlock calls
+
+lets debug mode throw a trace in order to potentially see where
+such calls are cascading from easier, but, generally, if you see one setBlock
+call, you're gonna see more, and this just potentially causes a flood of logs
+which can cause issues for slower terminals, etc.
+
+We can limit the flood by just allowing one for a single gen region,
+we'll also only gen a trace for the first one, I see no real pressing need
+to generate more, given that that would *massively* negate this patch otherwise
+
+diff --git a/src/main/java/net/minecraft/server/level/WorldGenRegion.java b/src/main/java/net/minecraft/server/level/WorldGenRegion.java
+index d11741d2618976bdb51f75d823f260f32d5bafc9..ff94af5be8de374f5cde2607eebbb23e65705581 100644
+--- a/src/main/java/net/minecraft/server/level/WorldGenRegion.java
++++ b/src/main/java/net/minecraft/server/level/WorldGenRegion.java
+@@ -275,6 +275,7 @@ public class WorldGenRegion implements WorldGenLevel {
+ }
+ }
+
++ private boolean hasSetFarWarned = false; // Paper - Buffer OOB setBlock calls
+ @Override
+ public boolean ensureCanWrite(BlockPos pos) {
+ int i = SectionPos.blockToSectionCoord(pos.getX());
+@@ -294,7 +295,15 @@ public class WorldGenRegion implements WorldGenLevel {
+
+ return true;
+ } else {
++ // Paper start - Buffer OOB setBlock calls
++ if (!hasSetFarWarned) {
+ Util.logAndPauseIfInIde("Detected setBlock in a far chunk [" + i + ", " + j + "], pos: " + pos + ", status: " + this.generatingStatus + (this.currentlyGenerating == null ? "" : ", currently generating: " + (String) this.currentlyGenerating.get()));
++ hasSetFarWarned = true;
++ if (this.getServer() != null && this.getServer().isDebugging()) {
++ io.papermc.paper.util.TraceUtil.dumpTraceForThread("far setBlock call");
++ }
++ }
++ // Paper end - Buffer OOB setBlock calls
+ return false;
+ }
+ }