diff options
author | Holger Hans Peter Freyther <zecke@selfish.org> | 2010-03-31 09:15:05 +0200 |
---|---|---|
committer | Holger Hans Peter Freyther <zecke@selfish.org> | 2010-06-15 20:24:09 +0800 |
commit | f4cfc4f2018d4a9b08f7713f9bc805ac80adb8ce (patch) | |
tree | f2058799594230e9dfb465b58617057694126c5f | |
parent | d131b79fc2348366d7da0ae005c143ce2a1f6a95 (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.c | 5 |
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 */ |