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

mimic:rgw: fix leak of curl handle on shutdown #24518

Merged
merged 1 commit into from Oct 15, 2018

Conversation

Projects
None yet
5 participants
@pdvian
Copy link

pdvian commented Oct 10, 2018

rgw: fix leak of curl handle on shutdown
addresses a valgrind leak reported from curl_multi_add_handle().
requests that are still linked to the curl_multi handle need to be
unlinked and finished with ECANCELED, rather than just finished

Fixes: http://tracker.ceph.com/issues/35715

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit dd47848)

@tchaikov tchaikov added this to the mimic milestone Oct 10, 2018

@smithfarm smithfarm requested review from yehudasa and cbodley Oct 10, 2018

@yuriw

This comment has been minimized.

Copy link
Contributor

yuriw commented Oct 12, 2018

@yuriw yuriw merged commit 50a78ef into ceph:mimic Oct 15, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment