<p>dexter has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24939">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp_endp: make wildcarded detection separate<br><br>osmo-mgw currently does only a very simple detection method for<br>wildcarded requests, but it makes sense to split this detection<br>off into a separate function so that it can be used from different code<br>locations and we still have it at one place only.<br><br>Change-Id: I27018c01afb8acabfcf5d435c996cc9806e52d6b<br>Related: SYS#5535<br>---<br>M include/osmocom/mgcp/mgcp_endp.h<br>M src/libosmo-mgcp/mgcp_endp.c<br>2 files changed, 14 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/39/24939/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmocom/mgcp/mgcp_endp.h b/include/osmocom/mgcp/mgcp_endp.h</span><br><span>index 8d19b44..db1a897 100644</span><br><span>--- a/include/osmocom/mgcp/mgcp_endp.h</span><br><span>+++ b/include/osmocom/mgcp/mgcp_endp.h</span><br><span>@@ -134,6 +134,7 @@</span><br><span> void mgcp_endp_release(struct mgcp_endpoint *endp);</span><br><span> int mgcp_endp_claim(struct mgcp_endpoint *endp, const char *callid);</span><br><span> void mgcp_endp_update(struct mgcp_endpoint *endp);</span><br><span style="color: hsl(120, 100%, 40%);">+bool mgcp_endp_is_wildcarded(const char *epname);</span><br><span> struct mgcp_endpoint *mgcp_endp_by_name_trunk(int *cause, const char *epname,</span><br><span>                                          const struct mgcp_trunk *trunk);</span><br><span> struct mgcp_endpoint *mgcp_endp_by_name(int *cause, const char *epname,</span><br><span>diff --git a/src/libosmo-mgcp/mgcp_endp.c b/src/libosmo-mgcp/mgcp_endp.c</span><br><span>index 9ebdd8e..ddfd6cf 100644</span><br><span>--- a/src/libosmo-mgcp/mgcp_endp.c</span><br><span>+++ b/src/libosmo-mgcp/mgcp_endp.c</span><br><span>@@ -237,6 +237,18 @@</span><br><span>        return NULL;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/*! Check if the given epname refers to a wildcarded request or to a specific</span><br><span style="color: hsl(120, 100%, 40%);">+ *  endpoint.</span><br><span style="color: hsl(120, 100%, 40%);">+ *  \param[in] epname endpoint name to check</span><br><span style="color: hsl(120, 100%, 40%);">+ *  \returns true if epname refers to wildcarded request, else false. */</span><br><span style="color: hsl(120, 100%, 40%);">+bool mgcp_endp_is_wildcarded(const char *epname)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+      if (strstr(epname, "*"))</span><br><span style="color: hsl(120, 100%, 40%);">+            return true;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+        return false;</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /*! Find an endpoint by its name on a specified trunk.</span><br><span>  *  \param[out] cause pointer to store cause code, can be NULL.</span><br><span>  *  \param[in] epname endpoint name to lookup.</span><br><span>@@ -253,7 +265,7 @@</span><br><span>      /* At the moment we only support a primitive ('*'-only) method of</span><br><span>     * wildcarded endpoint searches that picks the next free endpoint on</span><br><span>          * a trunk. */</span><br><span style="color: hsl(0, 100%, 40%);">-  if (strstr(epname, "*")) {</span><br><span style="color: hsl(120, 100%, 40%);">+  if (mgcp_endp_is_wildcarded(epname)) {</span><br><span>               endp = find_free_endpoint(trunk);</span><br><span>            if (endp) {</span><br><span>                  LOGPENDP(endp, DLMGCP, LOGL_DEBUG,</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24939">change 24939</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/osmo-mgw/+/24939"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I27018c01afb8acabfcf5d435c996cc9806e52d6b </div>
<div style="display:none"> Gerrit-Change-Number: 24939 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>