Skip to content

Commit d7c7776

Browse files
Add additional models; fix up tests
1 parent ae461bc commit d7c7776

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

java/ql/test/query-tests/security/CWE-927/SensitiveBroadcast.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ public void sendBroadcast6(Context context) {
7272
intent.setAction("com.example.custom_action");
7373
intent.putExtra("ticket", ticket);
7474
String perm = "com.example.user_permission";
75-
context.sendBroadcast(intent, perm); // $ hasTaintFlow
75+
context.sendBroadcast(intent, perm);
7676
}
7777

7878
// GOOD - Tests broadcast of sensitive user information to a specific application.
@@ -142,7 +142,6 @@ public void sendBroadcast11(Context context) {
142142

143143
/**
144144
* BAD - Tests broadcast of sensitive user information with multiple permissions using empty array initialization through two variables and `intent.getExtras().putString()`.
145-
* Note this case of `getExtras().putString(...)` is not yet detected thus is beyond what the query is capable of.
146145
*/
147146
public void sendBroadcast12(Context context) {
148147
String username = "test123";

0 commit comments

Comments
 (0)