Fix dev_t (for LP64).
32-bit Android's dev_t was wrong too. We can't fix that without ABI breakage,
but we can at least fix 64-bit Android. And add tests.
Bug: https://code.google.com/p/android/issues/detail?id=54966
Change-Id: Ie2e42cc042b78b669a1a44e55f959dbd9c52c5c9
diff --git a/tests/Android.mk b/tests/Android.mk
index 0a4db91..916d0b2 100644
--- a/tests/Android.mk
+++ b/tests/Android.mk
@@ -66,6 +66,7 @@
sys_stat_test.cpp \
sys_syscall_test.cpp \
sys_time_test.cpp \
+ sys_types_test.cpp \
system_properties_test.cpp \
time_test.cpp \
unistd_test.cpp \
diff --git a/tests/sys_types_test.cpp b/tests/sys_types_test.cpp
new file mode 100644
index 0000000..0793be2
--- /dev/null
+++ b/tests/sys_types_test.cpp
@@ -0,0 +1,43 @@
+/*
+ * Copyright (C) 2014 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <gtest/gtest.h>
+
+#include <sys/types.h>
+
+TEST(sys_types, type_sizes) {
+ // gids, pids, and uids should be 32-bit on all systems.
+ ASSERT_EQ(4U, sizeof(gid_t));
+ ASSERT_EQ(4U, sizeof(pid_t));
+ ASSERT_EQ(4U, sizeof(uid_t));
+ // id_t is the 'generic'.
+ ASSERT_EQ(4U, sizeof(id_t));
+
+ // Some types were too small on 32-bit Android by mistake,
+ // but are correct on 64-bit Android.
+#if defined(__LP64__)
+ ASSERT_EQ(8U, sizeof(dev_t));
+ ASSERT_EQ(8U, sizeof(off_t));
+ ASSERT_EQ(8U, sizeof(time_t));
+#else
+ ASSERT_EQ(4U, sizeof(dev_t));
+ ASSERT_EQ(4U, sizeof(off_t));
+ ASSERT_EQ(4U, sizeof(time_t));
+#endif
+ // These were right even on 32-bit Android.
+ ASSERT_EQ(8U, sizeof(loff_t));
+ ASSERT_EQ(8U, sizeof(off64_t));
+}