From b0e3ca43b8a2ccd9978ad42aa6f9f58b4d8f46f5 Mon Sep 17 00:00:00 2001 From: Jake Potrebic Date: Sun, 26 May 2024 14:41:30 -0700 Subject: [PATCH] Prevent NPE if hooked entity was cleared (#10712) --- ...ent-NPE-if-hooked-entity-was-cleared.patch | 24 +++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 patches/server/1049-Prevent-NPE-if-hooked-entity-was-cleared.patch diff --git a/patches/server/1049-Prevent-NPE-if-hooked-entity-was-cleared.patch b/patches/server/1049-Prevent-NPE-if-hooked-entity-was-cleared.patch new file mode 100644 index 000000000000..f1cbb0fcf37b --- /dev/null +++ b/patches/server/1049-Prevent-NPE-if-hooked-entity-was-cleared.patch @@ -0,0 +1,24 @@ +From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 +From: Jake Potrebic +Date: Sun, 12 May 2024 21:57:23 -0700 +Subject: [PATCH] Prevent NPE if hooked entity was cleared + + +diff --git a/src/main/java/net/minecraft/world/entity/projectile/FishingHook.java b/src/main/java/net/minecraft/world/entity/projectile/FishingHook.java +index 35299affb699d745804a3b60cb78b9323f3e068e..882de08963c72614a3d26cd917916e42b7136042 100644 +--- a/src/main/java/net/minecraft/world/entity/projectile/FishingHook.java ++++ b/src/main/java/net/minecraft/world/entity/projectile/FishingHook.java +@@ -498,11 +498,13 @@ public class FishingHook extends Projectile { + if (playerFishEvent.isCancelled()) { + return 0; + } ++ if (this.hookedIn != null) { // Paper - re-check to see if there is a hooked entity + // CraftBukkit end + this.pullEntity(this.hookedIn); + CriteriaTriggers.FISHING_ROD_HOOKED.trigger((ServerPlayer) entityhuman, usedItem, this, Collections.emptyList()); + this.level().broadcastEntityEvent(this, (byte) 31); + i = this.hookedIn instanceof ItemEntity ? 3 : 5; ++ } // Paper - re-check to see if there is a hooked entity + } else if (this.nibble > 0) { + LootParams lootparams = (new LootParams.Builder((ServerLevel) this.level())).withParameter(LootContextParams.ORIGIN, this.position()).withParameter(LootContextParams.TOOL, usedItem).withParameter(LootContextParams.THIS_ENTITY, this).withLuck((float) this.luck + entityhuman.getLuck()).create(LootContextParamSets.FISHING); + LootTable loottable = this.level().getServer().reloadableRegistries().getLootTable(BuiltInLootTables.FISHING);