falconre / falcon

Binary Analysis Framework in Rust
Apache License 2.0
551 stars 45 forks source link

Graph: Remove head field #69

Closed emmanuel099 closed 4 years ago

emmanuel099 commented 4 years ago

None of the functions in Graph really makes use of the head field.

Additionally, the head field may easily become out of sync with the CFG entry resulting in subtile bugs, such as removing the wrong unreachable vertices. This can be avoided by removing the head field altogether and instead require it as function argument.