Project

General

Profile

Actions

Bug #6176

closed

rgw: valgrind: leak in copy_obj

Added by Sage Weil over 10 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Target version:
-
% Done:

0%

Source:
Q/A
Tags:
Backport:
dumpling
Regression:
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

<error>
  <unique>0x277d75</unique>
  <tid>1</tid>
  <kind>Leak_DefinitelyLost</kind>
  <xwhat>
    <text>1,560 (48 direct, 1,512 indirect) bytes in 3 blocks are definitely lost in loss record 22 of 22</text>
    <leakedbytes>1560</leakedbytes>
    <leakedblocks>3</leakedblocks>
  </xwhat>
  <stack>
    <frame>
      <ip>0x4C2B1C7</ip>
      <obj>/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so</obj>
      <fn>operator new(unsigned long)</fn>
    </frame>
    <frame>
      <ip>0x4FD1DA</ip>
      <obj>/usr/bin/radosgw</obj>
      <fn>RGWRados::prepare_get_obj(void*, rgw_obj&amp;, long*, long*, std::map&lt;std::string, ceph::buffer::list, std::less&lt;std::string&gt;, std::allocator&lt;std::pair&lt;std::string const, ceph::buffer::list&gt; &gt; &gt;*, long const*, long const*, long*, char const*, char const*, unsigned long*, unsigned long*, RGWObjVersionTracker*, void**, rgw_err*)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-0.67-421-g56ff410/src/rgw</dir>
      <file>rgw_rados.cc</file>
      <line>3585</line>
    </frame>
    <frame>
      <ip>0x502DC5</ip>
      <obj>/usr/bin/radosgw</obj>
      <fn>RGWRados::copy_obj(void*, std::string const&amp;, std::string const&amp;, std::string const&amp;, req_info*, std::string const&amp;, rgw_obj&amp;, rgw_obj&amp;, RGWBucketInfo&amp;, RGWBucketInfo&amp;, long*, long const*, long const*, char const*, char const*, bool, std::map&lt;std::string, ceph::buffer::list, std::less&lt;std::string&gt;, std::allocator&lt;std::pair&lt;std::string const, ceph::buffer::list&gt; &gt; &gt;&amp;, RGWObjCategory, std::string*, rgw_err*, void (*)(long, void*), void*)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-0.67-421-g56ff410/src/rgw</dir>
      <file>rgw_rados.cc</file>
      <line>2542</line>
    </frame>
    <frame>
      <ip>0x5202C5</ip>
      <obj>/usr/bin/radosgw</obj>
      <fn>RGWCopyObj::execute()</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-0.67-421-g56ff410/src/rgw</dir>
      <file>rgw_op.cc</file>
      <line>1712</line>
    </frame>
    <frame>
      <ip>0x4A58A6</ip>
      <obj>/usr/bin/radosgw</obj>
      <fn>RGWProcess::handle_request(RGWRequest*)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-0.67-421-g56ff410/src/rgw</dir>
      <file>rgw_main.cc</file>
      <line>389</line>
    </frame>
    <frame>
      <ip>0x4A7685</ip>
      <obj>/usr/bin/radosgw</obj>
      <fn>RGWProcess::RGWWQ::_process(RGWRequest*)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-0.67-421-g56ff410/src/rgw</dir>
      <file>rgw_main.cc</file>
      <line>165</line>
    </frame>
    <frame>
      <ip>0x5043F75</ip>
      <obj>/usr/lib/librados.so.2.0.0</obj>
      <fn>ThreadPool::worker(ThreadPool::WorkThread*)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-0.67-421-g56ff410/src/common</dir>
      <file>WorkQueue.cc</file>
      <line>125</line>
    </frame>

Actions #1

Updated by Ian Colle over 10 years ago

  • Assignee set to Yehuda Sadeh
Actions #2

Updated by Yehuda Sadeh over 10 years ago

happened in commit:56ff4101a12e190caea9805dd5fb250ab5fa8e8c

Actions #3

Updated by Yehuda Sadeh over 10 years ago

just to clarify: teuthology run was against this commit, not when the bug was introduced.

Actions #4

Updated by Yehuda Sadeh over 10 years ago

  • Backport set to dumpling
Actions #5

Updated by Yehuda Sadeh over 10 years ago

  • Status changed from New to Fix Under Review
  • Assignee changed from Yehuda Sadeh to Josh Durgin

pushed a fix to wip-6176.

Actions #6

Updated by Yehuda Sadeh over 10 years ago

  • Status changed from Fix Under Review to 12
  • Assignee changed from Josh Durgin to Yehuda Sadeh

hmm.. found an issue with my fix, redoing.

Actions #7

Updated by Yehuda Sadeh over 10 years ago

  • Status changed from 12 to Fix Under Review
  • Assignee changed from Yehuda Sadeh to Josh Durgin
Actions #8

Updated by Josh Durgin over 10 years ago

  • Status changed from Fix Under Review to 12
  • Assignee changed from Josh Durgin to Yehuda Sadeh
Actions #9

Updated by Yehuda Sadeh over 10 years ago

  • Status changed from 12 to Pending Backport
Actions #10

Updated by Sage Weil over 10 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF