diff --git a/core/src/rpc.rs b/core/src/rpc.rs index 0a14e8cfe2..b9771984f9 100644 --- a/core/src/rpc.rs +++ b/core/src/rpc.rs @@ -31,6 +31,7 @@ use solana_transaction_status::{ }; use solana_vote_program::vote_state::{VoteState, MAX_LOCKOUT_HISTORY}; use std::{ + cmp::max, collections::HashMap, net::{SocketAddr, UdpSocket}, str::FromStr, @@ -381,7 +382,7 @@ impl JsonRpcRequestProcessor { } Ok(self .blockstore - .rooted_slot_iterator(start_slot) + .rooted_slot_iterator(max(start_slot, self.blockstore.lowest_slot())) .map_err(|_| Error::internal_error())? .filter(|&slot| slot <= end_slot) .collect()) @@ -2656,7 +2657,7 @@ pub mod tests { let result: Value = serde_json::from_str(&res.expect("actual response")) .expect("actual response deserialization"); let confirmed_blocks: Vec = serde_json::from_value(result["result"].clone()).unwrap(); - assert_eq!(confirmed_blocks, roots); + assert_eq!(confirmed_blocks, roots[1..].to_vec()); let req = format!(r#"{{"jsonrpc":"2.0","id":1,"method":"getConfirmedBlocks","params":[2]}}"#); @@ -2672,7 +2673,7 @@ pub mod tests { let result: Value = serde_json::from_str(&res.expect("actual response")) .expect("actual response deserialization"); let confirmed_blocks: Vec = serde_json::from_value(result["result"].clone()).unwrap(); - assert_eq!(confirmed_blocks, vec![0, 1, 3, 4]); + assert_eq!(confirmed_blocks, vec![1, 3, 4]); let req = format!(r#"{{"jsonrpc":"2.0","id":1,"method":"getConfirmedBlocks","params":[0, 7]}}"#); @@ -2680,7 +2681,7 @@ pub mod tests { let result: Value = serde_json::from_str(&res.expect("actual response")) .expect("actual response deserialization"); let confirmed_blocks: Vec = serde_json::from_value(result["result"].clone()).unwrap(); - assert_eq!(confirmed_blocks, vec![0, 1, 3, 4]); + assert_eq!(confirmed_blocks, vec![1, 3, 4]); let req = format!(r#"{{"jsonrpc":"2.0","id":1,"method":"getConfirmedBlocks","params":[9, 11]}}"#); diff --git a/ledger/src/blockstore.rs b/ledger/src/blockstore.rs index a87f691804..52c13e203a 100644 --- a/ledger/src/blockstore.rs +++ b/ledger/src/blockstore.rs @@ -1468,7 +1468,7 @@ impl Blockstore { } pub fn get_first_available_block(&self) -> Result { - let mut root_iterator = self.rooted_slot_iterator(0)?; + let mut root_iterator = self.rooted_slot_iterator(self.lowest_slot())?; Ok(root_iterator.next().unwrap_or_default()) }