Julie Qiu | 6040366 | 2021-12-17 18:11:51 -0500 | [diff] [blame] | 1 | // Copyright 2021 The Go Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style |
| 3 | // license that can be found in the LICENSE file. |
| 4 | |
| 5 | //go:build go1.17 && !windows |
| 6 | // +build go1.17,!windows |
| 7 | |
| 8 | package main |
| 9 | |
| 10 | import ( |
Tatiana Bradley | bca6ae2 | 2024-07-15 16:36:44 -0400 | [diff] [blame] | 11 | "context" |
Damien Neil | 74a8e01 | 2022-08-10 14:26:13 -0700 | [diff] [blame] | 12 | "errors" |
Julie Qiu | 6040366 | 2021-12-17 18:11:51 -0500 | [diff] [blame] | 13 | "os" |
| 14 | "os/exec" |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 15 | "path/filepath" |
| 16 | "runtime" |
Damien Neil | 74a8e01 | 2022-08-10 14:26:13 -0700 | [diff] [blame] | 17 | "sort" |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 18 | "strings" |
Julie Qiu | 6040366 | 2021-12-17 18:11:51 -0500 | [diff] [blame] | 19 | "testing" |
Damien Neil | 2c15bf7 | 2022-09-13 16:09:01 -0700 | [diff] [blame] | 20 | "time" |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 21 | |
Damien Neil | 2c15bf7 | 2022-09-13 16:09:01 -0700 | [diff] [blame] | 22 | "github.com/google/go-cmp/cmp" |
| 23 | "github.com/google/go-cmp/cmp/cmpopts" |
Tatiana Bradley | bca6ae2 | 2024-07-15 16:36:44 -0400 | [diff] [blame] | 24 | "golang.org/x/vulndb/cmd/vulnreport/priority" |
Tatiana Bradley | 2ff2c86 | 2024-05-06 17:34:29 -0400 | [diff] [blame] | 25 | "golang.org/x/vulndb/internal/cve5" |
Tatiana Bradley | bca6ae2 | 2024-07-15 16:36:44 -0400 | [diff] [blame] | 26 | "golang.org/x/vulndb/internal/gitrepo" |
Tatiana Bradley | 0cbf4ff | 2023-05-12 16:14:27 -0400 | [diff] [blame] | 27 | "golang.org/x/vulndb/internal/osvutils" |
Tatiana Bradley | 24e908f | 2023-08-29 14:10:18 -0400 | [diff] [blame] | 28 | "golang.org/x/vulndb/internal/proxy" |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 29 | "golang.org/x/vulndb/internal/report" |
Julie Qiu | 6040366 | 2021-12-17 18:11:51 -0500 | [diff] [blame] | 30 | ) |
| 31 | |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 32 | func TestChecksBash(t *testing.T) { |
Julie Qiu | 6040366 | 2021-12-17 18:11:51 -0500 | [diff] [blame] | 33 | bash, err := exec.LookPath("bash") |
| 34 | if err != nil { |
| 35 | t.Skipf("skipping: %v", err) |
| 36 | } |
| 37 | |
Tatiana Bradley | f75fb61 | 2023-10-11 10:57:25 -0400 | [diff] [blame] | 38 | // In short mode (used by presubmit checks), only do offline checks. |
| 39 | var cmd *exec.Cmd |
| 40 | if testing.Short() { |
| 41 | cmd = exec.Command(bash, "./checks.bash", "offline") |
| 42 | } else { |
| 43 | cmd = exec.Command(bash, "./checks.bash") |
| 44 | } |
| 45 | |
Julie Qiu | 6040366 | 2021-12-17 18:11:51 -0500 | [diff] [blame] | 46 | cmd.Stdout = os.Stdout |
| 47 | cmd.Stderr = os.Stderr |
| 48 | if err := cmd.Run(); err != nil { |
| 49 | t.Fatal(err) |
| 50 | } |
| 51 | } |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 52 | |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 53 | func TestLintReports(t *testing.T) { |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 54 | if runtime.GOOS == "android" { |
| 55 | t.Skipf("android builder does not have access to reports/") |
| 56 | } |
Damien Neil | 74a8e01 | 2022-08-10 14:26:13 -0700 | [diff] [blame] | 57 | allFiles := make(map[string]string) |
| 58 | var reports []string |
Tatiana Bradley | defb580 | 2022-11-18 16:26:39 -0500 | [diff] [blame] | 59 | for _, dir := range []string{report.YAMLDir, report.ExcludedDir} { |
cui fliter | c957d8f | 2022-09-14 06:13:32 +0000 | [diff] [blame] | 60 | files, err := os.ReadDir(dir) |
Damien Neil | 74a8e01 | 2022-08-10 14:26:13 -0700 | [diff] [blame] | 61 | if err != nil && !errors.Is(err, os.ErrNotExist) { |
| 62 | t.Fatalf("unable to read %v/: %s", dir, err) |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 63 | } |
Tatiana Bradley | 2551a34 | 2022-09-26 12:18:29 -0400 | [diff] [blame] | 64 | for _, file := range files { |
| 65 | if file.IsDir() { |
Damien Neil | 74a8e01 | 2022-08-10 14:26:13 -0700 | [diff] [blame] | 66 | continue |
| 67 | } |
Tatiana Bradley | 2551a34 | 2022-09-26 12:18:29 -0400 | [diff] [blame] | 68 | if filepath.Ext(file.Name()) != ".yaml" { |
Jonathan Amsterdam | 90283d9 | 2022-08-22 15:01:49 -0400 | [diff] [blame] | 69 | continue |
| 70 | } |
Tatiana Bradley | 2551a34 | 2022-09-26 12:18:29 -0400 | [diff] [blame] | 71 | filename := filepath.Join(dir, file.Name()) |
| 72 | if allFiles[file.Name()] != "" { |
| 73 | t.Errorf("report appears in multiple locations: %v, %v", allFiles[file.Name()], filename) |
Damien Neil | 74a8e01 | 2022-08-10 14:26:13 -0700 | [diff] [blame] | 74 | } |
Tatiana Bradley | 2551a34 | 2022-09-26 12:18:29 -0400 | [diff] [blame] | 75 | allFiles[file.Name()] = filename |
| 76 | reports = append(reports, filename) |
Damien Neil | 74a8e01 | 2022-08-10 14:26:13 -0700 | [diff] [blame] | 77 | } |
| 78 | } |
Tatiana Bradley | a435f71 | 2023-08-29 17:49:11 -0400 | [diff] [blame] | 79 | |
| 80 | // Skip network calls in short mode. |
| 81 | var lint func(r *report.Report) []string |
| 82 | if testing.Short() { |
| 83 | lint = func(r *report.Report) []string { |
| 84 | return r.LintOffline() |
| 85 | } |
| 86 | } else { |
Tatiana Bradley | 44ab8d2 | 2023-08-30 12:45:56 -0400 | [diff] [blame] | 87 | pc := proxy.NewDefaultClient() |
Tatiana Bradley | a435f71 | 2023-08-29 17:49:11 -0400 | [diff] [blame] | 88 | lint = func(r *report.Report) []string { |
| 89 | return r.Lint(pc) |
| 90 | } |
| 91 | } |
| 92 | |
Tatiana Bradley | bca6ae2 | 2024-07-15 16:36:44 -0400 | [diff] [blame] | 93 | vulndb, err := gitrepo.Open(context.Background(), ".") |
| 94 | if err != nil { |
| 95 | t.Fatal(err) |
| 96 | } |
| 97 | rc, err := report.NewClient(vulndb) |
| 98 | if err != nil { |
| 99 | t.Fatal(err) |
| 100 | } |
| 101 | modulesToImports, err := priority.LoadModuleMap() |
| 102 | if err != nil { |
| 103 | t.Fatal(err) |
| 104 | } |
| 105 | |
Tatiana Bradley | adabd30 | 2023-11-20 13:52:02 -0500 | [diff] [blame] | 106 | // Map from summaries to report paths, used to check for duplicate summaries. |
| 107 | summaries := make(map[string]string) |
Damien Neil | 74a8e01 | 2022-08-10 14:26:13 -0700 | [diff] [blame] | 108 | sort.Strings(reports) |
Tatiana Bradley | 2551a34 | 2022-09-26 12:18:29 -0400 | [diff] [blame] | 109 | for _, filename := range reports { |
| 110 | t.Run(filename, func(t *testing.T) { |
| 111 | r, err := report.Read(filename) |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 112 | if err != nil { |
Julie Qiu | e508e32 | 2022-01-04 15:12:43 -0500 | [diff] [blame] | 113 | t.Fatal(err) |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 114 | } |
Tatiana Bradley | 77da7b6 | 2023-05-24 13:07:54 -0400 | [diff] [blame] | 115 | if err := r.CheckFilename(filename); err != nil { |
| 116 | t.Error(err) |
| 117 | } |
Tatiana Bradley | a435f71 | 2023-08-29 17:49:11 -0400 | [diff] [blame] | 118 | lints := lint(r) |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 119 | if len(lints) > 0 { |
| 120 | t.Errorf(strings.Join(lints, "\n")) |
| 121 | } |
Tatiana Bradley | 0fee238 | 2024-06-14 14:25:47 -0400 | [diff] [blame] | 122 | duplicates := make(map[string][]string) |
Tatiana Bradley | 5c064e0 | 2023-05-23 14:29:53 -0400 | [diff] [blame] | 123 | for _, alias := range r.Aliases() { |
Tatiana Bradley | bca6ae2 | 2024-07-15 16:36:44 -0400 | [diff] [blame] | 124 | for _, r2 := range rc.ReportsByAlias(alias) { |
| 125 | if r2.ID != r.ID { |
| 126 | duplicates[r2.ID] = append(duplicates[r2.ID], alias) |
| 127 | } |
Maceo Thompson | e2eba22 | 2022-11-18 13:35:03 -0500 | [diff] [blame] | 128 | } |
| 129 | } |
Tatiana Bradley | bca6ae2 | 2024-07-15 16:36:44 -0400 | [diff] [blame] | 130 | for r2, aliases := range duplicates { |
| 131 | t.Errorf("report %s shares duplicate alias(es) %s with report %s", filename, aliases, r2) |
Tatiana Bradley | 0fee238 | 2024-06-14 14:25:47 -0400 | [diff] [blame] | 132 | } |
Tatiana Bradley | 922b5d4 | 2024-06-04 16:59:28 -0400 | [diff] [blame] | 133 | // Ensure that each reviewed report has a unique summary. |
| 134 | if r.IsReviewed() { |
| 135 | if summary := r.Summary.String(); summary != "" { |
| 136 | if report, ok := summaries[summary]; ok { |
| 137 | t.Errorf("report %s shares duplicate summary %q with report %s", filename, summary, report) |
| 138 | } else { |
| 139 | summaries[summary] = filename |
| 140 | } |
Tatiana Bradley | adabd30 | 2023-11-20 13:52:02 -0500 | [diff] [blame] | 141 | } |
| 142 | } |
Tatiana Bradley | bca6ae2 | 2024-07-15 16:36:44 -0400 | [diff] [blame] | 143 | // Ensure that no unreviewed reports are high priority. |
| 144 | // This can happen because the initial quick triage algorithm |
| 145 | // doesn't know about all affected modules - just the one |
| 146 | // listed in the Github issue. |
| 147 | if r.IsUnreviewed() { |
| 148 | pr, _ := priority.AnalyzeReport(r, rc, modulesToImports) |
| 149 | if pr.Priority == priority.High { |
| 150 | t.Errorf("UNREVIEWED report %s is high priority (should be REVIEWED) - reason: %s", filename, pr.Reason) |
| 151 | } |
| 152 | } |
Tatiana Bradley | 2551a34 | 2022-09-26 12:18:29 -0400 | [diff] [blame] | 153 | // Check that a correct OSV file was generated for each YAML report. |
Damien Neil | 2c15bf7 | 2022-09-13 16:09:01 -0700 | [diff] [blame] | 154 | if r.Excluded == "" { |
Tatiana Bradley | b259823 | 2024-06-21 18:36:29 -0400 | [diff] [blame] | 155 | generated, err := r.ToOSV(time.Time{}) |
| 156 | if err != nil { |
| 157 | t.Fatal(err) |
| 158 | } |
Tatiana Bradley | f762043 | 2023-05-25 11:11:56 -0400 | [diff] [blame] | 159 | osvFilename := r.OSVFilename() |
Tatiana Bradley | defb580 | 2022-11-18 16:26:39 -0500 | [diff] [blame] | 160 | current, err := report.ReadOSV(osvFilename) |
Damien Neil | 2c15bf7 | 2022-09-13 16:09:01 -0700 | [diff] [blame] | 161 | if err != nil { |
| 162 | t.Fatal(err) |
| 163 | } |
Tatiana Bradley | 2551a34 | 2022-09-26 12:18:29 -0400 | [diff] [blame] | 164 | if diff := cmp.Diff(generated, current, cmpopts.EquateEmpty()); diff != "" { |
Tatiana Bradley | defb580 | 2022-11-18 16:26:39 -0500 | [diff] [blame] | 165 | t.Errorf("%s does not match report:\n%v", osvFilename, diff) |
Damien Neil | 2c15bf7 | 2022-09-13 16:09:01 -0700 | [diff] [blame] | 166 | } |
Tatiana Bradley | 0cbf4ff | 2023-05-12 16:14:27 -0400 | [diff] [blame] | 167 | if err := osvutils.ValidateExceptTimestamps(¤t); err != nil { |
| 168 | t.Error(err) |
| 169 | } |
Damien Neil | 2c15bf7 | 2022-09-13 16:09:01 -0700 | [diff] [blame] | 170 | } |
Maceo Thompson | b8b87b1 | 2022-10-20 15:14:42 -0400 | [diff] [blame] | 171 | if r.CVEMetadata != nil { |
Tatiana Bradley | 2ff2c86 | 2024-05-06 17:34:29 -0400 | [diff] [blame] | 172 | generated, err := cve5.FromReport(r) |
Maceo Thompson | b8b87b1 | 2022-10-20 15:14:42 -0400 | [diff] [blame] | 173 | if err != nil { |
| 174 | t.Fatal(err) |
| 175 | } |
Tatiana Bradley | f762043 | 2023-05-25 11:11:56 -0400 | [diff] [blame] | 176 | cvePath := r.CVEFilename() |
Tatiana Bradley | 2ff2c86 | 2024-05-06 17:34:29 -0400 | [diff] [blame] | 177 | current, err := cve5.Read(cvePath) |
Maceo Thompson | b8b87b1 | 2022-10-20 15:14:42 -0400 | [diff] [blame] | 178 | if err != nil { |
| 179 | t.Fatal(err) |
| 180 | } |
| 181 | if diff := cmp.Diff(generated, current, cmpopts.EquateEmpty()); diff != "" { |
| 182 | t.Errorf("%s does not match report:\n%v", cvePath, diff) |
| 183 | } |
| 184 | |
| 185 | } |
Julie Qiu | 733040a | 2021-12-20 17:14:01 -0500 | [diff] [blame] | 186 | }) |
| 187 | } |
| 188 | } |