diff options
author | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2023-07-20 09:20:51 +0300 |
---|---|---|
committer | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2023-07-25 10:24:30 +0300 |
commit | ee630b29ea44d1851bb6c903f400956604834463 (patch) | |
tree | 2b4e3de59a0b9a320a9f11475d920258ba04db76 /drivers/media/tuners/qt1010.c | |
parent | d196d01c703d47c6a9b4c8ee57ce2c550dc7269a (diff) | |
download | linux-ee630b29ea44d1851bb6c903f400956604834463.tar.xz |
media: tuners: qt1010: replace BUG_ON with a regular error
BUG_ON is unnecessary here, and in addition it confuses smatch.
Replacing this with an error return help resolve this smatch
warning:
drivers/media/tuners/qt1010.c:350 qt1010_init() error: buffer overflow 'i2c_data' 34 <= 34
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Diffstat (limited to 'drivers/media/tuners/qt1010.c')
-rw-r--r-- | drivers/media/tuners/qt1010.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/drivers/media/tuners/qt1010.c b/drivers/media/tuners/qt1010.c index 3853a3d43d4f..a7b19863f489 100644 --- a/drivers/media/tuners/qt1010.c +++ b/drivers/media/tuners/qt1010.c @@ -345,11 +345,12 @@ static int qt1010_init(struct dvb_frontend *fe) else valptr = &tmpval; - BUG_ON(i >= ARRAY_SIZE(i2c_data) - 1); - - err = qt1010_init_meas1(priv, i2c_data[i+1].reg, - i2c_data[i].reg, - i2c_data[i].val, valptr); + if (i >= ARRAY_SIZE(i2c_data) - 1) + err = -EIO; + else + err = qt1010_init_meas1(priv, i2c_data[i + 1].reg, + i2c_data[i].reg, + i2c_data[i].val, valptr); i++; break; } |