More detailed MultiSellChoose warning messages.

L2J_Mobius_01.0_Ertheia/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -134,7 +134,24 @@ public class MultiSellChoose implements ClientPacket
 			}
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_02.5_Underground/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_03.0_Helios/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_04.0_GrandCrusade/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_05.0_Salvation/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_05.5_EtinasFate/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_06.0_Fafurion/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_07.0_PreludeOfWar/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_08.2_Homunculus/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_09.2_ReturnOfTheQueenAnt/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_10.2_MasterClass/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_10.3_MasterClass/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_1.0/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -134,7 +134,24 @@ public class MultiSellChoose implements ClientPacket
 			}
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_1.5_AgeOfSplendor/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -134,7 +134,24 @@ public class MultiSellChoose implements ClientPacket
 			}
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_2.0_Saviors/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_2.5_Zaken/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_2.7_Antharas/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_2.8_SevenSigns/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_2.9.5_Saviors/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_2.9_SecretOfEmpire/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_3.0_TheKamael/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Classic_Interlude/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -127,7 +127,24 @@ public class MultiSellChoose implements ClientPacket
 			}
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Essence_4.2_DwellingOfSpirits/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Essence_5.2_FrostLord/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Essence_6.2_Vanguard/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);

L2J_Mobius_Essence_6.3_Crusader/java/org/l2jmobius/gameserver/network/clientpackets/MultiSellChoose.java

@@ -158,7 +158,24 @@ public class MultiSellChoose implements ClientPacket
 			return;
 		}
 		
-		final MultisellEntryHolder entry = list.getEntries().get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
+		final List<MultisellEntryHolder> entries = list.getEntries();
+		if (entries == null)
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested null multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if (entries.isEmpty())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested empty multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		if ((_entryId - 1) >= entries.size())
+		{
+			PacketLogger.warning("Character: " + player.getName() + " requested out of bounds multisell entry. Multisell: " + _listId + " entry: " + _entryId);
+			return;
+		}
+		
+		final MultisellEntryHolder entry = entries.get(_entryId - 1); // Entry Id begins from 1. We currently use entry IDs as index pointer.
 		if (entry == null)
 		{
 			PacketLogger.warning("Character: " + player.getName() + " requested inexistant prepared multisell entry. Multisell: " + _listId + " entry: " + _entryId);