Skip to content

Commit 6699211

Browse files
committed
Work around problem with comments in heredocs
1 parent 5e606b7 commit 6699211

2 files changed

Lines changed: 195 additions & 192 deletions

File tree

ruby/ql/test/query-tests/security/cwe-089/ActiveRecordInjection.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,9 @@ def some_request_handler
6565

6666
# BAD: executes `SELECT "users".* FROM "users" WHERE id BETWEEN '#{params[:min_id]}' AND 100000`
6767
# where `params[:min_id]` is unsanitized
68+
min_id = params[:min_id] # $ Source
6869
User.where(<<-SQL, MAX_USER_ID) # $ Alert
69-
id BETWEEN '#{params[:min_id]}' AND ? # $ Source
70+
id BETWEEN '#{min_id}' AND ?
7071
SQL
7172

7273
# BAD: chained method case

0 commit comments

Comments
 (0)