From 52ea34f88b0592c661d3ba1e7ca192292fe26582 Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Thu, 27 Jun 2024 08:02:35 +0200 Subject: executor: handle features that fail in non-fatal way Coverage setup fails with exitf if not supported. Currently we consider it as transient error that needs to be retried. As the result we reach 20 attempts and crash the VM. Return an error in such case instead. --- executor/executor_runner.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'executor') diff --git a/executor/executor_runner.h b/executor/executor_runner.h index 0076f3a86..57d63f6a7 100644 --- a/executor/executor_runner.h +++ b/executor/executor_runner.h @@ -203,7 +203,7 @@ private: bool FailCurrentRequest(bool failed) { if (state_ == State::Handshaking) - return failed && IsSet(msg_->flags, rpc::RequestFlag::ReturnError); + return IsSet(msg_->flags, rpc::RequestFlag::ReturnError); if (state_ == State::Executing) return !failed || IsSet(msg_->flags, rpc::RequestFlag::ReturnError); return false; -- cgit mrf-deployment