hwmon: (aquacomputer_d5next) Fix length of speed_input array
authorAleksa Savic <savicaleksa83@gmail.com>
Sun, 24 Nov 2024 15:27:24 +0000 (16:27 +0100)
committerGuenter Roeck <linux@roeck-us.net>
Sun, 24 Nov 2024 15:36:47 +0000 (07:36 -0800)
Commit 120584c728a6 ("hwmon: (aquacomputer_d5next) Add support for Octo
flow sensor") added support for reading Octo flow sensor, but didn't
update the priv->speed_input array length. Since Octo has 8 fans, with
the addition of the flow sensor the proper length for speed_input is 9.

Reported by Arne Schwabe on Github [1], who received a UBSAN warning.

Fixes: 120584c728a6 ("hwmon: (aquacomputer_d5next) Add support for Octo flow sensor")
Closes: https://github.com/aleksamagicka/aquacomputer_d5next-hwmon/issues/100 [1]
Reported-by: Arne Schwabe <arne@rfc2549.org>
Signed-off-by: Aleksa Savic <savicaleksa83@gmail.com>
Message-ID: <20241124152725.7205-1-savicaleksa83@gmail.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/aquacomputer_d5next.c

index 34cac27..0dcb8a3 100644 (file)
@@ -597,7 +597,7 @@ struct aqc_data {
 
        /* Sensor values */
        s32 temp_input[20];     /* Max 4 physical and 16 virtual or 8 physical and 12 virtual */
-       s32 speed_input[8];
+       s32 speed_input[9];
        u32 speed_input_min[1];
        u32 speed_input_target[1];
        u32 speed_input_max[1];