From: Javier Carrasco <javier.carrasco.cruz@xxxxxxxxx>
commit 6007d10c5262f6f71479627c1216899ea7f09073 upstream.
The 'sample' local struct is used to push data to user space from a
triggered buffer, but it has a hole between the temperature and the
timestamp (u32 pressure, u16 temperature, GAP, u64 timestamp).
This hole is never initialized.
Initialize the struct to zero before using it to avoid pushing
uninitialized information to userspace.
Cc: stable@xxxxxxxxxxxxxxx
Fixes: 03b262f2bbf4 ("iio:pressure: initial zpa2326 barometer support")
Signed-off-by: Javier Carrasco <javier.carrasco.cruz@xxxxxxxxx>
Link: https://patch.msgid.link/20241125-iio_memset_scan_holes-v1-3-0cb6e98d895c@xxxxxxxxx
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
drivers/iio/pressure/zpa2326.c | 2 ++
1 file changed, 2 insertions(+)
--- a/drivers/iio/pressure/zpa2326.c
+++ b/drivers/iio/pressure/zpa2326.c
@@ -585,6 +585,8 @@ static int zpa2326_fill_sample_buffer(st
} sample;
int err;
+ memset(&sample, 0, sizeof(sample));
+
if (test_bit(0, indio_dev->active_scan_mask)) {
/* Get current pressure from hardware FIFO. */
err = zpa2326_dequeue_pressure(indio_dev, &sample.pressure);
Patches currently in stable-queue which might be from javier.carrasco.cruz@xxxxxxxxx are
queue-5.4/iio-pressure-zpa2326-fix-information-leak-in-triggered-buffer.patch
queue-5.4/iio-dummy-iio_simply_dummy_buffer-fix-information-leak-in-triggered-buffer.patch
queue-5.4/iio-imu-kmx61-fix-information-leak-in-triggered-buffer.patch
queue-5.4/iio-light-vcnl4035-fix-information-leak-in-triggered-buffer.patch
queue-5.4/iio-adc-ti-ads8688-fix-information-leak-in-triggered-buffer.patch
