| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <82d707b22803e2b363a2e0aa0bcf30bc1522f269.camel@gmail.com>
Date: Thu, 12 Jun 2025 07:09:04 +0100
From: Nuno Sá <noname.nuno@...il.com>
To: David Lechner <dlechner@...libre.com>, Michael Hennerich
<michael.hennerich@...log.com>, Lars-Peter Clausen <lars@...afoo.de>,
Jonathan Cameron <jic23@...nel.org>, Nuno Sá
<nuno.sa@...log.com>, Andy Shevchenko <andy@...nel.org>, Matthias Brugger
<matthias.bgg@...il.com>, AngeloGioacchino Del Regno
<angelogioacchino.delregno@...labora.com>, Heiko Stuebner
<heiko@...ech.de>, Maxime Coquelin <mcoquelin.stm32@...il.com>, Alexandre
Torgue <alexandre.torgue@...s.st.com>, Francesco Dolcini
<francesco@...cini.it>, João Paulo
Gonçalves <jpaulo.silvagoncalves@...il.com>, Leonard
Göhrs <l.goehrs@...gutronix.de>, kernel@...gutronix.de,
Oleksij Rempel <o.rempel@...gutronix.de>, Roan van Dijk <roan@...tonic.nl>,
Tomasz Duszynski <tomasz.duszynski@...akon.com>, Jacopo Mondi
<jacopo@...ndi.org>, Jean-Baptiste Maneyrol
<jean-baptiste.maneyrol@....com>, Mudit Sharma
<muditsharma.info@...il.com>, Javier Carrasco
<javier.carrasco.cruz@...il.com>, Ondřej Jirman
<megi@....cz>, Andreas Klinger <ak@...klinger.de>, Petre Rodan
<petre.rodan@...dimension.ro>
Cc: linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-mediatek@...ts.infradead.org,
linux-rockchip@...ts.infradead.org, linux-stm32@...md-mailman.stormreply.com
Subject: Re: [PATCH 00/28] iio: zero init stack with { } instead of memset()
On Wed, 2025-06-11 at 17:38 -0500, David Lechner wrote:
> Jonathan mentioned recently that he would like to get away from using
> memset() to zero-initialize stack memory in the IIO subsystem. And we
> have it on good authority that initializing a struct or array with = { }
> is the preferred way to do this in the kernel [1]. So here is a series
> to take care of that.
>
> [1]: https://lore.kernel.org/linux-iio/202505090942.48EBF01B@keescook/
>
> ---
Reviewed-by: Nuno Sá <nuno.sa@...log.com>
> David Lechner (28):
> iio: accel: adxl372: use = { } instead of memset()
> iio: accel: msa311: use = { } instead of memset()
> iio: adc: dln2-adc: use = { } instead of memset()
> iio: adc: mt6360-adc: use = { } instead of memset()
> iio: adc: rockchip_saradc: use = { } instead of memset()
> iio: adc: rtq6056: use = { } instead of memset()
> iio: adc: stm32-adc: use = { } instead of memset()
> iio: adc: ti-ads1015: use = { } instead of memset()
> iio: adc: ti-ads1119: use = { } instead of memset()
> iio: adc: ti-lmp92064: use = { } instead of memset()
> iio: adc: ti-tsc2046: use = { } instead of memset()
> iio: chemical: scd4x: use = { } instead of memset()
> iio: chemical: scd30: use = { } instead of memset()
> iio: chemical: sunrise_co2: use = { } instead of memset()
> iio: dac: ad3552r: use = { } instead of memset()
> iio: imu: inv_icm42600: use = { } instead of memset()
> iio: imu: inv_mpu6050: use = { } instead of memset()
> iio: light: bh1745: use = { } instead of memset()
> iio: light: ltr501: use = { } instead of memset()
> iio: light: opt4060: use = { } instead of memset()
> iio: light: veml6030: use = { } instead of memset()
> iio: magnetometer: af8133j: use = { } instead of memset()
> iio: pressure: bmp280: use = { } instead of memset()
> iio: pressure: mpl3115: use = { } instead of memset()
> iio: pressure: mprls0025pa: use = { } instead of memset()
> iio: pressure: zpa2326: use = { } instead of memset()
> iio: proximity: irsd200: use = { } instead of memset()
> iio: temperature: tmp006: use = { } instead of memset()
>
> drivers/iio/accel/adxl372.c | 3 +--
> drivers/iio/accel/msa311.c | 4 +---
> drivers/iio/adc/dln2-adc.c | 4 +---
> drivers/iio/adc/mt6360-adc.c | 3 +--
> drivers/iio/adc/rockchip_saradc.c | 4 +---
> drivers/iio/adc/rtq6056.c | 4 +---
> drivers/iio/adc/stm32-adc.c | 3 +--
> drivers/iio/adc/ti-ads1015.c | 4 +---
> drivers/iio/adc/ti-ads1119.c | 4 +---
> drivers/iio/adc/ti-lmp92064.c | 4 +---
> drivers/iio/adc/ti-tsc2046.c | 3 +--
> drivers/iio/chemical/scd30_core.c | 3 +--
> drivers/iio/chemical/scd4x.c | 3 +--
> drivers/iio/chemical/sunrise_co2.c | 6 ++----
> drivers/iio/dac/ad3552r.c | 3 +--
> drivers/iio/imu/inv_icm42600/inv_icm42600_accel.c | 5 ++---
> drivers/iio/imu/inv_icm42600/inv_icm42600_gyro.c | 5 ++---
> drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c | 4 +---
> drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c | 6 ++----
> drivers/iio/light/bh1745.c | 4 +---
> drivers/iio/light/ltr501.c | 4 +---
> drivers/iio/light/opt4060.c | 4 +---
> drivers/iio/light/veml6030.c | 4 +---
> drivers/iio/magnetometer/af8133j.c | 4 +---
> drivers/iio/pressure/bmp280-core.c | 5 +----
> drivers/iio/pressure/mpl3115.c | 3 +--
> drivers/iio/pressure/mprls0025pa_i2c.c | 5 +----
> drivers/iio/pressure/zpa2326.c | 4 +---
> drivers/iio/proximity/irsd200.c | 3 +--
> drivers/iio/temperature/tmp006.c | 4 +---
> 30 files changed, 34 insertions(+), 85 deletions(-)
> ---
> base-commit: 4c6073fec2fee4827fa0dd8a4ab4e6f7bbc05ee6
> change-id: 20250611-iio-zero-init-stack-with-instead-of-memset-0d12d41a7ecb
>
> Best regards,
Powered by blists - more mailing lists