HomeFreeBSD

acpi_battery: avoid divide-by-zero when no devices have capacity info

Description

acpi_battery: avoid divide-by-zero when no devices have capacity info

On laptops with builtin batteries, disconnecting the battery may show up
as a battery without any capacity information. (The theory is that one
is disconnecting the cells but the electronics identifying the battery
are still connected.) As a result, the loop over all batteries in
acpi_battery_get_battinfo results in total_lfcap == 0.

So, just check that total_lfcap is non-zero to avoid a division by zero
(triggerable by sysctl hw.acpi.battery).

Reported by: Stefano Marinelli
Tested by: Stefano Marinelli
Reviewed by: emaste
Differential Revision: https://reviews.freebsd.org/D44818

(cherry picked from commit 2e850b832f5d2adb9b230d191277d67c00caaab9)

Details

Provenance
jeffpc_josefsipek.netAuthored on Apr 16 2024, 11:01 PM
emasteCommitted on Apr 19 2024, 3:16 AM
Reviewer
emaste
Differential Revision
D44818: acpi_battery: avoid divide-by-zero when no devices have capacity info
Parents
rG7a2c2599a760: 14.1: update stable/14 to -PRERELEASE
Branches
Unknown
Tags
Unknown