jolly has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-msc/+/41504?usp=email )
Change subject: ASCI: Fix establishment of calls when ciphering is enabled ......................................................................
ASCI: Fix establishment of calls when ciphering is enabled
We do not support any VGCS/VBS ciphering. If the CM Service Request indicates VGCS or VBS service, encryption is not enabled for the channel to be set up. There will be no Ciphering Command sent to the ME.
Authentication is still performed with the initiator of the call, if authentication is enabled. Only authenticated subscribers will then be able to initiate ASCI calls.
Change-Id: Ic59221e6b0f64c9f19508ae31cc3672548d0666e --- M src/libvlr/vlr_access_req_fsm.c 1 file changed, 5 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/04/41504/1
diff --git a/src/libvlr/vlr_access_req_fsm.c b/src/libvlr/vlr_access_req_fsm.c index 91ff06e..4a1c854 100644 --- a/src/libvlr/vlr_access_req_fsm.c +++ b/src/libvlr/vlr_access_req_fsm.c @@ -301,9 +301,12 @@ LOGPFSM(fi, "%s()\n", __func__);
/* Continue with ciphering, if enabled. - * If auth/ciph is optional and the HLR returned no auth info, continue without ciphering. */ + * If auth/ciph is optional and the HLR returned no auth info, continue without ciphering. + * Also continue without ciphering, if ASCI call was requested. */ if (!is_cmc_smc_to_be_attempted(par) - || (vsub->sec_ctx == VLR_SEC_CTX_NONE && !par->is_ciphering_required)) { + || (vsub->sec_ctx == VLR_SEC_CTX_NONE && !par->is_ciphering_required) + || par->cm_service_type == GSM48_CMSERV_VGCS + || par->cm_service_type == GSM48_CMSERV_VBS) { _proc_arq_vlr_node2_post_ciph(fi); return; }