pespin has submitted this change. (
https://gerrit.osmocom.org/c/osmo-hnbgw/+/29546 )
Change subject: Fix handling of sctp SCTP_SHUTDOWN_EVENT notification
......................................................................
Fix handling of sctp SCTP_SHUTDOWN_EVENT notification
SCTP_SHUTDOWN_EVENT is a first class event, and not a subtype of
SCTP_ASSOC_CHANGE (such as SCTP_SHUTDOWN_COMP).
Related: SYS#6113
Change-Id: I7fa648142c07f63c55091d2a15b9d7312bcd4cec
---
M src/osmo-hnbgw/hnbgw.c
1 file changed, 7 insertions(+), 7 deletions(-)
Approvals:
Jenkins Builder: Verified
neels: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
diff --git a/src/osmo-hnbgw/hnbgw.c b/src/osmo-hnbgw/hnbgw.c
index 6d75525..0c59c7d 100644
--- a/src/osmo-hnbgw/hnbgw.c
+++ b/src/osmo-hnbgw/hnbgw.c
@@ -262,15 +262,15 @@
LOGHNB(hnb, DMAIN, LOGL_NOTICE, "HNB SCTP conn
RESTARTed, marking as HNBAP-unregistered\n");
hnb->hnb_registered = false;
break;
- case SCTP_SHUTDOWN_EVENT:
- LOGHNB(hnb, DMAIN, LOGL_NOTICE,
- "sctp_recvmsg(%s) = SCTP_SHUTDOWN_EVENT,
closing conn\n",
-
osmo_sock_get_name2(osmo_stream_srv_get_ofd(conn)->fd));
- osmo_stream_srv_destroy(conn);
- rc = -1;
- break;
}
break;
+ case SCTP_SHUTDOWN_EVENT:
+ LOGHNB(hnb, DMAIN, LOGL_NOTICE,
+ "sctp_recvmsg(%s) = SCTP_SHUTDOWN_EVENT, closing
conn\n",
+
osmo_sock_get_name2(osmo_stream_srv_get_ofd(conn)->fd));
+ osmo_stream_srv_destroy(conn);
+ rc = -1;
+ break;
}
goto out;
} else if (rc == -EAGAIN) {
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/29546
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I7fa648142c07f63c55091d2a15b9d7312bcd4cec
Gerrit-Change-Number: 29546
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
