Commit d9bcd462 authored by Heiner Kallweit's avatar Heiner Kallweit Committed by Bartosz Golaszewski

eeprom: at24: check at24_read/write arguments

So far we completely rely on the caller to provide valid arguments.
To be on the safe side perform an own sanity check.

Cc: stable@vger.kernel.org
Signed-off-by: default avatarHeiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: default avatarBartosz Golaszewski <brgl@bgdev.pl>
parent 644a1f19
...@@ -569,6 +569,9 @@ static int at24_read(void *priv, unsigned int off, void *val, size_t count) ...@@ -569,6 +569,9 @@ static int at24_read(void *priv, unsigned int off, void *val, size_t count)
if (unlikely(!count)) if (unlikely(!count))
return count; return count;
if (off + count > at24->chip.byte_len)
return -EINVAL;
client = at24_translate_offset(at24, &off); client = at24_translate_offset(at24, &off);
ret = pm_runtime_get_sync(&client->dev); ret = pm_runtime_get_sync(&client->dev);
...@@ -614,6 +617,9 @@ static int at24_write(void *priv, unsigned int off, void *val, size_t count) ...@@ -614,6 +617,9 @@ static int at24_write(void *priv, unsigned int off, void *val, size_t count)
if (unlikely(!count)) if (unlikely(!count))
return -EINVAL; return -EINVAL;
if (off + count > at24->chip.byte_len)
return -EINVAL;
client = at24_translate_offset(at24, &off); client = at24_translate_offset(at24, &off);
ret = pm_runtime_get_sync(&client->dev); ret = pm_runtime_get_sync(&client->dev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment