<p><a href="https://gerrit.osmocom.org/13515">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13515/7/src/hlr_ussd.c">File src/hlr_ussd.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13515/7/src/hlr_ussd.c@562">Patch Set #7, Line 562:</a> <code style="font-family:monospace,monospace">                         if (gsup->source_name) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Why do you think we can skip this patch chunk? This is how I've interpreted your comment [1]: […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">gsup->source_name and gsup->destination_name are only included in messages that need routing, i.e. currently exactly for inter-MSC E-interface messages. No other messages contain a source name IE.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Here we are in USSD negotiation land, a completely separate level from inter-MSC routing.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For USSD, you always have the VLR/MSC directly talking to the osmo-hlr, and there is the single GSUP connection on which the response should go back. So that would be stored on the MO USSD request coming in. Otherwise we would lookup the database's vlr_number for the subscriber, but IIUC that doesn't apply here. Definitely never source_name.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13515">change 13515</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/13515"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-hlr </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: If5a65f471672949192061c5fe396603611123bc1 </div>
<div style="display:none"> Gerrit-Change-Number: 13515 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@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-Comment-Date: Mon, 08 Apr 2019 15:04:57 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>