The client is sensitive to missing metadata in packets sent from the server.
A typical example is to strip out the metadata and only send the metadata structure with 0 keys.
Also adding extra values, leaving the existing ones there too does crash the client too. Example in the second attachment, there I added the int value of 123 to the key 73.
Attachments
Comments 6
I've developed a proxy that if nothing else passes on the messages as is. it has been working for over a year so I am rather sure the test changes I do to trigger these bugs are bugs in the client.
While I appreciate the vigor, unless it happens in vanilla and can be caused in vanilla, it isn't really a bug. The client side doesn't need to be robust enough to handle these issues, as they will never happen. If a client could trigger this behaviour on the server side though, then I would be worried.
Sounds like the proxy you have developed is causing the issues. Just because the proxy has been working for over a year doesn't mean it can't eventually cause an issue, especially when the MC protocols themselves change.
@Ray Well our proxy is only causing it when I tell it to such as when I want to generate the crash log for this report.
Yes the Minecraft protocol does change and we catch up all the time, but the development would have been a lot easier if the vanilla client simply disconnected rather than crashed and held the computer for half a minute.
I did send a similar report for another but that was closed since it did not affect vanilla client <-> vanilla server. I'm still stand that ones client should not crash for packets it receive over the internet, hover since you are of a different opinion you should close this issue too.
At least it makes a good tool for putting force into the /ban command 🙂
Does this bug occur in vanilla without interference? How did you "strip out the metadata" or "add extra values"?