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

Allowed an unreferenced Socket to be closed automatically #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fidlej
Copy link

@fidlej fidlej commented Aug 3, 2011

The event loop is holding a reference to the Socket.
I used weakref to release the Socket, when nobody else is referencing to it.

@tmc
Copy link
Owner

tmc commented Oct 11, 2011

Do you mind creating a test case or some sort of example to display this issue?

@fidlej
Copy link
Author

fidlej commented Oct 11, 2011

Thanks for the interest. The issue can be reproduced when testing gevent-zeromq:
python setup.py test

The "set/getsockopt roundtrip" test will be blocked forever. It is waiting for an unreferenced socket to be closed.
I have libevent-1.4.13.

@gtaylor
Copy link

gtaylor commented Apr 23, 2012

Could this be causing what looks like a memory leak? I'm baffled as to why my application is hogging the amount of RAM it is.

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

Successfully merging this pull request may close these issues.

3 participants