FS-6615 test this patch

src/include/switch_cpp.h

@@ -106,7 +106,6 @@ SWITCH_DECLARE(bool) email(char *to, char *from, char *headers = NULL, char *bod
 
 	 class API {
 	   protected:
-		 char *last_data;
 		 char time_buf[64];
 		 switch_core_session_t *session;
 	   public:

src/mod/languages/mod_lua/freeswitch.i

@@ -40,6 +40,8 @@
 %newobject Event;
 %newobject Stream;
 %newobject Dbh;
+%newobject API::execute;
+%newobject API::executeString;
 
 /**
  * tell swig to grok everything defined in these header files and

src/mod/languages/mod_lua/mod_lua_wrap.cpp

@@ -1980,6 +1980,7 @@ static int _wrap_API_execute(lua_State* L) {
   result = (char *)(arg1)->execute((char const *)arg2,(char const *)arg3);
   SWIG_arg=0;
   lua_pushstring(L,(const char*)result); SWIG_arg++;
+  free(result);
   return SWIG_arg;
   
   if(0) SWIG_fail;
@@ -2008,6 +2009,7 @@ static int _wrap_API_executeString(lua_State* L) {
   result = (char *)(arg1)->executeString((char const *)arg2);
   SWIG_arg=0;
   lua_pushstring(L,(const char*)result); SWIG_arg++;
+  free(result);
   return SWIG_arg;
   
   if(0) SWIG_fail;

src/mod/languages/mod_managed/freeswitch.i

@@ -82,6 +82,8 @@ char * SWIG_csharp_string_callback(const char * str) {
 %newobject CoreSession;
 %newobject Event;
 %newobject Stream;
+%newobject API::execute;
+%newobject API::executeString;
 
 // Allow bitwise compare on flag fields
 %typemap(csclassmodifiers) session_flag_t "[System.Flags] public enum"

src/mod/languages/mod_managed/freeswitch_wrap.cxx

@@ -1,11 +1,11 @@
 /* ----------------------------------------------------------------------------
  * This file was automatically generated by SWIG (http://www.swig.org).
- * Version 2.0.7
+ * Version 2.0.12
- * 
+ *
- * This file is not intended to be easily readable and contains a number of 
+ * This file is not intended to be easily readable and contains a number of
  * coding conventions designed to improve portability and efficiency. Do not make
- * changes to this file unless you know what you are doing--modify the SWIG 
+ * changes to this file unless you know what you are doing--modify the SWIG
- * interface file instead. 
+ * interface file instead.
  * ----------------------------------------------------------------------------- */
 
 #define SWIGCSHARP
@@ -65,28 +65,28 @@ template <typename T> T SwigValueInit() {
 #ifndef SWIGUNUSED
 # if defined(__GNUC__)
 #   if !(defined(__cplusplus)) || (__GNUC__ > 3 || (__GNUC__ == 3 && __GNUC_MINOR__ >= 4))
-#     define SWIGUNUSED __attribute__ ((__unused__)) 
+#     define SWIGUNUSED __attribute__ ((__unused__))
 #   else
 #     define SWIGUNUSED
 #   endif
 # elif defined(__ICC)
-#   define SWIGUNUSED __attribute__ ((__unused__)) 
+#   define SWIGUNUSED __attribute__ ((__unused__))
 # else
-#   define SWIGUNUSED 
+#   define SWIGUNUSED
 # endif
 #endif
 
 #ifndef SWIG_MSC_UNSUPPRESS_4505
 # if defined(_MSC_VER)
 #   pragma warning(disable : 4505) /* unreferenced local function has been removed */
-# endif 
+# endif
 #endif
 
 #ifndef SWIGUNUSEDPARM
 # ifdef __cplusplus
 #   define SWIGUNUSEDPARM(p)
 # else
-#   define SWIGUNUSEDPARM(p) p SWIGUNUSED 
+#   define SWIGUNUSEDPARM(p) p SWIGUNUSED
 # endif
 #endif
 
@@ -129,7 +129,7 @@ template <typename T> T SwigValueInit() {
 #   define SWIGSTDCALL __stdcall
 # else
 #   define SWIGSTDCALL
-# endif 
+# endif
 #endif
 
 /* Deal with Microsoft's attempt at deprecating C standard runtime functions */
@@ -41996,6 +41996,7 @@ SWIGEXPORT char * SWIGSTDCALL CSharp_Api_Execute(void * jarg1, char * jarg2, cha
   arg3 = (char *)jarg3; 
   result = (char *)(arg1)->execute((char const *)arg2,(char const *)arg3);
   jresult = SWIG_csharp_string_callback((const char *)result); 
+  free(result);
   return jresult;
 }
 
@@ -42010,6 +42011,7 @@ SWIGEXPORT char * SWIGSTDCALL CSharp_Api_ExecuteString(void * jarg1, char * jarg
   arg2 = (char *)jarg2; 
   result = (char *)(arg1)->executeString((char const *)arg2);
   jresult = SWIG_csharp_string_callback((const char *)result); 
+  free(result);
   return jresult;
 }
 

src/mod/languages/mod_perl/freeswitch.i

@@ -15,6 +15,8 @@
 %newobject CoreSession;
 %newobject Event;
 %newobject Stream;
+%newobject API::execute;
+%newobject API::executeString;
 
 /** insert the following includes into generated code so it compiles */
 %{

src/mod/languages/mod_perl/mod_perl_wrap.cpp

@@ -2679,6 +2679,7 @@ XS(_wrap_API_execute) {
     
     if (alloc2 == SWIG_NEWOBJ) delete[] buf2;
     if (alloc3 == SWIG_NEWOBJ) delete[] buf3;
+    free(result);
     XSRETURN(argvi);
   fail:
     
@@ -2719,6 +2720,7 @@ XS(_wrap_API_executeString) {
     ST(argvi) = SWIG_FromCharPtr((const char *)result); argvi++ ;
     
     if (alloc2 == SWIG_NEWOBJ) delete[] buf2;
+    free(result);
     XSRETURN(argvi);
   fail:
     

src/mod/legacy/languages/mod_lua/freeswitch.i

@@ -38,6 +38,8 @@
 %newobject Event;
 %newobject Stream;
 %newobject Dbh;
+%newobject API::execute;
+%newobject API::executeString;
 
 /**
  * tell swig to grok everything defined in these header files and

src/mod/legacy/languages/mod_lua/mod_lua_wrap.cpp

@@ -1980,6 +1980,7 @@ static int _wrap_API_execute(lua_State* L) {
   result = (char *)(arg1)->execute((char const *)arg2,(char const *)arg3);
   SWIG_arg=0;
   lua_pushstring(L,(const char*)result); SWIG_arg++;
+  free(result);
   return SWIG_arg;
   
   if(0) SWIG_fail;
@@ -2008,6 +2009,7 @@ static int _wrap_API_executeString(lua_State* L) {
   result = (char *)(arg1)->executeString((char const *)arg2);
   SWIG_arg=0;
   lua_pushstring(L,(const char*)result); SWIG_arg++;
+  free(result);
   return SWIG_arg;
   
   if(0) SWIG_fail;

src/switch_cpp.cpp

@@ -222,12 +222,11 @@ SWITCH_DECLARE_CONSTRUCTOR API::API(CoreSession *s)
 	} else {
 		session = NULL;
 	}
-	last_data = NULL;
 }
 
 SWITCH_DECLARE_CONSTRUCTOR API::~API()
 {
-	switch_safe_free(last_data);
+	return;
 }
 
 
@@ -237,9 +236,7 @@ SWITCH_DECLARE(const char *) API::execute(const char *cmd, const char *arg)
 	this_check("");
 	SWITCH_STANDARD_STREAM(stream);
 	switch_api_execute(cmd, arg, session, &stream);
-	switch_safe_free(last_data);
+	return (char *) stream.data;
-	last_data = (char *) stream.data;
-	return last_data;
 }
 
 
@@ -269,13 +266,10 @@ SWITCH_DECLARE(const char *) API::executeString(const char *cmd)
 		*arg++ = '\0';
 	}
 
-	switch_safe_free(last_data);
-	
 	SWITCH_STANDARD_STREAM(stream);
 	switch_api_execute(mycmd, arg, session, &stream);
-	last_data = (char *) stream.data;
 	switch_safe_free(mycmd);
-	return last_data;
+	return (char *) stream.data;
 }
 
 SWITCH_DECLARE_CONSTRUCTOR Event::Event(const char *type, const char *subclass_name)