diff options
| author | Heinrich Schuchardt <xypron.glpk@gmx.de> | 2021-02-01 03:01:54 +0100 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2021-02-03 03:38:41 -0700 |
| commit | 6f5edbaf938952e4a9860727c633dcaa36caf699 (patch) | |
| tree | daaf7db3ddddf0ff1504821691459ec254f6ad27 /drivers/firmware | |
| parent | c30a7093e81be0dbec963b6b347f678630c2eb8d (diff) | |
| download | u-boot-6f5edbaf938952e4a9860727c633dcaa36caf699.zip u-boot-6f5edbaf938952e4a9860727c633dcaa36caf699.tar.gz u-boot-6f5edbaf938952e4a9860727c633dcaa36caf699.tar.bz2 | |
firmware: smci: possible NULL dereference
sandbox_scmi_devices_ctx() may return NULL. We should not dereference this
value in sandbox_scmi_devices_remove().
The problem was indicated by 'gcc-11 -fanalyzer'.
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'drivers/firmware')
| -rw-r--r-- | drivers/firmware/scmi/sandbox-scmi_devices.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/firmware/scmi/sandbox-scmi_devices.c b/drivers/firmware/scmi/sandbox-scmi_devices.c index 414da6f..1a6fafb 100644 --- a/drivers/firmware/scmi/sandbox-scmi_devices.c +++ b/drivers/firmware/scmi/sandbox-scmi_devices.c @@ -50,6 +50,9 @@ static int sandbox_scmi_devices_remove(struct udevice *dev) int ret = 0; size_t n; + if (!devices) + return 0; + for (n = 0; n < SCMI_TEST_DEVICES_RD_COUNT; n++) { int ret2 = reset_free(devices->reset + n); |