commit 2d273d8e138834b5393c8c98ded516d8d787af1c
author Ricky Wai <rickywai@google.com> Tue Jun 16 12:10:15 2020 +0100
committer Ricky Wai <rickywai@google.com> Tue Jun 16 16:41:35 2020 +0000
tree 10040e4d95ca4ee067ef07a6edbcaaef04d7138d
parent 46dddd5057fa1b42a7f6680bc624b7e99e26bfd3

App data is not accessible as SELinux policy blocks it

SElinux change: aosp/1324511

Bug: 159050834
Bug: 158088415
Test: atest AppDataIsolationTests
Change-Id: Ib2f21a393fbd2e70d4536bb15131e858c6a2e420

hostsidetests/appsecurity/test-apps/AppDataIsolationTestApp/AppA/src/com/android/cts/appdataisolation/appa/IsolatedService.java

diff --git a/hostsidetests/appsecurity/test-apps/AppDataIsolationTestApp/AppA/src/com/android/cts/appdataisolation/appa/IsolatedService.java b/hostsidetests/appsecurity/test-apps/AppDataIsolationTestApp/AppA/src/com/android/cts/appdataisolation/appa/IsolatedService.java
index 5b1ec66..b2dd2df 100644
--- a/hostsidetests/appsecurity/test-apps/AppDataIsolationTestApp/AppA/src/com/android/cts/appdataisolation/appa/IsolatedService.java
+++ b/hostsidetests/appsecurity/test-apps/AppDataIsolationTestApp/AppA/src/com/android/cts/appdataisolation/appa/IsolatedService.java
@@ -16,7 +16,6 @@
 
 package com.android.cts.appdataisolation.appa;
 
-import static com.android.cts.appdataisolation.common.FileUtils.assertDirDoesNotExist;
 import static com.android.cts.appdataisolation.common.FileUtils.assertDirIsNotAccessible;
 
 import android.app.Service;
@@ -35,25 +34,25 @@
         public void assertDataIsolated() throws RemoteException {
             try {
                 ApplicationInfo applicationInfo = getApplicationInfo();
-                assertDirDoesNotExist(applicationInfo.dataDir);
-                assertDirDoesNotExist(applicationInfo.deviceProtectedDataDir);
-                assertDirDoesNotExist("/data/data/" + getPackageName());
-                assertDirDoesNotExist("/data/misc/profiles/cur/0/" + getPackageName());
+                assertDirIsNotAccessible(applicationInfo.dataDir);
+                assertDirIsNotAccessible(applicationInfo.deviceProtectedDataDir);
+                assertDirIsNotAccessible("/data/data/" + getPackageName());
+                assertDirIsNotAccessible("/data/misc/profiles/cur/0/" + getPackageName());
                 assertDirIsNotAccessible("/data/misc/profiles/ref");
 
-                assertDirDoesNotExist(FileUtils.replacePackageAWithPackageB(
+                assertDirIsNotAccessible(FileUtils.replacePackageAWithPackageB(
                         applicationInfo.dataDir));
-                assertDirDoesNotExist(FileUtils.replacePackageAWithPackageB(
+                assertDirIsNotAccessible(FileUtils.replacePackageAWithPackageB(
                         applicationInfo.deviceProtectedDataDir));
-                assertDirDoesNotExist("/data/data/" + FileUtils.APPB_PKG);
-                assertDirDoesNotExist("/data/misc/profiles/cur/0/" + FileUtils.APPB_PKG);
+                assertDirIsNotAccessible("/data/data/" + FileUtils.APPB_PKG);
+                assertDirIsNotAccessible("/data/misc/profiles/cur/0/" + FileUtils.APPB_PKG);
 
-                assertDirDoesNotExist(FileUtils.replacePackageAWithNotInstalledPkg(
+                assertDirIsNotAccessible(FileUtils.replacePackageAWithNotInstalledPkg(
                         applicationInfo.dataDir));
-                assertDirDoesNotExist(FileUtils.replacePackageAWithNotInstalledPkg(
+                assertDirIsNotAccessible(FileUtils.replacePackageAWithNotInstalledPkg(
                         applicationInfo.deviceProtectedDataDir));
-                assertDirDoesNotExist("/data/data/" + FileUtils.NOT_INSTALLED_PKG);
-                assertDirDoesNotExist("/data/misc/profiles/cur/0/" + FileUtils.NOT_INSTALLED_PKG);
+                assertDirIsNotAccessible("/data/data/" + FileUtils.NOT_INSTALLED_PKG);
+                assertDirIsNotAccessible("/data/misc/profiles/cur/0/" + FileUtils.NOT_INSTALLED_PKG);
             } catch (Throwable e) {
                 throw new IllegalStateException(e.getMessage());
             }