Change in osmo-mgw[for-1.9.0]: libosmo-mgcp: e1: fix memleaks in e1_recv_cb()

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/27820 ) Change subject: libosmo-mgcp: e1: fix memleaks in e1_recv_cb() ...................................................................... libosmo-mgcp: e1: fix memleaks in e1_recv_cb() Change-Id: I4be9e6d09b34e792f24c9f09d19dce15b9dfbe3f Fixes: OS#5533 --- M src/libosmo-mgcp/mgcp_e1.c 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/src/libosmo-mgcp/mgcp_e1.c b/src/libosmo-mgcp/mgcp_e1.c index e88b8c5..9a6efe2 100644 --- a/src/libosmo-mgcp/mgcp_e1.c +++ b/src/libosmo-mgcp/mgcp_e1.c @@ -338,6 +338,7 @@ trunk = mgcp_trunk_by_line_num(cfg, ts->line->num); if (!trunk) { LOGP(DE1, LOGL_ERROR, "E1-RX: unable to find a trunk for E1-line %u!\n", ts->line->num); + msgb_free(msg); return; } @@ -359,6 +360,9 @@ /* Trigger sending of pending E1 traffic */ e1_send(ts, trunk); + + /* e1inp_rx_ts() does not free() msgb */ + msgb_free(msg); } static int e1_init(struct mgcp_trunk *trunk, uint8_t ts_nr) -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/27820 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: for-1.9.0 Gerrit-Change-Id: I4be9e6d09b34e792f24c9f09d19dce15b9dfbe3f Gerrit-Change-Number: 27820 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: keith <keith(a)rhizomatica.org> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged