From b247b6b84a9681b3bb8bddd3a31d81d3424cc0a3 Mon Sep 17 00:00:00 2001
From: Aleksandr Nogikh <nogikh@google.com>
Date: Wed, 24 May 2023 15:01:43 +0200
Subject: pkg/report: don't record error for empty repro log case

It's not entirely normal, but it can still happen and it's not a big
problem by itself. Let's not pollute our error logs.
---
 pkg/repro/repro.go | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'pkg')

diff --git a/pkg/repro/repro.go b/pkg/repro/repro.go
index d614cdfb6..2887bcebd 100644
--- a/pkg/repro/repro.go
+++ b/pkg/repro/repro.go
@@ -5,6 +5,7 @@ package repro
 
 import (
 	"bytes"
+	"errors"
 	"fmt"
 	"sort"
 	"sync"
@@ -59,6 +60,8 @@ type context struct {
 	timeouts     targets.Timeouts
 }
 
+var ErrNoPrograms = errors.New("crash log does not contain any programs")
+
 func Run(crashLog []byte, cfg *mgrconfig.Config, features *host.Features, reporter *report.Reporter,
 	vmPool *vm.Pool, vmIndexes []int) (*Result, *Stats, error) {
 	if len(vmIndexes) == 0 {
@@ -66,7 +69,7 @@ func Run(crashLog []byte, cfg *mgrconfig.Config, features *host.Features, report
 	}
 	entries := cfg.Target.ParseLog(crashLog)
 	if len(entries) == 0 {
-		return nil, nil, fmt.Errorf("crash log does not contain any programs")
+		return nil, nil, ErrNoPrograms
 	}
 	crashStart := len(crashLog)
 	crashTitle, crashType := "", report.Unknown
-- 
cgit mrf-deployment