pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-pcu/+/27635 )
Change subject: ms: Remove unneeded bts!=NULL check ......................................................................
ms: Remove unneeded bts!=NULL check
The ms_alloc function is expected to always have a proper bts object passed, since an ms is always attached at a BTS.
Change-Id: I10b3c94dcb2fed4bff40b1ab7ea455175cf77ce4 --- M src/gprs_ms.c 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/35/27635/1
diff --git a/src/gprs_ms.c b/src/gprs_ms.c index 3167dee..63f454f 100644 --- a/src/gprs_ms.c +++ b/src/gprs_ms.c @@ -90,6 +90,7 @@ struct GprsMs *ms_alloc(struct gprs_rlcmac_bts *bts, uint32_t tlli) { struct GprsMs *ms = talloc_zero(tall_pcu_ctx, struct GprsMs); + OSMO_ASSERT(bts);
talloc_set_destructor(ms, ms_talloc_destructor);
@@ -116,8 +117,7 @@
ms_set_mode(ms, GPRS);
- if (ms->bts) - codel_interval = the_pcu->vty.llc_codel_interval_msec; + codel_interval = the_pcu->vty.llc_codel_interval_msec;
if (codel_interval != LLC_CODEL_DISABLE) { if (codel_interval == LLC_CODEL_USE_DEFAULT)