search

thedudeguy / JukeIt

A Bukkit plugin for Minecraft that allows you to "Burn" new music onto Music Discs and play them in the Jukeboxes.
GNU General Public License v3.0
10 stars 10 forks source link

Failed to handle packet null pointer exception again #101

Closed HinchyMinecraft closed 12 years ago

HinchyMinecraft commented 12 years ago

Happens on right click of any record player, regardless of held item. User is kicked with Internal Server Error. Disc burners work fine.

Running latest recommended versions of everything: JukeBukkit 1.6.1 CraftBukkit 1.2.5-R4.0 SpoutPlugin 1212 Spoutcraft 1503 for Minecraft 1.2.5.

Stack trace:

18:25:57 [WARNING] Failed to handle packet: java.lang.NullPointerException java.lang.NullPointerException at cc.thedudeguy.jukebukkit.materials.blocks.RecordPlayer.onBlockInteract(RecordPlayer.java:175) at org.getspout.spout.block.mcblock.CustomBlock.interact(CustomBlock.java:328) at net.minecraft.server.ItemInWorldManager.interact(ItemInWorldManager.java:296) at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:636) at net.minecraft.server.Packet15Place.handle(SourceFile:39) at net.minecraft.server.NetworkManager.b(NetworkManager.java:229) at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:113) at org.getspout.spout.SpoutNetServerHandler.a(SpoutNetServerHandler.java:172) at net.minecraft.server.NetworkListenThread.a(NetworkListenThread.java:78) at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:567) at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:459) at net.minecraft.server.ThreadServerApplication.run(SourceFile:492)

My config:

enableWebServer: false webServerPort: 8080 minecraftServerHostname: change.me.com serverName: JukeBukkit allowWebServerUploads: false allowExternalURLs: true debug: false webServerHost: 127.0.0.1

thedudeguy commented 12 years ago

looks like its coming from recordplayer though, not with disc burner, this error stack trace cant be produced by the disc burner, did you mean record player?

HinchyMinecraft commented 12 years ago

Yes I had a derp moment there. Disc burner is actually working 100% fine.

thedudeguy commented 12 years ago

does the error persist after destroying the record player and then replacing it? is it an old record player that existed in the map since before the update? i revamped the record player in the last release.

HinchyMinecraft commented 12 years ago

no, the record players that were in my map all turned back into wooden planks. I had to place a new one. regardless, the error persists after destroying and replacing it.

EDIT: just for the sake of confirmation, placing, interacting with, and destroying speaker wire also works just fine.

thedudeguy commented 12 years ago

happens every time you right click, regardless of what your holding? record? needle? nothing?

HinchyMinecraft commented 12 years ago

all of the above.

HinchyMinecraft commented 12 years ago

if necessary I'd be perfectly happy to show you the error in person; the server is 70.34.196.2.

thedudeguy commented 12 years ago

I know what the bug is, i just need to come up with a way to fix it. will have it fixed for the next release.

thedudeguy commented 12 years ago

well, it would seem as though if you remove the messed up block. and restart your server while there is air there, when you come back the data in that block space should be gone, and a record palyer in that spot should than work fine again.

can you verify this for me?

thedudeguy commented 12 years ago

going to assume it fixed itself for you too.