]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
rgw: Fix Browser POST content-length-range min value 49962/head
authorRobin H. Johnson <rjohnson@digitalocean.com>
Wed, 1 Feb 2023 06:29:51 +0000 (22:29 -0800)
committerRobin H. Johnson <rjohnson@digitalocean.com>
Wed, 1 Feb 2023 19:40:46 +0000 (11:40 -0800)
commit18533be1406997eedae16216447d9b4dcf460643
tree2fb6489cab528e9155ed9bebb142f1f26d36773f
parent89bad98ff59da867142408ba268c9b8bafd1a911
rgw: Fix Browser POST content-length-range min value

`ERR_TOO_SMALL` is wrongly returned if all of the following are true,
- the get_data returns multiple items (chunks)
- the length of the last item is smaller than the POST Policy's min
  value for content-length-range.

The check should be `(ofs < min_len)` instead of `(len < min_len)`

This is further confirmed by the next line of `s->obj_size = ofs`

Move the `int len` scope inside loop to try and prevent the bug in
future.

The bug was refactored in 2016, but was introduced in Oct 2012, when
this functionality was first added to RGW in commit 7bb3504d3f0974e9863f536e9af0ce8889d6888f.

Reference: https://github.com/ceph/ceph/blob/933a42f9af349b3b222270e7f19f1fe151d89e8e/src/rgw/rgw_op.cc#L4474-L4513
Reference: https://github.com/ceph/ceph/commit/7bb3504d3f0974e9863f536e9af0ce8889d6888f
Signed-off-by: Robin H. Johnson <rjohnson@digitalocean.com>
src/rgw/rgw_op.cc