gaspersic/solana
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

bank: don't panic when getting logs from malformed TransactionLogCollector

Browse Source
This commit is contained in:
Trent Nelson
2021-11-24 14:19:22 -07:00
committed by mergify[bot]
parent dafdc15dfe
commit e558ad4c71
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
runtime/src/bank.rs
View File

@ -582,10 +582,12 @@ impl TransactionLogCollector {
) -> Option<Vec<TransactionLogInfo>> { ) -> Option<Vec<TransactionLogInfo>> {
match address { match address {
None => Some(self.logs.clone()), None => Some(self.logs.clone()),
Some(address) => self Some(address) => self.mentioned_address_map.get(address).map(|log_indices| {
.mentioned_address_map log_indices
.get(address) .iter()
.map(|log_indices| log_indices.iter().map(|i| self.logs[*i].clone()).collect()), .filter_map(|i| self.logs.get(*i).cloned())
.collect()
}),
} }
} }
} }
@ -15397,7 +15399,6 @@ pub(crate) mod tests {
} }
#[test] #[test]
#[should_panic]
fn test_transaction_log_collector_get_logs_for_address() { fn test_transaction_log_collector_get_logs_for_address() {
let address = Pubkey::new_unique(); let address = Pubkey::new_unique();
let mut mentioned_address_map = HashMap::new(); let mut mentioned_address_map = HashMap::new();