<p>dexter <strong>uploaded patch set #7</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24941">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp_protocol: refactor MGCP request handling<br><br>At the moment the MGCP request handling and message parsing is not<br>clearly separated. The function mgcp_parse_header() in mgcp_msg.c is<br>also responsible for resolving an endpoint. This leads to unclear layer<br>separation. We eventually end up in a situation where we can not execute<br>any request handler without beeing able to resolve an endpoint, however<br>this is necessary if we want to implement wildcarded DLCX resquests.<br><br>In the current situation a wildcarded DLCX is not possible to implement<br>as we always have to resolve a an to get to the trunk which we need to<br>iterate. However, we just can't resolve a free endpoint in a situation<br>where all endpoints on te trunk are in use.<br><br>We have to refactor the request handler so that the parsing in mgcp_msg<br>only extracts us the endpoint name. The resolving is then done in<br>mgcp_handle_message() in mgcp_protocol.c. Then we are able to decide<br>what to do if we are unable to resolve an endpoint but still be able to<br>resolve the trunk.<br><br>This patch does not change the behaviour of osmo-mgw yet, but it lays<br>the foundation for request handler implementations that can still<br>perform useful actions if no endpoint but a trunk has been resolved. A<br>wilcarded DLCX is such a case. It does not need an endpoint, just the<br>trunk.<br><br>Change-Id: I9f519d8a0ee8a513fa1e74acf3ee7dbc0991cdde<br>Related: SYS#5535<br>---<br>M include/osmocom/mgcp/mgcp_protocol.h<br>M src/libosmo-mgcp/mgcp_msg.c<br>M src/libosmo-mgcp/mgcp_protocol.c<br>M src/libosmo-mgcp/mgcp_sdp.c<br>4 files changed, 208 insertions(+), 139 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/41/24941/7</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24941">change 24941</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-mgw/+/24941"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I9f519d8a0ee8a513fa1e74acf3ee7dbc0991cdde </div>
<div style="display:none"> Gerrit-Change-Number: 24941 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>