summaryrefslogtreecommitdiff
path: root/plugins/warp/src/plugin/ffi.rs
diff options
context:
space:
mode:
authorBrandon Miller <brandon@vector35.com>2025-09-26 08:33:48 -0400
committerBrandon Miller <brandon@vector35.com>2025-09-29 07:07:41 -0400
commitdf599006c0a805dda992910722ac69a9adf794fc (patch)
tree4d6b9643febf2d9d5f259d2d827846a13addec7a /plugins/warp/src/plugin/ffi.rs
parent9a9d7d98c2b49c734771397a8d37aca0279dfa21 (diff)
Fix Rust LowLevelILFunction owner function method
LowLevelILFunction objects can be constructed without a owner function. BNGetLowLevelILOwnerFunction can return NULL and must be checked before creating a Function object.
Diffstat (limited to 'plugins/warp/src/plugin/ffi.rs')
-rw-r--r--plugins/warp/src/plugin/ffi.rs10
1 files changed, 8 insertions, 2 deletions
diff --git a/plugins/warp/src/plugin/ffi.rs b/plugins/warp/src/plugin/ffi.rs
index c3a8051c..2826ed34 100644
--- a/plugins/warp/src/plugin/ffi.rs
+++ b/plugins/warp/src/plugin/ffi.rs
@@ -142,7 +142,10 @@ pub unsafe extern "C" fn BNWARPIsLiftedInstructionVariant(
unsafe { LowLevelILFunction::from_raw(analysis_function) };
match lifted_il.instruction_from_index(index) {
Some(instr) => {
- let relocatable_regions = relocatable_regions(&lifted_il.function().view());
+ let Some(owner_function) = lifted_il.function() else {
+ return false;
+ };
+ let relocatable_regions = relocatable_regions(&owner_function.view());
is_variant_instruction(&relocatable_regions, &instr)
}
None => false,
@@ -158,7 +161,10 @@ pub unsafe extern "C" fn BNWARPIsLowLevelInstructionComputedVariant(
unsafe { LowLevelILFunction::from_raw(analysis_function) };
match llil.instruction_from_index(index) {
Some(instr) => {
- let relocatable_regions = relocatable_regions(&llil.function().view());
+ let Some(owner_function) = llil.function() else {
+ return false;
+ };
+ let relocatable_regions = relocatable_regions(&owner_function.view());
is_computed_variant_instruction(&relocatable_regions, &instr)
}
None => false,