hyperlight-dev / hyperlight-dev/hyperlight

Replace generic HyperlightError::Error variant with specific error types to improve sandbox poisoning accuracy

未关闭
#998 2 条评论 0 个 reaction 已指派 1 人 已被 @ludfjig 认领 在 GitHub 查看
area/security lifecycle/confirmed
主要语言
Rust
星标
4.7k
派生
208
平均合并
1 天 7 小时
30 天内合并 PR
47

描述

The sandbox poisoning mechanism relies on accurate error information and strong typing to determine when a sandbox has entered an inconsistent state requiring recovery. Currently, the `HyperlightError::Error` variant is a generic catch-all error type that can be created using the `new_error!` macro.

The problem is that `HyperlightError::Error` can be (and is) used for situations that should poison the sandbox, but the error type doesn't provide enough information for the poisoning logic to make the correct determination. This creates a safety hole where:

- Critical errors that leave the sandbox in an inconsistent state might be wrapped in generic error variants
- The poisoning mechanism cannot distinguish between benign errors and those that compromise sandbox state
- Sandboxes that should be poisoned may continue to be used

## Solution
Replace all instances of `HyperlightError::Error` with specific, strongly-typed error variants that accurately describe the error condition.

As an aside, we should probably distinguish between internal errors and public errors. Right now all errors are pub because they all are part of the same enum. I think splitting errors by something like file/module/struct/method is a good idea in general.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。