diff options
| author | Yasin Lee <yasin.lee.x@gmail.com> | 2026-02-13 23:14:43 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2026-03-19 16:08:47 +0100 |
| commit | 05723a2c4b09dddc258fc7c496b7750c729d3fd9 (patch) | |
| tree | 2fbea561a1d2e4c1c97dd4c8e7c624d522f15dae | |
| parent | 70b455484a661ed6da5a3f6c8273d77dd95d4af6 (diff) | |
| download | linux-05723a2c4b09dddc258fc7c496b7750c729d3fd9.tar.gz linux-05723a2c4b09dddc258fc7c496b7750c729d3fd9.tar.bz2 linux-05723a2c4b09dddc258fc7c496b7750c729d3fd9.zip | |
iio: proximity: hx9023s: fix assignment order for __counted_by
commit 585b90c0161ab77416fe3acdbdc55b978e33e16c upstream.
Initialize fw_size before copying firmware data into the flexible
array member to match the __counted_by() annotation. This fixes the
incorrect assignment order that triggers runtime safety checks.
Fixes: e9ed97be4fcc ("iio: proximity: hx9023s: Added firmware file parsing functionality")
Signed-off-by: Yasin Lee <yasin.lee.x@gmail.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/iio/proximity/hx9023s.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/iio/proximity/hx9023s.c b/drivers/iio/proximity/hx9023s.c index 2918dfc0df54..ad839db6b326 100644 --- a/drivers/iio/proximity/hx9023s.c +++ b/drivers/iio/proximity/hx9023s.c @@ -1034,9 +1034,8 @@ static int hx9023s_send_cfg(const struct firmware *fw, struct hx9023s_data *data if (!bin) return -ENOMEM; - memcpy(bin->data, fw->data, fw->size); - bin->fw_size = fw->size; + memcpy(bin->data, fw->data, bin->fw_size); bin->fw_ver = bin->data[FW_VER_OFFSET]; bin->reg_count = get_unaligned_le16(bin->data + FW_REG_CNT_OFFSET); |