pkg/report: move TitleToCrashType to crash package

TitleToCrashType is a simple function with no heavy dependencies
that is used by the dashboard app.
Currnetly we have to import pkg/report into dashboard/app,
and this package has lots of heavy deps (symbolizer, demangler,
coverage report generation, etc).
Move TitleToCrashType to pkg/report/crash (where it arguably belongs anyway).

8 files changed, 63 insertions, 65 deletions

diff --git a/dashboard/app/graphs.go b/dashboard/app/graphs.go
index 77dcaf13a..69fd9b894 100644
--- a/dashboard/app/graphs.go
+++ b/dashboard/app/graphs.go
@@ -13,7 +13,6 @@ import (
 	"strconv"
 	"time"
 
-	"github.com/google/syzkaller/pkg/report"
 	"github.com/google/syzkaller/pkg/report/crash"
 	db "google.golang.org/appengine/v2/datastore"
 )
@@ -370,7 +369,7 @@ func createFoundBugs(c context.Context, bugs []*Bug) *uiGraph {
 	months := make(map[time.Time]map[string]int)
 	for _, bug := range bugs {
 		for _, typ := range types {
-			if !typ.pred(report.TitleToCrashType(bug.Title)) {
+			if !typ.pred(crash.TitleToType(bug.Title)) {
 				continue
 			}
 			t := bug.FirstTime
diff --git a/pkg/report/title_to_type.go b/pkg/report/crash/title_to_type.go
index a17edeeb9..b6d96c41a 100644
--- a/pkg/report/title_to_type.go
+++ b/pkg/report/crash/title_to_type.go
@@ -1,35 +1,31 @@
 // Copyright 2025 syzkaller project authors. All rights reserved.
 // Use of this source code is governed by Apache 2 LICENSE that can be found in the LICENSE file.
 
-package report
-
-import (
-	"github.com/google/syzkaller/pkg/report/crash"
-)
+package crash
 
 var titleToType = []struct {
 	includePrefixes []string
-	crashType       crash.Type
+	crashType       Type
 }{
 	{
 		includePrefixes: []string{
 			"KFENCE: use-after-free write",
 		},
-		crashType: crash.KFENCEUseAfterFreeWrite,
+		crashType: KFENCEUseAfterFreeWrite,
 	},
 	{
 		includePrefixes: []string{
 			"KFENCE: use-after-free read",
 			"KFENCE: use-after-free", // Read/Write is not clear. It is at least Read.
 		},
-		crashType: crash.KFENCEUseAfterFreeRead,
+		crashType: KFENCEUseAfterFreeRead,
 	},
 	{
 		includePrefixes: []string{
 			"KFENCE: invalid write",
 			"KFENCE: out-of-bounds write",
 		},
-		crashType: crash.KFENCEWrite,
+		crashType: KFENCEWrite,
 	},
 	{
 		includePrefixes: []string{
@@ -39,25 +35,25 @@ var titleToType = []struct {
 			"KFENCE: out-of-bounds", // Read/Write is not clear. It is at least Read.
 			// keep-sorted end
 		},
-		crashType: crash.KFENCERead,
+		crashType: KFENCERead,
 	},
 	{
 		includePrefixes: []string{
 			"KFENCE: memory corruption",
 		},
-		crashType: crash.KFENCEMemoryCorruption,
+		crashType: KFENCEMemoryCorruption,
 	},
 	{
 		includePrefixes: []string{
 			"KFENCE: invalid free",
 		},
-		crashType: crash.KFENCEInvalidFree,
+		crashType: KFENCEInvalidFree,
 	},
 	{
 		includePrefixes: []string{
 			"KMSAN: uninit-value",
 		},
-		crashType: crash.KMSANUninitValue,
+		crashType: KMSANUninitValue,
 	},
 	{
 		includePrefixes: []string{
@@ -67,32 +63,32 @@ var titleToType = []struct {
 			"KMSAN: use-after-free",
 			// keep-sorted end
 		},
-		crashType: crash.KMSANUseAfterFreeRead,
+		crashType: KMSANUseAfterFreeRead,
 	},
 	{
 		includePrefixes: []string{
 			"KMSAN: kernel-infoleak",
 			"KMSAN: kernel-usb-infoleak",
 		},
-		crashType: crash.KMSANInfoLeak,
+		crashType: KMSANInfoLeak,
 	},
 	{
 		includePrefixes: []string{
 			"KASAN: null-ptr-deref Write",
 		},
-		crashType: crash.KASANNullPtrDerefWrite,
+		crashType: KASANNullPtrDerefWrite,
 	},
 	{
 		includePrefixes: []string{
 			"KASAN: null-ptr-deref Read",
 		},
-		crashType: crash.KASANNullPtrDerefRead,
+		crashType: KASANNullPtrDerefRead,
 	},
 	{
 		includePrefixes: []string{
 			"BUG: unable to handle kernel NULL pointer dereference in",
 		},
-		crashType: crash.NullPtrDerefBUG,
+		crashType: NullPtrDerefBUG,
 	},
 	{
 		includePrefixes: []string{
@@ -106,7 +102,7 @@ var titleToType = []struct {
 			"KASAN: wild-memory-access Write",
 			// keep-sorting end
 		},
-		crashType: crash.KASANWrite,
+		crashType: KASANWrite,
 	},
 	{
 		includePrefixes: []string{
@@ -125,21 +121,21 @@ var titleToType = []struct {
 			"KASAN: wild-memory-access", // Read/Write is not clear. It is at least Read.
 			// keep-sorting end
 		},
-		crashType: crash.KASANRead,
+		crashType: KASANRead,
 	},
 	{
 		includePrefixes: []string{
 			"KASAN: double-free or invalid-free",
 			"KASAN: invalid-free",
 		},
-		crashType: crash.KASANInvalidFree,
+		crashType: KASANInvalidFree,
 	},
 	{
 		includePrefixes: []string{
 			"KASAN: slab-use-after-free Write",
 			"KASAN: use-after-free Write",
 		},
-		crashType: crash.KASANUseAfterFreeWrite,
+		crashType: KASANUseAfterFreeWrite,
 	},
 	{
 		includePrefixes: []string{
@@ -147,7 +143,7 @@ var titleToType = []struct {
 			"KASAN: use-after-free Read",
 			"KASAN: use-after-free", // Read/Write is not clear. It is at least Read.
 		},
-		crashType: crash.KASANUseAfterFreeRead,
+		crashType: KASANUseAfterFreeRead,
 	},
 	{
 		includePrefixes: []string{
@@ -156,27 +152,27 @@ var titleToType = []struct {
 			"BUG: unable to handle kernel paging request",
 			// keep-sorting end
 		},
-		crashType: crash.MemorySafetyBUG,
+		crashType: MemorySafetyBUG,
 	},
 	{
 		includePrefixes: []string{
 			"WARNING: refcount bug",
 		},
-		crashType: crash.RefcountWARNING,
+		crashType: RefcountWARNING,
 	},
 	{
 		includePrefixes: []string{
 			"UBSAN: array-index-out-of-bounds",
 		},
-		crashType: crash.MemorySafetyUBSAN,
+		crashType: MemorySafetyUBSAN,
 	},
 	{
 		includePrefixes: []string{"KCSAN: data-race"},
-		crashType:       crash.KCSANDataRace,
+		crashType:       KCSANDataRace,
 	},
 	{
 		includePrefixes: []string{"KCSAN: assert: race in"},
-		crashType:       crash.KCSANAssert,
+		crashType:       KCSANAssert,
 	},
 	{
 		includePrefixes: []string{
@@ -198,7 +194,7 @@ var titleToType = []struct {
 			"possible deadlock in",
 			// keep-sorted end
 		},
-		crashType: crash.LockdepBug,
+		crashType: LockdepBug,
 	},
 	{
 		includePrefixes: []string{
@@ -207,11 +203,11 @@ var titleToType = []struct {
 			"BUG: sleeping function called from invalid context in",
 			// keep-sorted end
 		},
-		crashType: crash.AtomicSleep,
+		crashType: AtomicSleep,
 	},
 	{
 		includePrefixes: []string{"memory leak in"},
-		crashType:       crash.MemoryLeak,
+		crashType:       MemoryLeak,
 	},
 	{
 		includePrefixes: []string{
@@ -220,11 +216,11 @@ var titleToType = []struct {
 			"kernel BUG",
 			// keep-sorted end
 		},
-		crashType: crash.Bug,
+		crashType: Bug,
 	},
 	{
 		includePrefixes: []string{"WARNING in"},
-		crashType:       crash.Warning,
+		crashType:       Warning,
 	},
 	{
 		includePrefixes: []string{
@@ -235,7 +231,7 @@ var titleToType = []struct {
 			"WARNING: kernel stack regs has bad",
 			// keep-sorted end
 		},
-		crashType: crash.UnknownType, // This is printk().
+		crashType: UnknownType, // This is printk().
 	},
 	{
 		includePrefixes: []string{
@@ -246,7 +242,7 @@ var titleToType = []struct {
 			"INFO: task hung in",
 			// keep-sorted end
 		},
-		crashType: crash.Hang,
+		crashType: Hang,
 	},
 	{
 		includePrefixes: []string{
@@ -271,51 +267,51 @@ var titleToType = []struct {
 			"unregister_netdevice: waiting for DEV to become free",
 			// keep-sorted end
 		},
-		crashType: crash.DoS,
+		crashType: DoS,
 	},
 	{
 		includePrefixes: []string{"unexpected kernel reboot"},
-		crashType:       crash.UnexpectedReboot,
+		crashType:       UnexpectedReboot,
 	},
 	{
 		includePrefixes: []string{
 			"SYZFAIL",
 			"SYZFATAL:",
 		},
-		crashType: crash.SyzFailure,
+		crashType: SyzFailure,
 	},
 
 	// DEFAULTS.
 	{
 		includePrefixes: []string{"WARNING:"},
-		crashType:       crash.Warning,
+		crashType:       Warning,
 	},
 	{
 		includePrefixes: []string{"BUG:"},
-		crashType:       crash.UnknownType,
+		crashType:       UnknownType,
 	},
 	{
 		includePrefixes: []string{"INFO:"},
-		crashType:       crash.UnknownType,
+		crashType:       UnknownType,
 	},
 	{
 		includePrefixes: []string{"KASAN:"},
-		crashType:       crash.KASANUnknown,
+		crashType:       KASANUnknown,
 	},
 	{
 		includePrefixes: []string{"KFENCE:"},
-		crashType:       crash.KFENCEUnknown,
+		crashType:       KFENCEUnknown,
 	},
 	{
 		includePrefixes: []string{"KMSAN:"},
-		crashType:       crash.KMSANUnknown,
+		crashType:       KMSANUnknown,
 	},
 	{
 		includePrefixes: []string{"UBSAN:"},
-		crashType:       crash.UBSAN,
+		crashType:       UBSAN,
 	},
 	{
 		includePrefixes: []string{"KCSAN:"},
-		crashType:       crash.KCSANUnknown,
+		crashType:       KCSANUnknown,
 	},
 }
diff --git a/pkg/report/title_to_type_test.go b/pkg/report/crash/title_to_type_test.go
index a152a3445..5ffa13412 100644
--- a/pkg/report/title_to_type_test.go
+++ b/pkg/report/crash/title_to_type_test.go
@@ -1,7 +1,7 @@
 // Copyright 2025 syzkaller project authors. All rights reserved.
 // Use of this source code is governed by Apache 2 LICENSE that can be found in the LICENSE file.
 
-package report
+package crash
 
 import (
 	"strings"
diff --git a/pkg/report/crash/types.go b/pkg/report/crash/types.go
index 4410230f2..915f3b4ae 100644
--- a/pkg/report/crash/types.go
+++ b/pkg/report/crash/types.go
@@ -3,7 +3,10 @@
 
 package crash
 
-import "slices"
+import (
+	"slices"
+	"strings"
+)
 
 type Type string
 
@@ -50,6 +53,17 @@ const (
 	UnexpectedReboot = Type("REBOOT")
 )
 
+func TitleToType(title string) Type {
+	for _, t := range titleToType {
+		for _, prefix := range t.includePrefixes {
+			if strings.HasPrefix(title, prefix) {
+				return t.crashType
+			}
+		}
+	}
+	return UnknownType
+}
+
 func (t Type) String() string {
 	if t == UnknownType {
 		return "UNKNOWN"
diff --git a/pkg/report/impact_score.go b/pkg/report/impact_score.go
index 8139644d2..dec727d1d 100644
--- a/pkg/report/impact_score.go
+++ b/pkg/report/impact_score.go
@@ -55,7 +55,7 @@ var impactOrder = []crash.Type{
 func TitlesToImpact(title string, otherTitles ...string) int {
 	maxImpact := -1
 	for _, t := range append([]string{title}, otherTitles...) {
-		typ := TitleToCrashType(t)
+		typ := crash.TitleToType(t)
 		for i, t := range impactOrder {
 			if typ == t {
 				maxImpact = max(maxImpact, len(impactOrder)-i)
diff --git a/pkg/report/linux.go b/pkg/report/linux.go
index f4266658c..01949f304 100644
--- a/pkg/report/linux.go
+++ b/pkg/report/linux.go
@@ -190,7 +190,7 @@ func (ctx *linux) Parse(output []byte) *Report {
 		}
 		rep.reportPrefixLen = len(rep.Report)
 		rep.Report = append(rep.Report, report...)
-		rep.Type = TitleToCrashType(rep.Title)
+		rep.Type = crash.TitleToType(rep.Title)
 		setExecutorInfo(rep)
 		if !rep.Corrupted {
 			rep.Corrupted, rep.CorruptedReason = isCorrupted(title, report, format)
diff --git a/pkg/report/report.go b/pkg/report/report.go
index b4f93e3ac..ce9c50f2c 100644
--- a/pkg/report/report.go
+++ b/pkg/report/report.go
@@ -800,7 +800,7 @@ func simpleLineParser(output []byte, oopses []*oops, params *stackParams, ignore
 	rep.Report = output[rep.StartPos:]
 	rep.Corrupted = corrupted != ""
 	rep.CorruptedReason = corrupted
-	rep.Type = TitleToCrashType(rep.Title)
+	rep.Type = crash.TitleToType(rep.Title)
 	return rep
 }
 
@@ -933,17 +933,6 @@ var groupGoRuntimeErrors = oops{
 	},
 }
 
-func TitleToCrashType(title string) crash.Type {
-	for _, t := range titleToType {
-		for _, prefix := range t.includePrefixes {
-			if strings.HasPrefix(title, prefix) {
-				return t.crashType
-			}
-		}
-	}
-	return crash.UnknownType
-}
-
 const reportSeparator = "\n<<<<<<<<<<<<<<< tail report >>>>>>>>>>>>>>>\n\n"
 
 func MergeReportBytes(reps []*Report) []byte {
diff --git a/pkg/report/report_test.go b/pkg/report/report_test.go
index 8a4a4975d..6a0a3f6c3 100644
--- a/pkg/report/report_test.go
+++ b/pkg/report/report_test.go
@@ -210,7 +210,7 @@ func testFromReport(rep *Report) *ParseTest {
 		Corrupted:       rep.Corrupted,
 		corruptedReason: rep.CorruptedReason,
 		Suppressed:      rep.Suppressed,
-		Type:            TitleToCrashType(rep.Title),
+		Type:            crash.TitleToType(rep.Title),
 		Frame:           rep.Frame,
 		Report:          rep.Report,
 	}