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

(re)Add support for ast only file #187

Open
montyly opened this issue Jun 16, 2021 · 2 comments
Open

(re)Add support for ast only file #187

montyly opened this issue Jun 16, 2021 · 2 comments

Comments

@montyly
Copy link
Member

montyly commented Jun 16, 2021

With crytic/slither#823, we removed the support for AST-only file in Slither.

We could re-add this support by adding a ast-only platform in crytic-compile.

Ideally the contract is compiled with abi,ast,bin,bin-runtime,srcmap,srcmap-runtime,userdoc,devdoc,hashes,compact-format, but we could allow a best-effort approach here.

The main difficulty that I can see here is that Slither now relies more on the source code being accessible, so we might need to disable some analyses in Slither.

Related crytic/slither#864

@yann300
Copy link

yann300 commented Mar 17, 2022

@montyly is this still something planned?

@joaquinlpereyra
Copy link

Voicing support for this to be re-enabled, as it allows for some quick-and-dirty analysis of some interesting properties. A very good usecase: if a bug is found on a widely used library, a quick detector could be written on slither to look for the vulnerability in contracts in smart-contract-sanctuary.

These contracts do not compile, but their AST can be generated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants