<h1>Project "FreeSWITCH Source" received a push.</h1>

<h2>branch: master updated</h2>
<pre>
       via: acd6333e33b9ff9e0f1a424cb7dc5e5b1d063b6a (commit)
      from: d8b11ce63b908ae893feda5345ed712c9125623a (commit)


</pre>= COMMIT LOG ===========================================================
<div class="highlight"><pre>committer: Moises Silva
comments: 
freetdm: disallow completely core variables for freetdm outbound variables

<span style="color: #000080; font-weight: bold">diff --git a/libs/freetdm/mod_freetdm/mod_freetdm.c b/libs/freetdm/mod_freetdm/mod_freetdm.c</span>
<span style="color: #000080; font-weight: bold">index c0fdaa6..78874de 100755</span>
<span style="color: #A00000">--- a/libs/freetdm/mod_freetdm/mod_freetdm.c</span>
<span style="color: #00A000">+++ b/libs/freetdm/mod_freetdm/mod_freetdm.c</span>
<span style="color: #800080; font-weight: bold">@@ -1055,28 +1055,18 @@ switch_io_routines_t freetdm_io_routines = {</span>
 static const char* channel_get_variable(switch_core_session_t *session, switch_event_t *var_event, const char *variable_name)
 {
        const char *variable = NULL;
<span style="color: #A00000">-</span>
        if (var_event) {
                if ((variable = switch_event_get_header(var_event, variable_name))) {
                        return variable;
                }
        }
<span style="color: #00A000">+</span>
        if (session) {
                switch_channel_t *channel = switch_core_session_get_channel(session);
                if ((variable = switch_channel_get_variable(channel, variable_name))) {
                        return variable;
                }
        }
<span style="color: #A00000">-</span>
<span style="color: #A00000">-</span>
<span style="color: #A00000">-           // This is unsafe, I don&#39;t see anywhere in the whole code where this is called with NULL session anyway.</span>
<span style="color: #A00000">-           // There is a new switch_core_get_variable_dup that will strdup it for you and then you must free it.</span>
<span style="color: #A00000">-           // That messes up the abstraction completely so I am just commenting it out for you.....</span>
<span style="color: #A00000">-           /*</span>
<span style="color: #A00000">-       if ((variable = switch_core_get_variable(variable_name))) {</span>
<span style="color: #A00000">-               return variable;</span>
<span style="color: #A00000">-       }</span>
<span style="color: #A00000">-           */</span>
        return NULL;
 }
 
</pre></div>
========================================================================<pre>

Summary of changes:
 libs/freetdm/mod_freetdm/mod_freetdm.c |   12 +-----------
 1 files changed, 1 insertions(+), 11 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>