diff options
| author | Kees Cook <[email protected]> | 2018-06-19 21:38:31 -0700 |
|---|---|---|
| committer | Kees Cook <[email protected]> | 2018-08-13 13:40:52 -0700 |
| commit | 699112f5e831c088ff6aeea594d23ebecf6bd806 (patch) | |
| tree | 812e84b4f90d1870258b94ca288f13a539ddcaa2 | |
| parent | 7daf201d7fe8334e2d2364d4e8ed3394ec9af819 (diff) | |
drm/i2c: tda9950: Remove VLA usage
In the quest to remove all stack VLA usage from the kernel[1], this
sets the buffer to maximum size and adds a sanity check.
[1] https://lkml.kernel.org/r/CA+55aFzCG-zNmZwX4A2FQpadafLfEzK6CC=qPXydAacU1RqZWA@mail.gmail.com
Cc: David Airlie <[email protected]>
Cc: Hans Verkuil <[email protected]>
Cc: Russell King <[email protected]>
Cc: [email protected]
Signed-off-by: Kees Cook <[email protected]>
| -rw-r--r-- | drivers/gpu/drm/i2c/tda9950.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i2c/tda9950.c b/drivers/gpu/drm/i2c/tda9950.c index 3f7396caad48..5d2f0d548469 100644 --- a/drivers/gpu/drm/i2c/tda9950.c +++ b/drivers/gpu/drm/i2c/tda9950.c @@ -76,9 +76,12 @@ struct tda9950_priv { static int tda9950_write_range(struct i2c_client *client, u8 addr, u8 *p, int cnt) { struct i2c_msg msg; - u8 buf[cnt + 1]; + u8 buf[CEC_MAX_MSG_SIZE + 3]; int ret; + if (WARN_ON(cnt > sizeof(buf) - 1)) + return -EINVAL; + buf[0] = addr; memcpy(buf + 1, p, cnt); |