Skip to content

Commit 43f54db

Browse files
authored
Merge pull request #17274 from asgerf/java/implicit-pending-intents-implicit-read
Java: Reveal false negative in test
2 parents d41d7c8 + 3aa32e4 commit 43f54db

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
failures
21
testFailures
2+
failures

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

+1-1
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ public static void testPendingIntentAsAnExtra(Context ctx)
3232
PendingIntent pi = PendingIntent.getActivity(ctx, 0, baseIntent, 0);
3333
Intent fwdIntent = new Intent();
3434
fwdIntent.putExtra("fwdIntent", pi);
35+
ctx.startActivities(new Intent[] {fwdIntent}); // $ MISSING: hasImplicitPendingIntent
3536
ctx.startActivity(fwdIntent); // $hasImplicitPendingIntent
36-
ctx.startActivities(new Intent[] {fwdIntent}); // $hasImplicitPendingIntent
3737
ctx.startService(fwdIntent); // Safe
3838
ctx.sendBroadcast(fwdIntent); // $hasImplicitPendingIntent
3939

0 commit comments

Comments
 (0)