mirror of
https://github.com/kidfromjupiter/nearby.git
synced 2026-09-14 14:46:12 -04:00
Juliet Levesque
8a78212bb8
During the destruction of NearbyConnections, Core (which owns ClientProxy) is destructed before ServiceController (which owns EndpointManager), which means any pending tasks on the EndpointManager executor that use ClientProxy will be using garbage memory. To fix this issue, EndpointManager::DiscardEndpoint will check for an `is_shutdown` boolean set during ~EndpointManager before accessing ClientProxy. The assumption is that any accessing of ClientProxy after the destruction will be invalid. PiperOrigin-RevId: 532581122
49 lines
1.5 KiB
C++
49 lines
1.5 KiB
C++
// Copyright 2023 Google LLC
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// https://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
#include "internal/test/fake_single_thread_executor.h"
|
|
|
|
#include <string>
|
|
#include <utility>
|
|
#include <vector>
|
|
|
|
namespace nearby {
|
|
|
|
FakeSingleThreadExecutor::FakeSingleThreadExecutor() = default;
|
|
|
|
FakeSingleThreadExecutor::~FakeSingleThreadExecutor() { DoShutdown(); }
|
|
|
|
void FakeSingleThreadExecutor::Execute(const std::string& name,
|
|
Runnable&& runnable) {
|
|
runnables_.push_back(std::make_pair(name, std::move(runnable)));
|
|
|
|
if (!run_executables_immediately_) return;
|
|
|
|
RunAllExecutables();
|
|
}
|
|
|
|
void FakeSingleThreadExecutor::RunAllExecutables() {
|
|
// Because `SingleThreadExecutor` ensures sequencing, run all pending
|
|
// executables in order they were added to the vector.
|
|
for (auto& runnable_pair : runnables_) {
|
|
runnable_pair.second();
|
|
}
|
|
|
|
runnables_.clear();
|
|
}
|
|
|
|
void FakeSingleThreadExecutor::DoShutdown() { RunAllExecutables(); }
|
|
|
|
} // namespace nearby
|