| From 26e9213898fc949923188ef0aeea31fc87708836 Mon Sep 17 00:00:00 2001 |
| From: Rupesh Majhi <zoone.rupert@gmail.com> |
| Date: Sun, 19 Jul 2026 03:07:52 +0300 |
| Subject: iio: pressure: dps310: fix NULL pointer dereference on ACPI probe |
| |
| From: Rupesh Majhi <zoone.rupert@gmail.com> |
| |
| commit 26e9213898fc949923188ef0aeea31fc87708836 upstream. |
| |
| When the device is enumerated through its ACPI HID (IFX3100), |
| i2c_client_get_device_id() returns NULL: the ACPI-derived client name |
| does not match the driver's i2c_device_id table. dps310_probe() then |
| dereferences that NULL pointer in "iio->name = id->name" and crashes the |
| kernel during probe. |
| |
| The IIO device name is always "dps310", so set it directly and drop the |
| now-unused device-id lookup. |
| |
| Fixes: 72ff282819d0 ("iio: pressure: dps310: Add ACPI HID table") |
| Cc: stable@vger.kernel.org |
| Signed-off-by: Rupesh Majhi <zoone.rupert@gmail.com> |
| Signed-off-by: Jonathan Cameron <jonathan.cameron@oss.qualcomm.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/iio/pressure/dps310.c | 3 +-- |
| 1 file changed, 1 insertion(+), 2 deletions(-) |
| |
| --- a/drivers/iio/pressure/dps310.c |
| +++ b/drivers/iio/pressure/dps310.c |
| @@ -845,7 +845,6 @@ static const struct iio_info dps310_info |
| |
| static int dps310_probe(struct i2c_client *client) |
| { |
| - const struct i2c_device_id *id = i2c_client_get_device_id(client); |
| struct dps310_data *data; |
| struct iio_dev *iio; |
| int rc; |
| @@ -858,7 +857,7 @@ static int dps310_probe(struct i2c_clien |
| data->client = client; |
| mutex_init(&data->lock); |
| |
| - iio->name = id->name; |
| + iio->name = DPS310_DEV_NAME; |
| iio->channels = dps310_channels; |
| iio->num_channels = ARRAY_SIZE(dps310_channels); |
| iio->info = &dps310_info; |
