Skip to content

Commit 5614631

Browse files
committed
JS: Fix mistranslated 'NOT OK' comment
1 parent 7b74be3 commit 5614631

File tree

1 file changed

+1
-1
lines changed
  • javascript/ql/test/query-tests/Security/CWE-400/RemovePropertyInjection

1 file changed

+1
-1
lines changed

javascript/ql/test/query-tests/Security/CWE-400/RemovePropertyInjection/tst.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ app.get('/user/:id', function(req, res) {
88
var prop = myCoolLocalFct(req.query.userControlled);
99
myObj[prop] = 23; // $ Alert
1010
myObj.prop = 23;
11-
var x = myObj[prop]; // $ Alert - but flagged by different query
11+
var x = myObj[prop]; // OK - flagged by different query
1212
x(23);
1313
delete myObj[prop]; // $ Alert
1414
Object.defineProperty(myObj, prop, {value: 24}); // $ Alert

0 commit comments

Comments
 (0)