diff options
| author | Mauro Carvalho Chehab <[email protected]> | 2024-10-15 10:33:10 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2024-10-18 10:43:03 +0200 |
| commit | 2aee207e5b3c94ef859316008119ea06d6798d49 (patch) | |
| tree | 1423ba116dca41558d2c927cee13f633eb3291af | |
| parent | 9883a4d41aba7612644e9bb807b971247cea9b9d (diff) | |
media: mgb4: protect driver against spectre
Frequency range is set from sysfs via frequency_range_store(),
being vulnerable to spectre, as reported by smatch:
drivers/media/pci/mgb4/mgb4_cmt.c:231 mgb4_cmt_set_vin_freq_range() warn: potential spectre issue 'cmt_vals_in' [r]
drivers/media/pci/mgb4/mgb4_cmt.c:238 mgb4_cmt_set_vin_freq_range() warn: possible spectre second half. 'reg_set'
Fix it.
Fixes: 0ab13674a9bd ("media: pci: mgb4: Added Digiteq Automotive MGB4 driver")
Cc: [email protected]
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Reviewed-by: Martin Tůma <[email protected]>
| -rw-r--r-- | drivers/media/pci/mgb4/mgb4_cmt.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/media/pci/mgb4/mgb4_cmt.c b/drivers/media/pci/mgb4/mgb4_cmt.c index 70dc78ef193c..a25b68403bc6 100644 --- a/drivers/media/pci/mgb4/mgb4_cmt.c +++ b/drivers/media/pci/mgb4/mgb4_cmt.c @@ -227,6 +227,8 @@ void mgb4_cmt_set_vin_freq_range(struct mgb4_vin_dev *vindev, u32 config; size_t i; + freq_range = array_index_nospec(freq_range, ARRAY_SIZE(cmt_vals_in)); + addr = cmt_addrs_in[vindev->config->id]; reg_set = cmt_vals_in[freq_range]; |