From 5d43fcd6b482123a7ac4c8e750c3eec07436ae27 Mon Sep 17 00:00:00 2001 From: MobiusDevelopment <8391001+MobiusDevelopment@users.noreply.github.com> Date: Sat, 12 Feb 2022 07:55:57 +0000 Subject: [PATCH] Castle NPE check for isAutoAttackable first mid victory. --- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 4 +++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 4 +++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 3 ++- .../java/org/l2jmobius/gameserver/model/actor/Player.java | 5 +++-- 23 files changed, 49 insertions(+), 24 deletions(-) diff --git a/L2J_Mobius_01.0_Ertheia/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_01.0_Ertheia/java/org/l2jmobius/gameserver/model/actor/Player.java index 299b6afc9c..e83c5ce781 100644 --- a/L2J_Mobius_01.0_Ertheia/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_01.0_Ertheia/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8276,7 +8276,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_02.5_Underground/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_02.5_Underground/java/org/l2jmobius/gameserver/model/actor/Player.java index 2b1554080d..7eacb0343f 100644 --- a/L2J_Mobius_02.5_Underground/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_02.5_Underground/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8279,7 +8279,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_03.0_Helios/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_03.0_Helios/java/org/l2jmobius/gameserver/model/actor/Player.java index d83be70a63..4da452de57 100644 --- a/L2J_Mobius_03.0_Helios/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_03.0_Helios/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8281,7 +8281,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_04.0_GrandCrusade/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_04.0_GrandCrusade/java/org/l2jmobius/gameserver/model/actor/Player.java index 6d41299f7c..0396769e2c 100644 --- a/L2J_Mobius_04.0_GrandCrusade/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_04.0_GrandCrusade/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8276,7 +8276,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_05.0_Salvation/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_05.0_Salvation/java/org/l2jmobius/gameserver/model/actor/Player.java index d06ce82e5d..331e8a49be 100644 --- a/L2J_Mobius_05.0_Salvation/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_05.0_Salvation/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8292,7 +8292,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_05.5_EtinasFate/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_05.5_EtinasFate/java/org/l2jmobius/gameserver/model/actor/Player.java index 8034ba587b..ef3368d4bd 100644 --- a/L2J_Mobius_05.5_EtinasFate/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_05.5_EtinasFate/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8292,7 +8292,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_06.0_Fafurion/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_06.0_Fafurion/java/org/l2jmobius/gameserver/model/actor/Player.java index 8dc9cf5aeb..01b7bec2ec 100644 --- a/L2J_Mobius_06.0_Fafurion/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_06.0_Fafurion/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8293,7 +8293,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_07.0_PreludeOfWar/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_07.0_PreludeOfWar/java/org/l2jmobius/gameserver/model/actor/Player.java index 965e3b0fb4..8688ab86b8 100644 --- a/L2J_Mobius_07.0_PreludeOfWar/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_07.0_PreludeOfWar/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8308,7 +8308,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_08.2_Homunculus/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_08.2_Homunculus/java/org/l2jmobius/gameserver/model/actor/Player.java index 610ebd74d3..999f1ed556 100644 --- a/L2J_Mobius_08.2_Homunculus/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_08.2_Homunculus/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8410,7 +8410,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_09.2_ReturnOfTheQueenAnt/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_09.2_ReturnOfTheQueenAnt/java/org/l2jmobius/gameserver/model/actor/Player.java index d7dba54109..aa77d4dc56 100644 --- a/L2J_Mobius_09.2_ReturnOfTheQueenAnt/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_09.2_ReturnOfTheQueenAnt/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8436,7 +8436,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_10.0_MasterClass/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_10.0_MasterClass/java/org/l2jmobius/gameserver/model/actor/Player.java index 1f42e79092..08b15cb4f5 100644 --- a/L2J_Mobius_10.0_MasterClass/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_10.0_MasterClass/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8473,7 +8473,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_CT_2.4_Epilogue/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_CT_2.4_Epilogue/java/org/l2jmobius/gameserver/model/actor/Player.java index 29f22e1579..7ca8835875 100644 --- a/L2J_Mobius_CT_2.4_Epilogue/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_CT_2.4_Epilogue/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8489,7 +8489,9 @@ public class Player extends Playable // Check if a siege is in progress and if attacker and the Player aren't in the Attacker clan. if (siege.checkIsAttacker(attackerPlayer.getClan()) && siege.checkIsAttacker(getClan())) { - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + // If first mid victory is achieved, attackers can attack attackers. + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_CT_2.6_HighFive/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_CT_2.6_HighFive/java/org/l2jmobius/gameserver/model/actor/Player.java index ca4be2bd2e..1646d96298 100644 --- a/L2J_Mobius_CT_2.6_HighFive/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_CT_2.6_HighFive/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8373,7 +8373,9 @@ public class Player extends Playable // Check if a siege is in progress and if attacker and the Player aren't in the Attacker clan. if (siege.checkIsAttacker(attackerPlayer.getClan()) && siege.checkIsAttacker(getClan())) { - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + // If first mid victory is achieved, attackers can attack attackers. + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Classic_2.0_Saviors/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Classic_2.0_Saviors/java/org/l2jmobius/gameserver/model/actor/Player.java index 7bafba504d..376c57cd03 100644 --- a/L2J_Mobius_Classic_2.0_Saviors/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Classic_2.0_Saviors/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8184,7 +8184,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Classic_2.1_Zaken/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Classic_2.1_Zaken/java/org/l2jmobius/gameserver/model/actor/Player.java index 9956a3e02c..45ee92dfe0 100644 --- a/L2J_Mobius_Classic_2.1_Zaken/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Classic_2.1_Zaken/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8184,7 +8184,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Classic_2.2_Antharas/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Classic_2.2_Antharas/java/org/l2jmobius/gameserver/model/actor/Player.java index 341b3ed89f..2066d892f7 100644 --- a/L2J_Mobius_Classic_2.2_Antharas/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Classic_2.2_Antharas/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8170,7 +8170,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Classic_2.3_SevenSigns/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Classic_2.3_SevenSigns/java/org/l2jmobius/gameserver/model/actor/Player.java index 2c59847990..fb87bd4b9a 100644 --- a/L2J_Mobius_Classic_2.3_SevenSigns/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Classic_2.3_SevenSigns/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8197,7 +8197,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Classic_2.4_SecretOfEmpire/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Classic_2.4_SecretOfEmpire/java/org/l2jmobius/gameserver/model/actor/Player.java index 91d308b2d0..cdd1b2bfcb 100644 --- a/L2J_Mobius_Classic_2.4_SecretOfEmpire/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Classic_2.4_SecretOfEmpire/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8197,7 +8197,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Classic_3.0_TheKamael/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Classic_3.0_TheKamael/java/org/l2jmobius/gameserver/model/actor/Player.java index 7bc611cb8e..5dc0aa1fe8 100644 --- a/L2J_Mobius_Classic_3.0_TheKamael/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Classic_3.0_TheKamael/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8151,7 +8151,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Classic_Interlude/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Classic_Interlude/java/org/l2jmobius/gameserver/model/actor/Player.java index 88323c7967..cd916db855 100644 --- a/L2J_Mobius_Classic_Interlude/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Classic_Interlude/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8162,7 +8162,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Essence_4.2_DwellingOfSpirits/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Essence_4.2_DwellingOfSpirits/java/org/l2jmobius/gameserver/model/actor/Player.java index c4fa80e4ac..7045e61297 100644 --- a/L2J_Mobius_Essence_4.2_DwellingOfSpirits/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Essence_4.2_DwellingOfSpirits/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8303,7 +8303,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Essence_5.2_FrostLord/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Essence_5.2_FrostLord/java/org/l2jmobius/gameserver/model/actor/Player.java index 507b0bb436..f537bdf2be 100644 --- a/L2J_Mobius_Essence_5.2_FrostLord/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Essence_5.2_FrostLord/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -8369,7 +8369,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } } diff --git a/L2J_Mobius_Essence_6.0_BattleChronicle/java/org/l2jmobius/gameserver/model/actor/Player.java b/L2J_Mobius_Essence_6.0_BattleChronicle/java/org/l2jmobius/gameserver/model/actor/Player.java index 2d4c1d50f6..767989fd38 100644 --- a/L2J_Mobius_Essence_6.0_BattleChronicle/java/org/l2jmobius/gameserver/model/actor/Player.java +++ b/L2J_Mobius_Essence_6.0_BattleChronicle/java/org/l2jmobius/gameserver/model/actor/Player.java @@ -73,10 +73,10 @@ import org.l2jmobius.gameserver.data.xml.AttendanceRewardData; import org.l2jmobius.gameserver.data.xml.CategoryData; import org.l2jmobius.gameserver.data.xml.ClassListData; import org.l2jmobius.gameserver.data.xml.CollectionData; -import org.l2jmobius.gameserver.data.xml.HennaPatternPotentialData; import org.l2jmobius.gameserver.data.xml.ElementalSpiritData; import org.l2jmobius.gameserver.data.xml.ExperienceData; import org.l2jmobius.gameserver.data.xml.HennaData; +import org.l2jmobius.gameserver.data.xml.HennaPatternPotentialData; import org.l2jmobius.gameserver.data.xml.NpcData; import org.l2jmobius.gameserver.data.xml.NpcNameLocalisationData; import org.l2jmobius.gameserver.data.xml.OptionData; @@ -8482,7 +8482,8 @@ public class Player extends Playable if (siege.checkIsAttacker(attackerClan) && siege.checkIsAttacker(clan)) { // If first mid victory is achieved, attackers can attack attackers. - return CastleManager.getInstance().getCastleById(_siegeSide).isFirstMidVictory(); + final Castle castle = CastleManager.getInstance().getCastleById(_siegeSide); + return (castle != null) && castle.isFirstMidVictory(); } }