Change in libosmo-sccp[master]: m3ua: fix leaky usage of m3ua_tx_xua_asp

neels gerrit-no-reply at
Mon Jul 12 14:25:23 UTC 2021

neels has submitted this change. ( )

Change subject: m3ua: fix leaky usage of m3ua_tx_xua_asp

m3ua: fix leaky usage of m3ua_tx_xua_asp

m3ua_tx_xua_asp will at some point convert the xua msg to a msgb by
copying and then send it, the xua msg still needs to be freed by the

Closes: OS#5185
Change-Id: Id8584b99f30f2db602d4d129e4114821697272ab
M src/m3ua.c
1 file changed, 5 insertions(+), 1 deletion(-)

  fixeria: Looks good to me, but someone else must approve
  pespin: Looks good to me, but someone else must approve
  neels: Looks good to me, approved
  Jenkins Builder: Verified

diff --git a/src/m3ua.c b/src/m3ua.c
index e2db6c1..4583683 100644
--- a/src/m3ua.c
+++ b/src/m3ua.c
@@ -752,8 +752,10 @@
 		err = m3ua_gen_error_msg(rc, msg);
-	if (err)
+	if (err) {
 		m3ua_tx_xua_asp(asp, err);
+		xua_msg_free(err);
+	}
@@ -856,6 +858,7 @@
 		xua = m3ua_encode_duna(rctx, num_rctx, aff_pc, num_aff_pc, info_string);
 	m3ua_tx_xua_asp(asp, xua);
+	xua_msg_free(xua);
 /*! Transmit SSNM DUPU message indicating user unavailability.
@@ -871,6 +874,7 @@
 	struct xua_msg *xua = m3ua_encode_dupu(rctx, num_rctx, dpc, user, cause, info_str);
 	m3ua_tx_xua_asp(asp, xua);
+	xua_msg_free(xua);
 /* received SNM message on ASP side */

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: Id8584b99f30f2db602d4d129e4114821697272ab
Gerrit-Change-Number: 24909
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen <ewild at>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy at>
Gerrit-Reviewer: laforge <laforge at>
Gerrit-Reviewer: neels <nhofmeyr at>
Gerrit-Reviewer: pespin <pespin at>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list