Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memory Leak #70

Closed
ninja- opened this issue Nov 7, 2014 · 7 comments
Closed

Memory Leak #70

ninja- opened this issue Nov 7, 2014 · 7 comments

Comments

@ninja-
Copy link

ninja- commented Nov 7, 2014

Hi, I am experiencing a memory leak with ProtocolLib.
Latest ProtocolLib build, latest available Spigot build.
I have done heap dumps and...debugging heap dumps is always tricky,
but it clearly shows that CraftPlayers are being leaked and somehow - it's ProtocolLib's ChannelInjector holding the CraftPlayer(and the player is holding other stuff such as now-unloaded worlds).

Removing ProtocolLib removes memory leaks.

Apart from the ChannelInjector, GC root also shows TinyProtocolLib instance from another plugin on the same path.
I am sure that the used TinyProtocolLib does not cause issues on itself - are there incompatibilities with "real" ProtocolLib that could cause CraftPlayers to leak?

Thanks for any help. Please tell me if you need more debugging etc. information.

@aadnk
Copy link
Owner

aadnk commented Nov 10, 2014

Sorry, but due to the CraftBukkit DCMA, I'm no longer maintaing ProtocolLib.

@Postremus
Copy link

This is very sad to hear, aadnk. :(

Is someone taking over this project, or will it die?

@Hellcode48
Copy link

Spigot is being Updated, MAINTAIN THE PLUGIN plz :D

@Janmm14
Copy link

Janmm14 commented Nov 14, 2014

@aadnk I would really like it if you would update protocollib to 1.8, because spigot is updated. I think you can get the spigot patch from md_5 for 1.8

@Postremus
Copy link

We, and many others, needs this lib for our servers.
Would you please at least recommend some alternative packet sending and
receiving lib?

2014-11-14 10:08 GMT+01:00 Jan Marian Meyer notifications@github.com:

@aadnk https://github.com/aadnk I would really like it if you would
update protocollib to 1.8, because spigot is updated. I think you can get
the spigot patch from md_5 for 1.8


Reply to this email directly or view it on GitHub
#70 (comment).

@ghost
Copy link

ghost commented Nov 15, 2014

@aadnk Is it alright if I continue ProtocolLib?

dmulloy2 added a commit to dmulloy2/ProtocolLib that referenced this issue Nov 15, 2014
@AmauryCarrade
Copy link

@aadnk Is it alright if I continue ProtocolLib?

This plugin is licensed under a GPLv2 license, so anyone can fork & continue the plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants