A critical error when handling a packet

[Briggybros]

Hello, I'm trying to set up a system which changes the item in your hand when you press a bound key.

However when I press the key it comes up with;

2012-09-15 20:58:02 [sEVERE] [ForgeModLoader] A critical server error occured handling a packet, kicking net.minecraft.src.NetServerHandler@4cd5bf

 

My packet handler is as follows;

package info.mineverse.willow.sabers;

import java.io.ByteArrayInputStream;
import java.io.DataInputStream;
import java.io.IOException;

import net.minecraft.src.EntityClientPlayerMP;
import net.minecraft.src.EntityPlayerMP;
import net.minecraft.src.ItemStack;
import net.minecraft.src.NetworkManager;
import net.minecraft.src.Packet250CustomPayload;
import cpw.mods.fml.common.FMLCommonHandler;
import cpw.mods.fml.common.Side;
import cpw.mods.fml.common.network.IPacketHandler;
import cpw.mods.fml.common.network.Player;

public class PacketHandler implements IPacketHandler
{

@Override
public void onPacketData(NetworkManager manager, Packet250CustomPayload packet, Player player)
{
	if (packet.channel.equals("SaberDraw"))
	{
		handleDraw(packet, player);
	}
}

private void handleDraw(Packet250CustomPayload packet, Player player)
{
	DataInputStream inputStream = new DataInputStream(new ByteArrayInputStream(packet.data));

	int id;

	try {
		id = inputStream.readInt();
	} catch (IOException e) {
		e.printStackTrace();
		return;
	}
	EntityClientPlayerMP packetSender = (EntityClientPlayerMP)player;
	packetSender.inventory.setInventorySlotContents(packetSender.inventory.currentItem, new ItemStack(id, 1, packetSender.inventory.getCurrentItem().getItemDamage()));
}

}

 

and the construction for the packet is;

ByteArrayOutputStream bos = new ByteArrayOutputStream(;
        		DataOutputStream outputStream = new DataOutputStream(bos);
        		try {
        			outputStream.writeInt(SaberBase.saberPurpleOn.shiftedIndex);
        		} catch (Exception ex) {
        			ex.printStackTrace();
        		}
        		
        		Packet250CustomPayload packet = new Packet250CustomPayload();
        		packet.channel = "SaberDraw";
        		packet.data = bos.toByteArray();
        		packet.length = bos.size();
        		
        		Side side = FMLCommonHandler.instance().getSide();
        		if (side == Side.CLIENT)
        		{
        			mc.thePlayer.sendQueue.addToSendQueue(packet);
        		}

where mc is Minecraft and initialized with;

mc = ModLoader.getMinecraftInstance();

 

As far as I can see the error is most likely to be how I tried and failed to update the item in the packet handler, can anyone point me in the right direction?

[LexManos]

I would go find the 'A critical server error' line and make it print its exception.

[Briggybros]

Firstly, would it be easier to use Packet5PlayerInventory to send the packet to update the inventory?

Finally, where would the line be likely to be? I can't seem to find it.

[SanAndreaP]

Maybe you could start a dedicated server with your mod and look what it does? I use this method to find such kind of errors in my mods.

[Ebilkill]

I hope you aren't using the ModLoader.getMinecraftInstance(); on the server side...

[Briggybros]

No, the construction of the packet is within a class that only runs on ClientSide