Skip to content

Commit f4691b1

Browse files
committed
Changed to more-modern Dataflow libraries
1 parent 81ef255 commit f4691b1

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

javascript/ql/src/Security/CWE-693/InsecureHelmet.ql

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,16 +13,18 @@
1313

1414
import semmle.javascript.frameworks.ExpressModules
1515

16-
class HelmetProperty extends Property {
16+
class HelmetProperty extends DataFlow::Node instanceof DataFlow::PropWrite {
1717
ExpressLibraries::HelmetRouteHandler helmet;
1818

1919
HelmetProperty() {
20-
helmet.(DataFlow::CallNode).getAnArgument().asExpr().(ObjectExpr).getAProperty() = this
20+
this = helmet.(DataFlow::CallNode).getAnArgument().getALocalSource().getAPropertyWrite()
2121
}
2222

2323
ExpressLibraries::HelmetRouteHandler getHelmet() { result = helmet }
2424

25-
predicate isFalse() { this.getInit().(BooleanLiteral).getBoolValue() = false }
25+
predicate isFalse() { DataFlow::PropWrite.super.getRhs().mayHaveBooleanValue(true) }
26+
27+
string getName() { result = DataFlow::PropWrite.super.getPropertyName() }
2628

2729
predicate isImportantSecuritySetting() {
2830
this.getName() in ["frameguard", "contentSecurityPolicy"]

0 commit comments

Comments
 (0)