Change in ...osmo-sip-connector[master]: avoid bogus error logs when no cmd_timer is set

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/.

keith gerrit-no-reply at lists.osmocom.org
Wed Aug 14 08:50:42 UTC 2019


keith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-sip-connector/+/15142 )

Change subject: avoid bogus error logs when no cmd_timer is set
......................................................................


Patch Set 1:

Looking closer, Log is kind of hard on the eyes for sure, I see that we got the response (RTP_CREATE) and stopped the timer. 
Maybe we should clear leg->rsp_wanted when we stop the timer, so that the LOG message can be more clear in that we got an UNEXPECTED or out of sequence command from the other side of the MNCC.

Maybe my knowledge of the MNCC sequence is missing something, but the osmo-sip-connector is only expecting MNCC_REL_CNF in response to it having sent a MNCC_REL_REQ. Is this wrong?


-- 
To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/15142
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sip-connector
Gerrit-Branch: master
Gerrit-Change-Id: I70f85a71df55ab8618ed78864cefb6fe5b26f581
Gerrit-Change-Number: 15142
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at gnumonks.org>
Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de>
Gerrit-CC: keith <keith at rhizomatica.org>
Gerrit-Comment-Date: Wed, 14 Aug 2019 08:50:42 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190814/69e1db6b/attachment.htm>


More information about the gerrit-log mailing list