mirror of
https://github.com/github/codeql.git
synced 2026-05-03 04:39:29 +02:00
JS: Fix lurking cartesian product in Express
This commit is contained in:
@@ -415,9 +415,6 @@ module Express {
|
||||
)
|
||||
)
|
||||
or
|
||||
kind = "body" and
|
||||
this.asExpr() = rh.getARequestBodyAccess()
|
||||
or
|
||||
exists(string propName |
|
||||
// `req.url` or `req.originalUrl`
|
||||
kind = "url" and
|
||||
@@ -432,6 +429,9 @@ module Express {
|
||||
this.(DataFlow::PropRef).accesses(request, "cookies")
|
||||
)
|
||||
or
|
||||
kind = "body" and
|
||||
this.asExpr() = rh.getARequestBodyAccess()
|
||||
or
|
||||
exists(RequestHeaderAccess access | this = access |
|
||||
rh = access.getRouteHandler() and
|
||||
kind = "header"
|
||||
|
||||
Reference in New Issue
Block a user