aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHolger Hans Peter Freyther <zecke@selfish.org>2010-03-31 09:15:05 +0200
committerHolger Hans Peter Freyther <zecke@selfish.org>2010-06-15 20:24:09 +0800
commitf4cfc4f2018d4a9b08f7713f9bc805ac80adb8ce (patch)
treef2058799594230e9dfb465b58617057694126c5f
parentd131b79fc2348366d7da0ae005c143ce2a1f6a95 (diff)
nat: bsc_mgcp_clear is called from within the remove function
No need to call it twice... All endpoints should be properly freed as part of the SCCP disconnection.
-rw-r--r--openbsc/src/nat/bsc_nat.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/openbsc/src/nat/bsc_nat.c b/openbsc/src/nat/bsc_nat.c
index b3754da43..a86895774 100644
--- a/openbsc/src/nat/bsc_nat.c
+++ b/openbsc/src/nat/bsc_nat.c
@@ -448,10 +448,7 @@ static int forward_sccp_to_msc(struct bsc_connection *bsc, struct msgb *msg)
break;
case SCCP_MSG_TYPE_RLC:
con = patch_sccp_src_ref_to_msc(msg, parsed, nat);
- if (con) {
- remove_sccp_src_ref(bsc, msg, parsed);
- bsc_mgcp_clear(con);
- }
+ remove_sccp_src_ref(bsc, msg, parsed);
break;
case SCCP_MSG_TYPE_UDT:
/* simply forward everything */