Hi,

there's a memory leak in SEMS master related to offer answer
handling. The local and remote telephone event payload types
are stored in the rtp stream object, but not freed when the
rtp stream object is deleted. The attached patch fixes this.

Regards,
Emil

-- 
Emil Kroymann
VoIP Services Engineer

Email: [email protected]
Tel: +49-30-203899885
Mobile: +49-176-38389303

ISACO GmbH
Kurfürstenstraße 79
10787 Berlin
Germany

Amtsgericht Charlottenburg, HRB 112464B
Geschäftsführer: Daniel Frommherz

From 2a40dd1248aac5ca7111c255051c4872fcde9669 Mon Sep 17 00:00:00 2001
From: Emil Kroymann <[email protected]>
Date: Thu, 23 Jun 2011 12:44:18 +0200
Subject: [PATCH] fix memleak

---
 core/AmRtpStream.cpp |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/core/AmRtpStream.cpp b/core/AmRtpStream.cpp
index 43d113b..d9065fd 100644
--- a/core/AmRtpStream.cpp
+++ b/core/AmRtpStream.cpp
@@ -452,6 +452,12 @@ AmRtpStream::~AmRtpStream()
 	AmRtpReceiver::instance()->removeStream(l_sd);
     close(l_sd);
   }
+
+  if (remote_telephone_event_pt != NULL)
+    delete remote_telephone_event_pt;
+
+  if (local_telephone_event_pt != NULL)
+    delete local_telephone_event_pt;
 }
 
 int AmRtpStream::getLocalPort()
-- 
1.7.2.5

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Semsdev mailing list
[email protected]
http://lists.iptel.org/mailman/listinfo/semsdev

Reply via email to