Merge pull request #65 from mcktr/bugfix/usage-calculation2

Fix dividing by zero
mcktr · Sep 19, 2019 · 8c329c4 · 8c329c4
2 parents ce9e69a + 3d5db02
commit 8c329c4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/cmd/check_fritz/check_downstream.go b/cmd/check_fritz/check_downstream.go
@@ -168,6 +168,12 @@ func CheckDownstreamUsage(aI ArgumentInformation) {
 
 	downstreamCurrent = downstreamCurrent * 8 / 1000000
 	downstreamMax = downstreamMax * 8 / 1000000
+
+	if downstreamMax == 0 {
+		fmt.Printf("UNKNOWN - Maximum Downstream is 0\n")
+		return
+	}
+
 	downstreamUsage := 100 / downstreamMax * downstreamCurrent
 	perfData := perfdata.CreatePerformanceData("downstream_usage", downstreamUsage, "")
 

diff --git a/cmd/check_fritz/check_upstream.go b/cmd/check_fritz/check_upstream.go
@@ -168,6 +168,12 @@ func CheckUpstreamUsage(aI ArgumentInformation) {
 
 	upstreamCurrent = upstreamCurrent * 8 / 1000000
 	upstreamMax = upstreamMax * 8 / 1000000
+
+	if upstreamMax == 0 {
+		fmt.Printf("UNKNOWN - Maximum Downstream is 0\n")
+		return
+	}
+
 	upstreamUsage := 100 / upstreamMax * upstreamCurrent
 	perfData := perfdata.CreatePerformanceData("upstream_usage", upstreamUsage, "")