diff options
| author | Roland Stigge <stigge@antcom.de> | 2012-04-04 02:02:58 +0200 |
|---|---|---|
| committer | Grant Likely <grant.likely@secretlab.ca> | 2012-04-10 22:20:56 +0200 |
| commit | 6270d830d030da48eddffbe31ed1e4444f203fc5 (patch) | |
| tree | bf2cfab7799ca158e65d9cf57c90d5fed395e6b2 /drivers/of | |
| parent | gpio: Fix uninitialized variable bit in adp5588_irq_handler (diff) | |
| download | linux-6270d830d030da48eddffbe31ed1e4444f203fc5.tar.xz linux-6270d830d030da48eddffbe31ed1e4444f203fc5.zip | |
gpio: Fix range check in of_gpio_simple_xlate()
of_gpio_simple_xlate() has an off-by-one bug where it checks to see if
args[0] is > ngpio instead of >=. args[0] must always be less than
ngpio because it is a zero-based enumeration.
Signed-off-by: Roland Stigge <stigge@antcom.de>
[grant.likely: beef up commit text]
Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
Diffstat (limited to 'drivers/of')
| -rw-r--r-- | drivers/of/gpio.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/of/gpio.c b/drivers/of/gpio.c index bba81216b4db..bf984b6dc477 100644 --- a/drivers/of/gpio.c +++ b/drivers/of/gpio.c @@ -140,7 +140,7 @@ int of_gpio_simple_xlate(struct gpio_chip *gc, if (WARN_ON(gpiospec->args_count < gc->of_gpio_n_cells)) return -EINVAL; - if (gpiospec->args[0] > gc->ngpio) + if (gpiospec->args[0] >= gc->ngpio) return -EINVAL; if (flags) |