<p>Pau Espin Pedrol <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/13433">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Replace broken ipa_ccm_idtag APIs with new ipa_ccm_id ones<br><br>ipa_ccm_idtag_parse_off is broken, and can only be used with<br>len_offset=1 on ID Request messages, otherwise won't work correctly.<br>Modify ipa_ccm_idtag_parse to at least parse those correctly, and<br>document the limitations.<br><br>Those two functions are already deprecated and only used in openbsc by 3<br>callers:<br>* ipa_ccm_idtag_parse in ussd_read_cb(): Broken, that function can only<br>work for Requests and it's used to parse a Response.<br>* ipa_ccm_idtag_parse_off in forward_sccp_to_msc (NAT): Broken, it can<br>only be used to parse Requests and it's used to parse a Response.<br>Furthermore, len_offset=2 is passed which makes no sense and most<br>probably it fails always, or can even make the program crash.<br>* ipa_ccm_idtag_parse_off in (answer_challenge): This one is fine and<br>could actually be replaced with ipa_ccm_id_get_parse after libosmocore<br>commit (see below) is merged.<br><br>See libosmocore I6efc852dfc041192f554e41a58290a0f63298021 for more information.<br><br>As a consequence of the fixes, osmo-bsc-nat now parses messages sent<br>from VTY test correctly and thus it goes into processing them instead of<br>silently dropping them. As a result, some VTY tests fail because they<br>are sending incorrect format (missing NULL char in unit id strings) and<br>osmo-bsc-nat closses its connection (due to bad auth).<br><br>Change-Id: I3b995f8ef0b48c0a5b3375e42926641934359cd2<br>---<br>M openbsc/src/osmo-bsc/osmo_bsc_msc.c<br>M openbsc/src/osmo-bsc_nat/bsc_nat.c<br>M openbsc/src/osmo-bsc_nat/bsc_ussd.c<br>M openbsc/tests/vty_test_runner.py<br>4 files changed, 10 insertions(+), 12 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/33/13433/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/13433">change 13433</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/13433"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: openbsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I3b995f8ef0b48c0a5b3375e42926641934359cd2 </div>
<div style="display:none"> Gerrit-Change-Number: 13433 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: Daniel Willmann <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Daniel Willmann <dwillmann@sysmocom.de> </div>