<p>dexter <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/9474">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">protocol: reject illegal lco options<br><br>At the moment osmo-mgw will accept multiple lco options. (e.g.<br>p:10, a:PCMU, p:10) If LCO appear multiple times, than the first<br>appearance of will be parsed and used, all following appearances<br>will be ignored. However, having multiple appearances of LCO is<br>illegal and affected requests should be rejected. Also osmo-mgw<br>should reject illegal formatted LCO strings<br><br>- make sure that multiple appearances of LCOs will be rejected<br>- make sure that illegal formated LCOs are rejected<br>- add testcases with garbeled LCO and valid LCO examples<br><br>Change-Id: Iae2fddfa5f2bcfc952f8ab217b3056694e5f7812<br>Closes: OS#3119<br>---<br>M include/osmocom/mgcp/mgcp_internal.h<br>M src/libosmo-mgcp/mgcp_protocol.c<br>M tests/mgcp/mgcp_test.c<br>M tests/mgcp/mgcp_test.ok<br>4 files changed, 244 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/74/9474/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/9474">change 9474</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/9474"/><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: Iae2fddfa5f2bcfc952f8ab217b3056694e5f7812 </div>
<div style="display:none"> Gerrit-Change-Number: 9474 </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-CC: Harald Welte <laforge@gnumonks.org> </div>