<HTML>
<HEAD>
<TITLE>Memory leak</TITLE>
</HEAD>
<BODY>
<FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'>Hi Ross,<BR>
<BR>
In teardownMediaSubsession in RTSPClient, the subsession’s sessionId variable is deleted and reset. However *only if*<B> </B> the teardown succeeds. If the teardown fails and the MediaSubsession is subsequently closed using the proper method, then the sessionId will not be freed. I suggest also delete the pointer in the destructor of MediaSubsession in case it has not already being deleted. I know that you are reworking RTSPClient, but I still think this is worth handling in MediaSubsession as an edge case.<BR>
<BR>
Regards,<BR>
Stuart<BR>
<BR>
<BR>
Patch:<BR>
--- MediaSession.cpp 2009-07-25 03:22:31.000000000 -0700<BR>
+++ MediaSession.cpp.new 2009-07-25 03:23:53.000000000 -0700<BR>
@@ -563,7 +563,8 @@<BR>
<BR>
delete[] fConnectionEndpointName; delete[] fSavedSDPLines;<BR>
delete[] fMediumName; delete[] fCodecName; delete[] fProtocolName;<BR>
- delete[] fControlPath; delete[] fConfig; delete[] fMode; delete[] fSpropParameterSets;<BR>
+ delete[] fControlPath; delete[] fConfig; delete[] fMode; delete[] fSpropParameterSets; <BR>
+ delete[] sessionId;<BR>
<BR>
delete fNext;<BR>
#ifdef SUPPORT_REAL_RTSP<BR>
</SPAN></FONT>
<BR>
- ------------------------------------------------------------------------------<BR>
Confidentiality Notice: The information contained in this transmission is legally privileged and confidential, intended only for the use of the individual(s) or entities named above. This email and any files transmitted with it are the property of Pelco. If the reader of this message is not the intended recipient, or an employee or agent responsible for delivering this message to the intended recipient, you are hereby notified that any review, disclosure, copying, distribution, retention, or any action taken or omitted to be taken in reliance on it is prohibited and may be unlawful. If you receive this communication in error, please notify us immediately by telephone call to +1-559-292-1981 or forward the e-mail to administrator@pelco.com and then permanently delete the e-mail and destroy all soft and hard copies of the message and any attachments. Thank you for your cooperation. <BR>
- ------------------------------------------------------------------------------<BR>
</BODY>
</HTML>