I'm getting some really strange crashes when I testplay my mod via Open-To-Lan.

I'm using forge build 1412

 

The client connecting gets this crash after 0-3 seconds after joining:

09:31:18] [Netty Client IO #2/ERROR] [FML/]: SimpleChannelHandlerWrapper exception
io.netty.handler.codec.DecoderException: java.lang.NullPointerException: Undefined message for discriminator 10 in channel vintagecraft

 

- However my mod using the vintagecraft channel registers only Message with the discriminators 0,1,2,3,4,5,6,7!

 

 

On the machine where i opened to lan I get this crash:

 

Caused by: java.lang.IndexOutOfBoundsException: readerIndex(165) + length(1) exceeds writerIndex(165): UnpooledHeapByteBuf(ridx: 165, widx: 165, cap: 256)
at io.netty.buffer.AbstractByteBuf.checkReadableBytes(AbstractByteBuf.java:1161) ~[AbstractByteBuf.class:4.0.15.Final]
at io.netty.buffer.AbstractByteBuf.readByte(AbstractByteBuf.java:563) ~[AbstractByteBuf.class:4.0.15.Final]
at net.minecraft.network.PacketBuffer.readByte(SourceFile:609) ~[hd.class:?]
at net.minecraftforge.fml.common.network.FMLIndexedMessageToMessageCodec.decode(FMLIndexedMessageToMessageCodec.java:73) ~[FMLIndexedMessageToMessageCodec.class:?]

(full crash report)

 

I checked the forge and minecraft source code. This crash apparently happens when Forge tries to read the package discriminator.

 

My best guess which Packet is causing the crash is MechanicalNetworkNBTPacket.java

 

Registered via: packetPipeline.registerMessage(MechanicalNetworkNBTPacket.ClientHandler.class, MechanicalNetworkNBTPacket.class, 4, Side.CLIENT);

 

The packet send command:

@SubscribeEvent
public void onServerTick(TickEvent.WorldTickEvent event) {
   [....other code....]

if (event.world.getWorldTime() % 40 == 0) {
		NBTTagCompound nbt = new NBTTagCompound();
		writeToNBT(nbt);
		VintageCraft.packetPipeline.sendToDimension(
			new MechanicalNetworkNBTPacket(nbt, networkId), 
			event.world.provider.getDimensionId()
		);
	}
}

 

What am I doing wrong here?

Hi

 

I've had this problem twice before-

one time, my toBytes() packet code was overwriting the discriminator.  (The buffer passed to toBytes() has the discriminator at position 0 and you're supposed to append to it.) 

 

another time, it was caused by a thread synchronisation bug (race condition) when sending to dimension or allNearbyPlayers.

https://github.com/MinecraftForge/MinecraftForge/issues/1908

 

Try changing this line

VintageCraft.packetPipeline.sendToDimension(

            new MechanicalNetworkNBTPacket(nbt, networkId),

            event.world.provider.getDimensionId()

        );

i.e. copy the code out of the forge sendToDimension to find the list of players and send them all a message one by one

 

for example

//    StartupCommon.simpleNetworkWrapper.sendToDimension(msg, sendingPlayer.dimension);  // DO NOT USE sendToDimension, it is buggy
//    as of build 1419 - see https://github.com/MinecraftForge/MinecraftForge/issues/1908

    int dimension = sendingPlayer.dimension;
    MinecraftServer minecraftServer = sendingPlayer.mcServer;
    for (EntityPlayerMP player : (List<EntityPlayerMP>)minecraftServer.getConfigurationManager().playerEntityList) {
      TargetEffectMessageToClient msg = new TargetEffectMessageToClient(message.getTargetCoordinates());   // must generate a fresh message for every player!
      if (dimension == player.dimension) {
        StartupCommon.simpleNetworkWrapper.sendTo(msg, player);
      }
    }

 

-TGG

 

Thank you very much, TheGreyGhost.

 

Sending the packet to the players individually helped with the crashes. Odd, I had the crash also under Forge Build 1450, but the changelog says its fixed from 1436 on.