Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LibAFL Dynamic Analysis #2208

Merged
merged 25 commits into from
May 24, 2024
Merged

LibAFL Dynamic Analysis #2208

merged 25 commits into from
May 24, 2024

Conversation

tokatoka
Copy link
Member

@tokatoka tokatoka commented May 17, 2024

follow up for #2178
the plan is

  • this pass logs all the hashed function signature.
  • and writes the hash into a global, this list should contain all the hashes of the functions that executions went through
  • an observer (built upon ListObserver) should observe the list
  • then do the lookup to find the static analysis info

@tokatoka tokatoka marked this pull request as draft May 17, 2024 15:10
@tokatoka tokatoka changed the title Function Logging Pass + Function Observer LibAFL Dynamic Analysis May 23, 2024
@tokatoka tokatoka marked this pull request as ready for review May 23, 2024 13:09
@tokatoka tokatoka requested a review from domenukk May 23, 2024 13:09
@tokatoka
Copy link
Member Author

@domenukk
done!


[dependencies]
env_logger = "0.10"
once_cell = "1.19"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

once_cell is in the standard library now, no need to use the trait anymore

Copy link
Member Author

@tokatoka tokatoka May 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the one i want to use, LazyCell is nightly only

https://doc.rust-lang.org/std/cell/struct.LazyCell.html

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then use OnceCell?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might look a tiny bit uglier but it's literally the same functionality

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extra line is worth it for one full dependency less, IMHO

libafl_cc/build.rs Outdated Show resolved Hide resolved
@tokatoka tokatoka merged commit e6eb6c4 into main May 24, 2024
99 checks passed
@tokatoka tokatoka deleted the logger_pass branch May 24, 2024 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants