Change in osmo-ttcn3-hacks[master]: bsc: Fix f_establish_fully() for cases where we expect ASSIGN FAIL

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Tue Jun 5 21:43:51 UTC 2018


Harald Welte has uploaded this change for review. ( https://gerrit.osmocom.org/9468


Change subject: bsc: Fix f_establish_fully() for cases where we expect ASSIGN FAIL
......................................................................

bsc: Fix f_establish_fully() for cases where we expect ASSIGN FAIL

When f_ass_patch_lcls() was recently introduced during LCLS support
patches, we broke any testcases that *expected* an ASSIGNMENT FAIL
by overwriting the ASSIGNMENT FAIL with an ASSIGNMENT COMPL.

This patch fixes f_ass_patch_lcls() to only patch 'assignmentComplete'
members, if this assignmentComplete is actually chosen.

This fixes BSC_Tests.TC_assignment_fr_a5_1_codec_missing

Change-Id: I64fbf4cc3178a91913143960475a0d3758779ced
---
M bsc/MSC_ConnectionHandler.ttcn
1 file changed, 6 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/68/9468/1

diff --git a/bsc/MSC_ConnectionHandler.ttcn b/bsc/MSC_ConnectionHandler.ttcn
index 225e86b..0846762 100644
--- a/bsc/MSC_ConnectionHandler.ttcn
+++ b/bsc/MSC_ConnectionHandler.ttcn
@@ -742,10 +742,12 @@
 	if (ispresent(g_pars.lcls.csc)) {
 		ass_tpl.pdu.bssmap.assignmentRequest.lCLS_ConnectionStatusControl := ts_BSSMAP_IE_LclsCsc(g_pars.lcls.csc);
 	}
-	if (ispresent(g_pars.lcls.exp_sts)) {
-		ass_cpl.pdu.bssmap.assignmentComplete.lCLS_BSS_Status := tr_BSSMAP_IE_LclsSts(g_pars.lcls.exp_sts);
-	} else {
-		ass_cpl.pdu.bssmap.assignmentComplete.lCLS_BSS_Status := omit;
+	if (ischosen(ass_cpl.pdu.bssmap.assignmentComplete)) {
+		if (ispresent(g_pars.lcls.exp_sts)) {
+			ass_cpl.pdu.bssmap.assignmentComplete.lCLS_BSS_Status := tr_BSSMAP_IE_LclsSts(g_pars.lcls.exp_sts);
+		} else {
+			ass_cpl.pdu.bssmap.assignmentComplete.lCLS_BSS_Status := omit;
+		}
 	}
 }
 

-- 
To view, visit https://gerrit.osmocom.org/9468
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I64fbf4cc3178a91913143960475a0d3758779ced
Gerrit-Change-Number: 9468
Gerrit-PatchSet: 1
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180605/5c1fb6e1/attachment.htm>


More information about the gerrit-log mailing list