ttcn3
/
osmo-ttcn3-hacks
mirror of https://gerrit.osmocom.org/osmo-ttcn3-hacks
10
0
Fork 1
Code Issues Releases Wiki Activity

bsc: fix TC_ho_into_this_bsc_a5_4 

This test should be failing as it is, because A5/4 is not permitted in
the BSC's configuration! It only passed all this time because osmo-bsc
is/was/is going to have been flawed and just takes the Chosen Encryption
Algorithm IE as the basis for choosing an encryption algorithm. Instead
it should intersect the pemitted algos with the BSC config. It will soon
do that, and then this test would fail as it should have. After fixing
the BSC config for the test with this patch, we'll not see it failing in
the transition.

Related: SYS#5839
Change-Id: Ic6bbbeea36e6ea26d16bbb510fd08f5c0defb955
This commit is contained in:
Neels Hofmeyr 2022-02-17 21:56:15 +01:00
parent 93182e0dfd
commit 731ddc558f
1 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
bsc/BSC_Tests.ttcn
View File

@ -3743,9 +3743,11 @@ private function f_vty_encryption_a5(charstring options) runs on test_CT {
f_vty_transceive(BSCVTY, "exit");
}
const charstring VTY_A5_DEFAULT := "0 1 3";
private function f_vty_encryption_a5_reset() runs on test_CT {
/* keep in sync with docker-playground.git ttcn3-bsc-test/osmo-bsc.cfg */
f_vty_encryption_a5("0 1 3");
f_vty_encryption_a5(VTY_A5_DEFAULT);
}
/* Establish signalling channel (non-assignment case) followed by cipher mode */
@ -6087,12 +6089,13 @@ private function f_tc_ho_into_this_bsc(charstring id) runs on MSC_ConnHdlr {
f_perform_clear(exp_rr_rel_tmpl := exp_rr_rel_tmpl);
setverdict(pass);
}
function f_tc_ho_into_this_bsc_main(TestHdlrParams pars) runs on test_CT {
function f_tc_ho_into_this_bsc_main(TestHdlrParams pars, charstring vty_a5_cfg := VTY_A5_DEFAULT) runs on test_CT {
var MSC_ConnHdlr vc_conn;
f_init(1, true);
f_sleep(1.0);
f_vty_encryption_a5(vty_a5_cfg);
f_ctrs_bsc_and_bts_handover_init();
pars.sccp_addr_msc := g_bssap[0].sccp_addr_own;
@ -6106,6 +6109,8 @@ function f_tc_ho_into_this_bsc_main(TestHdlrParams pars) runs on test_CT {
f_ctrs_bsc_and_bts_add(0, "interbsc_ho_in:attempted");
f_ctrs_bsc_and_bts_add(0, "interbsc_ho_in:completed");
f_ctrs_bsc_and_bts_verify();
f_vty_encryption_a5_reset();
}
testcase TC_ho_into_this_bsc() runs on test_CT {
@ -6114,10 +6119,10 @@ testcase TC_ho_into_this_bsc() runs on test_CT {
f_shutdown_helper();
}
function f_tc_ho_into_this_bsc_a5(TestHdlrEncrParams encr) runs on test_CT {
function f_tc_ho_into_this_bsc_a5(TestHdlrEncrParams encr, charstring vty_a5_cfg := VTY_A5_DEFAULT) runs on test_CT {
var TestHdlrParams pars := f_gen_test_hdlr_pars();
pars.encr := encr;
f_tc_ho_into_this_bsc_main(pars);
f_tc_ho_into_this_bsc_main(pars, vty_a5_cfg);
f_shutdown_helper();
}
@ -6134,7 +6139,7 @@ testcase TC_ho_into_this_bsc_a5_3() runs on test_CT {
}
testcase TC_ho_into_this_bsc_a5_4() runs on test_CT {
f_tc_ho_into_this_bsc_a5(f_encr_params('10'O, kc128 := true));
f_tc_ho_into_this_bsc_a5(f_encr_params('10'O, kc128 := true), "3 4");
}
/* Report: in inter-BSC incoming handover, when the MSC omits the Chosen Encryption Algorithm IE in the Handover Request