Skip to content

Commit 50b10be

Browse files
committed
Ruby: StackTraceExposure: add test for a specific rescue type
1 parent 1c407a2 commit 50b10be

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

ruby/ql/test/query-tests/security/cwe-209/StackTraceExposure.expected

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@ nodes
44
| StackTraceExposure.rb:6:18:6:28 | call to backtrace | semmle.label | call to backtrace |
55
| StackTraceExposure.rb:11:10:11:17 | call to caller : | semmle.label | call to caller : |
66
| StackTraceExposure.rb:12:18:12:19 | bt | semmle.label | bt |
7+
| StackTraceExposure.rb:18:18:18:28 | call to backtrace | semmle.label | call to backtrace |
78
subpaths
89
#select
910
| StackTraceExposure.rb:6:18:6:28 | call to backtrace | StackTraceExposure.rb:6:18:6:28 | call to backtrace | StackTraceExposure.rb:6:18:6:28 | call to backtrace | $@ can be exposed to an external user. | StackTraceExposure.rb:6:18:6:28 | call to backtrace | Error information |
1011
| StackTraceExposure.rb:12:18:12:19 | bt | StackTraceExposure.rb:11:10:11:17 | call to caller : | StackTraceExposure.rb:12:18:12:19 | bt | $@ can be exposed to an external user. | StackTraceExposure.rb:11:10:11:17 | call to caller | Error information |
12+
| StackTraceExposure.rb:18:18:18:28 | call to backtrace | StackTraceExposure.rb:18:18:18:28 | call to backtrace | StackTraceExposure.rb:18:18:18:28 | call to backtrace | $@ can be exposed to an external user. | StackTraceExposure.rb:18:18:18:28 | call to backtrace | Error information |

ruby/ql/test/query-tests/security/cwe-209/StackTraceExposure.rb

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,10 @@ def show2
1212
render body: bt, content_type: "text/plain"
1313
end
1414

15+
def show3
16+
not_a_method()
17+
rescue NoMethodError => e
18+
render body: e.backtrace, content_type: "text/plain"
19+
end
20+
1521
end

0 commit comments

Comments
 (0)