[Freeswitch-trunk] [freeswitch] FreeSWITCH Source branch master updated. git2svn-syncpoint-master-1690-gf60fdf6

[git at svn.freeswitch.org]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "FreeSWITCH Source".

The branch, master has been updated
       via  f60fdf653dd2d7f8d3eaa6a9086e1f68bd993c59 (commit)
      from  85913b70b43483d4d3d840d128549ddba75b56a6 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit f60fdf653dd2d7f8d3eaa6a9086e1f68bd993c59
Author: Anthony Minessale <anthm at freeswitch.org>
Date:   Wed Feb 2 16:22:43 2011 -0600

    fix possible bad pointer in global vars (please test)

diff --git a/fscomm/widgets/codecwidget.cpp b/fscomm/widgets/codecwidget.cpp
index 42b10fe..2f82e1a 100644
--- a/fscomm/widgets/codecwidget.cpp
+++ b/fscomm/widgets/codecwidget.cpp
@@ -131,6 +131,8 @@ void CodecWidget::setCodecString(QString codecList)
         QStringList parsed = codecList.split("{");
         QString var = parsed.at(1);
         var = var.split("}").at(0);
+		// warning switch_core_get_Variable may return an unsafe pointer in some cases.
+		// revise to use switch_core_get_variable_dup, and then free it after you are done.
         var = switch_core_get_variable(var.toAscii().data());
         if ( ! var.isEmpty() ) {
             codecList = var;

-----------------------------------------------------------------------

Summary of changes:
 fscomm/widgets/codecwidget.cpp |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)


hooks/post-receive
-- 
FreeSWITCH Source