resolved conflicts for merge of 99eefc24 to jb-mr1-dev-plus-aosp
Change-Id: If88e8260e8186b9825e5d1011213e06020d74277
diff --git a/tests/tests/permission/src/android/permission/cts/FileSystemPermissionTest.java b/tests/tests/permission/src/android/permission/cts/FileSystemPermissionTest.java
index 84e9e66..c5c158f 100644
--- a/tests/tests/permission/src/android/permission/cts/FileSystemPermissionTest.java
+++ b/tests/tests/permission/src/android/permission/cts/FileSystemPermissionTest.java
@@ -564,10 +564,7 @@
}
}
- // This set contains all exceptions for writable sysfs, if it is a
- // directory, all files below that directory are included, so be
- // careful.
- private static final Set<String> SYS_EXCEPTIONS = new HashSet<String>(
+ private static final Set<File> SYS_EXCEPTIONS = new HashSet<File>(
Arrays.asList(
new File("/sys/kernel/debug/tracing/trace_marker"),
new File("/sys/fs/selinux/member"),
@@ -578,40 +575,10 @@
new File("/sys/fs/selinux/context")
));
- private static Set <File> getIgnorablesFromPaths(Set <String> paths) {
-
- Set <File> ignorable = new HashSet <File> ();
-
- for(String ignore : paths) {
- File tmp = new File(ignore);
-
- File[] files = null;
- if(tmp.isDirectory()) {
- files = tmp.listFiles(new FileFilter() {
- @Override public boolean accept(File pathname) {
- return pathname.isFile();
- }
- });
- }
- else if(tmp.isFile()){
- files = new File[1];
- files [0] = tmp;
- }
- else {
- // Should this be an Exception?
- continue;
- }
- ignorable.addAll(Arrays.asList(files));
- }
- return ignorable;
- }
-
@LargeTest
public void testAllFilesInSysAreNotWritable() throws Exception {
Set<File> writable = getAllWritableFilesInDirAndSubDir(new File("/sys"));
- Set<File> ignorables = getIgnorablesFromPaths(SYS_EXCEPTIONS);
-
- writable.removeAll(ignorables);
+ writable.removeAll(SYS_EXCEPTIONS);
assertTrue("Found writable: " + writable.toString(),
writable.isEmpty());
}