keith has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-msc/+/28509
)
Change subject: Change CC_CAUSE returned on unanswered MT Call
......................................................................
Patch Set 1:
(1 comment)
File src/libmsc/gsm_04_08_cc.c:
https://gerrit.osmocom.org/c/osmo-msc/+/28509/comment/cf08d5bd_b34a264a
PS1, Line 281: (trans->cc.state == GSM_CSTATE_CALL_RECEIVED) ?
In adding this check, I should point out that I do not know what other CC state is
possible when we reach this code, but in order to at least try not use the wrong cause for
other situations, I decided to limit it to GSM_CSTATE_CALL_RECEIVED.
It might be better to know here if it was actually timer mgw X2 that expired. Maybe there
is a way to know that, I don't see it glancing at what seems to be immediately
available in trans or the msc_a. The timer expiration is quite some ways up the stack
through various FSM callbacks. Or maybe there is another state we can check, for example,
I thought to check for COMPLETE_LAYER3_PAGING_RESP. Even so, with those conditions in
place, I don't think there is any way to know if the call was simply unanswered or if
the MGW suffered some other problem?
--
To view, visit
https://gerrit.osmocom.org/c/osmo-msc/+/28509
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I4a9cfc388ec9ecb743d154a114a6db638eac4701
Gerrit-Change-Number: 28509
Gerrit-PatchSet: 1
Gerrit-Owner: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Sun, 03 Jul 2022 03:35:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment