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@sysmocom.de
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: msuraev
msuraev@sysmocom.de
Gerrit-Reviewer: pespin
pespin@sysmocom.de
Gerrit-CC: daniel
dwillmann@sysmocom.de
Gerrit-Attention: daniel
dwillmann@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@sysmocom.de
Gerrit-MessageType: comment