Fix: Add safety check to hwmon read
Some checks failed
golangci-lint / lint (push) Has been cancelled

Avoid panic for accessing slice out of range in hwmon.

Fixes: https://github.com/prometheus/node_exporter/issues/3108

Signed-off-by: Ben Kochie <superq@gmail.com>
This commit is contained in:
Ben Kochie 2024-09-26 00:00:04 +02:00 committed by Johannes 'fish' Ziemke
parent ebf3cbfbb3
commit 1b332edfe8

View file

@ -18,6 +18,7 @@ package collector
import (
"errors"
"fmt"
"log/slog"
"os"
"path/filepath"
@ -107,6 +108,9 @@ func sysReadFile(file string) ([]byte, error) {
if err != nil {
return nil, err
}
if n < 0 {
return nil, fmt.Errorf("failed to read file: %q, read returned negative bytes value: %d", file, n)
}
return b[:n], nil
}