<p>lynxis lazus has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/20557">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">gprs_ns2: const the return value of gprs_ns2_ip_vc_sockaddr / gprs_ns2_ip_bind_sockaddr<br><br>The sockaddr should not be changed.<br>free and create the bind/nsvc if the address should be changed.<br><br>Change-Id: I371ac2361b569e36722b02fc9cd82ec8da2fa9e3<br>---<br>M include/osmocom/gprs/gprs_ns2.h<br>M src/gb/gprs_ns2.c<br>M src/gb/gprs_ns2_sns.c<br>M src/gb/gprs_ns2_udp.c<br>M src/gb/gprs_ns2_vty.c<br>5 files changed, 14 insertions(+), 12 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/57/20557/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmocom/gprs/gprs_ns2.h b/include/osmocom/gprs/gprs_ns2.h</span><br><span>index 7531530..a3edc63 100644</span><br><span>--- a/include/osmocom/gprs/gprs_ns2.h</span><br><span>+++ b/include/osmocom/gprs/gprs_ns2.h</span><br><span>@@ -139,8 +139,8 @@</span><br><span>                         const struct osmo_sockaddr *remote,</span><br><span>                          uint16_t nsei);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-struct osmo_sockaddr *gprs_ns2_ip_vc_sockaddr(struct gprs_ns2_vc *nsvc);</span><br><span style="color: hsl(0, 100%, 40%);">-struct osmo_sockaddr *gprs_ns2_ip_bind_sockaddr(struct gprs_ns2_vc_bind *bind);</span><br><span style="color: hsl(120, 100%, 40%);">+const struct osmo_sockaddr *gprs_ns2_ip_vc_sockaddr(struct gprs_ns2_vc *nsvc);</span><br><span style="color: hsl(120, 100%, 40%);">+const struct osmo_sockaddr *gprs_ns2_ip_bind_sockaddr(struct gprs_ns2_vc_bind *bind);</span><br><span> int gprs_ns2_is_ip_bind(struct gprs_ns2_vc_bind *bind);</span><br><span> int gprs_ns2_ip_bind_set_dscp(struct gprs_ns2_vc_bind *bind, int dscp);</span><br><span> struct gprs_ns2_vc *gprs_ns2_nsvc_by_sockaddr_bind(</span><br><span>diff --git a/src/gb/gprs_ns2.c b/src/gb/gprs_ns2.c</span><br><span>index a476cff..4f9c5ac 100644</span><br><span>--- a/src/gb/gprs_ns2.c</span><br><span>+++ b/src/gb/gprs_ns2.c</span><br><span>@@ -210,8 +210,8 @@</span><br><span>  *  \return pointer to buf on success; NULL on error */</span><br><span> char *gprs_ns2_ll_str_buf(char *buf, size_t buf_len, struct gprs_ns2_vc *nsvc)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-     struct osmo_sockaddr *local;</span><br><span style="color: hsl(0, 100%, 40%);">-    struct osmo_sockaddr *remote;</span><br><span style="color: hsl(120, 100%, 40%);">+ const struct osmo_sockaddr *local;</span><br><span style="color: hsl(120, 100%, 40%);">+    const struct osmo_sockaddr *remote;</span><br><span>  struct osmo_sockaddr_str local_str;</span><br><span>  struct osmo_sockaddr_str remote_str;</span><br><span> </span><br><span>@@ -847,7 +847,7 @@</span><br><span>                                                 const struct osmo_sockaddr *sockaddr)</span><br><span> {</span><br><span>         struct gprs_ns2_vc *nsvc;</span><br><span style="color: hsl(0, 100%, 40%);">-       struct osmo_sockaddr *remote;</span><br><span style="color: hsl(120, 100%, 40%);">+ const struct osmo_sockaddr *remote;</span><br><span> </span><br><span>      OSMO_ASSERT(nse);</span><br><span>    OSMO_ASSERT(sockaddr);</span><br><span>diff --git a/src/gb/gprs_ns2_sns.c b/src/gb/gprs_ns2_sns.c</span><br><span>index ec6f31f..b476b46 100644</span><br><span>--- a/src/gb/gprs_ns2_sns.c</span><br><span>+++ b/src/gb/gprs_ns2_sns.c</span><br><span>@@ -473,7 +473,7 @@</span><br><span>        struct gprs_ns2_nse *nse = nse_inst_from_fi(fi);</span><br><span>     struct gprs_ns2_vc *nsvc;</span><br><span>    struct osmo_sockaddr sa = {};</span><br><span style="color: hsl(0, 100%, 40%);">-   struct osmo_sockaddr *remote;</span><br><span style="color: hsl(120, 100%, 40%);">+ const struct osmo_sockaddr *remote;</span><br><span>  uint8_t new_signal;</span><br><span>  uint8_t new_data;</span><br><span> </span><br><span>@@ -533,7 +533,7 @@</span><br><span>  struct ns2_sns_state *gss = (struct ns2_sns_state *) fi->priv;</span><br><span>    struct gprs_ns2_nse *nse = nse_inst_from_fi(fi);</span><br><span>     struct gprs_ns2_vc *nsvc, *tmp;</span><br><span style="color: hsl(0, 100%, 40%);">- struct osmo_sockaddr *remote;</span><br><span style="color: hsl(120, 100%, 40%);">+ const struct osmo_sockaddr *remote;</span><br><span>  struct osmo_sockaddr sa = {};</span><br><span> </span><br><span>    if (ip4) {</span><br><span>@@ -1239,7 +1239,9 @@</span><br><span>   struct gprs_ns_ie_ip6_elem *ip6_elems;</span><br><span>       struct gprs_ns2_vc_bind *bind;</span><br><span>       struct gprs_ns2_inst *nsi = nse->nsi;</span><br><span style="color: hsl(0, 100%, 40%);">-        struct osmo_sockaddr *sa, local;</span><br><span style="color: hsl(120, 100%, 40%);">+      const struct osmo_sockaddr *sa;</span><br><span style="color: hsl(120, 100%, 40%);">+       struct osmo_sockaddr local;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>        gss->ip = remote->u.sa.sa_family == AF_INET ? IPv4 : IPv6;</span><br><span> </span><br><span>         gss->initial = *remote;</span><br><span>diff --git a/src/gb/gprs_ns2_udp.c b/src/gb/gprs_ns2_udp.c</span><br><span>index 7ead71a..9aca80b 100644</span><br><span>--- a/src/gb/gprs_ns2_udp.c</span><br><span>+++ b/src/gb/gprs_ns2_udp.c</span><br><span>@@ -274,7 +274,7 @@</span><br><span>                                                  const struct osmo_sockaddr *sockaddr)</span><br><span> {</span><br><span>     struct gprs_ns2_vc_bind *bind;</span><br><span style="color: hsl(0, 100%, 40%);">-  struct osmo_sockaddr *local;</span><br><span style="color: hsl(120, 100%, 40%);">+  const struct osmo_sockaddr *local;</span><br><span> </span><br><span>       OSMO_ASSERT(nsi);</span><br><span>    OSMO_ASSERT(sockaddr);</span><br><span>@@ -396,7 +396,7 @@</span><br><span> /*! Return the socket address of the remote peer of a NS-VC.</span><br><span>  *  \param[in] nsvc NS-VC whose remote peer we want to know</span><br><span>  *  \return address of the remote peer; NULL in case of error */</span><br><span style="color: hsl(0, 100%, 40%);">-struct osmo_sockaddr *gprs_ns2_ip_vc_sockaddr(struct gprs_ns2_vc *nsvc)</span><br><span style="color: hsl(120, 100%, 40%);">+const struct osmo_sockaddr *gprs_ns2_ip_vc_sockaddr(struct gprs_ns2_vc *nsvc)</span><br><span> {</span><br><span>   struct priv_vc *priv;</span><br><span> </span><br><span>@@ -410,7 +410,7 @@</span><br><span> /*! Return the locally bound socket address of the bind.</span><br><span>  *  \param[in] bind The bind whose local address we want to know</span><br><span>  *  \return address of the local bind */</span><br><span style="color: hsl(0, 100%, 40%);">-struct osmo_sockaddr *gprs_ns2_ip_bind_sockaddr(struct gprs_ns2_vc_bind *bind)</span><br><span style="color: hsl(120, 100%, 40%);">+const struct osmo_sockaddr *gprs_ns2_ip_bind_sockaddr(struct gprs_ns2_vc_bind *bind)</span><br><span> {</span><br><span>         struct priv_bind *priv;</span><br><span> </span><br><span>diff --git a/src/gb/gprs_ns2_vty.c b/src/gb/gprs_ns2_vty.c</span><br><span>index 55e87ae..92c0095 100644</span><br><span>--- a/src/gb/gprs_ns2_vty.c</span><br><span>+++ b/src/gb/gprs_ns2_vty.c</span><br><span>@@ -247,7 +247,7 @@</span><br><span> {</span><br><span>      struct osmo_sockaddr_str remote;</span><br><span>     struct osmo_sockaddr_str local;</span><br><span style="color: hsl(0, 100%, 40%);">- struct osmo_sockaddr *sockaddr;</span><br><span style="color: hsl(120, 100%, 40%);">+       const struct osmo_sockaddr *sockaddr;</span><br><span> </span><br><span>    switch (nsvc->ll) {</span><br><span>       case GPRS_NS_LL_UDP: {</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/20557">change 20557</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/c/libosmocore/+/20557"/><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-Change-Id: I371ac2361b569e36722b02fc9cd82ec8da2fa9e3 </div>
<div style="display:none"> Gerrit-Change-Number: 20557 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>