<div dir="ltr">Daniel, would you mind backporting if you feel it is necessary? I have no idea howto... ;)<br><br>Cheers<div>Jason</div></div><br><div class="gmail_quote">On Fri, 20 Mar 2015 at 13:29 jaybeepee <<a href="mailto:jason.penton@gmail.com">jason.penton@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Module: kamailio<br>
Branch: master<br>
Commit: 0ee3dc5e3edc49cf62f97ddd87a40b<u></u>12c59b73ff<br>
URL: <a href="https://github.com/kamailio/kamailio/commit/0ee3dc5e3edc49cf62f97ddd87a40b12c59b73ff" target="_blank">https://github.com/kamailio/<u></u>kamailio/commit/<u></u>0ee3dc5e3edc49cf62f97ddd87a40b<u></u>12c59b73ff</a><br>
<br>
Author: jaybeepee <<a href="mailto:jason.penton@gmail.com" target="_blank">jason.penton@gmail.com</a>><br>
Committer: jaybeepee <<a href="mailto:jason.penton@gmail.com" target="_blank">jason.penton@gmail.com</a>><br>
Date: 2015-03-20T13:29:34+02:00<br>
<br>
modules/tm: restore code to set blind uac branch last_reply > 200<br>
<br>
---<br>
<br>
Modified: modules/tm/t_suspend.c<br>
<br>
---<br>
<br>
Diff:  <a href="https://github.com/kamailio/kamailio/commit/0ee3dc5e3edc49cf62f97ddd87a40b12c59b73ff.diff" target="_blank">https://github.com/kamailio/<u></u>kamailio/commit/<u></u>0ee3dc5e3edc49cf62f97ddd87a40b<u></u>12c59b73ff.diff</a><br>
Patch: <a href="https://github.com/kamailio/kamailio/commit/0ee3dc5e3edc49cf62f97ddd87a40b12c59b73ff.patch" target="_blank">https://github.com/kamailio/<u></u>kamailio/commit/<u></u>0ee3dc5e3edc49cf62f97ddd87a40b<u></u>12c59b73ff.patch</a><br>
<br>
---<br>
<br>
diff --git a/modules/tm/t_suspend.c b/modules/tm/t_suspend.c<br>
index 4abfe72..cc4dd61 100644<br>
--- a/modules/tm/t_suspend.c<br>
+++ b/modules/tm/t_suspend.c<br>
@@ -215,10 +215,15 @@ int t_continue(unsigned int hash_index, unsigned int label,<br>
                                return 1;<br>
                        }<br>
<br>
-                       /*we really don't need this next line anymore otherwise we will<br>
-                       never be able to forward replies after a (t_relay) on this branch.<br>
-                       We want to try and treat this branch as 'normal' (as if it were a normal req, not async)' */<br>
-                       //t->uac[branch].last_<u></u>received=500;<br>
+                       /* Set last_received to something >= 200,<br>
++                       * the actual value does not matter, the branch<br>
++                       * will never be picked up for response forwarding.<br>
++                       * If last_received is lower than 200,<br>
++                       * then the branch may tried to be cancelled later,<br>
++                       * for example when t_reply() is called from<br>
++                       * a failure route => deadlock, because both<br>
++                       * of them need the reply lock to be held. */<br>
+                       t->uac[branch].last_received=<u></u>500;<br>
                        uac = &t->uac[branch];<br>
                }<br>
                /* else<br>
<br>
<br>
______________________________<u></u>_________________<br>
sr-dev mailing list<br>
<a href="mailto:sr-dev@lists.sip-router.org" target="_blank">sr-dev@lists.sip-router.org</a><br>
<a href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev" target="_blank">http://lists.sip-router.org/<u></u>cgi-bin/mailman/listinfo/sr-<u></u>dev</a><br>
</blockquote></div>