mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/chenhuacai/linux-loongson
synced 2025-08-28 00:19:36 +00:00
iio: backend: make sure to NULL terminate stack buffer
Make sure to NULL terminate the buffer in
iio_backend_debugfs_write_reg() before passing it to sscanf(). It is a
stack variable so we should not assume it will 0 initialized.
Fixes: cdf01e0809
("iio: backend: add debugFs interface")
Signed-off-by: Nuno Sá <nuno.sa@analog.com>
Reviewed-by: David Lechner <dlechner@baylibre.com>
Link: https://patch.msgid.link/20250218-dev-iio-misc-v1-1-bf72b20a1eb8@analog.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
This commit is contained in:
parent
1a21a9842e
commit
035b498921
@ -155,10 +155,12 @@ static ssize_t iio_backend_debugfs_write_reg(struct file *file,
|
||||
ssize_t rc;
|
||||
int ret;
|
||||
|
||||
rc = simple_write_to_buffer(buf, sizeof(buf), ppos, userbuf, count);
|
||||
rc = simple_write_to_buffer(buf, sizeof(buf) - 1, ppos, userbuf, count);
|
||||
if (rc < 0)
|
||||
return rc;
|
||||
|
||||
buf[count] = '\0';
|
||||
|
||||
ret = sscanf(buf, "%i %i", &back->cached_reg_addr, &val);
|
||||
|
||||
switch (ret) {
|
||||
|
Loading…
Reference in New Issue
Block a user