| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 1 | /* | 
|  | 2 | * Copyright 2016 The Android Open Source Project | 
|  | 3 | * | 
|  | 4 | * Licensed under the Apache License, Version 2.0 (the "License"); | 
|  | 5 | * you may not use this file except in compliance with the License. | 
|  | 6 | * You may obtain a copy of the License at | 
|  | 7 | * | 
|  | 8 | *      http://www.apache.org/licenses/LICENSE-2.0 | 
|  | 9 | * | 
|  | 10 | * Unless required by applicable law or agreed to in writing, software | 
|  | 11 | * distributed under the License is distributed on an "AS IS" BASIS, | 
|  | 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | 
|  | 13 | * See the License for the specific language governing permissions and | 
|  | 14 | * limitations under the License. | 
|  | 15 | */ | 
|  | 16 |  | 
|  | 17 | #include <binder/SafeInterface.h> | 
|  | 18 |  | 
|  | 19 | #include <binder/IInterface.h> | 
|  | 20 | #include <binder/IPCThreadState.h> | 
|  | 21 | #include <binder/IServiceManager.h> | 
|  | 22 | #include <binder/Parcel.h> | 
|  | 23 | #include <binder/Parcelable.h> | 
|  | 24 | #include <binder/ProcessState.h> | 
|  | 25 |  | 
|  | 26 | #pragma clang diagnostic push | 
|  | 27 | #pragma clang diagnostic ignored "-Weverything" | 
|  | 28 | #include <gtest/gtest.h> | 
|  | 29 | #pragma clang diagnostic pop | 
|  | 30 |  | 
| Dan Stoza | 6dd325b | 2017-04-07 14:31:51 -0700 | [diff] [blame] | 31 | #include <utils/LightRefBase.h> | 
| Dan Stoza | 2537db7 | 2017-04-07 16:32:38 -0700 | [diff] [blame] | 32 | #include <utils/NativeHandle.h> | 
|  | 33 |  | 
|  | 34 | #include <cutils/native_handle.h> | 
| Dan Stoza | 6dd325b | 2017-04-07 14:31:51 -0700 | [diff] [blame] | 35 |  | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 36 | #include <optional> | 
|  | 37 |  | 
| Dan Stoza | 2537db7 | 2017-04-07 16:32:38 -0700 | [diff] [blame] | 38 | #include <sys/eventfd.h> | 
|  | 39 |  | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 40 | using namespace std::chrono_literals; // NOLINT - google-build-using-namespace | 
|  | 41 |  | 
|  | 42 | namespace android { | 
|  | 43 | namespace tests { | 
|  | 44 |  | 
| Dan Stoza | 81ea3ef | 2017-04-07 15:00:18 -0700 | [diff] [blame] | 45 | enum class TestEnum : uint32_t { | 
|  | 46 | INVALID = 0, | 
|  | 47 | INITIAL = 1, | 
|  | 48 | FINAL = 2, | 
|  | 49 | }; | 
|  | 50 |  | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 51 | // This class serves two purposes: | 
|  | 52 | //   1) It ensures that the implementation doesn't require copying or moving the data (for | 
|  | 53 | //      efficiency purposes) | 
|  | 54 | //   2) It tests that Parcelables can be passed correctly | 
|  | 55 | class NoCopyNoMove : public Parcelable { | 
|  | 56 | public: | 
|  | 57 | NoCopyNoMove() = default; | 
|  | 58 | explicit NoCopyNoMove(int32_t value) : mValue(value) {} | 
|  | 59 | ~NoCopyNoMove() override = default; | 
|  | 60 |  | 
|  | 61 | // Not copyable | 
|  | 62 | NoCopyNoMove(const NoCopyNoMove&) = delete; | 
|  | 63 | NoCopyNoMove& operator=(const NoCopyNoMove&) = delete; | 
|  | 64 |  | 
|  | 65 | // Not movable | 
|  | 66 | NoCopyNoMove(NoCopyNoMove&&) = delete; | 
|  | 67 | NoCopyNoMove& operator=(NoCopyNoMove&&) = delete; | 
|  | 68 |  | 
|  | 69 | // Parcelable interface | 
|  | 70 | status_t writeToParcel(Parcel* parcel) const override { return parcel->writeInt32(mValue); } | 
|  | 71 | status_t readFromParcel(const Parcel* parcel) override { return parcel->readInt32(&mValue); } | 
|  | 72 |  | 
|  | 73 | int32_t getValue() const { return mValue; } | 
|  | 74 | void setValue(int32_t value) { mValue = value; } | 
|  | 75 |  | 
|  | 76 | private: | 
|  | 77 | int32_t mValue = 0; | 
|  | 78 | uint8_t mPadding[4] = {}; // Avoids a warning from -Wpadded | 
|  | 79 | }; | 
|  | 80 |  | 
| Dan Stoza | df614ae | 2017-03-28 17:02:05 -0700 | [diff] [blame] | 81 | struct TestFlattenable : Flattenable<TestFlattenable> { | 
|  | 82 | TestFlattenable() = default; | 
|  | 83 | explicit TestFlattenable(int32_t v) : value(v) {} | 
|  | 84 |  | 
|  | 85 | // Flattenable protocol | 
|  | 86 | size_t getFlattenedSize() const { return sizeof(value); } | 
|  | 87 | size_t getFdCount() const { return 0; } | 
|  | 88 | status_t flatten(void*& buffer, size_t& size, int*& /*fds*/, size_t& /*count*/) const { | 
|  | 89 | FlattenableUtils::write(buffer, size, value); | 
|  | 90 | return NO_ERROR; | 
|  | 91 | } | 
|  | 92 | status_t unflatten(void const*& buffer, size_t& size, int const*& /*fds*/, size_t& /*count*/) { | 
|  | 93 | FlattenableUtils::read(buffer, size, value); | 
|  | 94 | return NO_ERROR; | 
|  | 95 | } | 
|  | 96 |  | 
|  | 97 | int32_t value = 0; | 
|  | 98 | }; | 
|  | 99 |  | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 100 | struct TestLightFlattenable : LightFlattenablePod<TestLightFlattenable> { | 
|  | 101 | TestLightFlattenable() = default; | 
|  | 102 | explicit TestLightFlattenable(int32_t v) : value(v) {} | 
|  | 103 | int32_t value = 0; | 
|  | 104 | }; | 
|  | 105 |  | 
| Dan Stoza | 6dd325b | 2017-04-07 14:31:51 -0700 | [diff] [blame] | 106 | // It seems like this should be able to inherit from TestFlattenable (to avoid duplicating code), | 
|  | 107 | // but the SafeInterface logic can't easily be extended to find an indirect Flattenable<T> | 
|  | 108 | // base class | 
|  | 109 | class TestLightRefBaseFlattenable : public Flattenable<TestLightRefBaseFlattenable>, | 
|  | 110 | public LightRefBase<TestLightRefBaseFlattenable> { | 
|  | 111 | public: | 
|  | 112 | TestLightRefBaseFlattenable() = default; | 
|  | 113 | explicit TestLightRefBaseFlattenable(int32_t v) : value(v) {} | 
|  | 114 |  | 
|  | 115 | // Flattenable protocol | 
|  | 116 | size_t getFlattenedSize() const { return sizeof(value); } | 
|  | 117 | size_t getFdCount() const { return 0; } | 
|  | 118 | status_t flatten(void*& buffer, size_t& size, int*& /*fds*/, size_t& /*count*/) const { | 
|  | 119 | FlattenableUtils::write(buffer, size, value); | 
|  | 120 | return NO_ERROR; | 
|  | 121 | } | 
|  | 122 | status_t unflatten(void const*& buffer, size_t& size, int const*& /*fds*/, size_t& /*count*/) { | 
|  | 123 | FlattenableUtils::read(buffer, size, value); | 
|  | 124 | return NO_ERROR; | 
|  | 125 | } | 
|  | 126 |  | 
|  | 127 | int32_t value = 0; | 
|  | 128 | }; | 
|  | 129 |  | 
| Dan Stoza | 1af8a88 | 2017-04-10 13:28:54 -0700 | [diff] [blame] | 130 | class TestParcelable : public Parcelable { | 
|  | 131 | public: | 
|  | 132 | TestParcelable() = default; | 
|  | 133 | explicit TestParcelable(int32_t value) : mValue(value) {} | 
|  | 134 | TestParcelable(const TestParcelable& other) : TestParcelable(other.mValue) {} | 
|  | 135 | TestParcelable(TestParcelable&& other) : TestParcelable(other.mValue) {} | 
|  | 136 |  | 
|  | 137 | // Parcelable interface | 
|  | 138 | status_t writeToParcel(Parcel* parcel) const override { return parcel->writeInt32(mValue); } | 
|  | 139 | status_t readFromParcel(const Parcel* parcel) override { return parcel->readInt32(&mValue); } | 
|  | 140 |  | 
|  | 141 | int32_t getValue() const { return mValue; } | 
|  | 142 | void setValue(int32_t value) { mValue = value; } | 
|  | 143 |  | 
|  | 144 | private: | 
|  | 145 | int32_t mValue = 0; | 
|  | 146 | }; | 
|  | 147 |  | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 148 | class ExitOnDeath : public IBinder::DeathRecipient { | 
|  | 149 | public: | 
|  | 150 | ~ExitOnDeath() override = default; | 
|  | 151 |  | 
|  | 152 | void binderDied(const wp<IBinder>& /*who*/) override { | 
|  | 153 | ALOG(LOG_INFO, "ExitOnDeath", "Exiting"); | 
|  | 154 | exit(0); | 
|  | 155 | } | 
|  | 156 | }; | 
|  | 157 |  | 
|  | 158 | // This callback class is used to test both one-way transactions and that sp<IInterface> can be | 
|  | 159 | // passed correctly | 
|  | 160 | class ICallback : public IInterface { | 
|  | 161 | public: | 
|  | 162 | DECLARE_META_INTERFACE(Callback) | 
|  | 163 |  | 
|  | 164 | enum class Tag : uint32_t { | 
|  | 165 | OnCallback = IBinder::FIRST_CALL_TRANSACTION, | 
|  | 166 | Last, | 
|  | 167 | }; | 
|  | 168 |  | 
|  | 169 | virtual void onCallback(int32_t aPlusOne) = 0; | 
|  | 170 | }; | 
|  | 171 |  | 
|  | 172 | class BpCallback : public SafeBpInterface<ICallback> { | 
|  | 173 | public: | 
|  | 174 | explicit BpCallback(const sp<IBinder>& impl) : SafeBpInterface<ICallback>(impl, getLogTag()) {} | 
|  | 175 |  | 
|  | 176 | void onCallback(int32_t aPlusOne) override { | 
|  | 177 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 178 | return callRemoteAsync<decltype(&ICallback::onCallback)>(Tag::OnCallback, aPlusOne); | 
|  | 179 | } | 
|  | 180 |  | 
|  | 181 | private: | 
|  | 182 | static constexpr const char* getLogTag() { return "BpCallback"; } | 
|  | 183 | }; | 
|  | 184 |  | 
|  | 185 | #pragma clang diagnostic push | 
|  | 186 | #pragma clang diagnostic ignored "-Wexit-time-destructors" | 
|  | 187 | IMPLEMENT_META_INTERFACE(Callback, "android.gfx.tests.ICallback"); | 
|  | 188 | #pragma clang diagnostic pop | 
|  | 189 |  | 
|  | 190 | class BnCallback : public SafeBnInterface<ICallback> { | 
|  | 191 | public: | 
|  | 192 | BnCallback() : SafeBnInterface("BnCallback") {} | 
|  | 193 |  | 
|  | 194 | status_t onTransact(uint32_t code, const Parcel& data, Parcel* reply, | 
|  | 195 | uint32_t /*flags*/) override { | 
|  | 196 | EXPECT_GE(code, IBinder::FIRST_CALL_TRANSACTION); | 
|  | 197 | EXPECT_LT(code, static_cast<uint32_t>(ICallback::Tag::Last)); | 
|  | 198 | ICallback::Tag tag = static_cast<ICallback::Tag>(code); | 
|  | 199 | switch (tag) { | 
|  | 200 | case ICallback::Tag::OnCallback: { | 
|  | 201 | return callLocalAsync(data, reply, &ICallback::onCallback); | 
|  | 202 | } | 
|  | 203 | case ICallback::Tag::Last: | 
|  | 204 | // Should not be possible because of the asserts at the beginning of the method | 
|  | 205 | [&]() { FAIL(); }(); | 
|  | 206 | return UNKNOWN_ERROR; | 
|  | 207 | } | 
|  | 208 | } | 
|  | 209 | }; | 
|  | 210 |  | 
|  | 211 | class ISafeInterfaceTest : public IInterface { | 
|  | 212 | public: | 
|  | 213 | DECLARE_META_INTERFACE(SafeInterfaceTest) | 
|  | 214 |  | 
|  | 215 | enum class Tag : uint32_t { | 
|  | 216 | SetDeathToken = IBinder::FIRST_CALL_TRANSACTION, | 
|  | 217 | ReturnsNoMemory, | 
|  | 218 | LogicalNot, | 
| Dan Stoza | 81ea3ef | 2017-04-07 15:00:18 -0700 | [diff] [blame] | 219 | ModifyEnum, | 
| Dan Stoza | df614ae | 2017-03-28 17:02:05 -0700 | [diff] [blame] | 220 | IncrementFlattenable, | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 221 | IncrementLightFlattenable, | 
| Dan Stoza | 6dd325b | 2017-04-07 14:31:51 -0700 | [diff] [blame] | 222 | IncrementLightRefBaseFlattenable, | 
| Dan Stoza | 2537db7 | 2017-04-07 16:32:38 -0700 | [diff] [blame] | 223 | IncrementNativeHandle, | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 224 | IncrementNoCopyNoMove, | 
| Dan Stoza | 1af8a88 | 2017-04-10 13:28:54 -0700 | [diff] [blame] | 225 | IncrementParcelableVector, | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 226 | ToUpper, | 
|  | 227 | CallMeBack, | 
|  | 228 | IncrementInt32, | 
|  | 229 | IncrementUint32, | 
| Dan Stoza | 662a899 | 2017-04-06 16:18:39 -0700 | [diff] [blame] | 230 | IncrementInt64, | 
|  | 231 | IncrementUint64, | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 232 | IncrementTwo, | 
|  | 233 | Last, | 
|  | 234 | }; | 
|  | 235 |  | 
|  | 236 | // This is primarily so that the remote service dies when the test does, but it also serves to | 
|  | 237 | // test the handling of sp<IBinder> and non-const methods | 
|  | 238 | virtual status_t setDeathToken(const sp<IBinder>& token) = 0; | 
|  | 239 |  | 
|  | 240 | // This is the most basic test since it doesn't require parceling any arguments | 
|  | 241 | virtual status_t returnsNoMemory() const = 0; | 
|  | 242 |  | 
|  | 243 | // These are ordered according to their corresponding methods in SafeInterface::ParcelHandler | 
|  | 244 | virtual status_t logicalNot(bool a, bool* notA) const = 0; | 
| Dan Stoza | 81ea3ef | 2017-04-07 15:00:18 -0700 | [diff] [blame] | 245 | virtual status_t modifyEnum(TestEnum a, TestEnum* b) const = 0; | 
| Dan Stoza | df614ae | 2017-03-28 17:02:05 -0700 | [diff] [blame] | 246 | virtual status_t increment(const TestFlattenable& a, TestFlattenable* aPlusOne) const = 0; | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 247 | virtual status_t increment(const TestLightFlattenable& a, | 
|  | 248 | TestLightFlattenable* aPlusOne) const = 0; | 
| Dan Stoza | 6dd325b | 2017-04-07 14:31:51 -0700 | [diff] [blame] | 249 | virtual status_t increment(const sp<TestLightRefBaseFlattenable>& a, | 
|  | 250 | sp<TestLightRefBaseFlattenable>* aPlusOne) const = 0; | 
| Dan Stoza | 2537db7 | 2017-04-07 16:32:38 -0700 | [diff] [blame] | 251 | virtual status_t increment(const sp<NativeHandle>& a, sp<NativeHandle>* aPlusOne) const = 0; | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 252 | virtual status_t increment(const NoCopyNoMove& a, NoCopyNoMove* aPlusOne) const = 0; | 
| Dan Stoza | 1af8a88 | 2017-04-10 13:28:54 -0700 | [diff] [blame] | 253 | virtual status_t increment(const std::vector<TestParcelable>& a, | 
|  | 254 | std::vector<TestParcelable>* aPlusOne) const = 0; | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 255 | virtual status_t toUpper(const String8& str, String8* upperStr) const = 0; | 
|  | 256 | // As mentioned above, sp<IBinder> is already tested by setDeathToken | 
|  | 257 | virtual void callMeBack(const sp<ICallback>& callback, int32_t a) const = 0; | 
|  | 258 | virtual status_t increment(int32_t a, int32_t* aPlusOne) const = 0; | 
|  | 259 | virtual status_t increment(uint32_t a, uint32_t* aPlusOne) const = 0; | 
| Dan Stoza | 662a899 | 2017-04-06 16:18:39 -0700 | [diff] [blame] | 260 | virtual status_t increment(int64_t a, int64_t* aPlusOne) const = 0; | 
|  | 261 | virtual status_t increment(uint64_t a, uint64_t* aPlusOne) const = 0; | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 262 |  | 
|  | 263 | // This tests that input/output parameter interleaving works correctly | 
|  | 264 | virtual status_t increment(int32_t a, int32_t* aPlusOne, int32_t b, | 
|  | 265 | int32_t* bPlusOne) const = 0; | 
|  | 266 | }; | 
|  | 267 |  | 
|  | 268 | class BpSafeInterfaceTest : public SafeBpInterface<ISafeInterfaceTest> { | 
|  | 269 | public: | 
|  | 270 | explicit BpSafeInterfaceTest(const sp<IBinder>& impl) | 
|  | 271 | : SafeBpInterface<ISafeInterfaceTest>(impl, getLogTag()) {} | 
|  | 272 |  | 
|  | 273 | status_t setDeathToken(const sp<IBinder>& token) override { | 
|  | 274 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 275 | return callRemote<decltype(&ISafeInterfaceTest::setDeathToken)>(Tag::SetDeathToken, token); | 
|  | 276 | } | 
|  | 277 | status_t returnsNoMemory() const override { | 
|  | 278 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 279 | return callRemote<decltype(&ISafeInterfaceTest::returnsNoMemory)>(Tag::ReturnsNoMemory); | 
|  | 280 | } | 
|  | 281 | status_t logicalNot(bool a, bool* notA) const override { | 
|  | 282 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 283 | return callRemote<decltype(&ISafeInterfaceTest::logicalNot)>(Tag::LogicalNot, a, notA); | 
|  | 284 | } | 
| Dan Stoza | 81ea3ef | 2017-04-07 15:00:18 -0700 | [diff] [blame] | 285 | status_t modifyEnum(TestEnum a, TestEnum* b) const override { | 
|  | 286 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 287 | return callRemote<decltype(&ISafeInterfaceTest::modifyEnum)>(Tag::ModifyEnum, a, b); | 
|  | 288 | } | 
| Dan Stoza | df614ae | 2017-03-28 17:02:05 -0700 | [diff] [blame] | 289 | status_t increment(const TestFlattenable& a, TestFlattenable* aPlusOne) const override { | 
|  | 290 | using Signature = | 
|  | 291 | status_t (ISafeInterfaceTest::*)(const TestFlattenable&, TestFlattenable*) const; | 
|  | 292 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 293 | return callRemote<Signature>(Tag::IncrementFlattenable, a, aPlusOne); | 
|  | 294 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 295 | status_t increment(const TestLightFlattenable& a, | 
|  | 296 | TestLightFlattenable* aPlusOne) const override { | 
|  | 297 | using Signature = status_t (ISafeInterfaceTest::*)(const TestLightFlattenable&, | 
|  | 298 | TestLightFlattenable*) const; | 
|  | 299 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 300 | return callRemote<Signature>(Tag::IncrementLightFlattenable, a, aPlusOne); | 
|  | 301 | } | 
| Dan Stoza | 6dd325b | 2017-04-07 14:31:51 -0700 | [diff] [blame] | 302 | status_t increment(const sp<TestLightRefBaseFlattenable>& a, | 
|  | 303 | sp<TestLightRefBaseFlattenable>* aPlusOne) const override { | 
|  | 304 | using Signature = status_t (ISafeInterfaceTest::*)(const sp<TestLightRefBaseFlattenable>&, | 
|  | 305 | sp<TestLightRefBaseFlattenable>*) const; | 
|  | 306 | return callRemote<Signature>(Tag::IncrementLightRefBaseFlattenable, a, aPlusOne); | 
|  | 307 | } | 
| Dan Stoza | 2537db7 | 2017-04-07 16:32:38 -0700 | [diff] [blame] | 308 | status_t increment(const sp<NativeHandle>& a, sp<NativeHandle>* aPlusOne) const override { | 
|  | 309 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 310 | using Signature = | 
|  | 311 | status_t (ISafeInterfaceTest::*)(const sp<NativeHandle>&, sp<NativeHandle>*) const; | 
|  | 312 | return callRemote<Signature>(Tag::IncrementNativeHandle, a, aPlusOne); | 
|  | 313 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 314 | status_t increment(const NoCopyNoMove& a, NoCopyNoMove* aPlusOne) const override { | 
|  | 315 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 316 | using Signature = status_t (ISafeInterfaceTest::*)(const NoCopyNoMove& a, | 
|  | 317 | NoCopyNoMove* aPlusOne) const; | 
|  | 318 | return callRemote<Signature>(Tag::IncrementNoCopyNoMove, a, aPlusOne); | 
|  | 319 | } | 
| Dan Stoza | 1af8a88 | 2017-04-10 13:28:54 -0700 | [diff] [blame] | 320 | status_t increment(const std::vector<TestParcelable>& a, | 
|  | 321 | std::vector<TestParcelable>* aPlusOne) const override { | 
|  | 322 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 323 | using Signature = status_t (ISafeInterfaceTest::*)(const std::vector<TestParcelable>&, | 
|  | 324 | std::vector<TestParcelable>*); | 
|  | 325 | return callRemote<Signature>(Tag::IncrementParcelableVector, a, aPlusOne); | 
|  | 326 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 327 | status_t toUpper(const String8& str, String8* upperStr) const override { | 
|  | 328 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 329 | return callRemote<decltype(&ISafeInterfaceTest::toUpper)>(Tag::ToUpper, str, upperStr); | 
|  | 330 | } | 
|  | 331 | void callMeBack(const sp<ICallback>& callback, int32_t a) const override { | 
|  | 332 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 333 | return callRemoteAsync<decltype(&ISafeInterfaceTest::callMeBack)>(Tag::CallMeBack, callback, | 
|  | 334 | a); | 
|  | 335 | } | 
|  | 336 | status_t increment(int32_t a, int32_t* aPlusOne) const override { | 
|  | 337 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 338 | using Signature = status_t (ISafeInterfaceTest::*)(int32_t, int32_t*) const; | 
|  | 339 | return callRemote<Signature>(Tag::IncrementInt32, a, aPlusOne); | 
|  | 340 | } | 
|  | 341 | status_t increment(uint32_t a, uint32_t* aPlusOne) const override { | 
|  | 342 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 343 | using Signature = status_t (ISafeInterfaceTest::*)(uint32_t, uint32_t*) const; | 
|  | 344 | return callRemote<Signature>(Tag::IncrementUint32, a, aPlusOne); | 
|  | 345 | } | 
| Dan Stoza | 662a899 | 2017-04-06 16:18:39 -0700 | [diff] [blame] | 346 | status_t increment(int64_t a, int64_t* aPlusOne) const override { | 
|  | 347 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 348 | using Signature = status_t (ISafeInterfaceTest::*)(int64_t, int64_t*) const; | 
|  | 349 | return callRemote<Signature>(Tag::IncrementInt64, a, aPlusOne); | 
|  | 350 | } | 
|  | 351 | status_t increment(uint64_t a, uint64_t* aPlusOne) const override { | 
|  | 352 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 353 | using Signature = status_t (ISafeInterfaceTest::*)(uint64_t, uint64_t*) const; | 
|  | 354 | return callRemote<Signature>(Tag::IncrementUint64, a, aPlusOne); | 
|  | 355 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 356 | status_t increment(int32_t a, int32_t* aPlusOne, int32_t b, int32_t* bPlusOne) const override { | 
|  | 357 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 358 | using Signature = | 
|  | 359 | status_t (ISafeInterfaceTest::*)(int32_t, int32_t*, int32_t, int32_t*) const; | 
|  | 360 | return callRemote<Signature>(Tag::IncrementTwo, a, aPlusOne, b, bPlusOne); | 
|  | 361 | } | 
|  | 362 |  | 
|  | 363 | private: | 
|  | 364 | static constexpr const char* getLogTag() { return "BpSafeInterfaceTest"; } | 
|  | 365 | }; | 
|  | 366 |  | 
|  | 367 | #pragma clang diagnostic push | 
|  | 368 | #pragma clang diagnostic ignored "-Wexit-time-destructors" | 
|  | 369 | IMPLEMENT_META_INTERFACE(SafeInterfaceTest, "android.gfx.tests.ISafeInterfaceTest"); | 
|  | 370 |  | 
|  | 371 | static sp<IBinder::DeathRecipient> getDeathRecipient() { | 
|  | 372 | static sp<IBinder::DeathRecipient> recipient = new ExitOnDeath; | 
|  | 373 | return recipient; | 
|  | 374 | } | 
|  | 375 | #pragma clang diagnostic pop | 
|  | 376 |  | 
|  | 377 | class BnSafeInterfaceTest : public SafeBnInterface<ISafeInterfaceTest> { | 
|  | 378 | public: | 
|  | 379 | BnSafeInterfaceTest() : SafeBnInterface(getLogTag()) {} | 
|  | 380 |  | 
|  | 381 | status_t setDeathToken(const sp<IBinder>& token) override { | 
|  | 382 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 383 | token->linkToDeath(getDeathRecipient()); | 
|  | 384 | return NO_ERROR; | 
|  | 385 | } | 
|  | 386 | status_t returnsNoMemory() const override { | 
|  | 387 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 388 | return NO_MEMORY; | 
|  | 389 | } | 
|  | 390 | status_t logicalNot(bool a, bool* notA) const override { | 
|  | 391 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 392 | *notA = !a; | 
|  | 393 | return NO_ERROR; | 
|  | 394 | } | 
| Dan Stoza | 81ea3ef | 2017-04-07 15:00:18 -0700 | [diff] [blame] | 395 | status_t modifyEnum(TestEnum a, TestEnum* b) const override { | 
|  | 396 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 397 | *b = (a == TestEnum::INITIAL) ? TestEnum::FINAL : TestEnum::INVALID; | 
|  | 398 | return NO_ERROR; | 
|  | 399 | } | 
| Dan Stoza | df614ae | 2017-03-28 17:02:05 -0700 | [diff] [blame] | 400 | status_t increment(const TestFlattenable& a, TestFlattenable* aPlusOne) const override { | 
|  | 401 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 402 | aPlusOne->value = a.value + 1; | 
|  | 403 | return NO_ERROR; | 
|  | 404 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 405 | status_t increment(const TestLightFlattenable& a, | 
|  | 406 | TestLightFlattenable* aPlusOne) const override { | 
|  | 407 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 408 | aPlusOne->value = a.value + 1; | 
|  | 409 | return NO_ERROR; | 
|  | 410 | } | 
| Dan Stoza | 6dd325b | 2017-04-07 14:31:51 -0700 | [diff] [blame] | 411 | status_t increment(const sp<TestLightRefBaseFlattenable>& a, | 
|  | 412 | sp<TestLightRefBaseFlattenable>* aPlusOne) const override { | 
|  | 413 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 414 | *aPlusOne = new TestLightRefBaseFlattenable(a->value + 1); | 
|  | 415 | return NO_ERROR; | 
|  | 416 | } | 
| Dan Stoza | 2537db7 | 2017-04-07 16:32:38 -0700 | [diff] [blame] | 417 | status_t increment(const sp<NativeHandle>& a, sp<NativeHandle>* aPlusOne) const override { | 
|  | 418 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 419 | native_handle* rawHandle = native_handle_create(1 /*numFds*/, 1 /*numInts*/); | 
|  | 420 | if (rawHandle == nullptr) return NO_MEMORY; | 
|  | 421 |  | 
|  | 422 | // Copy the fd over directly | 
|  | 423 | rawHandle->data[0] = dup(a->handle()->data[0]); | 
|  | 424 |  | 
|  | 425 | // Increment the int | 
|  | 426 | rawHandle->data[1] = a->handle()->data[1] + 1; | 
|  | 427 |  | 
|  | 428 | // This cannot fail, as it is just the sp<NativeHandle> taking responsibility for closing | 
|  | 429 | // the native_handle when it goes out of scope | 
|  | 430 | *aPlusOne = NativeHandle::create(rawHandle, true); | 
|  | 431 | return NO_ERROR; | 
|  | 432 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 433 | status_t increment(const NoCopyNoMove& a, NoCopyNoMove* aPlusOne) const override { | 
|  | 434 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 435 | aPlusOne->setValue(a.getValue() + 1); | 
|  | 436 | return NO_ERROR; | 
|  | 437 | } | 
| Dan Stoza | 1af8a88 | 2017-04-10 13:28:54 -0700 | [diff] [blame] | 438 | status_t increment(const std::vector<TestParcelable>& a, | 
|  | 439 | std::vector<TestParcelable>* aPlusOne) const override { | 
|  | 440 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 441 | aPlusOne->resize(a.size()); | 
|  | 442 | for (size_t i = 0; i < a.size(); ++i) { | 
|  | 443 | (*aPlusOne)[i].setValue(a[i].getValue() + 1); | 
|  | 444 | } | 
|  | 445 | return NO_ERROR; | 
|  | 446 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 447 | status_t toUpper(const String8& str, String8* upperStr) const override { | 
|  | 448 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 449 | *upperStr = str; | 
|  | 450 | upperStr->toUpper(); | 
|  | 451 | return NO_ERROR; | 
|  | 452 | } | 
|  | 453 | void callMeBack(const sp<ICallback>& callback, int32_t a) const override { | 
|  | 454 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 455 | callback->onCallback(a + 1); | 
|  | 456 | } | 
|  | 457 | status_t increment(int32_t a, int32_t* aPlusOne) const override { | 
|  | 458 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 459 | *aPlusOne = a + 1; | 
|  | 460 | return NO_ERROR; | 
|  | 461 | } | 
|  | 462 | status_t increment(uint32_t a, uint32_t* aPlusOne) const override { | 
|  | 463 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 464 | *aPlusOne = a + 1; | 
|  | 465 | return NO_ERROR; | 
|  | 466 | } | 
| Dan Stoza | 662a899 | 2017-04-06 16:18:39 -0700 | [diff] [blame] | 467 | status_t increment(int64_t a, int64_t* aPlusOne) const override { | 
|  | 468 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 469 | *aPlusOne = a + 1; | 
|  | 470 | return NO_ERROR; | 
|  | 471 | } | 
|  | 472 | status_t increment(uint64_t a, uint64_t* aPlusOne) const override { | 
|  | 473 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 474 | *aPlusOne = a + 1; | 
|  | 475 | return NO_ERROR; | 
|  | 476 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 477 | status_t increment(int32_t a, int32_t* aPlusOne, int32_t b, int32_t* bPlusOne) const override { | 
|  | 478 | ALOG(LOG_INFO, getLogTag(), "%s", __PRETTY_FUNCTION__); | 
|  | 479 | *aPlusOne = a + 1; | 
|  | 480 | *bPlusOne = b + 1; | 
|  | 481 | return NO_ERROR; | 
|  | 482 | } | 
|  | 483 |  | 
|  | 484 | // BnInterface | 
|  | 485 | status_t onTransact(uint32_t code, const Parcel& data, Parcel* reply, | 
|  | 486 | uint32_t /*flags*/) override { | 
|  | 487 | EXPECT_GE(code, IBinder::FIRST_CALL_TRANSACTION); | 
|  | 488 | EXPECT_LT(code, static_cast<uint32_t>(Tag::Last)); | 
|  | 489 | ISafeInterfaceTest::Tag tag = static_cast<ISafeInterfaceTest::Tag>(code); | 
|  | 490 | switch (tag) { | 
|  | 491 | case ISafeInterfaceTest::Tag::SetDeathToken: { | 
|  | 492 | return callLocal(data, reply, &ISafeInterfaceTest::setDeathToken); | 
|  | 493 | } | 
|  | 494 | case ISafeInterfaceTest::Tag::ReturnsNoMemory: { | 
|  | 495 | return callLocal(data, reply, &ISafeInterfaceTest::returnsNoMemory); | 
|  | 496 | } | 
|  | 497 | case ISafeInterfaceTest::Tag::LogicalNot: { | 
|  | 498 | return callLocal(data, reply, &ISafeInterfaceTest::logicalNot); | 
|  | 499 | } | 
| Dan Stoza | 81ea3ef | 2017-04-07 15:00:18 -0700 | [diff] [blame] | 500 | case ISafeInterfaceTest::Tag::ModifyEnum: { | 
|  | 501 | return callLocal(data, reply, &ISafeInterfaceTest::modifyEnum); | 
|  | 502 | } | 
| Dan Stoza | df614ae | 2017-03-28 17:02:05 -0700 | [diff] [blame] | 503 | case ISafeInterfaceTest::Tag::IncrementFlattenable: { | 
|  | 504 | using Signature = status_t (ISafeInterfaceTest::*)(const TestFlattenable& a, | 
|  | 505 | TestFlattenable* aPlusOne) const; | 
|  | 506 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 507 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 508 | case ISafeInterfaceTest::Tag::IncrementLightFlattenable: { | 
|  | 509 | using Signature = | 
|  | 510 | status_t (ISafeInterfaceTest::*)(const TestLightFlattenable& a, | 
|  | 511 | TestLightFlattenable* aPlusOne) const; | 
|  | 512 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 513 | } | 
| Dan Stoza | 6dd325b | 2017-04-07 14:31:51 -0700 | [diff] [blame] | 514 | case ISafeInterfaceTest::Tag::IncrementLightRefBaseFlattenable: { | 
|  | 515 | using Signature = | 
|  | 516 | status_t (ISafeInterfaceTest::*)(const sp<TestLightRefBaseFlattenable>&, | 
|  | 517 | sp<TestLightRefBaseFlattenable>*) const; | 
|  | 518 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 519 | } | 
| Dan Stoza | 2537db7 | 2017-04-07 16:32:38 -0700 | [diff] [blame] | 520 | case ISafeInterfaceTest::Tag::IncrementNativeHandle: { | 
|  | 521 | using Signature = status_t (ISafeInterfaceTest::*)(const sp<NativeHandle>&, | 
|  | 522 | sp<NativeHandle>*) const; | 
|  | 523 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 524 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 525 | case ISafeInterfaceTest::Tag::IncrementNoCopyNoMove: { | 
|  | 526 | using Signature = status_t (ISafeInterfaceTest::*)(const NoCopyNoMove& a, | 
|  | 527 | NoCopyNoMove* aPlusOne) const; | 
|  | 528 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 529 | } | 
| Dan Stoza | 1af8a88 | 2017-04-10 13:28:54 -0700 | [diff] [blame] | 530 | case ISafeInterfaceTest::Tag::IncrementParcelableVector: { | 
|  | 531 | using Signature = | 
|  | 532 | status_t (ISafeInterfaceTest::*)(const std::vector<TestParcelable>&, | 
|  | 533 | std::vector<TestParcelable>*) const; | 
|  | 534 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 535 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 536 | case ISafeInterfaceTest::Tag::ToUpper: { | 
|  | 537 | return callLocal(data, reply, &ISafeInterfaceTest::toUpper); | 
|  | 538 | } | 
|  | 539 | case ISafeInterfaceTest::Tag::CallMeBack: { | 
|  | 540 | return callLocalAsync(data, reply, &ISafeInterfaceTest::callMeBack); | 
|  | 541 | } | 
|  | 542 | case ISafeInterfaceTest::Tag::IncrementInt32: { | 
|  | 543 | using Signature = status_t (ISafeInterfaceTest::*)(int32_t, int32_t*) const; | 
|  | 544 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 545 | } | 
|  | 546 | case ISafeInterfaceTest::Tag::IncrementUint32: { | 
|  | 547 | using Signature = status_t (ISafeInterfaceTest::*)(uint32_t, uint32_t*) const; | 
|  | 548 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 549 | } | 
| Dan Stoza | 662a899 | 2017-04-06 16:18:39 -0700 | [diff] [blame] | 550 | case ISafeInterfaceTest::Tag::IncrementInt64: { | 
|  | 551 | using Signature = status_t (ISafeInterfaceTest::*)(int64_t, int64_t*) const; | 
|  | 552 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 553 | } | 
|  | 554 | case ISafeInterfaceTest::Tag::IncrementUint64: { | 
|  | 555 | using Signature = status_t (ISafeInterfaceTest::*)(uint64_t, uint64_t*) const; | 
|  | 556 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 557 | } | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 558 | case ISafeInterfaceTest::Tag::IncrementTwo: { | 
|  | 559 | using Signature = status_t (ISafeInterfaceTest::*)(int32_t, int32_t*, int32_t, | 
|  | 560 | int32_t*) const; | 
|  | 561 | return callLocal<Signature>(data, reply, &ISafeInterfaceTest::increment); | 
|  | 562 | } | 
|  | 563 | case ISafeInterfaceTest::Tag::Last: | 
|  | 564 | // Should not be possible because of the asserts at the beginning of the method | 
|  | 565 | [&]() { FAIL(); }(); | 
|  | 566 | return UNKNOWN_ERROR; | 
|  | 567 | } | 
|  | 568 | } | 
|  | 569 |  | 
|  | 570 | private: | 
|  | 571 | static constexpr const char* getLogTag() { return "BnSafeInterfaceTest"; } | 
|  | 572 | }; | 
|  | 573 |  | 
|  | 574 | class SafeInterfaceTest : public ::testing::Test { | 
|  | 575 | public: | 
|  | 576 | SafeInterfaceTest() : mSafeInterfaceTest(getRemoteService()) { | 
|  | 577 | ProcessState::self()->startThreadPool(); | 
|  | 578 | } | 
|  | 579 | ~SafeInterfaceTest() override = default; | 
|  | 580 |  | 
|  | 581 | protected: | 
|  | 582 | sp<ISafeInterfaceTest> mSafeInterfaceTest; | 
|  | 583 |  | 
|  | 584 | private: | 
|  | 585 | static constexpr const char* getLogTag() { return "SafeInterfaceTest"; } | 
|  | 586 |  | 
|  | 587 | sp<ISafeInterfaceTest> getRemoteService() { | 
|  | 588 | #pragma clang diagnostic push | 
|  | 589 | #pragma clang diagnostic ignored "-Wexit-time-destructors" | 
|  | 590 | static std::mutex sMutex; | 
|  | 591 | static sp<ISafeInterfaceTest> sService; | 
|  | 592 | static sp<IBinder> sDeathToken = new BBinder; | 
|  | 593 | #pragma clang diagnostic pop | 
|  | 594 |  | 
|  | 595 | std::unique_lock<decltype(sMutex)> lock; | 
|  | 596 | if (sService == nullptr) { | 
|  | 597 | ALOG(LOG_INFO, getLogTag(), "Forking remote process"); | 
|  | 598 | pid_t forkPid = fork(); | 
|  | 599 | EXPECT_NE(forkPid, -1); | 
|  | 600 |  | 
|  | 601 | const String16 serviceName("SafeInterfaceTest"); | 
|  | 602 |  | 
|  | 603 | if (forkPid == 0) { | 
|  | 604 | ALOG(LOG_INFO, getLogTag(), "Remote process checking in"); | 
|  | 605 | sp<ISafeInterfaceTest> nativeService = new BnSafeInterfaceTest; | 
|  | 606 | defaultServiceManager()->addService(serviceName, | 
|  | 607 | IInterface::asBinder(nativeService)); | 
|  | 608 | ProcessState::self()->startThreadPool(); | 
|  | 609 | IPCThreadState::self()->joinThreadPool(); | 
|  | 610 | // We shouldn't get to this point | 
|  | 611 | [&]() { FAIL(); }(); | 
|  | 612 | } | 
|  | 613 |  | 
|  | 614 | sp<IBinder> binder = defaultServiceManager()->getService(serviceName); | 
|  | 615 | sService = interface_cast<ISafeInterfaceTest>(binder); | 
|  | 616 | EXPECT_TRUE(sService != nullptr); | 
|  | 617 |  | 
|  | 618 | sService->setDeathToken(sDeathToken); | 
|  | 619 | } | 
|  | 620 |  | 
|  | 621 | return sService; | 
|  | 622 | } | 
|  | 623 | }; | 
|  | 624 |  | 
|  | 625 | TEST_F(SafeInterfaceTest, TestReturnsNoMemory) { | 
|  | 626 | status_t result = mSafeInterfaceTest->returnsNoMemory(); | 
|  | 627 | ASSERT_EQ(NO_MEMORY, result); | 
|  | 628 | } | 
|  | 629 |  | 
|  | 630 | TEST_F(SafeInterfaceTest, TestLogicalNot) { | 
|  | 631 | const bool a = true; | 
|  | 632 | bool notA = true; | 
|  | 633 | status_t result = mSafeInterfaceTest->logicalNot(a, ¬A); | 
|  | 634 | ASSERT_EQ(NO_ERROR, result); | 
|  | 635 | ASSERT_EQ(!a, notA); | 
|  | 636 | // Test both since we don't want to accidentally catch a default false somewhere | 
|  | 637 | const bool b = false; | 
|  | 638 | bool notB = false; | 
|  | 639 | result = mSafeInterfaceTest->logicalNot(b, ¬B); | 
|  | 640 | ASSERT_EQ(NO_ERROR, result); | 
|  | 641 | ASSERT_EQ(!b, notB); | 
|  | 642 | } | 
|  | 643 |  | 
| Dan Stoza | 81ea3ef | 2017-04-07 15:00:18 -0700 | [diff] [blame] | 644 | TEST_F(SafeInterfaceTest, TestModifyEnum) { | 
|  | 645 | const TestEnum a = TestEnum::INITIAL; | 
|  | 646 | TestEnum b = TestEnum::INVALID; | 
|  | 647 | status_t result = mSafeInterfaceTest->modifyEnum(a, &b); | 
|  | 648 | ASSERT_EQ(NO_ERROR, result); | 
|  | 649 | ASSERT_EQ(TestEnum::FINAL, b); | 
|  | 650 | } | 
|  | 651 |  | 
| Dan Stoza | df614ae | 2017-03-28 17:02:05 -0700 | [diff] [blame] | 652 | TEST_F(SafeInterfaceTest, TestIncrementFlattenable) { | 
|  | 653 | const TestFlattenable a{1}; | 
|  | 654 | TestFlattenable aPlusOne{0}; | 
|  | 655 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 656 | ASSERT_EQ(NO_ERROR, result); | 
|  | 657 | ASSERT_EQ(a.value + 1, aPlusOne.value); | 
|  | 658 | } | 
|  | 659 |  | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 660 | TEST_F(SafeInterfaceTest, TestIncrementLightFlattenable) { | 
|  | 661 | const TestLightFlattenable a{1}; | 
|  | 662 | TestLightFlattenable aPlusOne{0}; | 
|  | 663 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 664 | ASSERT_EQ(NO_ERROR, result); | 
|  | 665 | ASSERT_EQ(a.value + 1, aPlusOne.value); | 
|  | 666 | } | 
|  | 667 |  | 
| Dan Stoza | 6dd325b | 2017-04-07 14:31:51 -0700 | [diff] [blame] | 668 | TEST_F(SafeInterfaceTest, TestIncrementLightRefBaseFlattenable) { | 
|  | 669 | sp<TestLightRefBaseFlattenable> a = new TestLightRefBaseFlattenable{1}; | 
|  | 670 | sp<TestLightRefBaseFlattenable> aPlusOne; | 
|  | 671 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 672 | ASSERT_EQ(NO_ERROR, result); | 
|  | 673 | ASSERT_NE(nullptr, aPlusOne.get()); | 
|  | 674 | ASSERT_EQ(a->value + 1, aPlusOne->value); | 
|  | 675 | } | 
|  | 676 |  | 
| Dan Stoza | 2537db7 | 2017-04-07 16:32:38 -0700 | [diff] [blame] | 677 | namespace { // Anonymous namespace | 
|  | 678 |  | 
|  | 679 | bool fdsAreEquivalent(int a, int b) { | 
|  | 680 | struct stat statA {}; | 
|  | 681 | struct stat statB {}; | 
|  | 682 | if (fstat(a, &statA) != 0) return false; | 
|  | 683 | if (fstat(b, &statB) != 0) return false; | 
|  | 684 | return (statA.st_dev == statB.st_dev) && (statA.st_ino == statB.st_ino); | 
|  | 685 | } | 
|  | 686 |  | 
|  | 687 | } // Anonymous namespace | 
|  | 688 |  | 
|  | 689 | TEST_F(SafeInterfaceTest, TestIncrementNativeHandle) { | 
|  | 690 | // Create an fd we can use to send and receive from the remote process | 
|  | 691 | base::unique_fd eventFd{eventfd(0 /*initval*/, 0 /*flags*/)}; | 
|  | 692 | ASSERT_NE(-1, eventFd); | 
|  | 693 |  | 
|  | 694 | // Determine the maximum number of fds this process can have open | 
|  | 695 | struct rlimit limit {}; | 
|  | 696 | ASSERT_EQ(0, getrlimit(RLIMIT_NOFILE, &limit)); | 
|  | 697 | uint32_t maxFds = static_cast<uint32_t>(limit.rlim_cur); | 
|  | 698 |  | 
|  | 699 | // Perform this test enough times to rule out fd leaks | 
|  | 700 | for (uint32_t iter = 0; iter < (2 * maxFds); ++iter) { | 
|  | 701 | native_handle* handle = native_handle_create(1 /*numFds*/, 1 /*numInts*/); | 
|  | 702 | ASSERT_NE(nullptr, handle); | 
|  | 703 | handle->data[0] = dup(eventFd.get()); | 
|  | 704 | handle->data[1] = 1; | 
|  | 705 |  | 
|  | 706 | // This cannot fail, as it is just the sp<NativeHandle> taking responsibility for closing | 
|  | 707 | // the native_handle when it goes out of scope | 
|  | 708 | sp<NativeHandle> a = NativeHandle::create(handle, true); | 
|  | 709 |  | 
|  | 710 | sp<NativeHandle> aPlusOne; | 
|  | 711 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 712 | ASSERT_EQ(NO_ERROR, result); | 
|  | 713 | ASSERT_TRUE(fdsAreEquivalent(a->handle()->data[0], aPlusOne->handle()->data[0])); | 
|  | 714 | ASSERT_EQ(a->handle()->data[1] + 1, aPlusOne->handle()->data[1]); | 
|  | 715 | } | 
|  | 716 | } | 
|  | 717 |  | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 718 | TEST_F(SafeInterfaceTest, TestIncrementNoCopyNoMove) { | 
|  | 719 | const NoCopyNoMove a{1}; | 
|  | 720 | NoCopyNoMove aPlusOne{0}; | 
|  | 721 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 722 | ASSERT_EQ(NO_ERROR, result); | 
|  | 723 | ASSERT_EQ(a.getValue() + 1, aPlusOne.getValue()); | 
|  | 724 | } | 
|  | 725 |  | 
| Dan Stoza | 1af8a88 | 2017-04-10 13:28:54 -0700 | [diff] [blame] | 726 | TEST_F(SafeInterfaceTest, TestIncremementParcelableVector) { | 
|  | 727 | const std::vector<TestParcelable> a{TestParcelable{1}, TestParcelable{2}}; | 
|  | 728 | std::vector<TestParcelable> aPlusOne; | 
|  | 729 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 730 | ASSERT_EQ(a.size(), aPlusOne.size()); | 
|  | 731 | for (size_t i = 0; i < a.size(); ++i) { | 
|  | 732 | ASSERT_EQ(a[i].getValue() + 1, aPlusOne[i].getValue()); | 
|  | 733 | } | 
|  | 734 | } | 
|  | 735 |  | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 736 | TEST_F(SafeInterfaceTest, TestToUpper) { | 
|  | 737 | const String8 str{"Hello, world!"}; | 
|  | 738 | String8 upperStr; | 
|  | 739 | status_t result = mSafeInterfaceTest->toUpper(str, &upperStr); | 
|  | 740 | ASSERT_EQ(NO_ERROR, result); | 
|  | 741 | ASSERT_TRUE(upperStr == String8{"HELLO, WORLD!"}); | 
|  | 742 | } | 
|  | 743 |  | 
|  | 744 | TEST_F(SafeInterfaceTest, TestCallMeBack) { | 
|  | 745 | class CallbackReceiver : public BnCallback { | 
|  | 746 | public: | 
|  | 747 | void onCallback(int32_t aPlusOne) override { | 
|  | 748 | ALOG(LOG_INFO, "CallbackReceiver", "%s", __PRETTY_FUNCTION__); | 
|  | 749 | std::unique_lock<decltype(mMutex)> lock(mMutex); | 
|  | 750 | mValue = aPlusOne; | 
|  | 751 | mCondition.notify_one(); | 
|  | 752 | } | 
|  | 753 |  | 
|  | 754 | std::optional<int32_t> waitForCallback() { | 
|  | 755 | std::unique_lock<decltype(mMutex)> lock(mMutex); | 
|  | 756 | bool success = | 
|  | 757 | mCondition.wait_for(lock, 100ms, [&]() { return static_cast<bool>(mValue); }); | 
|  | 758 | return success ? mValue : std::nullopt; | 
|  | 759 | } | 
|  | 760 |  | 
|  | 761 | private: | 
|  | 762 | std::mutex mMutex; | 
|  | 763 | std::condition_variable mCondition; | 
|  | 764 | std::optional<int32_t> mValue; | 
|  | 765 | }; | 
|  | 766 |  | 
|  | 767 | sp<CallbackReceiver> receiver = new CallbackReceiver; | 
|  | 768 | const int32_t a = 1; | 
|  | 769 | mSafeInterfaceTest->callMeBack(receiver, a); | 
|  | 770 | auto result = receiver->waitForCallback(); | 
|  | 771 | ASSERT_TRUE(result); | 
|  | 772 | ASSERT_EQ(a + 1, *result); | 
|  | 773 | } | 
|  | 774 |  | 
|  | 775 | TEST_F(SafeInterfaceTest, TestIncrementInt32) { | 
|  | 776 | const int32_t a = 1; | 
|  | 777 | int32_t aPlusOne = 0; | 
|  | 778 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 779 | ASSERT_EQ(NO_ERROR, result); | 
|  | 780 | ASSERT_EQ(a + 1, aPlusOne); | 
|  | 781 | } | 
|  | 782 |  | 
|  | 783 | TEST_F(SafeInterfaceTest, TestIncrementUint32) { | 
|  | 784 | const uint32_t a = 1; | 
|  | 785 | uint32_t aPlusOne = 0; | 
|  | 786 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 787 | ASSERT_EQ(NO_ERROR, result); | 
|  | 788 | ASSERT_EQ(a + 1, aPlusOne); | 
|  | 789 | } | 
|  | 790 |  | 
| Dan Stoza | 662a899 | 2017-04-06 16:18:39 -0700 | [diff] [blame] | 791 | TEST_F(SafeInterfaceTest, TestIncrementInt64) { | 
|  | 792 | const int64_t a = 1; | 
|  | 793 | int64_t aPlusOne = 0; | 
|  | 794 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 795 | ASSERT_EQ(NO_ERROR, result); | 
|  | 796 | ASSERT_EQ(a + 1, aPlusOne); | 
|  | 797 | } | 
|  | 798 |  | 
|  | 799 | TEST_F(SafeInterfaceTest, TestIncrementUint64) { | 
|  | 800 | const uint64_t a = 1; | 
|  | 801 | uint64_t aPlusOne = 0; | 
|  | 802 | status_t result = mSafeInterfaceTest->increment(a, &aPlusOne); | 
|  | 803 | ASSERT_EQ(NO_ERROR, result); | 
|  | 804 | ASSERT_EQ(a + 1, aPlusOne); | 
|  | 805 | } | 
|  | 806 |  | 
| Dan Stoza | d630e52 | 2016-12-01 15:16:31 -0800 | [diff] [blame] | 807 | TEST_F(SafeInterfaceTest, TestIncrementTwo) { | 
|  | 808 | const int32_t a = 1; | 
|  | 809 | int32_t aPlusOne = 0; | 
|  | 810 | const int32_t b = 2; | 
|  | 811 | int32_t bPlusOne = 0; | 
|  | 812 | status_t result = mSafeInterfaceTest->increment(1, &aPlusOne, 2, &bPlusOne); | 
|  | 813 | ASSERT_EQ(NO_ERROR, result); | 
|  | 814 | ASSERT_EQ(a + 1, aPlusOne); | 
|  | 815 | ASSERT_EQ(b + 1, bPlusOne); | 
|  | 816 | } | 
|  | 817 |  | 
|  | 818 | } // namespace tests | 
|  | 819 | } // namespace android |