From 13f83ee4a4c8f28d0554755dd4e20e8b9379927b Mon Sep 17 00:00:00 2001 From: Bruce Luckcuck Date: Thu, 26 Mar 2020 14:01:18 -0400 Subject: [PATCH] Prevent division by 0 error in battery usage OSD element If the `bat_capacity` was set to 0 (the default) then it would result in a division by 0. --- src/main/osd/osd_elements.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/osd/osd_elements.c b/src/main/osd/osd_elements.c index 73ded33bcf..b6322b684d 100644 --- a/src/main/osd/osd_elements.c +++ b/src/main/osd/osd_elements.c @@ -1014,7 +1014,7 @@ static void osdElementMainBatteryUsage(osdElementParms_t *element) const float value = constrain(batteryConfig()->batteryCapacity - getMAhDrawn(), 0, batteryConfig()->batteryCapacity); // Calculate mAh used progress - const uint8_t mAhUsedProgress = ceilf((value / (batteryConfig()->batteryCapacity / MAIN_BATT_USAGE_STEPS))); + const uint8_t mAhUsedProgress = (batteryConfig()->batteryCapacity) ? ceilf((value / (batteryConfig()->batteryCapacity / MAIN_BATT_USAGE_STEPS))) : 0; // Create empty battery indicator bar element->buff[0] = SYM_PB_START;