neels has submitted this change. (
https://gerrit.osmocom.org/c/osmo-hnbgw/+/32526 )
Change subject: fix asn1 leaks in ps_rab_ass_resp_send_if_ready()
......................................................................
fix asn1 leaks in ps_rab_ass_resp_send_if_ready()
Related: SYS#6297
Change-Id: Icdb6eef0d1fde849c0b9121cc96d8221e72d45c8
---
M src/osmo-hnbgw/ps_rab_ass_fsm.c
1 file changed, 12 insertions(+), 0 deletions(-)
Approvals:
pespin: Looks good to me, approved
msuraev: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/src/osmo-hnbgw/ps_rab_ass_fsm.c b/src/osmo-hnbgw/ps_rab_ass_fsm.c
index 65a6116..464c4d0 100644
--- a/src/osmo-hnbgw/ps_rab_ass_fsm.c
+++ b/src/osmo-hnbgw/ps_rab_ass_fsm.c
@@ -569,6 +569,8 @@
/* Reencode this list item in the RANAP message */
rc = ANY_fromType_aper(&list_ie->value,
&asn_DEF_RANAP_RAB_SetupOrModifiedItem, rab_item);
+ ASN_STRUCT_FREE_CONTENTS_ONLY(asn_DEF_BIT_STRING,
&rab_item->transportLayerAddress);
+ ASN_STRUCT_FREE_CONTENTS_ONLY(asn_DEF_OCTET_STRING,
&rab_item->iuTransportAssociation->choice.gTP_TEI);
if (rc < 0) {
ASN_STRUCT_FREE_CONTENTS_ONLY(asn_DEF_RANAP_RAB_SetupOrModifiedItem, &item_ies);
LOG_PS_RAB_ASS(rab_ass, LOGL_ERROR, "Re-encoding RANAP PS RAB-AssignmentResponse
failed\n");
--
To view, visit
https://gerrit.osmocom.org/c/osmo-hnbgw/+/32526
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Icdb6eef0d1fde849c0b9121cc96d8221e72d45c8
Gerrit-Change-Number: 32526
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged