<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hello,<br>
<br>
<div class="moz-cite-prefix">On 6/23/12 7:58 PM, Avi Brender wrote:<br>
</div>
<blockquote cite="mid:4FE603DE.4020905@elitehosts.com" type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<div
style="font-family:Helvetica,Arial,sans-serif;font-size:13px;">
<div class="moz-cite-prefix">Hi,<br>
<br>
We've come across a problem where URIs that are in the IP
state (inactive - pinging) aren't being set to the 'active'
state after a successful ping probe. We've sifted through the
code and found that modules_k/dispatcher/dispatch.c has this
code in the ds_option_callback() function, on line 2328:<br>
if (*ps->param == NULL)<br>
{<br>
LM_DBG("No parameter provided, OPTIONS-Request was
finished"<br>
" with code %d\n", ps->code);<br>
return;<br>
}<br>
<br>
When the group (ie *ps->param) is 0 then that statement
evaluates to true and the function returns without executing
the callback. For groups > 0 the code works fine.<br>
<br>
To fix this we've changed the statement to:<br>
if (ps->param == NULL)<br>
<br>
If you have any questions, comments or feed back please let me
know. If that change gets accepted please let me know as well
so that we can remove the patch from our build systems.<br>
<br>
Thanks for the great software!<br>
</div>
</div>
</blockquote>
iirc, ps->param cannot be null as it is given as address of a
field in an existing structure.<br>
<br>
You are right regarding the group id 0 in this case. 0 should be
avoided usually when expecting to get the value in cofing file,
which means stop execution of the config file -- it happened with
permissions where is a function returning the group, now there 0 is
not allowed.<br>
<br>
I will commit your fix for now, but might be better to disallow set
id 0 to avoid possible troubles in the future.<br>
<br>
Thanks,<br>
Daniel<br>
<br>
<pre class="moz-signature" cols="72">--
Daniel-Constantin Mierla - <a class="moz-txt-link-freetext" href="http://www.asipto.com">http://www.asipto.com</a>
<a class="moz-txt-link-freetext" href="http://twitter.com/#!/miconda">http://twitter.com/#!/miconda</a> - <a class="moz-txt-link-freetext" href="http://www.linkedin.com/in/miconda">http://www.linkedin.com/in/miconda</a>
Kamailio Advanced Training, Seattle, USA, Sep 23-26, 2012 - <a class="moz-txt-link-freetext" href="http://asipto.com/u/katu">http://asipto.com/u/katu</a>
Kamailio Practical Workshop, Netherlands, Sep 10-12, 2012 - <a class="moz-txt-link-freetext" href="http://asipto.com/u/kpw">http://asipto.com/u/kpw</a></pre>
</body>
</html>