drivers/iio: Remove all strcpy() uses
authorLen Baker <len.baker@gmx.com>
Sun, 15 Aug 2021 17:42:04 +0000 (19:42 +0200)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Tue, 14 Sep 2021 11:00:30 +0000 (12:00 +0100)
commitd722f1e06fbc53eb369b39646945c1fa92068e74
treee309f6ab7ac4f43165952a083e9c4b8adfac1f1b
parent12ed27863ea3148239ec368e16c1a0f937e4d9bd
drivers/iio: Remove all strcpy() uses

strcpy() performs no bounds checking on the destination buffer. This
could result in linear overflows beyond the end of the buffer, leading
to all kinds of misbehaviors. So, remove all the uses and add
devm_kstrdup() or devm_kasprintf() instead.

Also, modify the "for" loop conditions to clarify the access to the
st->orientation.rotation buffer.

This patch is an effort to clean up the proliferation of str*()
functions in the kernel and a previous step in the path to remove
the strcpy function from the kernel entirely [1].

[1] https://github.com/KSPP/linux/issues/88

Signed-off-by: Len Baker <len.baker@gmx.com>
Link: https://lore.kernel.org/r/20210815174204.126593-1-len.baker@gmx.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c