From: J. Eric Ivancich Date: Wed, 12 Jan 2022 18:41:42 +0000 (-0500) Subject: rgw: fix bucket index list minor calculation bug X-Git-Tag: v17.1.0~21^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=badb8f1106980e30a938c44a37b22a91b920b5f6;p=ceph.git rgw: fix bucket index list minor calculation bug When "bucket index list" traverses the different regions in the bucket index assembling the output, it miscalculates how many entries to ask for at one point. This fixes that. This fixes previous "rgw: bucket index list can produce I/O errors". Credit for finding this bug goes to Soumya Koduri . Signed-off-by: J. Eric Ivancich (cherry picked from commit aa7605151f0a5f467d50f13f27c7aef42a40cc39) --- diff --git a/src/cls/rgw/cls_rgw.cc b/src/cls/rgw/cls_rgw.cc index 31cef6d06791..432b3aef71a4 100644 --- a/src/cls/rgw/cls_rgw.cc +++ b/src/cls/rgw/cls_rgw.cc @@ -2984,7 +2984,7 @@ static int rgw_bi_list_op(cls_method_context_t hctx, } if (!more) { - ret = list_plain_entries(hctx, op.name_filter, op.marker, max, + ret = list_plain_entries(hctx, op.name_filter, op.marker, max - count, &op_ret.entries, &more, PlainEntriesRegion::High); if (ret < 0) { CLS_LOG(0, "ERROR: %s: list_plain_entries (high) returned ret=%d, marker=\"%s\", filter=\"%s\", max=%d",