| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 1 | /* | 
|  | 2 | * Copyright (C) 2020 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 | #define LOG_TAG "RpcState" | 
|  | 18 |  | 
|  | 19 | #include "RpcState.h" | 
|  | 20 |  | 
| Steven Moreland | 6212901 | 2021-07-29 12:14:44 -0700 | [diff] [blame] | 21 | #include <android-base/hex.h> | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 22 | #include <android-base/macros.h> | 
| Steven Moreland | d730207 | 2021-05-15 01:32:04 +0000 | [diff] [blame] | 23 | #include <android-base/scopeguard.h> | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 24 | #include <binder/BpBinder.h> | 
| Steven Moreland | d730207 | 2021-05-15 01:32:04 +0000 | [diff] [blame] | 25 | #include <binder/IPCThreadState.h> | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 26 | #include <binder/RpcServer.h> | 
|  | 27 |  | 
|  | 28 | #include "Debug.h" | 
|  | 29 | #include "RpcWireFormat.h" | 
|  | 30 |  | 
| Steven Moreland | b817679 | 2021-06-22 20:29:21 +0000 | [diff] [blame] | 31 | #include <random> | 
|  | 32 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 33 | #include <inttypes.h> | 
|  | 34 |  | 
|  | 35 | namespace android { | 
|  | 36 |  | 
| Steven Moreland | d730207 | 2021-05-15 01:32:04 +0000 | [diff] [blame] | 37 | using base::ScopeGuard; | 
|  | 38 |  | 
| Devin Moore | 0825643 | 2021-07-02 13:03:49 -0700 | [diff] [blame] | 39 | #if RPC_FLAKE_PRONE | 
| Steven Moreland | b817679 | 2021-06-22 20:29:21 +0000 | [diff] [blame] | 40 | void rpcMaybeWaitToFlake() { | 
| Devin Moore | 0825643 | 2021-07-02 13:03:49 -0700 | [diff] [blame] | 41 | [[clang::no_destroy]] static std::random_device r; | 
|  | 42 | [[clang::no_destroy]] static std::mutex m; | 
| Steven Moreland | b817679 | 2021-06-22 20:29:21 +0000 | [diff] [blame] | 43 | unsigned num; | 
|  | 44 | { | 
|  | 45 | std::lock_guard<std::mutex> lock(m); | 
|  | 46 | num = r(); | 
|  | 47 | } | 
|  | 48 | if (num % 10 == 0) usleep(num % 1000); | 
|  | 49 | } | 
|  | 50 | #endif | 
|  | 51 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 52 | RpcState::RpcState() {} | 
|  | 53 | RpcState::~RpcState() {} | 
|  | 54 |  | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 55 | status_t RpcState::onBinderLeaving(const sp<RpcSession>& session, const sp<IBinder>& binder, | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 56 | uint64_t* outAddress) { | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 57 | bool isRemote = binder->remoteBinder(); | 
|  | 58 | bool isRpc = isRemote && binder->remoteBinder()->isRpcBinder(); | 
|  | 59 |  | 
| Steven Moreland | 9915762 | 2021-09-13 16:27:34 -0700 | [diff] [blame] | 60 | if (isRpc && binder->remoteBinder()->getPrivateAccessor().rpcSession() != session) { | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 61 | // We need to be able to send instructions over the socket for how to | 
|  | 62 | // connect to a different server, and we also need to let the host | 
|  | 63 | // process know that this is happening. | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 64 | ALOGE("Cannot send binder from unrelated binder RPC session."); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 65 | return INVALID_OPERATION; | 
|  | 66 | } | 
|  | 67 |  | 
|  | 68 | if (isRemote && !isRpc) { | 
|  | 69 | // Without additional work, this would have the effect of using this | 
|  | 70 | // process to proxy calls from the socket over to the other process, and | 
|  | 71 | // it would make those calls look like they come from us (not over the | 
|  | 72 | // sockets). In order to make this work transparently like binder, we | 
|  | 73 | // would instead need to send instructions over the socket for how to | 
|  | 74 | // connect to the host process, and we also need to let the host process | 
|  | 75 | // know this was happening. | 
|  | 76 | ALOGE("Cannot send binder proxy %p over sockets", binder.get()); | 
|  | 77 | return INVALID_OPERATION; | 
|  | 78 | } | 
|  | 79 |  | 
|  | 80 | std::lock_guard<std::mutex> _l(mNodeMutex); | 
| Steven Moreland | 7227c8a | 2021-06-02 00:24:32 +0000 | [diff] [blame] | 81 | if (mTerminated) return DEAD_OBJECT; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 82 |  | 
|  | 83 | // TODO(b/182939933): maybe move address out of BpBinder, and keep binder->address map | 
|  | 84 | // in RpcState | 
|  | 85 | for (auto& [addr, node] : mNodeForAddress) { | 
|  | 86 | if (binder == node.binder) { | 
|  | 87 | if (isRpc) { | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 88 | // check integrity of data structure | 
| Steven Moreland | 9915762 | 2021-09-13 16:27:34 -0700 | [diff] [blame] | 89 | uint64_t actualAddr = binder->remoteBinder()->getPrivateAccessor().rpcAddress(); | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 90 | LOG_ALWAYS_FATAL_IF(addr != actualAddr, "Address mismatch %" PRIu64 " vs %" PRIu64, | 
|  | 91 | addr, actualAddr); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 92 | } | 
|  | 93 | node.timesSent++; | 
|  | 94 | node.sentRef = binder; // might already be set | 
|  | 95 | *outAddress = addr; | 
|  | 96 | return OK; | 
|  | 97 | } | 
|  | 98 | } | 
|  | 99 | LOG_ALWAYS_FATAL_IF(isRpc, "RPC binder must have known address at this point"); | 
|  | 100 |  | 
| Steven Moreland | 9153824 | 2021-06-10 23:35:35 +0000 | [diff] [blame] | 101 | bool forServer = session->server() != nullptr; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 102 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 103 | // arbitrary limit for maximum number of nodes in a process (otherwise we | 
|  | 104 | // might run out of addresses) | 
|  | 105 | if (mNodeForAddress.size() > 100000) { | 
|  | 106 | return NO_MEMORY; | 
|  | 107 | } | 
|  | 108 |  | 
|  | 109 | while (true) { | 
|  | 110 | RpcWireAddress address{ | 
|  | 111 | .options = RPC_WIRE_ADDRESS_OPTION_CREATED, | 
|  | 112 | .address = mNextId, | 
|  | 113 | }; | 
|  | 114 | if (forServer) { | 
|  | 115 | address.options |= RPC_WIRE_ADDRESS_OPTION_FOR_SERVER; | 
|  | 116 | } | 
|  | 117 |  | 
|  | 118 | // avoid ubsan abort | 
|  | 119 | if (mNextId >= std::numeric_limits<uint32_t>::max()) { | 
|  | 120 | mNextId = 0; | 
|  | 121 | } else { | 
|  | 122 | mNextId++; | 
|  | 123 | } | 
|  | 124 |  | 
|  | 125 | auto&& [it, inserted] = mNodeForAddress.insert({RpcWireAddress::toRaw(address), | 
| Steven Moreland | 9153824 | 2021-06-10 23:35:35 +0000 | [diff] [blame] | 126 | BinderNode{ | 
|  | 127 | .binder = binder, | 
| Steven Moreland | 9153824 | 2021-06-10 23:35:35 +0000 | [diff] [blame] | 128 | .sentRef = binder, | 
| Andrei Homescu | 5a036f3 | 2022-03-08 22:54:40 +0000 | [diff] [blame] | 129 | .timesSent = 1, | 
| Steven Moreland | 9153824 | 2021-06-10 23:35:35 +0000 | [diff] [blame] | 130 | }}); | 
|  | 131 | if (inserted) { | 
|  | 132 | *outAddress = it->first; | 
|  | 133 | return OK; | 
|  | 134 | } | 
| Steven Moreland | 9153824 | 2021-06-10 23:35:35 +0000 | [diff] [blame] | 135 | } | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 136 | } | 
|  | 137 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 138 | status_t RpcState::onBinderEntering(const sp<RpcSession>& session, uint64_t address, | 
| Steven Moreland | 7227c8a | 2021-06-02 00:24:32 +0000 | [diff] [blame] | 139 | sp<IBinder>* out) { | 
| Steven Moreland | 9153824 | 2021-06-10 23:35:35 +0000 | [diff] [blame] | 140 | // ensure that: if we want to use addresses for something else in the future (for | 
|  | 141 | //   instance, allowing transitive binder sends), that we don't accidentally | 
|  | 142 | //   send those addresses to old server. Accidentally ignoring this in that | 
|  | 143 | //   case and considering the binder to be recognized could cause this | 
|  | 144 | //   process to accidentally proxy transactions for that binder. Of course, | 
|  | 145 | //   if we communicate with a binder, it could always be proxying | 
|  | 146 | //   information. However, we want to make sure that isn't done on accident | 
|  | 147 | //   by a client. | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 148 | RpcWireAddress addr = RpcWireAddress::fromRaw(address); | 
|  | 149 | constexpr uint32_t kKnownOptions = | 
|  | 150 | RPC_WIRE_ADDRESS_OPTION_CREATED | RPC_WIRE_ADDRESS_OPTION_FOR_SERVER; | 
|  | 151 | if (addr.options & ~kKnownOptions) { | 
|  | 152 | ALOGE("Address is of an unknown type, rejecting: %" PRIu64, address); | 
| Steven Moreland | 9153824 | 2021-06-10 23:35:35 +0000 | [diff] [blame] | 153 | return BAD_VALUE; | 
|  | 154 | } | 
|  | 155 |  | 
| Steven Moreland | d808331 | 2021-09-22 13:37:10 -0700 | [diff] [blame] | 156 | std::lock_guard<std::mutex> _l(mNodeMutex); | 
| Steven Moreland | 7227c8a | 2021-06-02 00:24:32 +0000 | [diff] [blame] | 157 | if (mTerminated) return DEAD_OBJECT; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 158 |  | 
|  | 159 | if (auto it = mNodeForAddress.find(address); it != mNodeForAddress.end()) { | 
| Steven Moreland | 7227c8a | 2021-06-02 00:24:32 +0000 | [diff] [blame] | 160 | *out = it->second.binder.promote(); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 161 |  | 
|  | 162 | // implicitly have strong RPC refcount, since we received this binder | 
|  | 163 | it->second.timesRecd++; | 
| Steven Moreland | d808331 | 2021-09-22 13:37:10 -0700 | [diff] [blame] | 164 | return OK; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 165 | } | 
|  | 166 |  | 
| Steven Moreland | 9153824 | 2021-06-10 23:35:35 +0000 | [diff] [blame] | 167 | // we don't know about this binder, so the other side of the connection | 
|  | 168 | // should have created it. | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 169 | if ((addr.options & RPC_WIRE_ADDRESS_OPTION_FOR_SERVER) == !!session->server()) { | 
|  | 170 | ALOGE("Server received unrecognized address which we should own the creation of %" PRIu64, | 
|  | 171 | address); | 
| Steven Moreland | 9153824 | 2021-06-10 23:35:35 +0000 | [diff] [blame] | 172 | return BAD_VALUE; | 
|  | 173 | } | 
|  | 174 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 175 | auto&& [it, inserted] = mNodeForAddress.insert({address, BinderNode{}}); | 
|  | 176 | LOG_ALWAYS_FATAL_IF(!inserted, "Failed to insert binder when creating proxy"); | 
|  | 177 |  | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 178 | // Currently, all binders are assumed to be part of the same session (no | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 179 | // device global binders in the RPC world). | 
| Steven Moreland | 9915762 | 2021-09-13 16:27:34 -0700 | [diff] [blame] | 180 | it->second.binder = *out = BpBinder::PrivateAccessor::create(session, it->first); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 181 | it->second.timesRecd = 1; | 
| Steven Moreland | 7227c8a | 2021-06-02 00:24:32 +0000 | [diff] [blame] | 182 | return OK; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 183 | } | 
|  | 184 |  | 
| Steven Moreland | d808331 | 2021-09-22 13:37:10 -0700 | [diff] [blame] | 185 | status_t RpcState::flushExcessBinderRefs(const sp<RpcSession>& session, uint64_t address, | 
|  | 186 | const sp<IBinder>& binder) { | 
| Steven Moreland | e96ed0e | 2021-09-27 17:43:53 -0700 | [diff] [blame] | 187 | // We can flush all references when the binder is destroyed. No need to send | 
|  | 188 | // extra reference counting packets now. | 
|  | 189 | if (binder->remoteBinder()) return OK; | 
|  | 190 |  | 
| Steven Moreland | d808331 | 2021-09-22 13:37:10 -0700 | [diff] [blame] | 191 | std::unique_lock<std::mutex> _l(mNodeMutex); | 
|  | 192 | if (mTerminated) return DEAD_OBJECT; | 
|  | 193 |  | 
|  | 194 | auto it = mNodeForAddress.find(address); | 
|  | 195 |  | 
|  | 196 | LOG_ALWAYS_FATAL_IF(it == mNodeForAddress.end(), "Can't be deleted while we hold sp<>"); | 
|  | 197 | LOG_ALWAYS_FATAL_IF(it->second.binder != binder, | 
|  | 198 | "Caller of flushExcessBinderRefs using inconsistent arguments"); | 
|  | 199 |  | 
| Steven Moreland | e96ed0e | 2021-09-27 17:43:53 -0700 | [diff] [blame] | 200 | LOG_ALWAYS_FATAL_IF(it->second.timesSent <= 0, "Local binder must have been sent %p", | 
|  | 201 | binder.get()); | 
| Steven Moreland | d808331 | 2021-09-22 13:37:10 -0700 | [diff] [blame] | 202 |  | 
| Steven Moreland | e96ed0e | 2021-09-27 17:43:53 -0700 | [diff] [blame] | 203 | // For a local binder, we only need to know that we sent it. Now that we | 
|  | 204 | // have an sp<> for this call, we don't need anything more. If the other | 
|  | 205 | // process is done with this binder, it needs to know we received the | 
|  | 206 | // refcount associated with this call, so we can acknowledge that we | 
|  | 207 | // received it. Once (or if) it has no other refcounts, it would reply with | 
|  | 208 | // its own decStrong so that it could be removed from this session. | 
|  | 209 | if (it->second.timesRecd != 0) { | 
| Steven Moreland | d808331 | 2021-09-22 13:37:10 -0700 | [diff] [blame] | 210 | _l.unlock(); | 
|  | 211 |  | 
| Steven Moreland | e96ed0e | 2021-09-27 17:43:53 -0700 | [diff] [blame] | 212 | return session->sendDecStrongToTarget(address, 0); | 
| Steven Moreland | d808331 | 2021-09-22 13:37:10 -0700 | [diff] [blame] | 213 | } | 
|  | 214 |  | 
|  | 215 | return OK; | 
|  | 216 | } | 
|  | 217 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 218 | size_t RpcState::countBinders() { | 
|  | 219 | std::lock_guard<std::mutex> _l(mNodeMutex); | 
|  | 220 | return mNodeForAddress.size(); | 
|  | 221 | } | 
|  | 222 |  | 
|  | 223 | void RpcState::dump() { | 
|  | 224 | std::lock_guard<std::mutex> _l(mNodeMutex); | 
| Steven Moreland | 583a14a | 2021-06-04 02:04:58 +0000 | [diff] [blame] | 225 | dumpLocked(); | 
|  | 226 | } | 
|  | 227 |  | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 228 | void RpcState::clear() { | 
| Steven Moreland | 583a14a | 2021-06-04 02:04:58 +0000 | [diff] [blame] | 229 | std::unique_lock<std::mutex> _l(mNodeMutex); | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 230 |  | 
|  | 231 | if (mTerminated) { | 
|  | 232 | LOG_ALWAYS_FATAL_IF(!mNodeForAddress.empty(), | 
|  | 233 | "New state should be impossible after terminating!"); | 
|  | 234 | return; | 
|  | 235 | } | 
|  | 236 |  | 
|  | 237 | if (SHOULD_LOG_RPC_DETAIL) { | 
|  | 238 | ALOGE("RpcState::clear()"); | 
|  | 239 | dumpLocked(); | 
|  | 240 | } | 
|  | 241 |  | 
|  | 242 | // if the destructor of a binder object makes another RPC call, then calling | 
|  | 243 | // decStrong could deadlock. So, we must hold onto these binders until | 
|  | 244 | // mNodeMutex is no longer taken. | 
|  | 245 | std::vector<sp<IBinder>> tempHoldBinder; | 
|  | 246 |  | 
|  | 247 | mTerminated = true; | 
|  | 248 | for (auto& [address, node] : mNodeForAddress) { | 
|  | 249 | sp<IBinder> binder = node.binder.promote(); | 
|  | 250 | LOG_ALWAYS_FATAL_IF(binder == nullptr, "Binder %p expected to be owned.", binder.get()); | 
|  | 251 |  | 
|  | 252 | if (node.sentRef != nullptr) { | 
|  | 253 | tempHoldBinder.push_back(node.sentRef); | 
|  | 254 | } | 
|  | 255 | } | 
|  | 256 |  | 
|  | 257 | mNodeForAddress.clear(); | 
|  | 258 |  | 
|  | 259 | _l.unlock(); | 
|  | 260 | tempHoldBinder.clear(); // explicit | 
| Steven Moreland | 583a14a | 2021-06-04 02:04:58 +0000 | [diff] [blame] | 261 | } | 
|  | 262 |  | 
|  | 263 | void RpcState::dumpLocked() { | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 264 | ALOGE("DUMP OF RpcState %p", this); | 
|  | 265 | ALOGE("DUMP OF RpcState (%zu nodes)", mNodeForAddress.size()); | 
|  | 266 | for (const auto& [address, node] : mNodeForAddress) { | 
|  | 267 | sp<IBinder> binder = node.binder.promote(); | 
|  | 268 |  | 
|  | 269 | const char* desc; | 
|  | 270 | if (binder) { | 
|  | 271 | if (binder->remoteBinder()) { | 
|  | 272 | if (binder->remoteBinder()->isRpcBinder()) { | 
|  | 273 | desc = "(rpc binder proxy)"; | 
|  | 274 | } else { | 
|  | 275 | desc = "(binder proxy)"; | 
|  | 276 | } | 
|  | 277 | } else { | 
|  | 278 | desc = "(local binder)"; | 
|  | 279 | } | 
|  | 280 | } else { | 
|  | 281 | desc = "(null)"; | 
|  | 282 | } | 
|  | 283 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 284 | ALOGE("- BINDER NODE: %p times sent:%zu times recd: %zu a: %" PRIu64 " type: %s", | 
|  | 285 | node.binder.unsafe_get(), node.timesSent, node.timesRecd, address, desc); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 286 | } | 
|  | 287 | ALOGE("END DUMP OF RpcState"); | 
|  | 288 | } | 
|  | 289 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 290 |  | 
| Steven Moreland | dbe7183 | 2021-05-12 23:31:00 +0000 | [diff] [blame] | 291 | RpcState::CommandData::CommandData(size_t size) : mSize(size) { | 
|  | 292 | // The maximum size for regular binder is 1MB for all concurrent | 
|  | 293 | // transactions. A very small proportion of transactions are even | 
|  | 294 | // larger than a page, but we need to avoid allocating too much | 
|  | 295 | // data on behalf of an arbitrary client, or we could risk being in | 
|  | 296 | // a position where a single additional allocation could run out of | 
|  | 297 | // memory. | 
|  | 298 | // | 
|  | 299 | // Note, this limit may not reflect the total amount of data allocated for a | 
|  | 300 | // transaction (in some cases, additional fixed size amounts are added), | 
|  | 301 | // though for rough consistency, we should avoid cases where this data type | 
|  | 302 | // is used for multiple dynamic allocations for a single transaction. | 
|  | 303 | constexpr size_t kMaxTransactionAllocation = 100 * 1000; | 
|  | 304 | if (size == 0) return; | 
|  | 305 | if (size > kMaxTransactionAllocation) { | 
|  | 306 | ALOGW("Transaction requested too much data allocation %zu", size); | 
|  | 307 | return; | 
|  | 308 | } | 
|  | 309 | mData.reset(new (std::nothrow) uint8_t[size]); | 
|  | 310 | } | 
|  | 311 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 312 | status_t RpcState::rpcSend(const sp<RpcSession::RpcConnection>& connection, | 
| Colin Cross | 9adfeaf | 2022-01-21 17:22:09 -0800 | [diff] [blame] | 313 | const sp<RpcSession>& session, const char* what, iovec* iovs, int niovs, | 
|  | 314 | const std::function<status_t()>& altPoll) { | 
|  | 315 | for (int i = 0; i < niovs; i++) { | 
| Andrei Homescu | 0a69235 | 2022-03-29 06:04:26 +0000 | [diff] [blame] | 316 | LOG_RPC_DETAIL("Sending %s (part %d of %d) on RpcTransport %p: %s", | 
|  | 317 | what, i + 1, niovs, connection->rpcTransport.get(), | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 318 | android::base::HexString(iovs[i].iov_base, iovs[i].iov_len).c_str()); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 319 | } | 
|  | 320 |  | 
| Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 321 | if (status_t status = | 
| Yifan Hong | 8c95042 | 2021-08-05 17:13:55 -0700 | [diff] [blame] | 322 | connection->rpcTransport->interruptableWriteFully(session->mShutdownTrigger.get(), | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 323 | iovs, niovs, altPoll); | 
| Steven Moreland | 798e0d1 | 2021-07-14 23:19:25 +0000 | [diff] [blame] | 324 | status != OK) { | 
| Colin Cross | 9adfeaf | 2022-01-21 17:22:09 -0800 | [diff] [blame] | 325 | LOG_RPC_DETAIL("Failed to write %s (%d iovs) on RpcTransport %p, error: %s", what, niovs, | 
| Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 326 | connection->rpcTransport.get(), statusToString(status).c_str()); | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 327 | (void)session->shutdownAndWait(false); | 
| Steven Moreland | 798e0d1 | 2021-07-14 23:19:25 +0000 | [diff] [blame] | 328 | return status; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 329 | } | 
|  | 330 |  | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 331 | return OK; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 332 | } | 
|  | 333 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 334 | status_t RpcState::rpcRec(const sp<RpcSession::RpcConnection>& connection, | 
| Colin Cross | 9adfeaf | 2022-01-21 17:22:09 -0800 | [diff] [blame] | 335 | const sp<RpcSession>& session, const char* what, iovec* iovs, int niovs) { | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 336 | if (status_t status = | 
| Yifan Hong | 8c95042 | 2021-08-05 17:13:55 -0700 | [diff] [blame] | 337 | connection->rpcTransport->interruptableReadFully(session->mShutdownTrigger.get(), | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 338 | iovs, niovs, {}); | 
| Steven Moreland | ee3f466 | 2021-05-22 01:07:33 +0000 | [diff] [blame] | 339 | status != OK) { | 
| Colin Cross | 9adfeaf | 2022-01-21 17:22:09 -0800 | [diff] [blame] | 340 | LOG_RPC_DETAIL("Failed to read %s (%d iovs) on RpcTransport %p, error: %s", what, niovs, | 
| Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 341 | connection->rpcTransport.get(), statusToString(status).c_str()); | 
| Steven Moreland | ae58f43 | 2021-08-05 17:53:16 -0700 | [diff] [blame] | 342 | (void)session->shutdownAndWait(false); | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 343 | return status; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 344 | } | 
|  | 345 |  | 
| Colin Cross | 9adfeaf | 2022-01-21 17:22:09 -0800 | [diff] [blame] | 346 | for (int i = 0; i < niovs; i++) { | 
| Andrei Homescu | 0a69235 | 2022-03-29 06:04:26 +0000 | [diff] [blame] | 347 | LOG_RPC_DETAIL("Received %s (part %d of %d) on RpcTransport %p: %s", | 
|  | 348 | what, i + 1, niovs, connection->rpcTransport.get(), | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 349 | android::base::HexString(iovs[i].iov_base, iovs[i].iov_len).c_str()); | 
|  | 350 | } | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 351 | return OK; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 352 | } | 
|  | 353 |  | 
| Steven Moreland | bf57bce | 2021-07-26 15:26:12 -0700 | [diff] [blame] | 354 | status_t RpcState::readNewSessionResponse(const sp<RpcSession::RpcConnection>& connection, | 
|  | 355 | const sp<RpcSession>& session, uint32_t* version) { | 
|  | 356 | RpcNewSessionResponse response; | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 357 | iovec iov{&response, sizeof(response)}; | 
|  | 358 | if (status_t status = rpcRec(connection, session, "new session response", &iov, 1); | 
| Steven Moreland | bf57bce | 2021-07-26 15:26:12 -0700 | [diff] [blame] | 359 | status != OK) { | 
|  | 360 | return status; | 
|  | 361 | } | 
|  | 362 | *version = response.version; | 
|  | 363 | return OK; | 
|  | 364 | } | 
|  | 365 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 366 | status_t RpcState::sendConnectionInit(const sp<RpcSession::RpcConnection>& connection, | 
|  | 367 | const sp<RpcSession>& session) { | 
| Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 368 | RpcOutgoingConnectionInit init{ | 
| Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 369 | .msg = RPC_CONNECTION_INIT_OKAY, | 
|  | 370 | }; | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 371 | iovec iov{&init, sizeof(init)}; | 
|  | 372 | return rpcSend(connection, session, "connection init", &iov, 1); | 
| Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 373 | } | 
|  | 374 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 375 | status_t RpcState::readConnectionInit(const sp<RpcSession::RpcConnection>& connection, | 
|  | 376 | const sp<RpcSession>& session) { | 
| Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 377 | RpcOutgoingConnectionInit init; | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 378 | iovec iov{&init, sizeof(init)}; | 
|  | 379 | if (status_t status = rpcRec(connection, session, "connection init", &iov, 1); status != OK) | 
| Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 380 | return status; | 
|  | 381 |  | 
|  | 382 | static_assert(sizeof(init.msg) == sizeof(RPC_CONNECTION_INIT_OKAY)); | 
|  | 383 | if (0 != strncmp(init.msg, RPC_CONNECTION_INIT_OKAY, sizeof(init.msg))) { | 
|  | 384 | ALOGE("Connection init message unrecognized %.*s", static_cast<int>(sizeof(init.msg)), | 
|  | 385 | init.msg); | 
|  | 386 | return BAD_VALUE; | 
|  | 387 | } | 
|  | 388 | return OK; | 
|  | 389 | } | 
|  | 390 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 391 | sp<IBinder> RpcState::getRootObject(const sp<RpcSession::RpcConnection>& connection, | 
|  | 392 | const sp<RpcSession>& session) { | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 393 | Parcel data; | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 394 | data.markForRpc(session); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 395 | Parcel reply; | 
|  | 396 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 397 | status_t status = | 
|  | 398 | transactAddress(connection, 0, RPC_SPECIAL_TRANSACT_GET_ROOT, data, session, &reply, 0); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 399 | if (status != OK) { | 
|  | 400 | ALOGE("Error getting root object: %s", statusToString(status).c_str()); | 
|  | 401 | return nullptr; | 
|  | 402 | } | 
|  | 403 |  | 
|  | 404 | return reply.readStrongBinder(); | 
|  | 405 | } | 
|  | 406 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 407 | status_t RpcState::getMaxThreads(const sp<RpcSession::RpcConnection>& connection, | 
|  | 408 | const sp<RpcSession>& session, size_t* maxThreadsOut) { | 
| Steven Moreland | f137de9 | 2021-04-24 01:54:26 +0000 | [diff] [blame] | 409 | Parcel data; | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 410 | data.markForRpc(session); | 
| Steven Moreland | f137de9 | 2021-04-24 01:54:26 +0000 | [diff] [blame] | 411 | Parcel reply; | 
|  | 412 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 413 | status_t status = transactAddress(connection, 0, RPC_SPECIAL_TRANSACT_GET_MAX_THREADS, data, | 
|  | 414 | session, &reply, 0); | 
| Steven Moreland | f137de9 | 2021-04-24 01:54:26 +0000 | [diff] [blame] | 415 | if (status != OK) { | 
|  | 416 | ALOGE("Error getting max threads: %s", statusToString(status).c_str()); | 
|  | 417 | return status; | 
|  | 418 | } | 
|  | 419 |  | 
| Steven Moreland | 7c5e6c2 | 2021-05-01 02:55:20 +0000 | [diff] [blame] | 420 | int32_t maxThreads; | 
|  | 421 | status = reply.readInt32(&maxThreads); | 
| Steven Moreland | f137de9 | 2021-04-24 01:54:26 +0000 | [diff] [blame] | 422 | if (status != OK) return status; | 
| Steven Moreland | 7c5e6c2 | 2021-05-01 02:55:20 +0000 | [diff] [blame] | 423 | if (maxThreads <= 0) { | 
|  | 424 | ALOGE("Error invalid max maxThreads: %d", maxThreads); | 
| Steven Moreland | f137de9 | 2021-04-24 01:54:26 +0000 | [diff] [blame] | 425 | return BAD_VALUE; | 
|  | 426 | } | 
|  | 427 |  | 
| Steven Moreland | 7c5e6c2 | 2021-05-01 02:55:20 +0000 | [diff] [blame] | 428 | *maxThreadsOut = maxThreads; | 
|  | 429 | return OK; | 
|  | 430 | } | 
|  | 431 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 432 | status_t RpcState::getSessionId(const sp<RpcSession::RpcConnection>& connection, | 
| Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame] | 433 | const sp<RpcSession>& session, std::vector<uint8_t>* sessionIdOut) { | 
| Steven Moreland | 7c5e6c2 | 2021-05-01 02:55:20 +0000 | [diff] [blame] | 434 | Parcel data; | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 435 | data.markForRpc(session); | 
| Steven Moreland | 7c5e6c2 | 2021-05-01 02:55:20 +0000 | [diff] [blame] | 436 | Parcel reply; | 
|  | 437 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 438 | status_t status = transactAddress(connection, 0, RPC_SPECIAL_TRANSACT_GET_SESSION_ID, data, | 
|  | 439 | session, &reply, 0); | 
| Steven Moreland | 7c5e6c2 | 2021-05-01 02:55:20 +0000 | [diff] [blame] | 440 | if (status != OK) { | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 441 | ALOGE("Error getting session ID: %s", statusToString(status).c_str()); | 
| Steven Moreland | 7c5e6c2 | 2021-05-01 02:55:20 +0000 | [diff] [blame] | 442 | return status; | 
|  | 443 | } | 
|  | 444 |  | 
| Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame] | 445 | return reply.readByteVector(sessionIdOut); | 
| Steven Moreland | f137de9 | 2021-04-24 01:54:26 +0000 | [diff] [blame] | 446 | } | 
|  | 447 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 448 | status_t RpcState::transact(const sp<RpcSession::RpcConnection>& connection, | 
|  | 449 | const sp<IBinder>& binder, uint32_t code, const Parcel& data, | 
|  | 450 | const sp<RpcSession>& session, Parcel* reply, uint32_t flags) { | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 451 | if (!data.isForRpc()) { | 
| Steven Moreland | caf14b2 | 2021-09-27 18:18:35 -0700 | [diff] [blame] | 452 | ALOGE("Refusing to send RPC with parcel not crafted for RPC call on binder %p code " | 
|  | 453 | "%" PRIu32, | 
|  | 454 | binder.get(), code); | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 455 | return BAD_TYPE; | 
|  | 456 | } | 
|  | 457 |  | 
|  | 458 | if (data.objectsCount() != 0) { | 
| Steven Moreland | caf14b2 | 2021-09-27 18:18:35 -0700 | [diff] [blame] | 459 | ALOGE("Parcel at %p has attached objects but is being used in an RPC call on binder %p " | 
|  | 460 | "code %" PRIu32, | 
|  | 461 | &data, binder.get(), code); | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 462 | return BAD_TYPE; | 
|  | 463 | } | 
|  | 464 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 465 | uint64_t address; | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 466 | if (status_t status = onBinderLeaving(session, binder, &address); status != OK) return status; | 
|  | 467 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 468 | return transactAddress(connection, address, code, data, session, reply, flags); | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 469 | } | 
|  | 470 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 471 | status_t RpcState::transactAddress(const sp<RpcSession::RpcConnection>& connection, | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 472 | uint64_t address, uint32_t code, const Parcel& data, | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 473 | const sp<RpcSession>& session, Parcel* reply, uint32_t flags) { | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 474 | LOG_ALWAYS_FATAL_IF(!data.isForRpc()); | 
|  | 475 | LOG_ALWAYS_FATAL_IF(data.objectsCount() != 0); | 
|  | 476 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 477 | uint64_t asyncNumber = 0; | 
|  | 478 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 479 | if (address != 0) { | 
| Steven Moreland | 583a14a | 2021-06-04 02:04:58 +0000 | [diff] [blame] | 480 | std::unique_lock<std::mutex> _l(mNodeMutex); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 481 | if (mTerminated) return DEAD_OBJECT; // avoid fatal only, otherwise races | 
|  | 482 | auto it = mNodeForAddress.find(address); | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 483 | LOG_ALWAYS_FATAL_IF(it == mNodeForAddress.end(), | 
|  | 484 | "Sending transact on unknown address %" PRIu64, address); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 485 |  | 
|  | 486 | if (flags & IBinder::FLAG_ONEWAY) { | 
| Steven Moreland | 583a14a | 2021-06-04 02:04:58 +0000 | [diff] [blame] | 487 | asyncNumber = it->second.asyncNumber; | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 488 | if (!nodeProgressAsyncNumber(&it->second)) { | 
|  | 489 | _l.unlock(); | 
|  | 490 | (void)session->shutdownAndWait(false); | 
|  | 491 | return DEAD_OBJECT; | 
|  | 492 | } | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 493 | } | 
|  | 494 | } | 
|  | 495 |  | 
| Steven Moreland | 77c3011 | 2021-06-02 20:45:46 +0000 | [diff] [blame] | 496 | LOG_ALWAYS_FATAL_IF(std::numeric_limits<int32_t>::max() - sizeof(RpcWireHeader) - | 
|  | 497 | sizeof(RpcWireTransaction) < | 
|  | 498 | data.dataSize(), | 
|  | 499 | "Too much data %zu", data.dataSize()); | 
|  | 500 |  | 
|  | 501 | RpcWireHeader command{ | 
|  | 502 | .command = RPC_COMMAND_TRANSACT, | 
|  | 503 | .bodySize = static_cast<uint32_t>(sizeof(RpcWireTransaction) + data.dataSize()), | 
|  | 504 | }; | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 505 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 506 | RpcWireTransaction transaction{ | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 507 | .address = RpcWireAddress::fromRaw(address), | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 508 | .code = code, | 
|  | 509 | .flags = flags, | 
|  | 510 | .asyncNumber = asyncNumber, | 
|  | 511 | }; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 512 |  | 
| Steven Moreland | 43921d5 | 2021-09-27 17:15:56 -0700 | [diff] [blame] | 513 | constexpr size_t kWaitMaxUs = 1000000; | 
|  | 514 | constexpr size_t kWaitLogUs = 10000; | 
|  | 515 | size_t waitUs = 0; | 
|  | 516 |  | 
|  | 517 | // Oneway calls have no sync point, so if many are sent before, whether this | 
|  | 518 | // is a twoway or oneway transaction, they may have filled up the socket. | 
|  | 519 | // So, make sure we drain them before polling. | 
|  | 520 | std::function<status_t()> drainRefs = [&] { | 
|  | 521 | if (waitUs > kWaitLogUs) { | 
|  | 522 | ALOGE("Cannot send command, trying to process pending refcounts. Waiting %zuus. Too " | 
|  | 523 | "many oneway calls?", | 
|  | 524 | waitUs); | 
|  | 525 | } | 
|  | 526 |  | 
|  | 527 | if (waitUs > 0) { | 
|  | 528 | usleep(waitUs); | 
|  | 529 | waitUs = std::min(kWaitMaxUs, waitUs * 2); | 
|  | 530 | } else { | 
|  | 531 | waitUs = 1; | 
|  | 532 | } | 
|  | 533 |  | 
|  | 534 | return drainCommands(connection, session, CommandType::CONTROL_ONLY); | 
|  | 535 | }; | 
|  | 536 |  | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 537 | iovec iovs[]{ | 
|  | 538 | {&command, sizeof(RpcWireHeader)}, | 
|  | 539 | {&transaction, sizeof(RpcWireTransaction)}, | 
|  | 540 | {const_cast<uint8_t*>(data.data()), data.dataSize()}, | 
|  | 541 | }; | 
|  | 542 | if (status_t status = | 
|  | 543 | rpcSend(connection, session, "transaction", iovs, arraysize(iovs), drainRefs); | 
| Steven Moreland | 43921d5 | 2021-09-27 17:15:56 -0700 | [diff] [blame] | 544 | status != OK) { | 
| Steven Moreland | a5036f0 | 2021-06-08 02:26:57 +0000 | [diff] [blame] | 545 | // TODO(b/167966510): need to undo onBinderLeaving - we know the | 
|  | 546 | // refcount isn't successfully transferred. | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 547 | return status; | 
| Steven Moreland | 43921d5 | 2021-09-27 17:15:56 -0700 | [diff] [blame] | 548 | } | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 549 |  | 
|  | 550 | if (flags & IBinder::FLAG_ONEWAY) { | 
| Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 551 | LOG_RPC_DETAIL("Oneway command, so no longer waiting on RpcTransport %p", | 
|  | 552 | connection->rpcTransport.get()); | 
| Steven Moreland | 52eee94 | 2021-06-03 00:59:28 +0000 | [diff] [blame] | 553 |  | 
|  | 554 | // Do not wait on result. | 
| Steven Moreland | 43921d5 | 2021-09-27 17:15:56 -0700 | [diff] [blame] | 555 | return OK; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 556 | } | 
|  | 557 |  | 
|  | 558 | LOG_ALWAYS_FATAL_IF(reply == nullptr, "Reply parcel must be used for synchronous transaction."); | 
|  | 559 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 560 | return waitForReply(connection, session, reply); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 561 | } | 
|  | 562 |  | 
| Steven Moreland | 438cce8 | 2021-04-02 18:04:08 +0000 | [diff] [blame] | 563 | static void cleanup_reply_data(Parcel* p, const uint8_t* data, size_t dataSize, | 
|  | 564 | const binder_size_t* objects, size_t objectsCount) { | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 565 | (void)p; | 
|  | 566 | delete[] const_cast<uint8_t*>(data - offsetof(RpcWireReply, data)); | 
|  | 567 | (void)dataSize; | 
|  | 568 | LOG_ALWAYS_FATAL_IF(objects != nullptr); | 
| Yifan Hong | 239a2ca | 2021-06-24 16:05:16 -0700 | [diff] [blame] | 569 | LOG_ALWAYS_FATAL_IF(objectsCount != 0, "%zu objects remaining", objectsCount); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 570 | } | 
|  | 571 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 572 | status_t RpcState::waitForReply(const sp<RpcSession::RpcConnection>& connection, | 
|  | 573 | const sp<RpcSession>& session, Parcel* reply) { | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 574 | RpcWireHeader command; | 
|  | 575 | while (true) { | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 576 | iovec iov{&command, sizeof(command)}; | 
|  | 577 | if (status_t status = rpcRec(connection, session, "command header (for reply)", &iov, 1); | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 578 | status != OK) | 
|  | 579 | return status; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 580 |  | 
|  | 581 | if (command.command == RPC_COMMAND_REPLY) break; | 
|  | 582 |  | 
| Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 583 | if (status_t status = processCommand(connection, session, command, CommandType::ANY); | 
| Steven Moreland | 52eee94 | 2021-06-03 00:59:28 +0000 | [diff] [blame] | 584 | status != OK) | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 585 | return status; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 586 | } | 
|  | 587 |  | 
| Steven Moreland | dbe7183 | 2021-05-12 23:31:00 +0000 | [diff] [blame] | 588 | CommandData data(command.bodySize); | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 589 | if (!data.valid()) return NO_MEMORY; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 590 |  | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 591 | iovec iov{data.data(), command.bodySize}; | 
|  | 592 | if (status_t status = rpcRec(connection, session, "reply body", &iov, 1); status != OK) | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 593 | return status; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 594 |  | 
|  | 595 | if (command.bodySize < sizeof(RpcWireReply)) { | 
|  | 596 | ALOGE("Expecting %zu but got %" PRId32 " bytes for RpcWireReply. Terminating!", | 
|  | 597 | sizeof(RpcWireReply), command.bodySize); | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 598 | (void)session->shutdownAndWait(false); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 599 | return BAD_VALUE; | 
|  | 600 | } | 
| Steven Moreland | e839334 | 2021-05-05 23:27:53 +0000 | [diff] [blame] | 601 | RpcWireReply* rpcReply = reinterpret_cast<RpcWireReply*>(data.data()); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 602 | if (rpcReply->status != OK) return rpcReply->status; | 
|  | 603 |  | 
| Steven Moreland | e839334 | 2021-05-05 23:27:53 +0000 | [diff] [blame] | 604 | data.release(); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 605 | reply->ipcSetDataReference(rpcReply->data, command.bodySize - offsetof(RpcWireReply, data), | 
| Steven Moreland | 438cce8 | 2021-04-02 18:04:08 +0000 | [diff] [blame] | 606 | nullptr, 0, cleanup_reply_data); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 607 |  | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 608 | reply->markForRpc(session); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 609 |  | 
|  | 610 | return OK; | 
|  | 611 | } | 
|  | 612 |  | 
| Steven Moreland | fd1e8a0 | 2021-07-21 23:30:29 +0000 | [diff] [blame] | 613 | status_t RpcState::sendDecStrongToTarget(const sp<RpcSession::RpcConnection>& connection, | 
|  | 614 | const sp<RpcSession>& session, uint64_t addr, | 
|  | 615 | size_t target) { | 
|  | 616 | RpcDecStrong body = { | 
|  | 617 | .address = RpcWireAddress::fromRaw(addr), | 
|  | 618 | }; | 
|  | 619 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 620 | { | 
|  | 621 | std::lock_guard<std::mutex> _l(mNodeMutex); | 
|  | 622 | if (mTerminated) return DEAD_OBJECT; // avoid fatal only, otherwise races | 
|  | 623 | auto it = mNodeForAddress.find(addr); | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 624 | LOG_ALWAYS_FATAL_IF(it == mNodeForAddress.end(), | 
|  | 625 | "Sending dec strong on unknown address %" PRIu64, addr); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 626 |  | 
| Steven Moreland | fd1e8a0 | 2021-07-21 23:30:29 +0000 | [diff] [blame] | 627 | LOG_ALWAYS_FATAL_IF(it->second.timesRecd < target, "Can't dec count of %zu to %zu.", | 
|  | 628 | it->second.timesRecd, target); | 
|  | 629 |  | 
|  | 630 | // typically this happens when multiple threads send dec refs at the | 
|  | 631 | // same time - the transactions will get combined automatically | 
|  | 632 | if (it->second.timesRecd == target) return OK; | 
|  | 633 |  | 
|  | 634 | body.amount = it->second.timesRecd - target; | 
|  | 635 | it->second.timesRecd = target; | 
|  | 636 |  | 
| Steven Moreland | 31bde7a | 2021-06-04 00:57:36 +0000 | [diff] [blame] | 637 | LOG_ALWAYS_FATAL_IF(nullptr != tryEraseNode(it), | 
|  | 638 | "Bad state. RpcState shouldn't own received binder"); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 639 | } | 
|  | 640 |  | 
|  | 641 | RpcWireHeader cmd = { | 
|  | 642 | .command = RPC_COMMAND_DEC_STRONG, | 
| Steven Moreland | fd1e8a0 | 2021-07-21 23:30:29 +0000 | [diff] [blame] | 643 | .bodySize = sizeof(RpcDecStrong), | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 644 | }; | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 645 | iovec iovs[]{{&cmd, sizeof(cmd)}, {&body, sizeof(body)}}; | 
|  | 646 | return rpcSend(connection, session, "dec ref", iovs, arraysize(iovs)); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 647 | } | 
|  | 648 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 649 | status_t RpcState::getAndExecuteCommand(const sp<RpcSession::RpcConnection>& connection, | 
|  | 650 | const sp<RpcSession>& session, CommandType type) { | 
| Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 651 | LOG_RPC_DETAIL("getAndExecuteCommand on RpcTransport %p", connection->rpcTransport.get()); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 652 |  | 
|  | 653 | RpcWireHeader command; | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 654 | iovec iov{&command, sizeof(command)}; | 
|  | 655 | if (status_t status = rpcRec(connection, session, "command header (for server)", &iov, 1); | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 656 | status != OK) | 
|  | 657 | return status; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 658 |  | 
| Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 659 | return processCommand(connection, session, command, type); | 
| Steven Moreland | 52eee94 | 2021-06-03 00:59:28 +0000 | [diff] [blame] | 660 | } | 
|  | 661 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 662 | status_t RpcState::drainCommands(const sp<RpcSession::RpcConnection>& connection, | 
|  | 663 | const sp<RpcSession>& session, CommandType type) { | 
| Steven Moreland | 52eee94 | 2021-06-03 00:59:28 +0000 | [diff] [blame] | 664 | uint8_t buf; | 
| Andrei Homescu | 5ad71b5 | 2022-03-11 03:49:12 +0000 | [diff] [blame] | 665 | while (true) { | 
|  | 666 | size_t num_bytes; | 
|  | 667 | status_t status = connection->rpcTransport->peek(&buf, sizeof(buf), &num_bytes); | 
|  | 668 | if (status == WOULD_BLOCK) break; | 
|  | 669 | if (status != OK) return status; | 
|  | 670 | if (!num_bytes) break; | 
|  | 671 |  | 
|  | 672 | status = getAndExecuteCommand(connection, session, type); | 
| Steven Moreland | 52eee94 | 2021-06-03 00:59:28 +0000 | [diff] [blame] | 673 | if (status != OK) return status; | 
|  | 674 | } | 
|  | 675 | return OK; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 676 | } | 
|  | 677 |  | 
| Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 678 | status_t RpcState::processCommand(const sp<RpcSession::RpcConnection>& connection, | 
|  | 679 | const sp<RpcSession>& session, const RpcWireHeader& command, | 
|  | 680 | CommandType type) { | 
| Steven Moreland | d730207 | 2021-05-15 01:32:04 +0000 | [diff] [blame] | 681 | IPCThreadState* kernelBinderState = IPCThreadState::selfOrNull(); | 
|  | 682 | IPCThreadState::SpGuard spGuard{ | 
|  | 683 | .address = __builtin_frame_address(0), | 
|  | 684 | .context = "processing binder RPC command", | 
|  | 685 | }; | 
|  | 686 | const IPCThreadState::SpGuard* origGuard; | 
|  | 687 | if (kernelBinderState != nullptr) { | 
|  | 688 | origGuard = kernelBinderState->pushGetCallingSpGuard(&spGuard); | 
|  | 689 | } | 
|  | 690 | ScopeGuard guardUnguard = [&]() { | 
|  | 691 | if (kernelBinderState != nullptr) { | 
|  | 692 | kernelBinderState->restoreGetCallingSpGuard(origGuard); | 
|  | 693 | } | 
|  | 694 | }; | 
|  | 695 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 696 | switch (command.command) { | 
|  | 697 | case RPC_COMMAND_TRANSACT: | 
| Steven Moreland | 52eee94 | 2021-06-03 00:59:28 +0000 | [diff] [blame] | 698 | if (type != CommandType::ANY) return BAD_TYPE; | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 699 | return processTransact(connection, session, command); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 700 | case RPC_COMMAND_DEC_STRONG: | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 701 | return processDecStrong(connection, session, command); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 702 | } | 
|  | 703 |  | 
|  | 704 | // We should always know the version of the opposing side, and since the | 
|  | 705 | // RPC-binder-level wire protocol is not self synchronizing, we have no way | 
|  | 706 | // to understand where the current command ends and the next one begins. We | 
|  | 707 | // also can't consider it a fatal error because this would allow any client | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 708 | // to kill us, so ending the session for misbehaving client. | 
|  | 709 | ALOGE("Unknown RPC command %d - terminating session", command.command); | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 710 | (void)session->shutdownAndWait(false); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 711 | return DEAD_OBJECT; | 
|  | 712 | } | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 713 | status_t RpcState::processTransact(const sp<RpcSession::RpcConnection>& connection, | 
|  | 714 | const sp<RpcSession>& session, const RpcWireHeader& command) { | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 715 | LOG_ALWAYS_FATAL_IF(command.command != RPC_COMMAND_TRANSACT, "command: %d", command.command); | 
|  | 716 |  | 
| Steven Moreland | dbe7183 | 2021-05-12 23:31:00 +0000 | [diff] [blame] | 717 | CommandData transactionData(command.bodySize); | 
| Steven Moreland | e839334 | 2021-05-05 23:27:53 +0000 | [diff] [blame] | 718 | if (!transactionData.valid()) { | 
|  | 719 | return NO_MEMORY; | 
|  | 720 | } | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 721 | iovec iov{transactionData.data(), transactionData.size()}; | 
|  | 722 | if (status_t status = rpcRec(connection, session, "transaction body", &iov, 1); status != OK) | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 723 | return status; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 724 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 725 | return processTransactInternal(connection, session, std::move(transactionData)); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 726 | } | 
|  | 727 |  | 
| Steven Moreland | 438cce8 | 2021-04-02 18:04:08 +0000 | [diff] [blame] | 728 | static void do_nothing_to_transact_data(Parcel* p, const uint8_t* data, size_t dataSize, | 
|  | 729 | const binder_size_t* objects, size_t objectsCount) { | 
|  | 730 | (void)p; | 
|  | 731 | (void)data; | 
|  | 732 | (void)dataSize; | 
|  | 733 | (void)objects; | 
|  | 734 | (void)objectsCount; | 
|  | 735 | } | 
|  | 736 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 737 | status_t RpcState::processTransactInternal(const sp<RpcSession::RpcConnection>& connection, | 
|  | 738 | const sp<RpcSession>& session, | 
| Steven Moreland | ada72bd | 2021-06-09 23:29:13 +0000 | [diff] [blame] | 739 | CommandData transactionData) { | 
|  | 740 | // for 'recursive' calls to this, we have already read and processed the | 
|  | 741 | // binder from the transaction data and taken reference counts into account, | 
|  | 742 | // so it is cached here. | 
| Steven Moreland | 3903bf0 | 2021-09-27 16:05:24 -0700 | [diff] [blame] | 743 | sp<IBinder> target; | 
| Steven Moreland | ada72bd | 2021-06-09 23:29:13 +0000 | [diff] [blame] | 744 | processTransactInternalTailCall: | 
|  | 745 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 746 | if (transactionData.size() < sizeof(RpcWireTransaction)) { | 
|  | 747 | ALOGE("Expecting %zu but got %zu bytes for RpcWireTransaction. Terminating!", | 
|  | 748 | sizeof(RpcWireTransaction), transactionData.size()); | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 749 | (void)session->shutdownAndWait(false); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 750 | return BAD_VALUE; | 
|  | 751 | } | 
|  | 752 | RpcWireTransaction* transaction = reinterpret_cast<RpcWireTransaction*>(transactionData.data()); | 
|  | 753 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 754 | uint64_t addr = RpcWireAddress::toRaw(transaction->address); | 
| Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 755 | bool oneway = transaction->flags & IBinder::FLAG_ONEWAY; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 756 |  | 
|  | 757 | status_t replyStatus = OK; | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 758 | if (addr != 0) { | 
| Steven Moreland | 3903bf0 | 2021-09-27 16:05:24 -0700 | [diff] [blame] | 759 | if (!target) { | 
| Steven Moreland | 7227c8a | 2021-06-02 00:24:32 +0000 | [diff] [blame] | 760 | replyStatus = onBinderEntering(session, addr, &target); | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 761 | } | 
|  | 762 |  | 
| Steven Moreland | 7227c8a | 2021-06-02 00:24:32 +0000 | [diff] [blame] | 763 | if (replyStatus != OK) { | 
|  | 764 | // do nothing | 
|  | 765 | } else if (target == nullptr) { | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 766 | // This can happen if the binder is remote in this process, and | 
|  | 767 | // another thread has called the last decStrong on this binder. | 
|  | 768 | // However, for local binders, it indicates a misbehaving client | 
|  | 769 | // (any binder which is being transacted on should be holding a | 
|  | 770 | // strong ref count), so in either case, terminating the | 
|  | 771 | // session. | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 772 | ALOGE("While transacting, binder has been deleted at address %" PRIu64 ". Terminating!", | 
|  | 773 | addr); | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 774 | (void)session->shutdownAndWait(false); | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 775 | replyStatus = BAD_VALUE; | 
|  | 776 | } else if (target->localBinder() == nullptr) { | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 777 | ALOGE("Unknown binder address or non-local binder, not address %" PRIu64 | 
|  | 778 | ". Terminating!", | 
|  | 779 | addr); | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 780 | (void)session->shutdownAndWait(false); | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 781 | replyStatus = BAD_VALUE; | 
| Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 782 | } else if (oneway) { | 
| Steven Moreland | d45be62 | 2021-06-04 02:19:37 +0000 | [diff] [blame] | 783 | std::unique_lock<std::mutex> _l(mNodeMutex); | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 784 | auto it = mNodeForAddress.find(addr); | 
|  | 785 | if (it->second.binder.promote() != target) { | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 786 | ALOGE("Binder became invalid during transaction. Bad client? %" PRIu64, addr); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 787 | replyStatus = BAD_VALUE; | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 788 | } else if (transaction->asyncNumber != it->second.asyncNumber) { | 
|  | 789 | // we need to process some other asynchronous transaction | 
|  | 790 | // first | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 791 | it->second.asyncTodo.push(BinderNode::AsyncTodo{ | 
|  | 792 | .ref = target, | 
|  | 793 | .data = std::move(transactionData), | 
|  | 794 | .asyncNumber = transaction->asyncNumber, | 
|  | 795 | }); | 
| Steven Moreland | d45be62 | 2021-06-04 02:19:37 +0000 | [diff] [blame] | 796 |  | 
|  | 797 | size_t numPending = it->second.asyncTodo.size(); | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 798 | LOG_RPC_DETAIL("Enqueuing %" PRIu64 " on %" PRIu64 " (%zu pending)", | 
|  | 799 | transaction->asyncNumber, addr, numPending); | 
| Steven Moreland | d45be62 | 2021-06-04 02:19:37 +0000 | [diff] [blame] | 800 |  | 
|  | 801 | constexpr size_t kArbitraryOnewayCallTerminateLevel = 10000; | 
|  | 802 | constexpr size_t kArbitraryOnewayCallWarnLevel = 1000; | 
|  | 803 | constexpr size_t kArbitraryOnewayCallWarnPer = 1000; | 
|  | 804 |  | 
|  | 805 | if (numPending >= kArbitraryOnewayCallWarnLevel) { | 
|  | 806 | if (numPending >= kArbitraryOnewayCallTerminateLevel) { | 
|  | 807 | ALOGE("WARNING: %zu pending oneway transactions. Terminating!", numPending); | 
|  | 808 | _l.unlock(); | 
|  | 809 | (void)session->shutdownAndWait(false); | 
|  | 810 | return FAILED_TRANSACTION; | 
|  | 811 | } | 
|  | 812 |  | 
|  | 813 | if (numPending % kArbitraryOnewayCallWarnPer == 0) { | 
|  | 814 | ALOGW("Warning: many oneway transactions built up on %p (%zu)", | 
|  | 815 | target.get(), numPending); | 
|  | 816 | } | 
|  | 817 | } | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 818 | return OK; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 819 | } | 
|  | 820 | } | 
|  | 821 | } | 
|  | 822 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 823 | Parcel reply; | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 824 | reply.markForRpc(session); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 825 |  | 
|  | 826 | if (replyStatus == OK) { | 
| Steven Moreland | eff77c1 | 2021-04-15 00:37:19 +0000 | [diff] [blame] | 827 | Parcel data; | 
|  | 828 | // transaction->data is owned by this function. Parcel borrows this data and | 
|  | 829 | // only holds onto it for the duration of this function call. Parcel will be | 
|  | 830 | // deleted before the 'transactionData' object. | 
|  | 831 | data.ipcSetDataReference(transaction->data, | 
|  | 832 | transactionData.size() - offsetof(RpcWireTransaction, data), | 
|  | 833 | nullptr /*object*/, 0 /*objectCount*/, | 
|  | 834 | do_nothing_to_transact_data); | 
| Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 835 | data.markForRpc(session); | 
| Steven Moreland | eff77c1 | 2021-04-15 00:37:19 +0000 | [diff] [blame] | 836 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 837 | if (target) { | 
| Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 838 | bool origAllowNested = connection->allowNested; | 
|  | 839 | connection->allowNested = !oneway; | 
|  | 840 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 841 | replyStatus = target->transact(transaction->code, data, &reply, transaction->flags); | 
| Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 842 |  | 
|  | 843 | connection->allowNested = origAllowNested; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 844 | } else { | 
|  | 845 | LOG_RPC_DETAIL("Got special transaction %u", transaction->code); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 846 |  | 
| Steven Moreland | 103424e | 2021-06-02 18:16:19 +0000 | [diff] [blame] | 847 | switch (transaction->code) { | 
|  | 848 | case RPC_SPECIAL_TRANSACT_GET_MAX_THREADS: { | 
| Yifan Hong | 1042306 | 2021-10-08 16:26:32 -0700 | [diff] [blame] | 849 | replyStatus = reply.writeInt32(session->getMaxIncomingThreads()); | 
| Steven Moreland | 103424e | 2021-06-02 18:16:19 +0000 | [diff] [blame] | 850 | break; | 
|  | 851 | } | 
|  | 852 | case RPC_SPECIAL_TRANSACT_GET_SESSION_ID: { | 
|  | 853 | // for client connections, this should always report the value | 
| Steven Moreland | 01a6bad | 2021-06-11 00:59:20 +0000 | [diff] [blame] | 854 | // originally returned from the server, so this is asserting | 
|  | 855 | // that it exists | 
| Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame] | 856 | replyStatus = reply.writeByteVector(session->mId); | 
| Steven Moreland | 103424e | 2021-06-02 18:16:19 +0000 | [diff] [blame] | 857 | break; | 
|  | 858 | } | 
|  | 859 | default: { | 
| Steven Moreland | 7b8bc4c | 2021-06-10 22:50:27 +0000 | [diff] [blame] | 860 | sp<RpcServer> server = session->server(); | 
| Steven Moreland | 103424e | 2021-06-02 18:16:19 +0000 | [diff] [blame] | 861 | if (server) { | 
|  | 862 | switch (transaction->code) { | 
|  | 863 | case RPC_SPECIAL_TRANSACT_GET_ROOT: { | 
| Steven Moreland | 51c44a9 | 2021-10-14 16:50:35 -0700 | [diff] [blame] | 864 | sp<IBinder> root = session->mSessionSpecificRootObject | 
|  | 865 | ?: server->getRootObject(); | 
|  | 866 | replyStatus = reply.writeStrongBinder(root); | 
| Steven Moreland | 103424e | 2021-06-02 18:16:19 +0000 | [diff] [blame] | 867 | break; | 
|  | 868 | } | 
|  | 869 | default: { | 
|  | 870 | replyStatus = UNKNOWN_TRANSACTION; | 
|  | 871 | } | 
|  | 872 | } | 
|  | 873 | } else { | 
|  | 874 | ALOGE("Special command sent, but no server object attached."); | 
| Steven Moreland | f137de9 | 2021-04-24 01:54:26 +0000 | [diff] [blame] | 875 | } | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 876 | } | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 877 | } | 
|  | 878 | } | 
|  | 879 | } | 
|  | 880 |  | 
| Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 881 | if (oneway) { | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 882 | if (replyStatus != OK) { | 
|  | 883 | ALOGW("Oneway call failed with error: %d", replyStatus); | 
|  | 884 | } | 
|  | 885 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 886 | LOG_RPC_DETAIL("Processed async transaction %" PRIu64 " on %" PRIu64, | 
|  | 887 | transaction->asyncNumber, addr); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 888 |  | 
|  | 889 | // Check to see if there is another asynchronous transaction to process. | 
|  | 890 | // This behavior differs from binder behavior, since in the binder | 
|  | 891 | // driver, asynchronous transactions will be processed after existing | 
|  | 892 | // pending binder transactions on the queue. The downside of this is | 
|  | 893 | // that asynchronous transactions can be drowned out by synchronous | 
|  | 894 | // transactions. However, we have no easy way to queue these | 
|  | 895 | // transactions after the synchronous transactions we may want to read | 
|  | 896 | // from the wire. So, in socket binder here, we have the opposite | 
|  | 897 | // downside: asynchronous transactions may drown out synchronous | 
|  | 898 | // transactions. | 
|  | 899 | { | 
|  | 900 | std::unique_lock<std::mutex> _l(mNodeMutex); | 
|  | 901 | auto it = mNodeForAddress.find(addr); | 
|  | 902 | // last refcount dropped after this transaction happened | 
|  | 903 | if (it == mNodeForAddress.end()) return OK; | 
|  | 904 |  | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 905 | if (!nodeProgressAsyncNumber(&it->second)) { | 
|  | 906 | _l.unlock(); | 
|  | 907 | (void)session->shutdownAndWait(false); | 
|  | 908 | return DEAD_OBJECT; | 
|  | 909 | } | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 910 |  | 
|  | 911 | if (it->second.asyncTodo.size() == 0) return OK; | 
|  | 912 | if (it->second.asyncTodo.top().asyncNumber == it->second.asyncNumber) { | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 913 | LOG_RPC_DETAIL("Found next async transaction %" PRIu64 " on %" PRIu64, | 
|  | 914 | it->second.asyncNumber, addr); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 915 |  | 
|  | 916 | // justification for const_cast (consider avoiding priority_queue): | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 917 | // - AsyncTodo operator< doesn't depend on 'data' or 'ref' objects | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 918 | // - gotta go fast | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 919 | auto& todo = const_cast<BinderNode::AsyncTodo&>(it->second.asyncTodo.top()); | 
|  | 920 |  | 
| Steven Moreland | ada72bd | 2021-06-09 23:29:13 +0000 | [diff] [blame] | 921 | // reset up arguments | 
|  | 922 | transactionData = std::move(todo.data); | 
| Steven Moreland | 3903bf0 | 2021-09-27 16:05:24 -0700 | [diff] [blame] | 923 | LOG_ALWAYS_FATAL_IF(target != todo.ref, | 
|  | 924 | "async list should be associated with a binder"); | 
| Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 925 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 926 | it->second.asyncTodo.pop(); | 
| Steven Moreland | ada72bd | 2021-06-09 23:29:13 +0000 | [diff] [blame] | 927 | goto processTransactInternalTailCall; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 928 | } | 
|  | 929 | } | 
| Steven Moreland | d808331 | 2021-09-22 13:37:10 -0700 | [diff] [blame] | 930 |  | 
|  | 931 | // done processing all the async commands on this binder that we can, so | 
|  | 932 | // write decstrongs on the binder | 
|  | 933 | if (addr != 0 && replyStatus == OK) { | 
|  | 934 | return flushExcessBinderRefs(session, addr, target); | 
|  | 935 | } | 
|  | 936 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 937 | return OK; | 
|  | 938 | } | 
|  | 939 |  | 
| Steven Moreland | 6709cf4 | 2021-09-30 15:21:54 -0700 | [diff] [blame] | 940 | // Binder refs are flushed for oneway calls only after all calls which are | 
|  | 941 | // built up are executed. Otherwise, they fill up the binder buffer. | 
|  | 942 | if (addr != 0 && replyStatus == OK) { | 
|  | 943 | replyStatus = flushExcessBinderRefs(session, addr, target); | 
|  | 944 | } | 
|  | 945 |  | 
| Steven Moreland | 77c3011 | 2021-06-02 20:45:46 +0000 | [diff] [blame] | 946 | LOG_ALWAYS_FATAL_IF(std::numeric_limits<int32_t>::max() - sizeof(RpcWireHeader) - | 
|  | 947 | sizeof(RpcWireReply) < | 
|  | 948 | reply.dataSize(), | 
|  | 949 | "Too much data for reply %zu", reply.dataSize()); | 
|  | 950 |  | 
|  | 951 | RpcWireHeader cmdReply{ | 
|  | 952 | .command = RPC_COMMAND_REPLY, | 
|  | 953 | .bodySize = static_cast<uint32_t>(sizeof(RpcWireReply) + reply.dataSize()), | 
|  | 954 | }; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 955 | RpcWireReply rpcReply{ | 
|  | 956 | .status = replyStatus, | 
|  | 957 | }; | 
|  | 958 |  | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 959 | iovec iovs[]{ | 
|  | 960 | {&cmdReply, sizeof(RpcWireHeader)}, | 
|  | 961 | {&rpcReply, sizeof(RpcWireReply)}, | 
|  | 962 | {const_cast<uint8_t*>(reply.data()), reply.dataSize()}, | 
|  | 963 | }; | 
|  | 964 | return rpcSend(connection, session, "reply", iovs, arraysize(iovs)); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 965 | } | 
|  | 966 |  | 
| Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 967 | status_t RpcState::processDecStrong(const sp<RpcSession::RpcConnection>& connection, | 
|  | 968 | const sp<RpcSession>& session, const RpcWireHeader& command) { | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 969 | LOG_ALWAYS_FATAL_IF(command.command != RPC_COMMAND_DEC_STRONG, "command: %d", command.command); | 
|  | 970 |  | 
| Steven Moreland | dbe7183 | 2021-05-12 23:31:00 +0000 | [diff] [blame] | 971 | CommandData commandData(command.bodySize); | 
| Steven Moreland | e839334 | 2021-05-05 23:27:53 +0000 | [diff] [blame] | 972 | if (!commandData.valid()) { | 
|  | 973 | return NO_MEMORY; | 
|  | 974 | } | 
| Andrei Homescu | a39e4ed | 2021-12-10 08:41:54 +0000 | [diff] [blame] | 975 | iovec iov{commandData.data(), commandData.size()}; | 
|  | 976 | if (status_t status = rpcRec(connection, session, "dec ref body", &iov, 1); status != OK) | 
| Steven Moreland | 1e4c2b8 | 2021-05-25 01:51:31 +0000 | [diff] [blame] | 977 | return status; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 978 |  | 
| Steven Moreland | fd1e8a0 | 2021-07-21 23:30:29 +0000 | [diff] [blame] | 979 | if (command.bodySize != sizeof(RpcDecStrong)) { | 
|  | 980 | ALOGE("Expecting %zu but got %" PRId32 " bytes for RpcDecStrong. Terminating!", | 
|  | 981 | sizeof(RpcDecStrong), command.bodySize); | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 982 | (void)session->shutdownAndWait(false); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 983 | return BAD_VALUE; | 
|  | 984 | } | 
| Steven Moreland | fd1e8a0 | 2021-07-21 23:30:29 +0000 | [diff] [blame] | 985 | RpcDecStrong* body = reinterpret_cast<RpcDecStrong*>(commandData.data()); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 986 |  | 
| Steven Moreland | fd1e8a0 | 2021-07-21 23:30:29 +0000 | [diff] [blame] | 987 | uint64_t addr = RpcWireAddress::toRaw(body->address); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 988 | std::unique_lock<std::mutex> _l(mNodeMutex); | 
|  | 989 | auto it = mNodeForAddress.find(addr); | 
|  | 990 | if (it == mNodeForAddress.end()) { | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 991 | ALOGE("Unknown binder address %" PRIu64 " for dec strong.", addr); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 992 | return OK; | 
|  | 993 | } | 
|  | 994 |  | 
|  | 995 | sp<IBinder> target = it->second.binder.promote(); | 
|  | 996 | if (target == nullptr) { | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 997 | ALOGE("While requesting dec strong, binder has been deleted at address %" PRIu64 | 
|  | 998 | ". Terminating!", | 
|  | 999 | addr); | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 1000 | _l.unlock(); | 
|  | 1001 | (void)session->shutdownAndWait(false); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 1002 | return BAD_VALUE; | 
|  | 1003 | } | 
|  | 1004 |  | 
| Steven Moreland | fd1e8a0 | 2021-07-21 23:30:29 +0000 | [diff] [blame] | 1005 | if (it->second.timesSent < body->amount) { | 
|  | 1006 | ALOGE("Record of sending binder %zu times, but requested decStrong for %" PRIu64 " of %u", | 
|  | 1007 | it->second.timesSent, addr, body->amount); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 1008 | return OK; | 
|  | 1009 | } | 
|  | 1010 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 1011 | LOG_ALWAYS_FATAL_IF(it->second.sentRef == nullptr, "Inconsistent state, lost ref for %" PRIu64, | 
|  | 1012 | addr); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 1013 |  | 
| Steven Moreland | fd1e8a0 | 2021-07-21 23:30:29 +0000 | [diff] [blame] | 1014 | LOG_RPC_DETAIL("Processing dec strong of %" PRIu64 " by %u from %zu", addr, body->amount, | 
|  | 1015 | it->second.timesSent); | 
|  | 1016 |  | 
|  | 1017 | it->second.timesSent -= body->amount; | 
| Steven Moreland | 31bde7a | 2021-06-04 00:57:36 +0000 | [diff] [blame] | 1018 | sp<IBinder> tempHold = tryEraseNode(it); | 
|  | 1019 | _l.unlock(); | 
|  | 1020 | tempHold = nullptr; // destructor may make binder calls on this session | 
|  | 1021 |  | 
|  | 1022 | return OK; | 
|  | 1023 | } | 
|  | 1024 |  | 
| Steven Moreland | 5623d1a | 2021-09-10 15:45:34 -0700 | [diff] [blame] | 1025 | sp<IBinder> RpcState::tryEraseNode(std::map<uint64_t, BinderNode>::iterator& it) { | 
| Steven Moreland | 31bde7a | 2021-06-04 00:57:36 +0000 | [diff] [blame] | 1026 | sp<IBinder> ref; | 
|  | 1027 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 1028 | if (it->second.timesSent == 0) { | 
| Steven Moreland | 31bde7a | 2021-06-04 00:57:36 +0000 | [diff] [blame] | 1029 | ref = std::move(it->second.sentRef); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 1030 |  | 
|  | 1031 | if (it->second.timesRecd == 0) { | 
| Steven Moreland | a6e11cf | 2021-06-04 00:58:31 +0000 | [diff] [blame] | 1032 | LOG_ALWAYS_FATAL_IF(!it->second.asyncTodo.empty(), | 
|  | 1033 | "Can't delete binder w/ pending async transactions"); | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 1034 | mNodeForAddress.erase(it); | 
|  | 1035 | } | 
|  | 1036 | } | 
|  | 1037 |  | 
| Steven Moreland | 31bde7a | 2021-06-04 00:57:36 +0000 | [diff] [blame] | 1038 | return ref; | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 1039 | } | 
|  | 1040 |  | 
| Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 1041 | bool RpcState::nodeProgressAsyncNumber(BinderNode* node) { | 
| Steven Moreland | 583a14a | 2021-06-04 02:04:58 +0000 | [diff] [blame] | 1042 | // 2**64 =~ 10**19 =~ 1000 transactions per second for 585 million years to | 
|  | 1043 | // a single binder | 
|  | 1044 | if (node->asyncNumber >= std::numeric_limits<decltype(node->asyncNumber)>::max()) { | 
|  | 1045 | ALOGE("Out of async transaction IDs. Terminating"); | 
| Steven Moreland | 583a14a | 2021-06-04 02:04:58 +0000 | [diff] [blame] | 1046 | return false; | 
|  | 1047 | } | 
|  | 1048 | node->asyncNumber++; | 
|  | 1049 | return true; | 
|  | 1050 | } | 
|  | 1051 |  | 
| Steven Moreland | 5553ac4 | 2020-11-11 02:14:45 +0000 | [diff] [blame] | 1052 | } // namespace android |