From: William King <william.king@quentustech.com>
Date: Thu, 9 May 2013 01:45:24 +0000 (-0700)
Subject: FS-5395: False alarm on a reported valgrind memory leak. The io_private is alloc... 
X-Git-Tag: v1.2.10~38
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=569da261aaf749b9f66f58e03fb9ec166765cb21;p=thirdparty%2Ffreeswitch.git

FS-5395: False alarm on a reported valgrind memory leak. The io_private is alloc'd from a memory pool which valgrind sucks at tracking. No need to free this, because it'll get cleared when the memory pool on the rsession is destroyed.
---

diff --git a/src/mod/endpoints/mod_rtmp/rtmp_tcp.c b/src/mod/endpoints/mod_rtmp/rtmp_tcp.c
index 62caf23185..2624a1a927 100644
--- a/src/mod/endpoints/mod_rtmp/rtmp_tcp.c
+++ b/src/mod/endpoints/mod_rtmp/rtmp_tcp.c
@@ -183,11 +183,6 @@ static switch_status_t rtmp_tcp_close(rtmp_session_t *rsession)
 		switch_buffer_destroy(&(io_pvt->sendq));
 	}
 	
-	if ( rsession->io_private ) {
-		free(rsession->io_private);
-		rsession->io_private = NULL;
-	}
-
 	return SWITCH_STATUS_SUCCESS;
 }