Skip to content

Update rust ffi code to use C-unwind #3151

Description

@stevenengler

Most of our rust-C ffi code uses pub extern "C" fn, but these functions are not allowed by rust to panic (a panic is not allowed to cross the ffi boundary). While Shadow does do this and this is technically UB, it has seemed to work fine for us. But this may break in the future, so we should switch to pub extern "C-unwind" fn instead.

See:

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Fields

    Component

    None yet

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions