protocol: remove panic::catch_unwind for PoS VP

anoma · Nov 10, 2023 · 82d8c23 · 82d8c23
1 parent 193ca95
commit 82d8c23
Showing 1 changed file with 7 additions and 18 deletions.
diff --git a/shared/src/ledger/protocol/mod.rs b/shared/src/ledger/protocol/mod.rs
@@ -842,24 +842,13 @@ where
                             // and `RefUnwindSafe` in
                             // shared/src/ledger/pos/vp.rs)
                             let keys_changed_ref = &keys_changed;
-                            let result = match panic::catch_unwind(move || {
-                                pos_ref
-                                    .validate_tx(
-                                        tx,
-                                        keys_changed_ref,
-                                        verifiers_addr_ref,
-                                    )
-                                    .map_err(Error::PosNativeVpError)
-                            }) {
-                                Ok(result) => result,
-                                Err(err) => {
-                                    tracing::error!(
-                                        "PoS native VP failed with {:#?}",
-                                        err
-                                    );
-                                    Err(Error::PosNativeVpRuntime)
-                                }
-                            };
+                            let result = pos_ref
+                                .validate_tx(
+                                    tx,
+                                    keys_changed_ref,
+                                    verifiers_addr_ref,
+                                )
+                                .map_err(Error::PosNativeVpError);
                             // Take the gas meter back out of the context
                             gas_meter = pos.ctx.gas_meter.into_inner();
                             result