v1.17: Remove unnecessary unwrap from `simulate_transaction_unchecked…

…()` (backport of solana-labs#35375) Remove unnecessary unwrap from `simulate_transaction_unchecked()` (solana-labs#35375) Remove unnecessary unwrap from simulate_transaction_unchecked() (cherry picked from commit cb260f1) Co-authored-by: Pankaj Garg <[email protected]>
pgarg66 · Mar 5, 2024 · 0c77757 · 0c77757
1 parent 527b75b
commit 0c77757
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/runtime/src/bank.rs b/runtime/src/bank.rs
@@ -4384,9 +4384,7 @@ impl Bank {
         let post_simulation_accounts = loaded_transactions
             .into_iter()
             .next()
-            .unwrap()
-            .0
-            .ok()
+            .and_then(|(loaded_transactions_res, _)| loaded_transactions_res.ok())
             .map(|loaded_transaction| {
                 loaded_transaction
                     .accounts
@@ -4406,7 +4404,12 @@ impl Bank {
 
         debug!("simulate_transaction: {:?}", timings);
 
-        let execution_result = execution_results.pop().unwrap();
+        let execution_result =
+            execution_results
+                .pop()
+                .unwrap_or(TransactionExecutionResult::NotExecuted(
+                    TransactionError::InvalidProgramForExecution,
+                ));
         let flattened_result = execution_result.flattened_result();
         let (logs, return_data) = match execution_result {
             TransactionExecutionResult::Executed { details, .. } => {