neels submitted this change.

View Change


Approvals: pespin: Looks good to me, approved msuraev: Looks good to me, but someone else must approve Jenkins Builder: Verified
fix asn1 leak in handle_cn_ranap()

Related: SYS#6297
Change-Id: I307f65ae083aba16f75510bcd0a7d1f124e62be4
---
M src/osmo-hnbgw/hnbgw_cn.c
1 file changed, 11 insertions(+), 0 deletions(-)

diff --git a/src/osmo-hnbgw/hnbgw_cn.c b/src/osmo-hnbgw/hnbgw_cn.c
index e2dccf9..fdc5cde 100644
--- a/src/osmo-hnbgw/hnbgw_cn.c
+++ b/src/osmo-hnbgw/hnbgw_cn.c
@@ -276,6 +276,7 @@
}

rc = _cn_ranap_rx(cnlink, unitdata, pdu, data, len);
+ ASN_STRUCT_FREE_CONTENTS_ONLY(asn_DEF_RANAP_RANAP_PDU, pdu);

return rc;
}

To view, visit change 32523. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I307f65ae083aba16f75510bcd0a7d1f124e62be4
Gerrit-Change-Number: 32523
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <msuraev@sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged