hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2026-05-03 04:39:29 +02:00
Code Issues Packages Projects Releases Wiki Activity

autoformat, and apply naming suggestion

Browse Source
This commit is contained in:
Erik Krogh Kristensen
2020-05-01 18:35:34 +02:00
parent 87365357ba
commit 2a1095abcc
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
javascript/ql/src/semmle/javascript/frameworks/jQuery.qll
View File

@@ -37,19 +37,19 @@ private class OrdinaryJQueryObject extends JQueryObjectInternal {
OrdinaryJQueryObject() {
exists(JQuery::MethodCall jq |
this.flow().getALocalSource() = jq and
returnsAJqueryObject(jq, jq.getMethodName())
returnsAJQueryObject(jq, jq.getMethodName())
)
}
}
/**
* Holds if the jQuery method call `call`, with name `methodName`, returns a JQuery object.
*
* The `call` parameter has type `DataFlow::CallNode` instead of `JQuery::MethodCall` to avoid non-monotonic recursion.
* The not is placed inside the predicate to avoid non-monotonic recursion.
*
* The `call` parameter has type `DataFlow::CallNode` instead of `JQuery::MethodCall` to avoid non-monotonic recursion.
* The not is placed inside the predicate to avoid non-monotonic recursion.
*/
bindingset[methodName, call]
private predicate returnsAJqueryObject(DataFlow::CallNode call, string methodName) {
private predicate returnsAJQueryObject(DataFlow::CallNode call, string methodName) {
not (
methodName = "val" // `jQuery.val()`
or
@@ -70,7 +70,7 @@ private predicate returnsAJqueryObject(DataFlow::CallNode call, string methodNam
or
methodName = "trim" // $.trim()
or
// `$.ajax`, and related methods.
// `$.ajax`, and related methods.
// note: there are 2 different `get` methods, and none of them return a jQuery object.
methodName = ["ajax", "get", "getJSON", "getScript", "post", "load"]
)