Merge pull request #78 from markusressel/enhancement/validate_hwmon_i…

…d_config added validation check for hwmon index
markusressel · Dec 19, 2021 · 8ba9871 · 8ba9871
2 parents 6b628f1 + 7e426f2
commit 8ba9871
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/internal/configuration/config.go b/internal/configuration/config.go
@@ -114,6 +114,12 @@ func validateSensors(config *Configuration) {
 		if !isSensorConfigInUse(sensorConfig, config.Curves) {
 			ui.Warning("Unused sensor configuration: %s", sensorConfig.ID)
 		}
+
+		if sensorConfig.HwMon != nil {
+			if sensorConfig.HwMon.Index <= 0 {
+				ui.Fatal("Sensor %s: invalid index, must be >= 1", sensorConfig.ID)
+			}
+		}
 	}
 }
 
@@ -209,5 +215,11 @@ func validateFans(config *Configuration) {
 		if len(fanConfig.Curve) <= 0 {
 			ui.Fatal("Fan %s: missing curve definition in configuration entry", fanConfig.ID)
 		}
+
+		if fanConfig.HwMon != nil {
+			if fanConfig.HwMon.Index <= 0 {
+				ui.Fatal("Fan %s: invalid index, must be >= 1", fanConfig.ID)
+			}
+		}
 	}
 }