<h1>Project "FreeSWITCH Source" received a push.</h1>
<h2>branch: master updated</h2>
<pre>
via: 3116334d277b0f69305434388bd56f32ae1cb2e6 (commit)
from: f28c211c0c69c8f34a8c483cbb5de379a2b96b84 (commit)
</pre>= COMMIT LOG ===========================================================
<div class="highlight"><pre>committer: Anthony Minessale
comments:
FS-3086 --comment-only This should fix the 'hupall' based lockups
<span style="color: #000080; font-weight: bold">diff --git a/src/switch_core_session.c b/src/switch_core_session.c</span>
<span style="color: #000080; font-weight: bold">index d2e28de..dfe419e 100644</span>
<span style="color: #A00000">--- a/src/switch_core_session.c</span>
<span style="color: #00A000">+++ b/src/switch_core_session.c</span>
<span style="color: #800080; font-weight: bold">@@ -203,16 +203,12 @@ SWITCH_DECLARE(void) switch_core_session_hupall_matching_var(const char *var_nam</span>
        for (hi = switch_hash_first(NULL, session_manager.session_table); hi; hi = switch_hash_next(hi)) {
                switch_hash_this(hi, NULL, NULL, &val);
                if (val) {
<span style="color: #A00000">-                        const char *this_val;</span>
                        session = (switch_core_session_t *) val;
                        if (switch_core_session_read_lock(session) == SWITCH_STATUS_SUCCESS) {
<span style="color: #A00000">-                                if (switch_channel_up(session->channel) &&</span>
<span style="color: #A00000">-                                        (this_val = switch_channel_get_variable(session->channel, var_name)) && (!strcmp(this_val, var_val))) {</span>
<span style="color: #A00000">-                                        np = switch_core_alloc(pool, sizeof(*np));</span>
<span style="color: #A00000">-                                        np->str = switch_core_strdup(pool, session->uuid_str);</span>
<span style="color: #A00000">-                                        np->next = head;</span>
<span style="color: #A00000">-                                        head = np;</span>
<span style="color: #A00000">-                                }</span>
<span style="color: #00A000">+                                np = switch_core_alloc(pool, sizeof(*np));</span>
<span style="color: #00A000">+                                np->str = switch_core_strdup(pool, session->uuid_str);</span>
<span style="color: #00A000">+                                np->next = head;</span>
<span style="color: #00A000">+                                head = np;</span>
                                switch_core_session_rwunlock(session);
                        }
                }
<span style="color: #800080; font-weight: bold">@@ -221,7 +217,11 @@ SWITCH_DECLARE(void) switch_core_session_hupall_matching_var(const char *var_nam</span>
        for(np = head; np; np = np->next) {
                if ((session = switch_core_session_locate(np->str))) {
<span style="color: #A00000">-                        switch_channel_hangup(session->channel, cause);</span>
<span style="color: #00A000">+                        const char *this_val;</span>
<span style="color: #00A000">+                        if (switch_channel_up(session->channel) &&</span>
<span style="color: #00A000">+                                (this_val = switch_channel_get_variable(session->channel, var_name)) && (!strcmp(this_val, var_val))) {                        </span>
<span style="color: #00A000">+                                switch_channel_hangup(session->channel, cause);</span>
<span style="color: #00A000">+                        }</span>
                        switch_core_session_rwunlock(session);
                }
        }
</pre></div>
========================================================================<pre>
Summary of changes:
src/switch_core_session.c | 18 +++++++++---------
1 files changed, 9 insertions(+), 9 deletions(-)
</pre>
<p>this email was generated because of /git/your-repo.git/hooks/post-receive by the file /git-core/contrib/hooks/post-receive-email<br />
For more info, see <a href="http://blog.chomperstomp.com/?p=630">http://blog.chomperstomp.com/?p=630</a>
-- <br />
FreeSWITCH Source</p>