dme1737: Keep index within pwm_config[]
authorRoel Kluin <[email protected]>
Tue, 22 Sep 2009 23:43:41 +0000 (16:43 -0700)
committerLinus Torvalds <[email protected]>
Wed, 23 Sep 2009 14:39:28 +0000 (07:39 -0700)
The static code scanner "Parfait" reported this because pwm_config is
only 3 bytes - pwm_config[3] is out of range.

Since this code path is never called with ix == 3 (the device has no PWM4
output) this doesn't change anything in practice.  But to encourage
testing with Parfait, lets make the warning go away...

Signed-off-by: Roel Kluin <[email protected]>
Acked-by: Jean Delvare <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
drivers/hwmon/dme1737.c

index 9814d51b3af42893b607a5106b2e4030b3b87ecf..2c2cb1ec94c5f1a22ea47a6607c8408a4fe588c7 100644 (file)
@@ -1134,7 +1134,7 @@ static ssize_t show_pwm(struct device *dev, struct device_attribute *attr,
                res = PWM_FREQ_FROM_REG(data->pwm_freq[ix]);
                break;
        case SYS_PWM_ENABLE:
-               if (ix > 3) {
+               if (ix >= 3) {
                        res = 1; /* pwm[5-6] hard-wired to manual mode */
                } else {
                        res = PWM_EN_FROM_REG(data->pwm_config[ix]);