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 /src | |
| 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 'src')
| -rw-r--r-- | src/libvlr/vlr.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/libvlr/vlr.c b/src/libvlr/vlr.c index 6094c9c43..adfe258db 100644 --- a/src/libvlr/vlr.c +++ b/src/libvlr/vlr.c @@ -804,6 +804,7 @@ int vlr_gsupc_read_cb(struct gsup_client *gsupc, struct msgb *msg) osmo_hexdump_nospc(msgb_l2(msg), msgb_l2len(msg))); rc = osmo_gsup_decode(msgb_l2(msg), msgb_l2len(msg), &gsup); + msgb_free(msg); if (rc < 0) { LOGP(DVLR, LOGL_ERROR, "decoding GSUP message fails with error '%s' (%d)\n", |