<p>Vadim Yanitskiy has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/10323">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">hlr_ussd.c: avoid using CR and NL in IUSE responses<br><br>According to GSM TS 03.38, section 6.1.2.1, CR symbol at the end<br>is optional, and moreover libosmogsm encoding API will carry<br>about the bit padding itself.<br><br>Change-Id: I09e8a67758698f3b7a578eab956311e269d091ee<br>---<br>M src/hlr_ussd.c<br>1 file changed, 2 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-hlr refs/changes/23/10323/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/hlr_ussd.c b/src/hlr_ussd.c</span><br><span>index 7b981bb..d8bbcb5 100644</span><br><span>--- a/src/hlr_ussd.c</span><br><span>+++ b/src/hlr_ussd.c</span><br><span>@@ -297,7 +297,7 @@</span><br><span>                if (strlen(subscr.msisdn) == 0)</span><br><span>                      snprintf(buf, sizeof(buf), "You have no MSISDN!");</span><br><span>                 else</span><br><span style="color: hsl(0, 100%, 40%);">-                    snprintf(buf, sizeof(buf), "Your extension is %s\r", subscr.msisdn);</span><br><span style="color: hsl(120, 100%, 40%);">+                        snprintf(buf, sizeof(buf), "Your extension is %s", subscr.msisdn);</span><br><span>                 ss_tx_ussd_7bit(ss, true, req->invoke_id, buf);</span><br><span>           break;</span><br><span>       case -ENOENT:</span><br><span>@@ -315,7 +315,7 @@</span><br><span>                          const struct osmo_gsup_message *gsup, const struct ss_request *req)</span><br><span> {</span><br><span>     char buf[GSM0480_USSD_7BIT_STRING_LEN+1];</span><br><span style="color: hsl(0, 100%, 40%);">-       snprintf(buf, sizeof(buf), "Your IMSI is %s!\n", ss->imsi);</span><br><span style="color: hsl(120, 100%, 40%);">+      snprintf(buf, sizeof(buf), "Your IMSI is %s!", ss->imsi);</span><br><span>       ss_tx_ussd_7bit(ss, true, req->invoke_id, buf);</span><br><span>   return 0;</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/10323">change 10323</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/10323"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I09e8a67758698f3b7a578eab956311e269d091ee </div>
<div style="display:none"> Gerrit-Change-Number: 10323 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>