Make Bigtable::get_confirmed_blocks inclusive of requested start_slot and end_slot (#14651)

* Fix off-by-one error * Filter out blocks greater than end slot
2021-01-18 19:14:10 -07:00
parent d63b2baf0e
commit cbf8ef7480
1 changed files with 5 additions and 1 deletions
--- a/core/src/rpc.rs
+++ b/core/src/rpc.rs
@ -732,8 +732,12 @@ impl JsonRpcRequestProcessor {
                    .runtime_handle
                    .block_on(
                        bigtable_ledger_storage
-                            .get_confirmed_blocks(start_slot, (end_slot - start_slot) as usize),
+                            .get_confirmed_blocks(start_slot, (end_slot - start_slot) as usize + 1), // increment limit by 1 to ensure returned range is inclusive of both start_slot and end_slot
                    )
+                    .map(|mut bigtable_blocks| {
+                        bigtable_blocks.retain(|&slot| slot <= end_slot);
+                        bigtable_blocks
+                    })
                    .unwrap_or_else(|_| vec![]));
            }
        }