From 973d0076958ec9416cbe9ebc1cec03ee147ca4f9 Mon Sep 17 00:00:00 2001 From: Casey Bodley Date: Wed, 12 Oct 2016 08:55:52 -0400 Subject: [PATCH] rgw: get_system_obj does not use result of get_system_obj_state get_system_obj() calls get_system_obj_state() to send a [getxattrs,stat] request and fill out an RGWObjState - but the results are not used Fixes: http://tracker.ceph.com/issues/17580 Signed-off-by: Casey Bodley --- src/rgw/rgw_rados.cc | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/rgw/rgw_rados.cc b/src/rgw/rgw_rados.cc index 80ebc273c0fc..3a9f8fc1408f 100644 --- a/src/rgw/rgw_rados.cc +++ b/src/rgw/rgw_rados.cc @@ -9373,14 +9373,8 @@ int RGWRados::get_system_obj(RGWObjectCtx& obj_ctx, RGWRados::SystemObject::Read uint64_t len; ObjectReadOperation op; - RGWObjState *astate = NULL; - get_obj_bucket_and_oid_loc(obj, bucket, oid, key); - int r = get_system_obj_state(&obj_ctx, obj, &astate, NULL); - if (r < 0) - return r; - if (end < 0) len = 0; else @@ -9398,7 +9392,7 @@ int RGWRados::get_system_obj(RGWObjectCtx& obj_ctx, RGWRados::SystemObject::Read } librados::IoCtx *io_ctx; - r = read_state.get_ioctx(this, obj, &io_ctx); + int r = read_state.get_ioctx(this, obj, &io_ctx); if (r < 0) { ldout(cct, 20) << "get_ioctx() on obj=" << obj << " returned " << r << dendl; return r; -- 2.47.3