| Age | Commit message (Collapse) | Author |
|
|
|
|
|
This change allows architecture plugins to override the LiftFunction
callback to iterate a function's basic block list and lift entire
functions at once. This is required for architectures such as TMS320
C6x, which have non-traditional "delay slots" in that branches, loads,
and other instructions take multiple cycles to complete, and branch
instructions can reside within the delay slots of other branches.
|
|
|
|
inlining during analysis
Previously the address of the instruction in the function being inlined
was used as the new instruction's address when copying it during
inlining. Now there is an additional option: use the address of the
call instruction that is being replaced as the new instruction's
address.
This new mode is useful when inlining thunks or stub functions, but care
must be taken if using it beyond that.
The benefit is that it ensures that when a function contains multiple
calls to the same stub function, each inlined copy ends up with distinct
addresses. This ensures that call type adjustments and other overrides
that are stored on the function and keyed by address can be applied
independently to each callsite that was inlined.
The trade-off is that if the function being inlined contains non-trivial
logic, all of the inlined instructions sharing an address will limit
what type of adjustments can be applied to them.
The Objective-C and shared cache workflows are updated to take advantage
of this new mode when they enable inlining of stub functions. This will
make it possible for multiple calls to the same runtime function within
a single function to have separate call type adjustments applied in the
future.
|
|
|
|
|
|
Assign the `session_id` span field before calling into the relocation handler so logs get scoped to the specific view.
|
|
|
|
view callbacks
|
|
- Added more documentation
- Replaced global named logger for plugins, fixing the issue when the CU has multiple (e.g. statically linked demo)
- Simplified some misc code
This is a breaking change, but I believe there is no better time to make it, we cannot continue to use the `log` crate, it is too limited for our needs.
|
|
|
|
separator
|
|
|
|
|
|
|
|
|
|
|
|
- Added some unit tests
- Improved documentation
- Made `QualifiedName::new` and `QualifiedName::new_with_separator` generic over `Into<String>`
|
|
This is used in more places than types, so its best we keep it separate.
|
|
|
|
This helps with documentation, giving a single module for those working with types to find related APIs
Also split out enumeration and structure APIs into their own file, since they have their own backing data separate from `Type`.
|
|
function symbol is not warranted
|
|
|
|
|
|
|
|
Prevents type confusions considering there are at times, three different id types being referred to in the type archive API
|
|
|
|
`BinaryViewExt::analysis_progress`
|
|
These have been deprecated since last stable, so they will be removed now.
I do believe that there might be some external users of this api so we should alert them.
|
|
|
|
|
|
- `analysis_info` was causing a double free with function refs, also did not need to be wrapped in a Result
|
|
|
|
There was a TODO there to document that it is optional, we should just wrap it in `Option` instead.
|
|
|
|
|
|
|
|
|
|
To keep backwards compatibility for commonly referenced code we re-export them within the architecture module.
Also does some light refactoring of some newly added APIs to keep them more consistent with other parts of the codebase.
|
|
Use `Location` instead, `arch` can be a nullptr
|
|
Forgot to call `BNFreeArchitectureAndAddressList`, also use `Location` instead of `ArchAndAddr`.
|
|
|
|
without an owner
You cannot generate SSA form of an LLIL function without a backing function unfortunately.
|
|
These are introduced after changing to Rust 1.91.1
|
|
Co-authored-by: Mason Reed <mason@vector35.com>
|
|
|
|
|
|
|
|
|