Skip to content

Commit 01b20b3

Browse files
committed
Added external storage test case
1 parent c448481 commit 01b20b3

File tree

2 files changed

+18
-8
lines changed

2 files changed

+18
-8
lines changed
Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
11
edges
22
nodes
3-
| APKInstallation.java:13:31:13:58 | fromFile(...) | semmle.label | fromFile(...) |
4-
| APKInstallation.java:20:31:20:44 | parse(...) | semmle.label | parse(...) |
5-
| APKInstallation.java:28:24:28:38 | parse(...) | semmle.label | parse(...) |
6-
| APKInstallation.java:35:24:35:51 | fromFile(...) | semmle.label | fromFile(...) |
3+
| APKInstallation.java:14:31:14:58 | fromFile(...) | semmle.label | fromFile(...) |
4+
| APKInstallation.java:21:31:21:44 | parse(...) | semmle.label | parse(...) |
5+
| APKInstallation.java:29:24:29:38 | parse(...) | semmle.label | parse(...) |
6+
| APKInstallation.java:36:24:36:51 | fromFile(...) | semmle.label | fromFile(...) |
7+
| APKInstallation.java:43:31:43:48 | fromFile(...) | semmle.label | fromFile(...) |
78
subpaths
89
#select
9-
| APKInstallation.java:13:31:13:58 | fromFile(...) | APKInstallation.java:13:31:13:58 | fromFile(...) | APKInstallation.java:13:31:13:58 | fromFile(...) | Arbitrary Android APK installation. |
10-
| APKInstallation.java:20:31:20:44 | parse(...) | APKInstallation.java:20:31:20:44 | parse(...) | APKInstallation.java:20:31:20:44 | parse(...) | Arbitrary Android APK installation. |
11-
| APKInstallation.java:28:24:28:38 | parse(...) | APKInstallation.java:28:24:28:38 | parse(...) | APKInstallation.java:28:24:28:38 | parse(...) | Arbitrary Android APK installation. |
12-
| APKInstallation.java:35:24:35:51 | fromFile(...) | APKInstallation.java:35:24:35:51 | fromFile(...) | APKInstallation.java:35:24:35:51 | fromFile(...) | Arbitrary Android APK installation. |
10+
| APKInstallation.java:14:31:14:58 | fromFile(...) | APKInstallation.java:14:31:14:58 | fromFile(...) | APKInstallation.java:14:31:14:58 | fromFile(...) | Arbitrary Android APK installation. |
11+
| APKInstallation.java:21:31:21:44 | parse(...) | APKInstallation.java:21:31:21:44 | parse(...) | APKInstallation.java:21:31:21:44 | parse(...) | Arbitrary Android APK installation. |
12+
| APKInstallation.java:29:24:29:38 | parse(...) | APKInstallation.java:29:24:29:38 | parse(...) | APKInstallation.java:29:24:29:38 | parse(...) | Arbitrary Android APK installation. |
13+
| APKInstallation.java:36:24:36:51 | fromFile(...) | APKInstallation.java:36:24:36:51 | fromFile(...) | APKInstallation.java:36:24:36:51 | fromFile(...) | Arbitrary Android APK installation. |
14+
| APKInstallation.java:43:31:43:48 | fromFile(...) | APKInstallation.java:43:31:43:48 | fromFile(...) | APKInstallation.java:43:31:43:48 | fromFile(...) | Arbitrary Android APK installation. |

java/ql/test/query-tests/security/CWE-094/APKInstallation.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import android.app.Activity;
22
import android.content.Intent;
33
import android.net.Uri;
4+
import android.os.Environment;
45

56
import java.io.File;
67

@@ -35,4 +36,11 @@ public void installAPK3(String path) {
3536
intent.setData(Uri.fromFile(new File(path)));
3637
startActivity(intent);
3738
}
39+
40+
public void installAPK4(String path) {
41+
File file = new File(Environment.getExternalStorageDirectory(), path);
42+
Intent intent = new Intent(Intent.ACTION_VIEW);
43+
intent.setDataAndType(Uri.fromFile(file), APK_MIMETYPE);
44+
startActivity(intent);
45+
}
3846
}

0 commit comments

Comments
 (0)