Cleanup of CheckSQL
 - CVE checking is now a simple hash
 - Removed implicit returns
 - Removed duplicate code from find_scope_calls
 - Removed assignments from if statements
 - Bracketed ambiguos statements
 - Onelined simple if statements
Add back controller report for -m and -d
Add tests for -m option in reports
Code cleanup
 - Removed brackets where possible.
 - No more assignments in if-statements
 - Refactored out unsafe argument array checking
Merge pull request #357 from presidentbeef/report_controllers_again_oops

Report controllers again for -m and -d
Merge pull request #356 from barttenbrinke/check_sql_refactor

Cleanup of CheckSQL because of the F rating in code climate
Update CHANGES
Don't use auto-vivifying hashes in CallIndex

because it creates unnecessary values when searching
the index for keys that don't exist yet
Add tests for CallIndex
Merge pull request #358 from presidentbeef/no_default_hash_values_in_call_index

No default hash values in CallIndex
add check for dangerous attributes defined in attr_accessible
Merge branch 'master' of https://github.com/presidentbeef/brakeman into check_model_attr_accessible
update vulnerability type and add tests for rails32
fix attr_accessible tests assert_warning butrescanner still failing
Move NoApplication exception to Brakeman::

because it is masking some errors when the scanner has
not been loaded yet and bin/brakeman tries to rescue an
exception that has not been defined
Track calls with blocks in CallIndex
Track calls with blocks in CallIndex
Add check for authenticate_or_request_with_http_basic

and a password literal
Output JSON diff to file if -o option is used

and also output regular reports for other -o options
Clone mixin methods before processing

in case they are mixed into multiple controllers