Fixes an issue where errors do not get propogated correctly when the hypervisor handler thread returns an error.

The hypervisor handler thread can return an error rather than send a message, in this case the error returned would be a timeout rather than the real error, this change ensures that we check for an error in the hypervisor handler thread when we time out and return that if it exists

Signed-off-by: Simon Davies <[email protected]>

Author: simongdavies

src/hyperlight_host/src/hypervisor/hypervisor_handler.rs

@@ -592,7 +592,29 @@ impl HypervisorHandler {
                 HandlerMsg::Error(e) => Err(e),
                 HandlerMsg::FinishedHypervisorHandlerAction => Ok(()),
             },
-            Err(_) => Err(HyperlightError::HypervisorHandlerMessageReceiveTimedout()),
+            Err(_) => {
+                // If we have timed out it may be that the handler thread returned an error before it sent a message, so rather than just timeout here
+                // we will try and get the join handle for the thread and if it has finished check to see if it returned an error
+                // if it did then we will return that error, otherwise we wil return the timeout error
+                match self
+                    .execution_variables
+                    .join_handle
+                    .try_lock()
+                    .map_err(|_| HyperlightError::HypervisorHandlerMessageReceiveTimedout())?
+                    .take()
+                {
+                    Some(handle) => {
+                        if handle.is_finished() {
+                            let res = handle.join();
+                            if res.as_ref().is_ok_and(|inner_res| inner_res.is_err()) {
+                                return Err(res.unwrap().unwrap_err());
+                            }
+                        }
+                        Err(HyperlightError::HypervisorHandlerMessageReceiveTimedout())
+                    }
+                    None => Err(HyperlightError::HypervisorHandlerMessageReceiveTimedout()),
+                }
+            }
         }
     }