Skip to content

Commit de2f323

Browse files
committed
JS: Mark unused parameter nodes as incomplete
1 parent 69dbbcf commit de2f323

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

javascript/ql/src/semmle/javascript/dataflow/DataFlow.qll

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1280,6 +1280,8 @@ module DataFlow {
12801280
exists(PropertyPattern p | nd = TPropNode(p)) and cause = "heap"
12811281
or
12821282
nd instanceof TElementPatternNode and cause = "heap"
1283+
or
1284+
nd instanceof UnusedParameterNode and cause = "call"
12831285
}
12841286

12851287
/**

javascript/ql/test/library-tests/TaintTracking/BasicTaintTracking.expected

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@ typeInferenceMismatch
33
| addexpr.js:4:10:4:17 | source() | addexpr.js:6:3:6:14 | x |
44
| addexpr.js:11:15:11:22 | source() | addexpr.js:17:5:17:18 | value |
55
| addexpr.js:11:15:11:22 | source() | addexpr.js:19:3:19:14 | value |
6-
| partialCalls.js:4:17:4:24 | source() | partialCalls.js:6:23:6:23 | y |
7-
| partialCalls.js:4:17:4:24 | source() | partialCalls.js:9:20:9:20 | x |
86
#select
97
| access-path-sanitizer.js:2:18:2:25 | source() | access-path-sanitizer.js:4:8:4:12 | obj.x |
108
| addexpr.js:4:10:4:17 | source() | addexpr.js:7:8:7:8 | x |

0 commit comments

Comments
 (0)