neels has uploaded this change for review.

View Change

mark two possible asn1c leaks with FIXME

Change-Id: Ic13ca13bf3c95eb7ab20fad508d03bc899f34f15
---
M src/osmo-hnbgw/hnbgw_hnbap.c
1 file changed, 3 insertions(+), 0 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/27/31027/1
diff --git a/src/osmo-hnbgw/hnbgw_hnbap.c b/src/osmo-hnbgw/hnbgw_hnbap.c
index 4aefe2b..07612c7 100644
--- a/src/osmo-hnbgw/hnbgw_hnbap.c
+++ b/src/osmo-hnbgw/hnbgw_hnbap.c
@@ -147,6 +147,7 @@
memset(&accept_out, 0, sizeof(accept_out));
rc = hnbap_encode_ueregisteraccepties(&accept_out, &accept);
if (rc < 0) {
+ // FIXME: leaking 'accept'
return rc;
}

@@ -343,6 +344,8 @@
* never be a UE DE-REGISTER for this UE from the HNB, and the ue_context would linger forever. */
if (ue_allocated)
ue_context_free(ue_allocated);
+
+ // FIXME: leaking 'accept'
return rc;
}


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

Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Ic13ca13bf3c95eb7ab20fad508d03bc899f34f15
Gerrit-Change-Number: 31027
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr@sysmocom.de>
Gerrit-MessageType: newchange