<p>dexter <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/10172">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp_network: translate payload type numbers in RTP packets<br><br>Since no transcoding is in place osmo-mgw forwards the incoming rtp<br>packets as they are (there may be minor modifications of the header) from<br>an ingress connection to an egress connection.<br><br>This works without problems as long as both connections use the same<br>payload type. For IANA defined fixed payload type numbers this is<br>usually the case, but for dynemic payload type numbers both ends may set<br>up the same codecs but with different payload type numbers.<br><br>When different payload type numbers are set up, and the packet is passed<br>through without modification, it will have the wrong payload type when<br>it is sent. The receiving end may then toss the packet since it expects<br>packets with the payload type it has configured.<br><br>The machanism, which is introduced with this patch looks up actual codec<br>inside the struct data of the ingress connection and then looks for the<br>matching codec in the struct data of the egress connection. When it<br>finds the codec there it looks up the payload type of this codec. The<br>header of the RTP packet is then patched with the correct payoad type.<br><br>- Add function mgcp_codec_pt_translate() to look up the payload type<br>- Add unit-test for function mgcp_codec_pt_translate()<br>- Add payload type translation to mgcp_network.c<br><br>Change-Id: I3a874e59fa07bcc2a67c376cafa197360036f539<br>Related: OS#2728<br>Related: OS#3384<br>---<br>M include/osmocom/mgcp/mgcp_codec.h<br>M src/libosmo-mgcp/mgcp_codec.c<br>M src/libosmo-mgcp/mgcp_network.c<br>M tests/mgcp/mgcp_test.c<br>4 files changed, 204 insertions(+), 0 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/72/10172/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/10172">change 10172</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/10172"/><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-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I3a874e59fa07bcc2a67c376cafa197360036f539 </div>
<div style="display:none"> Gerrit-Change-Number: 10172 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>