From 396a9f8ce52c351e0e5179b86ab4cf2f82f5091d Mon Sep 17 00:00:00 2001 From: Holger Obermaier <40787752+ho-ob@users.noreply.github.com> Date: Fri, 6 Feb 2026 13:09:19 +0100 Subject: [PATCH] Fixed Error return value of `...` is not checked (errcheck) --- collectors/cpustatMetric.go | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/collectors/cpustatMetric.go b/collectors/cpustatMetric.go index d66f957..5530e73 100644 --- a/collectors/cpustatMetric.go +++ b/collectors/cpustatMetric.go @@ -80,9 +80,17 @@ func (m *CpustatCollector) Init(config json.RawMessage) error { // Check input file file, err := os.Open(string(CPUSTATFILE)) if err != nil { - cclog.ComponentError(m.name, err.Error()) + cclog.ComponentError( + m.name, + fmt.Sprintf("Init(): Failed to open file '%s': %v", string(CPUSTATFILE), err)) } - defer file.Close() + defer func() { + if err := file.Close(); err != nil { + cclog.ComponentError( + m.name, + fmt.Sprintf("Init(): Failed to close file '%s': %v", string(CPUSTATFILE), err)) + } + }() // Pre-generate tags for all CPUs num_cpus := 0