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

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

Merged
1 commit merged into from Aug 30, 2015

Conversation

Projects
None yet
3 participants
@ghost
Copy link

commented Jul 28, 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)

@ghost ghost self-assigned this Jul 28, 2015

@ghost ghost added this to the hammer milestone Jul 28, 2015

@ghost ghost added bug fix core labels Jul 28, 2015

ghost pushed a commit that referenced this pull request Jul 29, 2015

Merge pull request #5378: Memory leak in Mutex.cc, pthread_mutexattr_…
…init without pthread_mutexattr_destroy

Reviewed-by:

oritwas added a commit to oritwas/ceph that referenced this pull request Aug 28, 2015

rgw: make radosgw-admin user rm idempotent
Fixes: ceph#5378
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
@ghost

This comment has been minimized.

Copy link
Author

commented Aug 30, 2015

@tchaikov does this backport look good to merge ? It passed a run of the hammer rados suite ( see http://tracker.ceph.com/issues/12701#rados for details ).

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5378: Memory leak in Mutex.cc, pthread_mutexattr_…
…init without pthread_mutexattr_destroy

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5378: Memory leak in Mutex.cc, pthread_mutexattr_…
…init without pthread_mutexattr_destroy

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5378: Memory leak in Mutex.cc, pthread_mutexattr_…
…init without pthread_mutexattr_destroy

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@tchaikov

This comment has been minimized.

Copy link
Contributor

commented Aug 30, 2015

@dachary lgtm
On Sun, 30 Aug 2015 at 18:09 Loic Dachary notifications@github.com wrote:

@tchaikov https://github.com/tchaikov does this backport look good to
merge ? It passed a run of the hammer rados suite ( see
http://tracker.ceph.com/issues/12701#rados for details ).


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

ghost pushed a commit that referenced this pull request Aug 30, 2015

Loic Dachary
Merge pull request #5378 from dachary/wip-12394-hammer
Memory leak in Mutex.cc, pthread_mutexattr_init without pthread_mutexattr_destroy

Reviewed-by: Kefu Chai <kchai@redhat.com>

@ghost ghost merged commit dfcdba7 into ceph:hammer Aug 30, 2015

@ghost ghost changed the title Memory leak in Mutex.cc, pthread_mutexattr_init without pthread_mutexattr_destroy common: Memory leak in Mutex.cc, pthread_mutexattr_init without pthread_mutexattr_destroy Sep 13, 2015

oritwas added a commit to oritwas/ceph that referenced this pull request Sep 14, 2015

rgw: make radosgw-admin user rm idempotent
Fixes: ceph#5378
Signed-off-by: Orit Wasserman <owasserm@redhat.com>

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.