diff options
| author | Taras Madan <tarasmadan@google.com> | 2025-04-02 10:59:58 +0200 |
|---|---|---|
| committer | Aleksandr Nogikh <nogikh@google.com> | 2025-04-02 14:58:11 +0000 |
| commit | 514a302df7b2437100847d85646426c56a359e3b (patch) | |
| tree | cfff023ffeb97ad858663a774dc6415fa0d809b0 /tools | |
| parent | b0cc4801a294a67648c4f2b8232e3e1212c0fe43 (diff) | |
tools/syz-covermerger: fix nil deref
Empty flagToGCS means nil WriteCloser.
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/syz-covermerger/syz_covermerger.go | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tools/syz-covermerger/syz_covermerger.go b/tools/syz-covermerger/syz_covermerger.go index 196c4982d..6cd714421 100644 --- a/tools/syz-covermerger/syz_covermerger.go +++ b/tools/syz-covermerger/syz_covermerger.go @@ -124,10 +124,11 @@ func do() error { if err != nil { return fmt.Errorf("covermerger.MergeCSVWriteJSONL: %w", err) } - if err := wc.Close(); err != nil { - return fmt.Errorf("wc.Close: %w", err) + if wc != nil { + if err := wc.Close(); err != nil { + return fmt.Errorf("wc.Close: %w", err) + } } - printCoverage(totalInstrumentedLines, totalCoveredLines) if *flagToDashAPI != "" { // Merging may take hours. It is better to create new connection instead of reuse. |
