Ошибка при депозите в ВХ (Хранилище)

[Mifesto]

Когда ложу вещи в КХ ил ив КВХ, то мне ГС выдает

Код:

01:21:40.115  Client: L2GameClient: player 'Rosetta'@127.0.0.1 from IP: 127.0.0.
1 - Failed running: [C] SendWareHouseDepositList - L2P Server Version: 1
java.lang.NullPointerException
        at l2p.gameserver.model.items.Warehouse.addItem(Warehouse.java:96)
        at l2p.gameserver.clientpackets.SendWareHouseDepositList.runImpl(SendWar
eHouseDepositList.java:132)
        at l2p.gameserver.clientpackets.L2GameClientPacket.run(L2GameClientPacke
t.java:44)
        at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(Unknown Source
)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
        at java.lang.Thread.run(Unknown Source)
01:21:40.349  Packet not completed. Maybe cheater. IP:127.0.0.1, account:kasik,
character:Rosetta

сам код SendWareHouseDepositLis:

package l2p.gameserver.clientpackets;

Код:

import l2p.Config;
import l2p.commons.collections.GArray;
import l2p.gameserver.cache.Msg;
import l2p.gameserver.model.L2Character;
import l2p.gameserver.model.L2Player;
import l2p.gameserver.model.instances.L2NpcInstance;
import l2p.gameserver.model.items.L2ItemInstance;
import l2p.gameserver.model.items.L2ItemInstance.ItemClass;
import l2p.gameserver.model.items.PcInventory;
import l2p.gameserver.model.items.Warehouse;
import l2p.gameserver.model.items.Warehouse.WarehouseType;

import java.util.HashMap;

public class SendWareHouseDepositList extends L2GameClientPacket
{
    private static final int _WAREHOUSE_FEE = 30;
    private HashMap<Integer, Long> _items;

    @Override
    public void readImpl()
    {
        int itemsCount = readD();
        if(itemsCount * 12 > _buf.remaining() || itemsCount > Short.MAX_VALUE || itemsCount < 0)
        {
            _items = null;
            return;
        }
        _items = new HashMap<Integer, Long>(itemsCount + 1, 0.999f);
        for(int i = 0; i < itemsCount; i++)
        {
            int obj_id = readD();
            long itemQuantity = readQ();
            if(obj_id < 1 || itemQuantity < 0)
            {
                _items = null;
                return;
            }
            _items.put(obj_id, itemQuantity);
        }
    }

    @Override
    public void runImpl()
    {
        L2Player activeChar = getClient().getActiveChar();
        if(activeChar == null || _items == null)
            return;

        // Проверяем наличие npc и расстояние до него
        L2NpcInstance whkeeper = activeChar.getLastNpc();
        if(whkeeper == null || !activeChar.isInRange(whkeeper.getLoc(), L2Character.INTERACTION_DISTANCE))
        {
            activeChar.sendPacket(Msg.WAREHOUSE_IS_TOO_FAR);
            return;
        }
        if(activeChar.isInStoreMode())
        {
            activeChar.sendPacket(Msg.WHILE_OPERATING_A_PRIVATE_STORE_OR_WORKSHOP_YOU_CANNOT_DISCARD_DESTROY_OR_TRADE_AN_ITEM);
            return;
        }
        Warehouse warehouse;
        PcInventory inventory = activeChar.getInventory();
        boolean privatewh = activeChar.getUsingWarehouseType() != WarehouseType.CLAN;
        int slotsleft = 0;
        long adenaDeposit = 0;
        // Список предметов, уже находящихся на складе
        L2ItemInstance[] itemsOnWarehouse;
        if(privatewh)
        {
            warehouse = activeChar.getWarehouse();
            itemsOnWarehouse = warehouse.listItems(ItemClass.ALL);
            slotsleft = activeChar.getWarehouseLimit() - itemsOnWarehouse.length;
        }
        else
        {
            warehouse = activeChar.getClan().getWarehouse();
            itemsOnWarehouse = warehouse.listItems(ItemClass.ALL);
            slotsleft = activeChar.getClan().getWhBonus() + Config.WAREHOUSE_SLOTS_CLAN - itemsOnWarehouse.length;
        }
        // Список стекуемых предметов, уже находящихся на складе
        GArray<Integer> stackableList = new GArray<Integer>();
        for(L2ItemInstance i : itemsOnWarehouse)
        {
            if(i.isStackable())
                stackableList.add(i.getItemId());
        }
        // Создаем новый список передаваемых предметов, на основе полученных данных
        GArray<L2ItemInstance> itemsToStoreList = new GArray<L2ItemInstance>(_items.size() + 1);
        for(Integer itemObjectId : _items.keySet())
        {
            L2ItemInstance item = inventory.getItemByObjectId(itemObjectId);
            if(item == null || !item.canBeStored(activeChar, privatewh)) // а его вообще положить можно?
                continue;
            if(!item.isStackable() || !stackableList.contains(item.getItemId())) // вещь требует слота
            {
                if(slotsleft <= 0) // если слоты кончились нестекуемые вещи и отсутствующие стекуемые пропускаем
                    continue;
                slotsleft--; // если слот есть то его уже нет
            }
            if(item.getItemId() == 57)
                adenaDeposit = _items.get(itemObjectId);
            itemsToStoreList.add(item);
        }
        // Проверяем, хватит ли у нас денег на уплату налога
        int fee = itemsToStoreList.size() * _WAREHOUSE_FEE;
        if(fee + adenaDeposit > activeChar.getAdena())
        {
            activeChar.sendPacket(Msg.YOU_LACK_THE_FUNDS_NEEDED_TO_PAY_FOR_THIS_TRANSACTION);
            return;
        }
        // Сообщаем о том, что слоты кончились
        if(slotsleft <= 0)
            activeChar.sendPacket(Msg.YOUR_WAREHOUSE_IS_FULL);

        // Перекидываем
        for(L2ItemInstance itemToStore : itemsToStoreList)
            warehouse.addItem(inventory.dropItem(itemToStore, _items.get(itemToStore.getObjectId()), false), activeChar.getName());

        // Платим налог
        activeChar.reduceAdena(fee, true);
        // Обновляем параметры персонажа
        activeChar.updateStats();
    }
}

не пойму в чем ошибка, менял патчи (думал из-за протокола), не помогло, сравнивал пакеты SendWareHouseDepositList. Еще в яве новичок и не знаю всех нюансов. Исходники сборки Sun-Dev (Pheonix). Помогите разобраться что не так работает.

П.С.

как люди узнают

Код:

sddddd

из пакета

Код:

2B 6B 00 61 00 73 00 69 00 6B 00 00 00 50 33 32 66 49 F0 FE 556 1A 35 B4 1D FF 36 50 40 и тд.....(пример на пакете AuthLogin)

[Ro_0TT]

Пакет не причем, ошибка здесь:

at l2p.gameserver.model.items.Warehouse.addItem(Warehouse.java:96)

[Mifesto]

спс щас проверю

[aliza]

Описание: Возвращает имя файла, в котором находится активный скрипт (либо это файл скриптов по умолчанию, либо это скрипты активного приложения). Для файла скриптов приложения функция возвращает то же имя, что и у исполняемого файла приложения. исключение составляют те случаи, когда загружаемый при старте приложения файл со скриптами во время выполнения заменяет сам себя другим файлом посредством функции SwitchToScriptFile.Аналогичным образом для файла скриптов по умолчанию возвращается имя файла, определенного в JFW.INI, исключая те случаи, когда происходит замена файла скриптов при помощи функции SwitchToScriptFile.

[ProDev]

Судя по всему фениксы? Выкладывайте из класса Warehouse метод addItem(), у меня код сильно перепилен, номера строк, разумеется, уже не совпадают.

Ошибка может быть и в пакете в вызове inventory.dropItem(), если этот метод вернет null, т.к. в addItem() параметр на null не проверяется, что и может привести к NPE.