Change in libosmo-abis[master]: trau_sync: Check return value of osmo_fsm_register()
laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/19539 ) Change subject: trau_sync: Check return value of osmo_fsm_register() .. trau_sync: Check return value of osmo_fsm_register() Change-Id: Ieaf524423033864a8b79ea1b04b62568d10ffbd3 Closes: CID#211593 --- M src/trau/trau_sync.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/src/trau/trau_sync.c b/src/trau/trau_sync.c index dfd7ce4..730cf0c 100644 --- a/src/trau/trau_sync.c +++ b/src/trau/trau_sync.c @@ -524,5 +524,5 @@ for (i = 0; i < ARRAY_SIZE(sync_patterns); i++) sync_pattern_register(_patterns[i]); - osmo_fsm_register(_sync_fsm); + OSMO_ASSERT(osmo_fsm_register(_sync_fsm) == 0); } -- To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/19539 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-abis Gerrit-Branch: master Gerrit-Change-Id: Ieaf524423033864a8b79ea1b04b62568d10ffbd3 Gerrit-Change-Number: 19539 Gerrit-PatchSet: 1 Gerrit-Owner: laforge Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge Gerrit-MessageType: merged
Change in libosmo-abis[master]: trau_sync: Check return value of osmo_fsm_register()
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/19539 ) Change subject: trau_sync: Check return value of osmo_fsm_register() .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/19539 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-abis Gerrit-Branch: master Gerrit-Change-Id: Ieaf524423033864a8b79ea1b04b62568d10ffbd3 Gerrit-Change-Number: 19539 Gerrit-PatchSet: 1 Gerrit-Owner: laforge Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge Gerrit-Comment-Date: Thu, 06 Aug 2020 10:05:17 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in libosmo-abis[master]: trau_sync: Check return value of osmo_fsm_register()
laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-abis/+/19539 ) Change subject: trau_sync: Check return value of osmo_fsm_register() .. trau_sync: Check return value of osmo_fsm_register() Change-Id: Ieaf524423033864a8b79ea1b04b62568d10ffbd3 Closes: CID#211593 --- M src/trau/trau_sync.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/39/19539/1 diff --git a/src/trau/trau_sync.c b/src/trau/trau_sync.c index dfd7ce4..730cf0c 100644 --- a/src/trau/trau_sync.c +++ b/src/trau/trau_sync.c @@ -524,5 +524,5 @@ for (i = 0; i < ARRAY_SIZE(sync_patterns); i++) sync_pattern_register(_patterns[i]); - osmo_fsm_register(_sync_fsm); + OSMO_ASSERT(osmo_fsm_register(_sync_fsm) == 0); } -- To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/19539 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-abis Gerrit-Branch: master Gerrit-Change-Id: Ieaf524423033864a8b79ea1b04b62568d10ffbd3 Gerrit-Change-Number: 19539 Gerrit-PatchSet: 1 Gerrit-Owner: laforge Gerrit-MessageType: newchange