Merge pull request #747 from pventuzelo/pventuzelo/724-fix-panics-exe…

…cute-wasm [fix issue #724] replace panic! with return Err()
wasmerio · Sep 3, 2019 · 80c1359 · 80c1359
2 parents 777833e + 7dd496c
commit 80c1359
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/bin/wasmer.rs b/src/bin/wasmer.rs
@@ -634,7 +634,9 @@ fn execute_wasm(options: &Run) -> Result<(), String> {
 
                 if let Err(ref err) = result {
                     match err {
-                        RuntimeError::Trap { msg } => panic!("wasm trap occured: {}", msg),
+                        RuntimeError::Trap { msg } => {
+                            return Err(format!("wasm trap occured: {}", msg))
+                        }
                         #[cfg(feature = "wasi")]
                         RuntimeError::Error { data } => {
                             if let Some(error_code) = data.downcast_ref::<wasmer_wasi::ExitCode>() {
@@ -644,7 +646,7 @@ fn execute_wasm(options: &Run) -> Result<(), String> {
                         #[cfg(not(feature = "wasi"))]
                         RuntimeError::Error { .. } => (),
                     }
-                    panic!("error: {:?}", err)
+                    return Err(format!("error: {:?}", err));
                 }
             }
         } else {