diff options
| author | Michal Simek <[email protected]> | 2026-04-02 17:36:58 +0200 |
|---|---|---|
| committer | Heinrich Schuchardt <[email protected]> | 2026-04-17 08:02:42 +0200 |
| commit | 583efb5040587f06bab5f5d3eb321186c3a39fe4 (patch) | |
| tree | 70565a3c72924415363e60deb40785cdd4dc1ce3 /test | |
| parent | 57eae396ec91729d5655c26456f8ac3bac439202 (diff) | |
event: Check return value from event_notify_null()
event_notify_null() returns int but its return value is not
checked in run_main_loop() and in fwu_mdata tests.
Add proper error checking to all unchecked call sites.
Signed-off-by: Michal Simek <[email protected]>
Reviewed-by: Simon Glass <[email protected]>
Acked-by: Ilias Apalodimas <[email protected]>
Diffstat (limited to 'test')
| -rw-r--r-- | test/dm/fwu_mdata.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/dm/fwu_mdata.c b/test/dm/fwu_mdata.c index b7680632f95..643b647af0a 100644 --- a/test/dm/fwu_mdata.c +++ b/test/dm/fwu_mdata.c @@ -100,7 +100,7 @@ static int dm_test_fwu_mdata_read(struct unit_test_state *uts) * Trigger lib/fwu_updates/fwu.c fwu_boottime_checks() * to populate g_dev global pointer in that library. */ - event_notify_null(EVT_MAIN_LOOP); + ut_assertok(event_notify_null(EVT_MAIN_LOOP)); ut_assertok(uclass_first_device_err(UCLASS_FWU_MDATA, &dev)); ut_assertok(fwu_init()); @@ -127,7 +127,7 @@ static int dm_test_fwu_mdata_write(struct unit_test_state *uts) * Trigger lib/fwu_updates/fwu.c fwu_boottime_checks() * to populate g_dev global pointer in that library. */ - event_notify_null(EVT_MAIN_LOOP); + ut_assertok(event_notify_null(EVT_MAIN_LOOP)); ut_assertok(uclass_first_device_err(UCLASS_FWU_MDATA, &dev)); |