More rust cleanup

- Added main thread handler api
- Register a headless main thread handler by default in initialization
- Refactor QualifiedName to be properly owned
- Loosened some type constraints on some apis involving QualifiedName
- Fixed some apis that were crashing due to incorrect param types
- Removed extern crate cruft for log crate
- Simplified headless initialization using more wrapper apis
- Fixed segments leaking because of no ref wrapper, see BinaryViewExt::segment_at
- Added rstest to manage headless init in unit tests
- Added some more unit tests
- Refactored demangler api to be more ergonomic
- Fixed minidump plugin not building

plugins/minidump/src/command.rs

@@ -7,7 +7,7 @@ use binaryninja::binaryview::{BinaryView, BinaryViewBase, BinaryViewExt};
 
 pub fn print_memory_information(bv: &BinaryView) {
     debug!("Printing memory information");
-    if let Ok(minidump_bv) = bv.parent_view() {
+    if let Some(minidump_bv) = bv.parent_view() {
         if let Ok(read_buffer) = minidump_bv.read_buffer(0, minidump_bv.len()) {
             if let Ok(minidump_obj) = Minidump::read(read_buffer.get_data()) {
                 if let Ok(memory_info_list) = minidump_obj.get_stream::<MinidumpMemoryInfoList>() {

plugins/minidump/src/view.rs

@@ -116,7 +116,7 @@ impl MinidumpBinaryView {
     }
 
     fn init(&self) -> BinaryViewResult<()> {
-        let parent_view = self.parent_view()?;
+        let parent_view = self.parent_view().ok_or(())?;
         let read_buffer = parent_view.read_buffer(0, parent_view.len())?;
 
         if let Ok(minidump_obj) = Minidump::read(read_buffer.get_data()) {

rust/Cargo.toml

@@ -16,4 +16,5 @@ rayon = { version = "1.8", optional = true }
 binaryninjacore-sys = { path = "binaryninjacore-sys" }
 
 [dev-dependencies]
-rstest = "0.23.0"
+rstest = "0.24.0"
+tempfile = "3"

rust/examples/demangler.rs

@@ -16,11 +16,11 @@ impl CustomDemangler for TestDemangler {
         _arch: &CoreArchitecture,
         name: &str,
         _view: Option<Ref<BinaryView>>,
-    ) -> Result<(Option<Ref<Type>>, QualifiedName), ()> {
+    ) -> Option<(QualifiedName, Option<Ref<Type>>)> {
         match name {
-            "test_name" => Ok((Some(Type::bool()), QualifiedName::from(vec!["test_name"]))),
-            "test_name2" => Ok((None, QualifiedName::from(vec!["test_name2", "aaa"]))),
-            _ => Err(()),
+            "test_name" => Some((QualifiedName::from(vec!["test_name"]), Some(Type::bool()))),
+            "test_name2" => Some((QualifiedName::from(vec!["test_name2", "aaa"]), None)),
+            _ => None,
         }
     }
 }

rust/src/architecture.rs

@@ -2588,7 +2588,7 @@ where
                     return expr.index.0;
                 }
             } else {
-                warn!(
+                log::warn!(
                     "unable to unpack flag write op: {:?} with {} operands",
                     op,
                     operands.len()