Harald Welte has submitted this change and it was merged. Change subject: BSC_Tests: Add TC_outbound_connect to test if BSC rejects inbound SCCP connections ......................................................................
BSC_Tests: Add TC_outbound_connect to test if BSC rejects inbound SCCP connections Change-Id: Ie9087df9ff9b48c8be242b1ae66cbca936b07121 --- M bsc/BSC_Tests.ttcn M library/BSSAP_CodecPort.ttcn 2 files changed, 18 insertions(+), 2 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn index 090ed0c..41e3a84 100644 --- a/bsc/BSC_Tests.ttcn +++ b/bsc/BSC_Tests.ttcn @@ -381,6 +381,20 @@ } } +/*********************************************************************** + * Assignment Testing + ***********************************************************************/ + +/* Verify that the BSC refuses any BSSAP connection from the MSC (They are all BSC->MSC direction) */ +testcase TC_outbound_connect() runs on test_CT { + f_init(); + f_bssap_reset(); + + BSSAP.send(ts_BSSAP_CONNECT_req(g_sccp_addr_peer, g_sccp_addr_own, 2342, ts_BSSMAP_AssignmentReq)); + BSSAP.receive(tr_BSSAP_DISC_ind(2342, ?, ?)); + setverdict(pass); +} + /* Test behavior if MSC answers with CREF to CR */ testcase TC_assignment_cic_only() runs on test_CT { var BSSAP_N_CONNECT_ind rx_c_ind; @@ -1110,6 +1124,7 @@ execute( TC_chan_rel_hard_rlsd() ); execute( TC_chan_rel_a_reset() ); + execute( TC_outbound_connect() ); execute( TC_assignment_cic_only() ); execute( TC_rll_est_ind_inact_lchan() ); diff --git a/library/BSSAP_CodecPort.ttcn b/library/BSSAP_CodecPort.ttcn index 5e43608..7470dd4 100644 --- a/library/BSSAP_CodecPort.ttcn +++ b/library/BSSAP_CodecPort.ttcn @@ -21,13 +21,14 @@ template BSSAP_N_CONNECT_req ts_BSSAP_CONNECT_req(SCCP_PAR_Address called, SCCP_PAR_Address calling, - template PDU_BSSAP bssap := *) := { + SCCP_PAR_Connection_Id conn_id, + template PDU_BSSAP bssap := omit) := { calledAddress := called, callingAddress := calling, expeditedDataSel := omit, qualityOfService := omit, userData := bssap, - connectionId := omit, + connectionId := conn_id, importance := omit } -- To view, visit https://gerrit.osmocom.org/5394 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9087df9ff9b48c8be242b1ae66cbca936b07121 Gerrit-PatchSet: 1 Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Owner: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder