pespin has submitted this change. (
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(-)
Approvals:
laforge: Looks good to me, approved
daniel: Looks good to me, but someone else must approve
Jenkins Builder: Verified
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);
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/41505?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I2977ddd4aa476172e61b76f7a6501f0af9d25062
Gerrit-Change-Number: 41505
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
