pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/41505?usp=email )
Change subject: xua_rkm: remove unneeded check rkm_dyn_allocated ......................................................................
xua_rkm: remove unneeded check rkm_dyn_allocated
This is always true, as per checks done further above in the same function.
Change-Id: I2977ddd4aa476172e61b76f7a6501f0af9d25062 --- M src/xua_rkm.c 1 file changed, 4 insertions(+), 7 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/05/41505/1
diff --git a/src/xua_rkm.c b/src/xua_rkm.c index 44a0eb3..d88fda7 100644 --- a/src/xua_rkm.c +++ b/src/xua_rkm.c @@ -428,13 +428,10 @@ * we should refuse RKM DEREG if the ASP is still ACTIVE */ osmo_fsm_inst_dispatch(as->fi, XUA_ASPAS_ASP_DOWN_IND, asp);
- /* if we were dynamically allocated, release the associated - * route and destroy the AS */ - if (as->rkm_dyn_allocated) { - /* remove route + AS definition */ - ss7_route_destroy(rt); - osmo_ss7_as_destroy(as); - } + /* Release the associated route and destroy the dynamically allocated AS */ + ss7_route_destroy(rt); + osmo_ss7_as_destroy(as); + /* report success */ msgb_append_dereg_res(resp, M3UA_RKM_DEREG_SUCCESS, rctx);