diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2025-05-28 23:22:14 +0300 |
|---|---|---|
| committer | Hans Verkuil <hverkuil@xs4all.nl> | 2025-06-17 10:35:15 +0200 |
| commit | aef89c0b2417da79cb2062a95476288f9f203ab0 (patch) | |
| tree | 831a8db342cff80a67fc2aea6ff069813093e523 /tools/perf/scripts/python/bin/stackcollapse-record | |
| parent | d8fbfcab95ffab466aba475127ae677a4c7160fb (diff) | |
| download | linux-aef89c0b2417da79cb2062a95476288f9f203ab0.tar.gz linux-aef89c0b2417da79cb2062a95476288f9f203ab0.zip | |
media: gspca: Add bounds checking to firmware parser
This sd_init() function reads the firmware. The firmware data holds a
series of records and the function reads each record and sends the data
to the device. The request_ihex_firmware() function
calls ihex_validate_fw() which ensures that the total length of all the
records won't read out of bounds of the fw->data[].
However, a potential issue is if there is a single very large
record (larger than PAGE_SIZE) and that would result in memory
corruption. Generally we trust the firmware, but it's always better to
double check.
Fixes: 49b61ec9b5af ("[media] gspca: Add new vicam subdriver")
Cc: stable@vger.kernel.org
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Diffstat (limited to 'tools/perf/scripts/python/bin/stackcollapse-record')
0 files changed, 0 insertions, 0 deletions
