dashboard/app: fix infinite emails
We override crash with the crash used for bisection
to make the information more consistent.
However if bisection crash only have syz repro
and there is now another crash with C repro,
then we always think that we have not reported C repro
and continue sending the same report again and again.
Don't override the crash with bisection crash in such case.
diff --git a/dashboard/app/bisect_test.go b/dashboard/app/bisect_test.go
index 00f87ed..57ce55c 100644
--- a/dashboard/app/bisect_test.go
+++ b/dashboard/app/bisect_test.go
@@ -7,6 +7,7 @@
import (
"fmt"
+ "strings"
"testing"
"time"
@@ -547,3 +548,83 @@
c.expectEQ(bisect.Type, dashapi.ReportBisectCause)
c.expectEQ(bisect.Title, rep.Title)
}
+
+func TestBisectCauseReproSyz(t *testing.T) {
+ c := NewCtx(t)
+ defer c.Close()
+
+ build := testBuild(1)
+ c.client2.UploadBuild(build)
+ crash := testCrashWithRepro(build, 1)
+ crash.ReproC = nil
+ c.client2.ReportCrash(crash)
+
+ pollResp := c.client2.pollJobs(build.Manager)
+ jobID := pollResp.ID
+ done := &dashapi.JobDoneReq{
+ ID: jobID,
+ Build: *build,
+ Log: []byte("bisect log"),
+ CrashTitle: "bisect crash title",
+ CrashLog: []byte("bisect crash log"),
+ }
+ done.Build.ID = jobID
+ c.expectOK(c.client2.JobDone(done))
+
+ crash.ReproC = []byte("int main")
+ c.client2.ReportCrash(crash)
+
+ msg := c.client2.pollEmailBug()
+ if !strings.Contains(msg.Body, "syzbot found the following crash") {
+ t.Fatalf("wrong email header:\n%v", msg.Body)
+ }
+ if !strings.Contains(msg.Body, "Bisection is inconclusive") {
+ t.Fatalf("report does not contain bisection results:\n%v", msg.Body)
+ }
+}
+
+func TestBisectCauseReproSyz2(t *testing.T) {
+ c := NewCtx(t)
+ defer c.Close()
+
+ build := testBuild(1)
+ c.client2.UploadBuild(build)
+ crash := testCrashWithRepro(build, 1)
+ crash.ReproC = nil
+ c.client2.ReportCrash(crash)
+
+ pollResp := c.client2.pollJobs(build.Manager)
+ jobID := pollResp.ID
+ done := &dashapi.JobDoneReq{
+ ID: jobID,
+ Build: *build,
+ Log: []byte("bisect log"),
+ CrashTitle: "bisect crash title",
+ CrashLog: []byte("bisect crash log"),
+ }
+ done.Build.ID = jobID
+ c.expectOK(c.client2.JobDone(done))
+
+ msg := c.client2.pollEmailBug()
+ if !strings.Contains(msg.Body, "syzbot found the following crash") {
+ t.Fatalf("wrong email header:\n%v", msg.Body)
+ }
+ if !strings.Contains(msg.Body, "Bisection is inconclusive") {
+ t.Fatalf("report does not contain bisection results:\n%v", msg.Body)
+ }
+
+ crash.ReproC = []byte("int main")
+ c.client2.ReportCrash(crash)
+
+ msg = c.client2.pollEmailBug()
+ if !strings.Contains(msg.Body, "syzbot has found a reproducer for the following crash") {
+ t.Fatalf("wrong email header:\n%v", msg.Body)
+ }
+ // Do we need bisection results in this email as well?
+ // We already mailed them, so we could not mail them here.
+ // But if we don't include bisection results, need to check that CC is correct
+ // (includes bisection CC).
+ if !strings.Contains(msg.Body, "Bisection is inconclusive") {
+ t.Fatalf("report still contains bisection results:\n%v", msg.Body)
+ }
+}
diff --git a/dashboard/app/reporting.go b/dashboard/app/reporting.go
index b05d651..5020219 100644
--- a/dashboard/app/reporting.go
+++ b/dashboard/app/reporting.go
@@ -303,10 +303,16 @@
var job *Job
if bug.BisectCause == BisectYes {
// If we have bisection results, report the crash/repro used for bisection.
- job, crash, _, crashKey, err = loadBisectJob(c, bug)
+ job1, crash1, _, crashKey1, err := loadBisectJob(c, bug)
if err != nil {
return nil, err
}
+ job = job1
+ if crash1.ReproC != 0 || crash.ReproC == 0 {
+ // Don't override the crash in this case,
+ // otherwise we will always think that we haven't reported the C repro.
+ crash, crashKey = crash1, crashKey1
+ }
}
crashLog, _, err := getText(c, textCrashLog, crash.Log)
if err != nil {
