Bug #15528
rgw: leaks RGWStatObjCR
Status:
Resolved
Priority:
Urgent
Assignee:
-
Target version:
-
% Done:
0%
Source:
Q/A
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
<kind>Leak_DefinitelyLost</kind> <xwhat> <text>172,653 (100,352 direct, 72,301 indirect) bytes in 64 blocks are definitely lost in loss record 379 of 381</text> <leakedbytes>172653</leakedbytes> <leakedblocks>64</leakedblocks> </xwhat> <stack> <frame> <ip>0x4C2B0E0</ip> <obj>/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so</obj> <fn>operator new(unsigned long)</fn> </frame> <frame> <ip>0x5167C52</ip> <obj>/usr/lib/librgw.so.2.0.0</obj> <fn>(anonymous namespace)::FindAnyShardCR::operate()</fn> <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir> <file>rgw_metadata.cc</file> <line>387</line> </frame> <frame> <ip>0x50FCA3A</ip> <obj>/usr/lib/librgw.so.2.0.0</obj> <fn>RGWCoroutinesStack::operate(RGWCoroutinesEnv*)</fn> <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir> <file>rgw_coroutine.cc</file> <line>187</line> </frame> <frame> <ip>0x50FE8DA</ip> <obj>/usr/lib/librgw.so.2.0.0</obj> <fn>RGWCoroutinesManager::run(std::list<RGWCoroutinesStack*, std::allocator<RGWCoroutinesStack*> >&)</fn> <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir> <file>rgw_coroutine.cc</file> <line>468</line> </frame> <frame> <ip>0x50FF44C</ip> <obj>/usr/lib/librgw.so.2.0.0</obj> <fn>RGWCoroutinesManager::run(RGWCoroutine*)</fn> <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir> <file>rgw_coroutine.cc</file> <line>595</line> </frame> <frame> <ip>0x516B92E</ip> <obj>/usr/lib/librgw.so.2.0.0</obj> <fn>find_shards_for_period</fn> <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir> <file>rgw_metadata.cc</file> <line>418</line> </frame> <frame> <ip>0x516B92E</ip> <obj>/usr/lib/librgw.so.2.0.0</obj> <fn>(anonymous namespace)::find_oldest_log_period(RGWRados*)</fn> <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir> <file>rgw_metadata.cc</file> <line>435</line> </frame> <frame> <ip>0x516C374</ip> <obj>/usr/lib/librgw.so.2.0.0</obj> <fn>RGWMetadataManager::init(std::string const&)</fn> <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir> <file>rgw_metadata.cc</file> <line>482</line> </frame> <frame>
/a/sage-2016-04-16_08:41:53-rgw-master---basic-smithi/134502
History
#1 Updated by Sage Weil almost 8 years ago
/a/sage-2016-04-16_08:41:53-rgw-master---basic-smithi/134490
/a/sage-2016-04-16_08:41:53-rgw-master---basic-smithi/134526
#2 Updated by Yehuda Sadeh almost 8 years ago
There are a couple of other leaks there as well, some related to extra reference taken (introduced in 1a0e74bbd1e81d483a90d37ab088744c4aa7c788), and the error handling.
#3 Updated by Yehuda Sadeh almost 8 years ago
looking at it a bit more, the change in that commit seems ok. The refcounting seems a bit off at the async cr code.
#4 Updated by Yehuda Sadeh almost 8 years ago
- Status changed from New to Resolved
multiple leak fixes went into 10.2.1, this one is included.