diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2024-06-27 08:02:35 +0200 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2024-06-27 07:46:15 +0000 |
| commit | 52ea34f88b0592c661d3ba1e7ca192292fe26582 (patch) | |
| tree | 954da51d13b8a57956445a2581169aa0d6af3add | |
| parent | 5c045c049cdc7a5c511f4b387909ba2729843dce (diff) | |
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.
| -rw-r--r-- | executor/executor_runner.h | 2 |
1 files changed, 1 insertions, 1 deletions
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; |