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

luminous: rgw: fix leak of curl handle on shutdown #24519

Merged
merged 1 commit into from Nov 1, 2018

Conversation

Projects
None yet
5 participants
@pdvian

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 luminous milestone Oct 10, 2018

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

@yuriw

This comment has been minimized.

Contributor

yuriw commented Oct 31, 2018

@yuriw yuriw merged commit c738678 into ceph:luminous Nov 1, 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