]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
rgw_file: unify allocations of request id 28648/head
authorMatt Benjamin <mbenjamin@redhat.com>
Wed, 15 May 2019 13:31:03 +0000 (09:31 -0400)
committerPrashant D <pdhange@redhat.com>
Tue, 18 Jun 2019 23:10:27 +0000 (19:10 -0400)
commit6794279e840d15cf62d391a4fca68b120334fc2d
tree9be6060258faddc8c0a4985953ec7affd2319f95
parent91314fef668f7a838d8c1e2497a6e67027ad9bed
rgw_file: unify allocations of request id

The fix by Tao Chen <chentao@umcloud.com> for construction of the
request id in RGWWriteRequests/RGWContinuedReq paths--just writes,
effectively--revealed that RGWRequest::id was being set to zero
in general, while a valid request id was being assigned to
req_state.

This implies that code which the RGWRequest::id of non-write requests
would still use an invalid value, so avoid that by initializing
RGWRequest::id in all RGWLibRequests, and propagating its value
to req_state.  This still works as intended for RGWWriteRequest.

Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
(cherry picked from commit 51356347a3400bbf7ea846c3e1196aeb7be0ab60)
src/rgw/librgw.cc
src/rgw/rgw_file.h
src/rgw/rgw_lib.h