Task #3334
openrust: cleanup registration of C function pointers in SuricataContext
Description
In Suricata's main() a SuricataContext is setup with no comments on what it is for. This is used to send some C functions as function pointers inside Rust.
This solves a chicken and egg problem when running the Rust unit tests. When the Rust code is simply compiled we can reference any of the C functions, they will get resolved when the resulting Rust library is linked into the C executable. However, when running the Rust unit tests (via cargo test), these C functions don't exist, so the tests fail to compile. To get around this we pass some functions to Rust as pointers wrapped in an Option and resolve it. This process is at best a hack.
At a minimum just clean this up behind some more explicit Rust initialization, but it may be worth researching alternatives.
Updated by Jason Ish about 5 years ago
- Project changed from Suricata-Update to Suricata
Updated by Victor Julien almost 5 years ago
- Priority changed from Normal to High
Updated by Jason Ish over 3 years ago
- Target version changed from 7.0.0-beta1 to 8.0.0-beta1
Retargeting to 8. I don't think we can get rid of this as long as we have C that calls Rust, and Rust that calls C.
Updated by Victor Julien 5 months ago
- Subject changed from Cleanup registration of C function pointers in SuricataContext in main() to rust: cleanup registration of C function pointers in SuricataContext
- Priority changed from High to Normal
Updated by Victor Julien 5 months ago
- Related to Story #7141: misc: general improvements and cleanups added