pespin has submitted this change. (
https://gerrit.osmocom.org/c/osmo-pcu/+/33274 )
Change subject: tbf_{ul,dl}_fsm: Abort on unexpected path
......................................................................
tbf_{ul,dl}_fsm: Abort on unexpected path
That path should not really happen. It happening would mean a totally wrong
code being used. Hence, just abort() to simplify the code.
Change-Id: Iaf65e909e45cc279e5be4b045c15174f910ffc12
---
M src/tbf_dl_fsm.c
M src/tbf_ul_fsm.c
2 files changed, 14 insertions(+), 8 deletions(-)
Approvals:
Jenkins Builder: Verified
osmith: Looks good to me, approved
diff --git a/src/tbf_dl_fsm.c b/src/tbf_dl_fsm.c
index 9cc2007..abcd0c4 100644
--- a/src/tbf_dl_fsm.c
+++ b/src/tbf_dl_fsm.c
@@ -60,10 +60,7 @@
ch = "PACCH";
break;
default:
- LOGPTBFDL(ctx->dl_tbf, LOGL_ERROR,
- "attempted to %sset unexpected ass. type %d - FIXME!\n",
- set ? "" : "un", t);
- return;
+ OSMO_ASSERT(0);
}
if (set && prev_set)
diff --git a/src/tbf_ul_fsm.c b/src/tbf_ul_fsm.c
index f6615a8..5d02032 100644
--- a/src/tbf_ul_fsm.c
+++ b/src/tbf_ul_fsm.c
@@ -59,10 +59,7 @@
ch = "PACCH";
break;
default:
- LOGPTBFUL(ctx->ul_tbf, LOGL_ERROR,
- "attempted to %sset unexpected ass. type %d - FIXME!\n",
- set ? "" : "un", t);
- return;
+ OSMO_ASSERT(0);
}
if (set && prev_set)
--
To view, visit
https://gerrit.osmocom.org/c/osmo-pcu/+/33274
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Iaf65e909e45cc279e5be4b045c15174f910ffc12
Gerrit-Change-Number: 33274
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged