laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve laforge: Looks good to me, approved
bsc: Early shutdown upon BSSMAP RESET-ACK timeout

Change-Id: I75e098d387562267d5851bc753e27e40186827ed
---
M bsc/BSC_Tests.ttcn
1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn
index 1fc42ba..5e4ebbc 100644
--- a/bsc/BSC_Tests.ttcn
+++ b/bsc/BSC_Tests.ttcn
@@ -941,13 +941,15 @@
}
[] BSSAP.receive { repeat; }
[] T.timeout {
- log("BSSMAP: Timeout waiting for RESET-ACK after sending RESET");
/* If we received a RESET after ours was sent, it
may be a race condition where the other peer beacame
available after we sent it, but we are in a desired
state anyway, so go forward. */
if (not reset_received) {
- setverdict(fail);
+ Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail,
+ "BSSMAP: Timeout waiting for RESET-ACK after sending RESET");
+ } else {
+ log("BSSMAP: Timeout waiting for RESET-ACK after sending RESET");
}
}
}

To view, visit change 36578. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I75e098d387562267d5851bc753e27e40186827ed
Gerrit-Change-Number: 36578
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged