aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Cherry <tomcherry@google.com>2017-08-04 21:25:57 +0000
committerandroid-build-merger <android-build-merger@google.com>2017-08-04 21:25:57 +0000
commitfe1dc33b28525a503da6c052acaa5894c0dae6e8 (patch)
treefa2a0f95e9959af3360c049fa15fc465de030224
parent9a1df36a07ddc6f143beb1c0a6c70337d2f9b9c6 (diff)
parentc2c36b661cf80f88305fc15907a6df16a0342862 (diff)
downloadbionic-fe1dc33b28525a503da6c052acaa5894c0dae6e8.tar.gz
Remove a check for AIDs in the OEM range.oreo-dr1-dev
am: c2c36b661c Change-Id: Ic99c543a3cf5eaad251fc07957262f144c5f7b97
-rw-r--r--tests/grp_pwd_test.cpp18
1 files changed, 16 insertions, 2 deletions
diff --git a/tests/grp_pwd_test.cpp b/tests/grp_pwd_test.cpp
index f8232aa73..b8df8e790 100644
--- a/tests/grp_pwd_test.cpp
+++ b/tests/grp_pwd_test.cpp
@@ -207,7 +207,14 @@ TEST(pwd, getpwent_iterate) {
application = true;
} else {
ASSERT_STREQ("/", pwd->pw_dir);
- ASSERT_FALSE(exist[pwd->pw_uid]);
+ // TODO(b/27999086): fix this check with the OEM range
+ // If OEMs add their own AIDs to private/android_filesystem_config.h, this check will fail.
+ // Long term we want to create a better solution for OEMs adding AIDs, but we're not there
+ // yet, so therefore we do not check for uid's in the OEM range.
+ if (!(pwd->pw_uid >= 2900 && pwd->pw_uid <= 2999) &&
+ !(pwd->pw_uid >= 5000 && pwd->pw_uid <= 5999)) {
+ ASSERT_FALSE(exist[pwd->pw_uid]);
+ }
exist[pwd->pw_uid] = true;
}
}
@@ -453,7 +460,14 @@ TEST(grp, getgrent_iterate) {
if (grp->gr_gid >= exist.size()) {
application = true;
} else {
- ASSERT_FALSE(exist[grp->gr_gid]);
+ // TODO(b/27999086): fix this check with the OEM range
+ // If OEMs add their own AIDs to private/android_filesystem_config.h, this check will fail.
+ // Long term we want to create a better solution for OEMs adding AIDs, but we're not there
+ // yet, so therefore we do not check for gid's in the OEM range.
+ if (!(grp->gr_gid >= 2900 && grp->gr_gid <= 2999) &&
+ !(grp->gr_gid >= 5000 && grp->gr_gid <= 5999)) {
+ ASSERT_FALSE(exist[grp->gr_gid]);
+ }
exist[grp->gr_gid] = true;
}
}