Attention is currently required from: daniel.
neels has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-iuh/+/32613
)
Change subject: fix leak in ranap_new_transp_layer_addr()
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Maybe also rename the function to
ranap_replace_transp_layer_addr() ?
would be a good name, but this is public API of
libosmo-ranap. imho the current name is not bad enough to do the work of changing public
API and adjusting all callers...
--
To view, visit
https://gerrit.osmocom.org/c/osmo-iuh/+/32613
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: Ic92fa317644c3a1f2be9badf455d34fe0b8ea2a4
Gerrit-Change-Number: 32613
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: daniel <dwillmann(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 08 May 2023 13:28:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: daniel <dwillmann(a)sysmocom.de>
Gerrit-MessageType: comment