From 40d58a50e9c90cf136202d56bb14bd8709d3c2fb Mon Sep 17 00:00:00 2001
From: Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com>
Date: Thu, 4 Jan 2024 16:53:02 +0100
Subject: bugfix: Add missing return in SetRemoteExecutionDispatch

This was a source of occasional std::bad_variant_access exceptions.
---
 src/buildtool/execution_api/remote/config.cpp | 1 +
 1 file changed, 1 insertion(+)

(limited to 'src/buildtool/execution_api/remote/config.cpp')

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;
-- 
cgit v1.2.3