From: Dan Carpenter Date: Thu, 3 Apr 2014 07:22:54 +0000 (+0300) Subject: i2c: qup: off by ones in qup_i2c_probe() X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=199c1df28d41ec057d92aaa6bd8bf9ebd5686c57;p=GitHub%2Fmoto-9609%2Fandroid_kernel_motorola_exynos9610.git i2c: qup: off by ones in qup_i2c_probe() These should ">= ARRAY_SIZE()" instead of "> ARRAY_SIZE()". Fixes: 10c5a8425968 ('i2c: qup: New bus driver for the Qualcomm QUP I2C controller') Signed-off-by: Dan Carpenter Signed-off-by: Wolfram Sang --- diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c index c9d5f788e36b..ee409803fd94 100644 --- a/drivers/i2c/busses/i2c-qup.c +++ b/drivers/i2c/busses/i2c-qup.c @@ -633,12 +633,12 @@ static int qup_i2c_probe(struct platform_device *pdev) * associated with each byte written/received */ size = QUP_OUTPUT_BLOCK_SIZE(io_mode); - if (size > ARRAY_SIZE(blk_sizes)) + if (size >= ARRAY_SIZE(blk_sizes)) return -EIO; qup->out_blk_sz = blk_sizes[size] / 2; size = QUP_INPUT_BLOCK_SIZE(io_mode); - if (size > ARRAY_SIZE(blk_sizes)) + if (size >= ARRAY_SIZE(blk_sizes)) return -EIO; qup->in_blk_sz = blk_sizes[size] / 2;