summaryrefslogtreecommitdiff
path: root/src/buildtool/execution_api/remote/config.cpp
diff options
context:
space:
mode:
authorPaul Cristian Sarbu <paul.cristian.sarbu@huawei.com>2024-01-04 16:53:02 +0100
committerPaul Cristian Sarbu <paul.cristian.sarbu@huawei.com>2024-01-08 15:00:15 +0100
commit40d58a50e9c90cf136202d56bb14bd8709d3c2fb (patch)
tree482e217e1b8b77ca27f3a0dc74275984ad2ff148 /src/buildtool/execution_api/remote/config.cpp
parent5edcbf32453455b7be06e63629a0fc78aedd4e06 (diff)
downloadjustbuild-40d58a50e9c90cf136202d56bb14bd8709d3c2fb.tar.gz
bugfix: Add missing return in SetRemoteExecutionDispatch
This was a source of occasional std::bad_variant_access exceptions.
Diffstat (limited to 'src/buildtool/execution_api/remote/config.cpp')
-rw-r--r--src/buildtool/execution_api/remote/config.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/buildtool/execution_api/remote/config.cpp b/src/buildtool/execution_api/remote/config.cpp
index 39bcebea..d878b93b 100644
--- a/src/buildtool/execution_api/remote/config.cpp
+++ b/src/buildtool/execution_api/remote/config.cpp
@@ -30,6 +30,7 @@
auto parsed = ParseDispatch(*dispatch_info);
if (parsed.index() == 0) {
Logger::Log(LogLevel::Warning, std::get<0>(parsed));
+ return false;
}
Instance().dispatch_ = std::move(std::get<1>(parsed));
return true;