-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
13 changed files
with
150 additions
and
474 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,91 @@ | ||
--- | ||
slug: / | ||
--- | ||
|
||
|
||
# CyScout | ||
|
||
## Run queries and detect vulnerabilities in your smart contracts using CodeQL-Solidity | ||
|
||
![alt text](docs/static/img/output.gif)) | ||
|
||
This repository contains CoinFabrik's ongoing research and development to extend CodeQL support to the Solidity smart contract language. By leveraging the foundational work done by the CodeQL team for Ruby, we have adapted and expanded their approach to create a powerful toolset for analyzing Solidity code. | ||
|
||
## 🔍 Overview | ||
|
||
Our goal is to provide a comprehensive set of tools for querying and detecting vulnerabilities in Solidity smart contracts. We build upon the work of [Joran Honig's Solidity Tree-sitter grammar](https://github.com/JoranHonig/tree-sitter-solidity) and the CodeQL team's [Ruby implementation](https://github.blog/security/web-application-security/code-scanning-and-ruby-turning-source-code-into-a-queryable-database/). The project includes an extractor, database schema generation, and abstractions such as a cleaner Abstract Syntax Tree (AST), Control Flow Graph (CFG), and Dataflow analysis. These elements enable complex vulnerability detection and querying, similar to the C++ libraries in CodeQL. | ||
|
||
## 🚀 Project Status | ||
|
||
- **Extractor and Database Schema**: Usable and functional for Solidity codebases. | ||
- **Current Work**: We are actively developing a cleaner AST, CFG, and Dataflow support to enhance the detection of vulnerabilities. | ||
- **Proof of Concept (PoC)**: Nine simple detector examples are provided to demonstrate basic usage and potential. | ||
|
||
## 🏁 Getting Started | ||
|
||
To get started with analyzing Solidity smart contracts using CodeQL, follow these steps: | ||
|
||
### Setting Up CodeQL Solidity | ||
|
||
Work in progress | ||
|
||
### Install CodeQL Solidity Extension | ||
|
||
|
||
### Run Sample Detectors | ||
|
||
Once the database is created, you can run sample detectors written in the QL language: | ||
|
||
```bash | ||
codeql query run /path-to-detector/ -d /path-to-created-database/ | ||
``` | ||
|
||
For instance: | ||
|
||
```shell | ||
[1/1] Found in cache: /home/user/codeql/solidity/ql/lib/detector3.ql. | ||
detector3.ql: Evaluation completed (191ms). | ||
| col0 | col1 | | ||
+-----------------+------------------------------------------------------------------------------+ | ||
| YulFunctionCall | /home/user/codeql/solidity-test/bitshift-order-test/vulnerable.sol@4:18:4:26 | | ||
Shutting down query evaluator. | ||
``` | ||
|
||
## Detectors | ||
|
||
| Num | Detector | What it Detects | | ||
| --- | ----------- | -------------------------------------------------------------------- | | ||
| 1 | `transfer-from` | transferFrom uses arbitrary `from` | | ||
| 2 | `incorrect-shift` | incorrect order of arguments in bit shift operations | | ||
| 3 | `msg-valu-in-for-loop` | Detects the use of msg.value inside a loop | | ||
| 4 | `bad-prng` | Detects bad randomness | | ||
| 5 | `divide-before-multiply` | Detects loss of precision | | ||
| 6 | `incorrect-exp` | Detects use of bitwise xor instead of exponential | | ||
| 7 | `unchecked-send` | The return value of a send is not checked | | ||
| 8 | `is-unreachable` | Detects dead code | | ||
| 9 | `unprotected-self-destruct` | Detects unprotected call to selfdestruct/suicidal | | ||
|
||
|
||
### Further Documentation | ||
|
||
## Work In Progress | ||
|
||
We are actively working on enhancing the capabilities of our Solidity CodeQL implementation. Our current focus and status is on: | ||
|
||
- AST: usable | ||
- CFG: experimental | ||
- DFA: soon | ||
|
||
For more detailed instructions on using CodeQL, refer to the [official CodeQL documentation](https://codeql.github.com/docs/). | ||
|
||
## 🤝 Contributing | ||
|
||
We welcome contributions to enhance and expand the support for Solidity in CodeQL. Feel free to submit issues, feature requests, or pull requests. | ||
|
||
For more information, please refer to the [Contribution Guidelines](/docs/Contributing.md). | ||
|
||
## License | ||
|
||
The code in this repository is licensed under the [MIT License](LICENSE.md) by CoinFabrik. | ||
|
||
For further information on CodeQL and CodeQL CLI licensing, please refer to the official [repo](https://github.com/github/codeql-cli-binaries). |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,29 @@ | ||
# Contribute | ||
|
||
Thank you for your interest in contributing to the development of new detectors. | ||
|
||
### Getting Started | ||
|
||
Create a new issue on our [repository](https://github.com/CoinFabrik/CyScout) with the name of the new detector or test case you wish to contribute. Then, link a new branch to that issue. | ||
|
||
### Detectors | ||
|
||
In order to build a custom detector, you need to first import all static analysis primitives you need to use. | ||
|
||
- `import codeql.solidity.ast.AST` for AST language constructs (binary, unary and tertiary operations, control statements, loops, functions, contracts, etc.) | ||
|
||
- `import codeql.solidity.controlflow.ControlFlowGraph` for CFG constructs (SSA, basic blocks, control flow nodes, etc.) | ||
|
||
- `import codeql.solidity.dataflow.DataFlow` for DFA constructs (to define sinks and sources for local flow analysis) | ||
|
||
For global data flow analysis, you must write a class that implements DataFlow::ConfigSig. Refer to our global flow example or to the official codeQL guide for more information on this specific case. | ||
|
||
### Test Cases | ||
|
||
When you create a new detector, please also add a new test case. To add a new one: | ||
|
||
1. Create a new folder in the [`tests`](https://github.com/CoinFabrik/stacy/tree/main/tests) directory. **Remember to follow the snake_case/underscore naming convention for the folder name**. | ||
|
||
2. Within this folder, create two directories: `vulnerable-example` and `remediated-example`. Fill each with the relevant files for their respective test cases. | ||
|
||
For guidance, refer to the `detector_name` template in [`templates/test-case`](https://github.com/CoinFabrik/stacy/tree/main/templates/tests). |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
{ | ||
"name": "codeql-research", | ||
"name": "cyscout", | ||
"version": "0.0.0", | ||
"private": true, | ||
"scripts": { | ||
|
Binary file not shown.
Binary file not shown.
Binary file not shown.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.