From 9f06e2c16e409a9a8284af9964f0cfff6e222164 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Mon, 30 Jun 2014 12:55:45 -0700 Subject: [PATCH] rgw: fix uninit ofs in RGWObjManifect::obj_iterator Valgrind picked this up: UninitCondition Conditional jump or move depends on uninitialised value(s) 0x5145B8 /usr/bin/radosgw RGWObjManifest::obj_iterator::seek(unsigned long) /srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-0.82-354-g62027ec/src/rgw rgw_rados.cc 562 0x5672A4 /usr/bin/radosgw list_multipart_parts(RGWRados*, req_state*, std::string const&, std::string&, int, int, std::map<unsigned int, RGWUploadPartInfo, std::less<unsigned int>, std::allocator<std::pair<unsigned int const, RGWUploadPartInfo> > >&, int*, bool*, bool) /srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-0.82-354-g62027ec/src/rgw rgw_rados.h 217 0x5688EE /usr/bin/radosgw RGWListMultipart::execute() /srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-0.82-354-g62027ec/src/rgw rgw_op.cc 2956 ... Fixes: #8699 Backport: firefly Signed-off-by: Sage Weil (cherry picked from commit e0d3b78b6af523ae77b0cee385cd40f67f7a0ab6) --- src/rgw/rgw_rados.h | 1 + 1 file changed, 1 insertion(+) diff --git a/src/rgw/rgw_rados.h b/src/rgw/rgw_rados.h index 51ebccc2c5e1b..7aa91d0eeef0a 100644 --- a/src/rgw/rgw_rados.h +++ b/src/rgw/rgw_rados.h @@ -373,6 +373,7 @@ public: void init() { part_ofs = 0; stripe_ofs = 0; + ofs = 0; stripe_size = 0; cur_part_id = 0; cur_stripe = 0; -- 2.39.5