pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39484?usp=email )
Change subject: MGCP_Emulation: Make sure peer is running before Tx
......................................................................
MGCP_Emulation: Make sure peer is running before Tx
This avoid DTE with "Broken pipe" if messages are being transmitted
while tear down process has already started, even if components are
created as "alive".
Change-Id: I8724870b9b967ffa49c3da03e1cdb2941f81653a
---
M library/MGCP_Emulation.ttcn
1 file changed, 10 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/84/39484/1
diff --git a/library/MGCP_Emulation.ttcn b/library/MGCP_Emulation.ttcn
index 3167579..7106f63 100644
--- a/library/MGCP_Emulation.ttcn
+++ b/library/MGCP_Emulation.ttcn
@@ -252,9 +252,17 @@
private function f_forward_to_client(MGCP_RecvFrom mrf, MGCP_ConnHdlr vc_conn) runs on MGCP_Emulation_CT {
if (g_pars.multi_conn_mode) {
- MGCP_CLIENT_MULTI.send(mrf) to vc_conn;
+ if (MGCP_CLIENT_MULTI.checkstate("Connected") and vc_conn.running) {
+ MGCP_CLIENT_MULTI.send(mrf) to vc_conn;
+ } else {
+ log("MGCP_CLIENT_MULTI: Discarding msg due to peer unavailable: ", mrf);
+ }
} else {
- MGCP_CLIENT.send(mrf.msg.command) to vc_conn;
+ if (MGCP_CLIENT.checkstate("Connected") and vc_conn.running) {
+ MGCP_CLIENT.send(mrf.msg.command) to vc_conn;
+ } else {
+ log("MGCP_CLIENT: Discarding msg due to peer unavailable: ", mrf);
+ }
}
}
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39484?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I8724870b9b967ffa49c3da03e1cdb2941f81653a
Gerrit-Change-Number: 39484
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39017?usp=email )
Change subject: SGSN_Tests: rework TC_sgsn_context_req_out to actual function
......................................................................
Patch Set 3:
This change is ready for review.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39017?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I0666a518aed57c2594653ab6b0f801c4193fd2f3
Gerrit-Change-Number: 39017
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Thu, 06 Feb 2025 16:53:24 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Attention is currently required from: pespin.
lynxis lazus has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/39482?usp=email )
Change subject: peek_l3_ul_nas: Parse PTMSI MI from GMM RAU req
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/39482?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I570eb31dcee94a3ebef25e15ac7cd7199ec9a79e
Gerrit-Change-Number: 39482
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 06 Feb 2025 16:47:58 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: lynxis lazus.
pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/39480?usp=email )
Change subject: peek_l3_ul_nas: Improve RAU req parsing checks
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> Same code as I did in sgsn. […]
In here we only want a minimal parsing, so no need to have the full parsing in here imho.
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/39480?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I1a6c642e7017e159a2035869c75475d0de7fbb75
Gerrit-Change-Number: 39480
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Thu, 06 Feb 2025 16:42:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>