diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2017-11-14 09:41:55 +0100 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2017-11-14 09:41:55 +0100 |
| commit | 7a53e7e35da7468b5a6291fa3b5e1db4bcdf402f (patch) | |
| tree | 62d7db4c53b8ab4da2fab89c25f528904d8bf517 /pkg/repro | |
| parent | f9a8d567eb3388d0909e0d3cb6df23d345911850 (diff) | |
pkg/report: combine report data into a struct
Parse returns 5 variables now. Later we may want to add crash "priority".
Introduce Report struct that holds all report data.
Diffstat (limited to 'pkg/repro')
| -rw-r--r-- | pkg/repro/repro.go | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/pkg/repro/repro.go b/pkg/repro/repro.go index 699f0d7c3..b6b6a908e 100644 --- a/pkg/repro/repro.go +++ b/pkg/repro/repro.go @@ -74,10 +74,11 @@ func Run(crashLog []byte, cfg *mgrconfig.Config, reporter report.Reporter, vmPoo if len(entries) == 0 { return nil, fmt.Errorf("crash log does not contain any programs") } - crashDesc, _, crashStart, _, _ := reporter.Parse(crashLog) - if crashDesc == "" { - crashStart = len(crashLog) // assuming VM hanged - crashDesc = "hang" + crashStart := len(crashLog) // assuming VM hanged + crashDesc := "hang" + if rep := reporter.Parse(crashLog); rep != nil { + crashStart = rep.Start + crashDesc = rep.Desc } ctx := &context{ |