/* * Copyright (C) 2020 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. */ #define LOG_TAG "dumpstate_1_1_hidl_hal_test" #include #include #include #include #include #include #include #include #include #include #include #include #include namespace { using ::android::sp; using ::android::hardware::Return; using ::android::hardware::dumpstate::V1_1::DumpstateMode; using ::android::hardware::dumpstate::V1_1::DumpstateStatus; using ::android::hardware::dumpstate::V1_1::IDumpstateDevice; using ::android::hardware::dumpstate::V1_1::toString; // Base class common to all dumpstate HAL v1.1 tests. template class DumpstateHidl1_1TestBase : public ::testing::TestWithParam { protected: virtual void SetUp() override { GetService(); } virtual std::string GetInstanceName() = 0; void GetService() { const std::string instance_name = GetInstanceName(); dumpstate = IDumpstateDevice::getService(instance_name); ASSERT_NE(dumpstate, nullptr) << "Could not get HIDL instance " << instance_name; } void ToggleVerboseLogging(bool enable) { Return status = dumpstate->setVerboseLoggingEnabled(enable); ASSERT_TRUE(status.isOk()) << "Status should be ok: " << status.description(); if (!dumpstate->ping().isOk()) { ALOGW("IDumpstateDevice service appears to have exited lazily, attempting to get " "again"); GetService(); } Return logging_enabled = dumpstate->getVerboseLoggingEnabled(); ASSERT_TRUE(logging_enabled.isOk()) << "Status should be ok: " << logging_enabled.description(); ASSERT_EQ(logging_enabled, enable) << "Verbose logging should now be " << (enable ? "enabled" : "disabled"); if (!dumpstate->ping().isOk()) { ALOGW("IDumpstateDevice service appears to have exited lazily, attempting to get " "again"); GetService(); } } void EnableVerboseLogging() { ToggleVerboseLogging(true); } void DisableVerboseLogging() { ToggleVerboseLogging(false); } sp dumpstate; }; // Tests that don't need to iterate every single DumpstateMode value for dumpstateBoard_1_1. class DumpstateHidl1_1GeneralTest : public DumpstateHidl1_1TestBase { protected: virtual std::string GetInstanceName() override { return GetParam(); } }; // Tests that iterate every single DumpstateMode value for dumpstateBoard_1_1. class DumpstateHidl1_1PerModeTest : public DumpstateHidl1_1TestBase> { protected: virtual std::string GetInstanceName() override { return std::get<0>(GetParam()); } DumpstateMode GetMode() { return std::get<1>(GetParam()); } // Will only execute additional_assertions when status == expected. void AssertStatusForMode(const Return& status, const DumpstateStatus expected, std::function additional_assertions = nullptr) { ASSERT_TRUE(status.isOk()) << "Status should be ok and return a more specific DumpstateStatus: " << status.description(); if (GetMode() == DumpstateMode::DEFAULT) { ASSERT_EQ(expected, status) << "Required mode (DumpstateMode::" << toString(GetMode()) << "): status should be DumpstateStatus::" << toString(expected) << ", but got DumpstateStatus::" << toString(status); } else { // The rest of the modes are optional to support, but they MUST return either the // expected value or UNSUPPORTED_MODE. ASSERT_TRUE(status == expected || status == DumpstateStatus::UNSUPPORTED_MODE) << "Optional mode (DumpstateMode::" << toString(GetMode()) << "): status should be DumpstateStatus::" << toString(expected) << " or DumpstateStatus::UNSUPPORTED_MODE, but got DumpstateStatus::" << toString(status); } if (status == expected && additional_assertions != nullptr) { additional_assertions(); } } }; constexpr uint64_t kDefaultTimeoutMillis = 30 * 1000; // 30 seconds // Negative test: make sure dumpstateBoard() doesn't crash when passed a null pointer. TEST_P(DumpstateHidl1_1PerModeTest, TestNullHandle) { EnableVerboseLogging(); Return status = dumpstate->dumpstateBoard_1_1(nullptr, GetMode(), kDefaultTimeoutMillis); AssertStatusForMode(status, DumpstateStatus::ILLEGAL_ARGUMENT); } // Negative test: make sure dumpstateBoard() ignores a handle with no FD. TEST_P(DumpstateHidl1_1PerModeTest, TestHandleWithNoFd) { EnableVerboseLogging(); native_handle_t* handle = native_handle_create(0, 0); ASSERT_NE(handle, nullptr) << "Could not create native_handle"; Return status = dumpstate->dumpstateBoard_1_1(handle, GetMode(), kDefaultTimeoutMillis); AssertStatusForMode(status, DumpstateStatus::ILLEGAL_ARGUMENT); native_handle_close(handle); native_handle_delete(handle); } // Positive test: make sure dumpstateBoard() writes something to the FD. TEST_P(DumpstateHidl1_1PerModeTest, TestOk) { EnableVerboseLogging(); // Index 0 corresponds to the read end of the pipe; 1 to the write end. int fds[2]; ASSERT_EQ(0, pipe2(fds, O_NONBLOCK)) << errno; native_handle_t* handle = native_handle_create(1, 0); ASSERT_NE(handle, nullptr) << "Could not create native_handle"; handle->data[0] = fds[1]; Return status = dumpstate->dumpstateBoard_1_1(handle, GetMode(), kDefaultTimeoutMillis); AssertStatusForMode(status, DumpstateStatus::OK, [&fds]() { // Check that at least one byte was written. char buff; ASSERT_EQ(1, read(fds[0], &buff, 1)) << "Dumped nothing"; }); native_handle_close(handle); native_handle_delete(handle); } // Positive test: make sure dumpstateBoard() doesn't crash with two FDs. TEST_P(DumpstateHidl1_1PerModeTest, TestHandleWithTwoFds) { EnableVerboseLogging(); int fds1[2]; int fds2[2]; ASSERT_EQ(0, pipe2(fds1, O_NONBLOCK)) << errno; ASSERT_EQ(0, pipe2(fds2, O_NONBLOCK)) << errno; native_handle_t* handle = native_handle_create(2, 0); ASSERT_NE(handle, nullptr) << "Could not create native_handle"; handle->data[0] = fds1[1]; handle->data[1] = fds2[1]; Return status = dumpstate->dumpstateBoard_1_1(handle, GetMode(), kDefaultTimeoutMillis); AssertStatusForMode(status, DumpstateStatus::OK, [&fds1, &fds2]() { // Check that at least one byte was written to one of the FDs. char buff; size_t read1 = read(fds1[0], &buff, 1); size_t read2 = read(fds2[0], &buff, 1); // Sometimes read returns -1, so we can't just add them together and expect >= 1. ASSERT_TRUE(read1 == 1 || read2 == 1) << "Dumped nothing"; }); native_handle_close(handle); native_handle_delete(handle); } // Make sure dumpstateBoard_1_1 actually validates its arguments. TEST_P(DumpstateHidl1_1GeneralTest, TestInvalidModeArgument_Negative) { EnableVerboseLogging(); int fds[2]; ASSERT_EQ(0, pipe2(fds, O_NONBLOCK)) << errno; native_handle_t* handle = native_handle_create(1, 0); ASSERT_NE(handle, nullptr) << "Could not create native_handle"; handle->data[0] = fds[1]; Return status = dumpstate->dumpstateBoard_1_1( handle, static_cast(-100), kDefaultTimeoutMillis); ASSERT_TRUE(status.isOk()) << "Status should be ok and return a more specific DumpstateStatus: " << status.description(); ASSERT_EQ(status, DumpstateStatus::ILLEGAL_ARGUMENT) << "Should return DumpstateStatus::ILLEGAL_ARGUMENT for invalid mode param"; native_handle_close(handle); native_handle_delete(handle); } TEST_P(DumpstateHidl1_1GeneralTest, TestInvalidModeArgument_Undefined) { EnableVerboseLogging(); int fds[2]; ASSERT_EQ(0, pipe2(fds, O_NONBLOCK)) << errno; native_handle_t* handle = native_handle_create(1, 0); ASSERT_NE(handle, nullptr) << "Could not create native_handle"; handle->data[0] = fds[1]; Return status = dumpstate->dumpstateBoard_1_1( handle, static_cast(9001), kDefaultTimeoutMillis); ASSERT_TRUE(status.isOk()) << "Status should be ok and return a more specific DumpstateStatus: " << status.description(); ASSERT_EQ(status, DumpstateStatus::ILLEGAL_ARGUMENT) << "Should return DumpstateStatus::ILLEGAL_ARGUMENT for invalid mode param"; native_handle_close(handle); native_handle_delete(handle); } // Positive test: make sure dumpstateBoard() from 1.0 doesn't fail. TEST_P(DumpstateHidl1_1GeneralTest, Test1_0MethodOk) { EnableVerboseLogging(); int fds[2]; ASSERT_EQ(0, pipe2(fds, O_NONBLOCK)) << errno; native_handle_t* handle = native_handle_create(1, 0); ASSERT_NE(handle, nullptr) << "Could not create native_handle"; handle->data[0] = fds[1]; Return status = dumpstate->dumpstateBoard(handle); ASSERT_TRUE(status.isOk()) << "Status should be ok: " << status.description(); // Check that at least one byte was written. char buff; ASSERT_EQ(1, read(fds[0], &buff, 1)) << "Dumped nothing"; native_handle_close(handle); native_handle_delete(handle); } // Make sure disabling verbose logging behaves correctly. Some info is still allowed to be emitted, // but it can't have privacy/storage/battery impacts. TEST_P(DumpstateHidl1_1PerModeTest, TestDeviceLoggingDisabled) { DisableVerboseLogging(); // Index 0 corresponds to the read end of the pipe; 1 to the write end. int fds[2]; ASSERT_EQ(0, pipe2(fds, O_NONBLOCK)) << errno; native_handle_t* handle = native_handle_create(1, 0); ASSERT_NE(handle, nullptr) << "Could not create native_handle"; handle->data[0] = fds[1]; Return status = dumpstate->dumpstateBoard_1_1(handle, GetMode(), kDefaultTimeoutMillis); // We don't include additional assertions here about the file passed in. If verbose logging is // disabled, the OEM may choose to include nothing at all, but it is allowed to include some // essential information based on the mode as long as it isn't private user information. AssertStatusForMode(status, DumpstateStatus::OK); native_handle_close(handle); native_handle_delete(handle); } // Double-enable is perfectly valid, but the second call shouldn't do anything. TEST_P(DumpstateHidl1_1GeneralTest, TestRepeatedEnable) { EnableVerboseLogging(); EnableVerboseLogging(); } // Double-disable is perfectly valid, but the second call shouldn't do anything. TEST_P(DumpstateHidl1_1GeneralTest, TestRepeatedDisable) { DisableVerboseLogging(); DisableVerboseLogging(); } // Toggling in short order is perfectly valid. TEST_P(DumpstateHidl1_1GeneralTest, TestRepeatedToggle) { EnableVerboseLogging(); DisableVerboseLogging(); EnableVerboseLogging(); DisableVerboseLogging(); } INSTANTIATE_TEST_SUITE_P( PerInstance, DumpstateHidl1_1GeneralTest, testing::ValuesIn(android::hardware::getAllHalInstanceNames(IDumpstateDevice::descriptor)), android::hardware::PrintInstanceNameToString); // Includes the mode's name as part of the description string. static inline std::string PrintInstanceNameToStringWithMode( const testing::TestParamInfo>& info) { return android::hardware::PrintInstanceNameToString( testing::TestParamInfo(std::get<0>(info.param), info.index)) + "_" + toString(std::get<1>(info.param)); } INSTANTIATE_TEST_SUITE_P( PerInstanceAndMode, DumpstateHidl1_1PerModeTest, testing::Combine(testing::ValuesIn(android::hardware::getAllHalInstanceNames( IDumpstateDevice::descriptor)), testing::ValuesIn(android::hardware::hidl_enum_range())), PrintInstanceNameToStringWithMode); } // namespace