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

common: Memory leak in Mutex.cc, pthread_mutexattr_init without pthread_mutexattr_destroy #5403

Merged
merged 1 commit into from Sep 7, 2015

Conversation

Projects
None yet
4 participants
@smithfarm
Contributor

smithfarm commented Jul 29, 2015

Mutex: fix leak of pthread_mutexattr
Need pthread_mutexattr_destroy after pthread_mutexattr_init

Fixes: #111762
Signed-off-by: Ketor Meng <d.ketor@gmail.com>
(cherry picked from commit 2b23327)

@smithfarm smithfarm self-assigned this Jul 29, 2015

@smithfarm smithfarm added this to the firefly milestone Jul 29, 2015

@smithfarm smithfarm added bug fix core common and removed core labels Jul 29, 2015

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Sep 4, 2015

@tchaikov This has passed a rados suite (see http://tracker.ceph.com/issues/11644#rados for details). OK to merge, do you think?

@smithfarm smithfarm assigned tchaikov and unassigned smithfarm Sep 7, 2015

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Sep 7, 2015

lgtm

@tchaikov tchaikov assigned smithfarm and unassigned tchaikov Sep 7, 2015

smithfarm added a commit that referenced this pull request Sep 7, 2015

Merge pull request #5403 from SUSE/wip-12393-firefly
Memory leak in Mutex.cc, pthread_mutexattr_init without pthread_mutexattr_destroy

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Nathan Cutler <ncutler@suse.com>

@smithfarm smithfarm merged commit c4fa5ef into ceph:firefly Sep 7, 2015

@smithfarm smithfarm deleted the SUSE:wip-12393-firefly branch Sep 7, 2015

@ghost ghost changed the title from Memory leak in Mutex.cc, pthread_mutexattr_init without pthread_mutexattr_destroy to common: Memory leak in Mutex.cc, pthread_mutexattr_init without pthread_mutexattr_destroy Oct 24, 2015

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