diff options
author | Neels Hofmeyr <neels@hofmeyr.de> | 2017-09-28 03:07:16 +0200 |
---|---|---|
committer | Neels Hofmeyr <nhofmeyr@sysmocom.de> | 2017-10-03 20:30:17 +0000 |
commit | 834f94a2cb9dee69031228a82196265bf541de6d (patch) | |
tree | 40e9870640d5abb28dacb68e2340f082441ebcdf /tests | |
parent | 5c76e0aa1a32d09d1be8a9ca9461ecebd38cad87 (diff) |
fix memory leak: vlr: vlr_gsupc_read_cb() must msgb_free()
Add required msgb_free() to vlr_gsupc_read_cb().
Adjust msc_vlr_tests.c gsup_rx() to *not* free the msgb again after
vlr_gsupc_read_cb() did.
Related: OS#2476
Change-Id: I347c53f57a7fa79921aed3f6e42599841acf27c0
Diffstat (limited to 'tests')
-rw-r--r-- | tests/msc_vlr/msc_vlr_tests.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/msc_vlr/msc_vlr_tests.c b/tests/msc_vlr/msc_vlr_tests.c index a6e0f29a0..f9dc27856 100644 --- a/tests/msc_vlr/msc_vlr_tests.c +++ b/tests/msc_vlr/msc_vlr_tests.c @@ -130,12 +130,12 @@ void gsup_rx(const char *rx_hex, const char *expect_tx_hex) label = osmo_gsup_message_type_name(msg->l2h[0]); fprintf(stderr, "<-- GSUP rx %s: %s\n", label, osmo_hexdump_nospc(msgb_l2(msg), msgb_l2len(msg))); + /* GSUP read cb takes ownership of msgb */ rc = vlr_gsupc_read_cb(net->vlr->gsup_client, msg); fprintf(stderr, "<-- GSUP rx %s: vlr_gsupc_read_cb() returns %d\n", label, rc); if (expect_tx_hex) OSMO_ASSERT(gsup_tx_confirmed); - talloc_free(msg); } bool conn_exists(struct gsm_subscriber_connection *conn) |