<p>neels <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/15137">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">ptmap: implicitly match  '/8000' and '/8000/1'<br><br>In codecs_same(), do not compare the complete audio_name. The parts of it are<br>already checked individually:<br>- subtype_name ("AMR"),<br>- rate ("8000"; defaults to 8000 if omitted) and<br>- channels ("1"; defaults to 1 if omitted)<br>So by also checking the complete audio_name, we brushed over the match of<br>implicit "/8000" and "/8000/1", which otherwise works out fine.<br><br>As a result, translating payload type numbers in RTP headers now also works if<br>one conn of an endpoint set an rtpmap with "AMR/8000" and the other conn set<br>"AMR/8000/1".<br><br>It seems to me that most PBX out there generate ptmaps omitting the "/1", so<br>fixing this should make us more interoperable with third party SDP.<br><br>See IETF RFC4566 section 6. SDP Attributes:<br>  For audio streams, <encoding parameters> indicates the number<br>  of audio channels.  This parameter is OPTIONAL and may be<br>  omitted if the number of channels is one, provided that no<br>  additional parameters are needed.<br><br>Also allowing to omit the "/8000" is a mere side effect of this patch.<br>Omitting the rate does not seem to be specified in an RFC, but is logical for<br>audio codecs defined to require exactly 8000 set as rate (most GSM codecs).<br><br>Add tests in mgcp_test.c.<br><br>Change-Id: Iab00bf9a55b1847f85999077114b37e70fb677c2<br>---<br>M src/libosmo-mgcp/mgcp_codec.c<br>M tests/mgcp/mgcp_test.c<br>M tests/mgcp/mgcp_test.ok<br>3 files changed, 108 insertions(+), 2 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/37/15137/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/15137">change 15137</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/+/15137"/><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: Iab00bf9a55b1847f85999077114b37e70fb677c2 </div>
<div style="display:none"> Gerrit-Change-Number: 15137 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>