Skip to content

Commit a8f93ca

Browse files
committed
JS: Remove obsolete comment
The test case actually has the correct result now
1 parent dd37c47 commit a8f93ca

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

javascript/ql/lib/semmle/javascript/security/dataflow/DomBasedXssQuery.qll

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,6 @@ module DomBasedXssConfig implements DataFlow::StateConfigSig {
105105
state1.isTaintedUrlSuffix() and
106106
state2.isTaintedPrefix()
107107
or
108-
// FIXME: this fails to work in the test case at jquery.js:37
109108
exists(DataFlow::FunctionNode callback, DataFlow::Node arg |
110109
any(JQuery::MethodCall c).interpretsArgumentAsHtml(arg) and
111110
callback = arg.getABoundFunctionValue(_) and

0 commit comments

Comments
 (0)