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.

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

kvs: memory leak in namespace_create() #1483

Closed
garlick opened this issue Apr 23, 2018 · 0 comments
Closed

kvs: memory leak in namespace_create() #1483

garlick opened this issue Apr 23, 2018 · 0 comments
Milestone
release 0.9.0

Comments

@garlick
Copy link
Member

garlick commented Apr 23, 2018

There's a missing free() and json_decref() in namespace_create()'s successful return path.

I had this fixed in a a branch that was "paused" and didn't want to lose track of it.

See cc44fe5
@garlick garlick added this to the release 0.9.0 milestone Apr 23, 2018
garlick added a commit to garlick/flux-core that referenced this issue Apr 24, 2018
@garlick
modules/kvs: fix memory leak in namespace_create()
b9d616a 
Problem: valgrind test fails when private KVS namespace
is made part of job setup.

Looks like some memory allocated in namespace_create()
that was freed in the error path, was not freed in
the non-error path.  Fix that.

Fixes flux-framework#1483
@garlick garlick mentioned this issue Apr 24, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
release 0.9.0
Development

No branches or pull requests
1 participant
@garlick