fixeria has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/28660 )
Change subject: tests/tdef: assert pointer returned by osmo_tdef_get_entry() ......................................................................
tests/tdef: assert pointer returned by osmo_tdef_get_entry()
Coverity complains that we do check if osmo_tdef_get_entry() returns NULL 39 out of 40 times. Check it in test_tdef_set_and_get() too.
Change-Id: I96041eab2786d850a49cb38a60a368cef2e476d3 Related: CID#274729 --- M tests/tdef/tdef_test.c 1 file changed, 1 insertion(+), 0 deletions(-)
Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved
diff --git a/tests/tdef/tdef_test.c b/tests/tdef/tdef_test.c index a2beb6a..c348e69 100644 --- a/tests/tdef/tdef_test.c +++ b/tests/tdef/tdef_test.c @@ -143,6 +143,7 @@
printf("setting 7 = 42\n"); t = osmo_tdef_get_entry(tdefs, 7); + OSMO_ASSERT(t != NULL); OSMO_ASSERT(osmo_tdef_val_in_range(t, 42)); t->val = 42; print_tdef_info(7);