Skip to content

Commit 4012866

Browse files
Allow arbitrary read steps at the sink
1 parent bae0da8 commit 4012866

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

java/ql/lib/semmle/code/java/dataflow/ExternalFlow.qll

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ private module Frameworks {
8484
private import semmle.code.java.frameworks.apache.Collections
8585
private import semmle.code.java.frameworks.apache.Lang
8686
private import semmle.code.java.frameworks.Flexjson
87+
private import semmle.code.java.frameworks.android.Intent
8788
private import semmle.code.java.frameworks.guava.Guava
8889
private import semmle.code.java.frameworks.jackson.JacksonSerializability
8990
private import semmle.code.java.frameworks.javaee.jsf.JSFRenderer

java/ql/src/semmle/code/java/security/AndroidSensitiveBroadcastQuery.qll

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,4 +119,10 @@ class SensitiveBroadcastConfig extends TaintTracking::Configuration {
119119
setReceiverMa.getQualifier().(VarAccess).getVariable().getAnAccess() = node.asExpr()
120120
)
121121
}
122+
123+
override predicate allowImplicitRead(DataFlow::Node node, DataFlow::Content c) {
124+
super.allowImplicitRead(node, c)
125+
or
126+
this.isSink(node)
127+
}
122128
}

0 commit comments

Comments
 (0)