TechzoneMC / SonarPet

A fork of EchoPet, updated to 1.11
https://ci.techcable.net/job/SonarPet/
GNU General Public License v3.0
19 stars 16 forks source link

Build #15 Pet Hat Crashes Java Binary #20

Closed Internetpilot closed 8 years ago

Internetpilot commented 8 years ago

With Build #15 attempting to switch out of the Pet Hat feature causes the Java binary to crash on the client (not the server). There is no console message or error message in Minecraft other than reporting the Java binary has stopped responding. If you toggle the pet instead of switching off the Pet Hat feature, the problem doesn't occur.

Techcable commented 8 years ago

Can you get me logs from the client and the server? What version of minecraft and java are you using?

Techcable commented 8 years ago

Hats work just fine for me on TacoSpigot 1.9 with a forge 1.9 client. Horse hat

Internetpilot commented 8 years ago

I'm using the latest build (as of today, 4/11/16) of Spigot 1.9.2, the latest vanilla Minecraft client (1.9.2), and I have no idea what version of Java I'm running on the client because I've never installed it on this gaming laptop (it's brand new) so it must just be whatever the Minecraft client installs/downloads.

There's nothing in either the Spigot logs or the Minecraft client logs. I don't know where I would find any Java client logs since I never actually installed Java on this laptop???

The pet hat feature definitely works for me, too. But when I click on the pet hat icon again (which is supposed to remove the pet from your head), that's when Java crashes. If I just toggle the pet off instead of attempting to remove the pet from my head with the pet hat icon, it doesn't crash.

andrewkm commented 8 years ago

Using Spigot 800, SonarPets 15. Client Using 1.9.2 with Forge and WorldEdit CUI.

Cannot reproduce this issue at all and am not experiencing any crashing.

Internetpilot commented 8 years ago

I can reproduce this exact error on all seven PCs that I have in this house running the latest Minecraft client (which have either Java 8 Update 77 manually installed or whatever is automatically installed with Minecraft these days). I'm running the latest build of a self-hosted Spigot 1.9.2 server (also with Java 8 Update 77). So far, four of my users are also encountering the exact same issue. I have no idea what Java version they're running (because neither do they -- haha).

To definitely reproduce the problem every time, activate the pet hat feature and then directly activate the ride your pet feature. It always causes the Java SE to stop responding, locking up the Minecraft client. Sometimes if you activate the pet hat feature and then press the pet hat feature again (like to take the pet off your head) it will also lock up, but not every time -- probably about 75% of the time.

The Spigot server is unaffected with nothing noted in the console log, even if I run the Minecraft client on that server (remember, I'm self-hosted). There's nothing noted in the Minecraft client log either. Seems to be a 100% Java issue like a bad programming call or something.

Techcable commented 8 years ago

Yes, but what errors show up on the client?

Internetpilot commented 8 years ago

It's simply says that Java SE had stopped responding and asks if you want to close it. If you do, Minecraft closes if you don't it just stays locked up. There are no other messages or errors.

Techcable commented 8 years ago

No I mean what are the contents of the debug log? You go into profile settings and set the launcher visibility options to 'Keep the launcher open'. Then you should see the launcher stay open showing debug logs. Once the client crashes, the launcher should stay open (they're seperate processes). Then right click on the logs, click 'Copy All Text', and paste it into a github gist (the logs may be very large, and github gist is probably one of the best pasting services).

Internetpilot commented 8 years ago

There is nothing there. It's not the Minecraft client that's was crashing. Java SE is. It doesn't happen anymore with the latest build (#17).

Techcable commented 8 years ago

Okay, so I guess this is fixed :tada:

Internetpilot commented 8 years ago

Aren't those the very best kind of issues? :smile: