<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/13293">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  Harald Welte: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">gprs_ns_sns: Use "correct" remote IP address for local IP endpoint<br><br>we cannot use "nsi->nsip.remote_ip", as this address is not set<br>when SNS is in use.  We can only have a valid nsi->nsip.remote_ip<br>if there's only a single NS-VC inside the NS Instance, as this would<br>connect() the UDP socket to the remote IP/port, breaking any possibility<br>to have multiple NS-VCs to different SGNS-side IP addresses.<br><br>Closes: OS#3845<br>Change-Id: Ic094621eb01d7458063f531289d5eeadf52bf330<br>---<br>M src/gb/gprs_ns_sns.c<br>1 file changed, 2 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/gb/gprs_ns_sns.c b/src/gb/gprs_ns_sns.c</span><br><span>index b0ee5d7..cbaa468 100644</span><br><span>--- a/src/gb/gprs_ns_sns.c</span><br><span>+++ b/src/gb/gprs_ns_sns.c</span><br><span>@@ -657,8 +657,8 @@</span><br><span>              /* unspecified local address. Figure out which address the kernel would use if we</span><br><span>             * wanted to send a packet to the remote_ip */</span><br><span>               char local_ip[32];</span><br><span style="color: hsl(0, 100%, 40%);">-              struct in_addr in = { .s_addr = htonl(nsi->nsip.remote_ip) };</span><br><span style="color: hsl(0, 100%, 40%);">-                osmo_sock_local_ip(local_ip, inet_ntoa(in));</span><br><span style="color: hsl(120, 100%, 40%);">+          struct sockaddr_in *daddr = &nsvc->ip.bts_addr;</span><br><span style="color: hsl(120, 100%, 40%);">+                osmo_sock_local_ip(local_ip, inet_ntoa(daddr->sin_addr));</span><br><span>                 ip4->ip_addr = inet_addr(local_ip);</span><br><span>       }</span><br><span>    ip4->udp_port = htons(gss->nsi->nsip.local_port);</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/13293">change 13293</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/13293"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ic094621eb01d7458063f531289d5eeadf52bf330 </div>
<div style="display:none"> Gerrit-Change-Number: 13293 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Chemeris <Alexander.Chemeris@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>