Change in osmo-mgw[master]: fix: multiple initial CRCX

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Fri Apr 26 20:26:14 UTC 2019


Hello Harald Welte, Jenkins Builder, 

I'd like you to reexamine a change. Please visit

    https://gerrit.osmocom.org/13591

to look at the new patch set (#4).

Change subject: fix: multiple initial CRCX
......................................................................

fix: multiple initial CRCX

The first CRCX responds with the actual MGW endpoint name that is assigned (at
least for rtpbridge/*@mgw requests).

If multiple CRCX are scheduled at the same time on a fresh MGW endpoint, both
get fired with a '*' and each creates a separate MGW endpoint.

Make mgcp_client_endpoint_fsm avoid this, and schedule only one CRCX at first,
and the rest once the MGW endpoint name is fixated. It is thus possible to
safely issue two osmo_mgcpc_ep_ci_request() at the same time.

Change-Id: I92a9944acc96398acd6649f9c3c5badec5dd6dcc
---
M src/libosmo-mgcp-client/mgcp_client_endpoint_fsm.c
1 file changed, 90 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/91/13591/4
-- 
To view, visit https://gerrit.osmocom.org/13591
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I92a9944acc96398acd6649f9c3c5badec5dd6dcc
Gerrit-Change-Number: 13591
Gerrit-PatchSet: 4
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190426/eca4b130/attachment.html>


More information about the gerrit-log mailing list