diff options
| author | Taras Madan <tarasmadan@google.com> | 2025-07-03 12:29:03 +0200 |
|---|---|---|
| committer | Taras Madan <tarasmadan@google.com> | 2025-07-03 13:44:30 +0000 |
| commit | 1dff53dc25a42aeeca592d5701a6740f3144211d (patch) | |
| tree | 1dad5eb8b48f1b8daae4e2e59c3cabb9f2dec0e0 /pkg/report | |
| parent | a9cc82a1dd1c40e23a3f26a29f14b2d8f60c65e2 (diff) | |
pkg/report: use crash.KASANUnknown instead of crash.KASANOther
What we need is the category for "matched unknown KASAN bug".
This king on bugs should be recategorised.
The final goal is to keep this category empty.
Diffstat (limited to 'pkg/report')
| -rw-r--r-- | pkg/report/crash/types.go | 4 | ||||
| -rw-r--r-- | pkg/report/title_to_type.go | 16 |
2 files changed, 11 insertions, 9 deletions
diff --git a/pkg/report/crash/types.go b/pkg/report/crash/types.go index cae824161..21bc472a2 100644 --- a/pkg/report/crash/types.go +++ b/pkg/report/crash/types.go @@ -15,8 +15,8 @@ const ( DoS = Type("DoS") Hang = Type("HANG") KASANInvalidFree = Type("KASAN-INVALID-FREE") - KASANOther = Type("KASAN-OTHER") KASANRead = Type("KASAN-READ") + KASANUnknown = Type("KASAN-UNKNOWN") KASANUseAfterFreeRead = Type("KASAN-USE-AFTER-FREE-READ") KASANUseAfterFreeWrite = Type("KASAN-USE-AFTER-FREE-WRITE") KASANWrite = Type("KASAN-WRITE") @@ -51,7 +51,7 @@ type TypeGroupPred func(Type) bool func (t Type) IsKASAN() bool { return slices.Contains([]Type{ - KASANRead, KASANWrite, KASANUseAfterFreeRead, KASANUseAfterFreeWrite, KASANInvalidFree, KASANOther}, t) + KASANRead, KASANWrite, KASANUseAfterFreeRead, KASANUseAfterFreeWrite, KASANInvalidFree, KASANUnknown}, t) } func (t Type) IsKMSAN() bool { diff --git a/pkg/report/title_to_type.go b/pkg/report/title_to_type.go index fc87e4069..f5fb30e68 100644 --- a/pkg/report/title_to_type.go +++ b/pkg/report/title_to_type.go @@ -64,6 +64,7 @@ var titleToType = []struct { "KASAN: user-memory-access Read", "KASAN: vmalloc-out-of-bounds Read", "KASAN: wild-memory-access Read", + "KASAN: wild-memory-access ", // Read/Write is not clear. It is at least Read. // keep-sorting end }, crashType: crash.KASANRead, @@ -77,17 +78,18 @@ var titleToType = []struct { }, { includePrefixes: []string{ - "KASAN: slab-use-after-free Read", - "KASAN: use-after-free Read", + "KASAN: slab-use-after-free Write", + "KASAN: use-after-free Write", }, - crashType: crash.KASANUseAfterFreeRead, + crashType: crash.KASANUseAfterFreeWrite, }, { includePrefixes: []string{ - "KASAN: slab-use-after-free Write", - "KASAN: use-after-free Write", + "KASAN: slab-use-after-free Read", + "KASAN: use-after-free Read", + "KASAN: use-after-free ", // Read/Write is not clear. It is at least Read. }, - crashType: crash.KASANUseAfterFreeWrite, + crashType: crash.KASANUseAfterFreeRead, }, { includePrefixes: []string{ @@ -236,7 +238,7 @@ var titleToType = []struct { }, { includePrefixes: []string{"KASAN: "}, - crashType: crash.KASANOther, + crashType: crash.KASANUnknown, }, { includePrefixes: []string{"KFENCE: "}, |