diff options
| author | Taras Madan <tarasmadan@google.com> | 2025-07-15 16:50:26 +0200 |
|---|---|---|
| committer | Taras Madan <tarasmadan@google.com> | 2025-07-17 08:31:25 +0000 |
| commit | 94243f0c885f79147a0c19cb2e11dae119683f81 (patch) | |
| tree | 8cd43f579959083313fbcc97e90c9a1b67cb9087 /dashboard/app | |
| parent | abd11cfd08430ec5f9d2c6dbd0e0f798816922d1 (diff) | |
all: manual linter fixes
1. recover the removed comment
2. unnecessary leading newline
3. unnecessary brackets
4. restore dropped "..."
5. use bytes.Equal instead of conversion to string
Diffstat (limited to 'dashboard/app')
| -rw-r--r-- | dashboard/app/kcidb.go | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/dashboard/app/kcidb.go b/dashboard/app/kcidb.go index 7203c6dfa..f2cfeaacb 100644 --- a/dashboard/app/kcidb.go +++ b/dashboard/app/kcidb.go @@ -69,7 +69,10 @@ func publishKcidbBug(c context.Context, client *kcidb.Client, bug *Bug, bugKey * if err != nil { return false, err } - publish := !(rep.KernelCommit == "" || len(rep.KernelConfig) == 0) + // publish == false happens only for syzkaller build/test errors. + // But if this ever happens for a kernel bug, then we also don't want to publish such bugs + // with missing critical info. + publish := rep.KernelCommit != "" && len(rep.KernelConfig) != 0 if publish { if err := client.Publish(rep); err != nil { |