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?

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

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.

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.

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

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