]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
rgw_rest_s3.cc: reorder checks in while loop
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Tue, 21 Feb 2017 14:28:46 +0000 (15:28 +0100)
committerKefu Chai <kchai@redhat.com>
Mon, 10 Apr 2017 10:16:17 +0000 (18:16 +0800)
Fix for:

[src/rgw/rgw_rest_s3.cc:1224]: (style) Array index 'chunk_offset'
 is used before limits check.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
src/rgw/rgw_rest_s3.cc

index e498c504303e81a7d5516fec6e3a9b7b46e314e3..7fd3c6a951844da979884a8e6aeb065c0feb814d 100644 (file)
@@ -1234,7 +1234,7 @@ int RGWPutObj_ObjStore_S3::validate_and_unwrap_available_aws4_chunked_data(buffe
 
     /* grab chunk size */
 
-    while ((*(chunk_str+chunk_offset) != ';') && (chunk_offset < chunk_str_min_len))
+    while ((chunk_offset < chunk_str_min_len) && (chunk_str[chunk_offset] != ';'))
       chunk_offset++;
     string str = string(chunk_str, chunk_offset);
     unsigned int chunk_data_size;