Ecosyste.ms: Awesome
An open API service indexing awesome lists of open source software.
https://github.com/emanuil/php-reaper
PHP tool to scan ADOdb code for SQL Injections
https://github.com/emanuil/php-reaper
Last synced: 15 days ago
JSON representation
PHP tool to scan ADOdb code for SQL Injections
- Host: GitHub
- URL: https://github.com/emanuil/php-reaper
- Owner: emanuil
- Created: 2014-11-12T12:01:19.000Z (about 10 years ago)
- Default Branch: master
- Last Pushed: 2016-08-09T13:07:21.000Z (over 8 years ago)
- Last Synced: 2024-08-02T00:25:58.327Z (4 months ago)
- Language: PHP
- Size: 286 KB
- Stars: 32
- Watchers: 2
- Forks: 11
- Open Issues: 2
-
Metadata Files:
- Readme: README.md
- Security: SecurityChecks.php
Awesome Lists containing this project
README
[![Build Status](https://travis-ci.org/emanuil/php-reaper.svg?branch=master)](https://travis-ci.org/emanuil/php-reaper)
PHP-Reaper
==========
PHP tool to scan [ADOdb](http://adodb.sourceforge.net) code for SQL InjectionsWhy
===
The main idea is to be able to detect problems as early as possible, when the code is fresh in your mind. Shift as much checks as possible to the left. Automate as much as possible.Running PHP-Reaper is far less time consuming than running full fledged automated security scanner at your application. The web security scanner might not locate all possible SQL Injections vulnerabilities, because of hard to reach code from the UI (or needs to set rare conditions). PHP-Reaper is fast and pinpoints the exact line where the problem lies, scanning all your PHP [ADOdb](http://adodb.sourceforge.net) source code.
You'll get the most out of PHP-Reaper if you run it on every commit. It's made to be CI friendly and fast.
Examples
========Because of laziness, pressure or just ignorance, php developers using ADOdb are making such mistakes.
Vulnerable SQL query #1:
```php
$dbConn->GetRow("SELECT * FROM users WHERE id = $user_id");
```Correct SQL query #1:
```php
$dbConn->GetRow("SELECT * FROM users WHERE id = ?", array(‘$user_id’));
```Vulnerable SQL Query #2:
```php
$ids = join(',', $ids);
$dbConn->GetAll("SELECT * FROM campaigns WHERE id IN ({$ids})");
```Correct SQL query #2:
```php
$dbConn->GetAll('SELECT * FROM campaigns WHERE FIND_IN_SET (id, ' . $dbConn->Param('') . ')', array(join(',', $ids)));
```Usage
=====
Recursively scan directory with php files:```bash
php php-reaper -d directory_with_php_files
```or scan a single file:
```bash
php php-reaper -f single_file.php
```Tests
=====
The tests are located in [tests](https://github.com/emanuil/php-reaper/tree/master/tests) directory. To run them, once in tests directory, type:
```bash
phpunit .
```
If you extend this tool, make sure that the tests are passing before submitting pull request. Better yet, add new test files and unit tests. Look at [example files](https://github.com/emanuil/php-reaper/tree/master/tests/SecurityChecks/exampleFiles) directory, what types of SQL Injections are detected.Continuous Integration
======================
PHP-Reaper is CI friendly. On error it will exit with -1 status, so it's easy to hook it to your CI jobs.Exclude from warnings
======================
You can ignore the warnings by PHP-Reaper, if you're absolutely sure that the code does not contain SQL Injection. Comment the line above the ADOdb function with:
```php
// safesql
$result_set = $dbConn->getAll('SELECT * FROM ' . $this->usersTable);
```
You need to be absolutely sure `$this->usersTable` variable cannot be controller by an attacker.Dangerous ADOdb Methods
=======================
The following [ADOdb](http://adodb.sourceforge.net) methods are considered dangerous and are scanned for potential SQL
injections: getone(), getrow(), getall(), getcol(), getassoc(), execute(), replace(). Note that autoexecute() is immune,
because it automatically escapes all the parameters. If you have methods in your code base with the same names e.g.
execute() - non ADOdb method, you may see false positives. The solution is to rename your methods to be with names
different than the default ADODb methods - e.g. executeTask(). PHP-Reaper is written in such a way because PHP is pretty
dynamic and static analysis cannot reliably tell us the class of the instantiated object.PHP Parser
==========
PHP-Reaper is using the excellent php parser with the same name: [PHP-Parser](https://github.com/nikic/PHP-Parser). It currently uses version 1.4.1.