diff options
| author | Eric Sesterhenn <snakebyte@gmx.de> | 2006-09-26 08:22:00 -0400 |
|---|---|---|
| committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2006-09-27 18:17:54 -0400 |
| commit | d1d8f7dec179a421d6d449ddcd1713ed55db40be (patch) | |
| tree | 46dff83ffc5618eb22b60add847434b7e03ada53 /arch/arm/common/icst525.c | |
| parent | 576b3ef2495c732a56509febd5de5144f3ebccf6 (diff) | |
[ARM] Off-by-one in arch/arm/common/icst*
hi,
a quick find -iname \*.[ch] | xargs grep "> ARRAY_SIZE(", revealed
these in the icst drivers.
If i == ARRAY_SIZE, we get past the idx2s array.
Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/common/icst525.c')
| -rw-r--r-- | arch/arm/common/icst525.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/arm/common/icst525.c b/arch/arm/common/icst525.c index 943ef88c0379..3d377c5bdef6 100644 --- a/arch/arm/common/icst525.c +++ b/arch/arm/common/icst525.c | |||
| @@ -55,7 +55,7 @@ icst525_khz_to_vco(const struct icst525_params *p, unsigned long freq) | |||
| 55 | break; | 55 | break; |
| 56 | } while (i < ARRAY_SIZE(idx2s)); | 56 | } while (i < ARRAY_SIZE(idx2s)); |
| 57 | 57 | ||
| 58 | if (i > ARRAY_SIZE(idx2s)) | 58 | if (i >= ARRAY_SIZE(idx2s)) |
| 59 | return vco; | 59 | return vco; |
| 60 | 60 | ||
| 61 | vco.s = idx2s[i]; | 61 | vco.s = idx2s[i]; |
| @@ -118,7 +118,7 @@ icst525_ps_to_vco(const struct icst525_params *p, unsigned long period) | |||
| 118 | break; | 118 | break; |
| 119 | } while (i < ARRAY_SIZE(idx2s)); | 119 | } while (i < ARRAY_SIZE(idx2s)); |
| 120 | 120 | ||
| 121 | if (i > ARRAY_SIZE(idx2s)) | 121 | if (i >= ARRAY_SIZE(idx2s)) |
| 122 | return vco; | 122 | return vco; |
| 123 | 123 | ||
| 124 | vco.s = idx2s[i]; | 124 | vco.s = idx2s[i]; |