laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve laforge: Looks good to me, approved
sua: Dispatch M-NOTIFY.ind to ASP

This was already done in M3UA counterpart m3ua_rx_mgmt_ntfy(), but was
missing in the SUA one.

Change-Id: I62255aea1448bb76181b5f6799c3b9bf8c7daa79
---
M src/sua.c
1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/sua.c b/src/sua.c
index 3b54c22..572e77b 100644
--- a/src/sua.c
+++ b/src/sua.c
@@ -34,7 +34,6 @@
#include <osmocom/core/fsm.h>

#include <osmocom/netif/stream.h>
-#include "xua_msg.h"

#include <osmocom/sigtran/sccp_sap.h>
#include <osmocom/sigtran/protocol/mtp.h>
@@ -50,6 +49,8 @@
#include "ss7_as.h"
#include "ss7_asp.h"
#include "ss7_internal.h"
+#include "xua_msg.h"
+#include "xua_lm_sap.h"

/* Appendix C.4 of Q.714 (all in milliseconds) */
#define CONNECTION_TIMER ( 1 * 60 * 100)
@@ -653,6 +654,7 @@
{
struct osmo_xlm_prim_notify ntfy;
const char *type_name, *info_name;
+ struct osmo_xlm_prim *prim;

m3ua_decode_notify(&ntfy, asp, xua);

@@ -662,10 +664,13 @@
type_name, info_name,
ntfy.info_string ? ntfy.info_string : "");

+ /* report this to layer manager */
+ prim = xua_xlm_prim_alloc_m_notify_ind(&ntfy);
+ xua_asp_send_xlm_prim(asp, prim);
+
if (ntfy.info_string)
talloc_free(ntfy.info_string);

- /* TODO: should we report this somewhere? */
return 0;
}


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

Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I62255aea1448bb76181b5f6799c3b9bf8c7daa79
Gerrit-Change-Number: 43219
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>