diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-11-08 12:50:24 +0300 |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2013-11-13 18:49:43 +0530 |
| commit | 02808b427689d46136d9662c5013145f70119f42 (patch) | |
| tree | 5c2e323b7d9504fe7368d2ac4e175033a6ad850c | |
| parent | 2dcbdce3610a1ba190a74b782b9f7f1f7f976325 (diff) | |
| download | linux-02808b427689d46136d9662c5013145f70119f42.tar.bz2 | |
dma: pl330: off by one in pl330_probe()
There are only AMBA_NR_IRQS (2) elements in adev->irq[]. This code
maybe works if the there is a zero directly after the array.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
| -rw-r--r-- | drivers/dma/pl330.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c index 75328f9870d2..ebd23b036a69 100644 --- a/drivers/dma/pl330.c +++ b/drivers/dma/pl330.c @@ -2922,7 +2922,7 @@ pl330_probe(struct amba_device *adev, const struct amba_id *id) amba_set_drvdata(adev, pdmac); - for (i = 0; i <= AMBA_NR_IRQS; i++) { + for (i = 0; i < AMBA_NR_IRQS; i++) { irq = adev->irq[i]; if (irq) { ret = devm_request_irq(&adev->dev, irq, |