diff options
| author | Patrice Chotard <[email protected]> | 2026-02-11 15:32:23 +0100 |
|---|---|---|
| committer | Patrice Chotard <[email protected]> | 2026-02-24 14:14:15 +0100 |
| commit | 6a6f2eb3e6a78abf76566df1e57b85a2a3d35ee2 (patch) | |
| tree | bd3538cf055f49a8037eaf9d7557cc4ffdb77c3f | |
| parent | 32912d0908bbdb56a8863e835a2edc6e13c7159a (diff) | |
board: st: common: add uclass_get_device_by_driver()'s return value check
class_get_device_by_driver()'s return value is not checked, in case of BSEC
driver is not probed, dev is not set and used just after as parameter of
misc_read() which leads to a Synchronous Abort.
Add uclass_get_device_by_driver()'s return value check to fix it.
Signed-off-by: Patrice Chotard <[email protected]>
Reviewed-by: Patrick Delaunay <[email protected]>
| -rw-r--r-- | board/st/common/cmd_stboard.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/board/st/common/cmd_stboard.c b/board/st/common/cmd_stboard.c index d77046499a0..58d9f205b92 100644 --- a/board/st/common/cmd_stboard.c +++ b/board/st/common/cmd_stboard.c @@ -92,6 +92,11 @@ static int do_stboard(struct cmd_tbl *cmdtp, int flag, int argc, DM_DRIVER_GET(stm32mp_bsec), &dev); + if (ret) { + puts("Can't get BSEC device\n"); + return CMD_RET_FAILURE; + } + ret = misc_read(dev, STM32_BSEC_OTP(BSEC_OTP_BOARD), &otp, sizeof(otp)); |