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.

Sign up for GitHub

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

Jump to bottom

Check for Memory Leaks #13

Open

theonewolf opened this issue Jan 18, 2014 · 1 comment

Assignees

Labels

gray-inferencer

Contributor

theonewolf commented Jan 18, 2014

While checking some numbers, found the following new memory leaks (in new code) in various places throughout deep_inspection.c:

channel name construction, but not freeing before reassignment
getting lists from Redis, but not freeing lists
creating BSON handles, but not cleaning them up

Valgrind provides good insight into all of these.

ghost assigned theonewolf

Contributor Author

theonewolf commented Jan 28, 2014

Partially fixed in 3be1aad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment