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

Add slither integration #530

Merged
merged 20 commits into from
Jan 8, 2025
Merged

Add slither integration #530

merged 20 commits into from
Jan 8, 2025

Conversation

anishnaik
Copy link
Collaborator

@anishnaik anishnaik commented Jan 6, 2025

Closes #504
Closes #503
Closes #52

This PR integrates Slither into medusa. In addition to the basic integration where we capture the constants found by Slither, we also implement caching of the Slither results. Thus, when you run medusa the first time on a codebase, the results from slither are cached. These results are then re-used in subsequent fuzzing campaigns.

We do not run slither again for two reasons:

  1. Re-running Slither on large projects every single time medusa starts up can save up to many minutes. Using the cache makes the process relatively instantaneous.
  2. We are gambling that, if you are fuzzing the codebase, that the underlying system is no longer changing. Thus, the constants remain...constant...

@anishnaik anishnaik requested a review from Xenomega as a code owner January 6, 2025 21:23
@anishnaik anishnaik changed the title WIP: Add slither integration Add slither integration Jan 8, 2025
@anishnaik anishnaik merged commit 0112ddc into master Jan 8, 2025
12 checks passed
@anishnaik anishnaik deleted the feat/slither branch January 8, 2025 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants