Skip to content

Review use of expect/panic/assert/unwrap across the hyperlight codebase #320

Closed
@simongdavies

Description

@simongdavies

Since hyperlight is a library the use of expect/panic/assert/unwrap especially in release builds is problematic since it use implies that we are happy to terminate the process, in most cases these usages should be turned into errors returned to the host program, the only cases where we should expect/panic/assert/unwrap are where we detect out of bounds memory access attempts or actions that would corrupt data.

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

Status

In Progress

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions