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 <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to