[prev in list] [next in list] [prev in thread] [next in thread] 

List:       helix-datatype-cvs
Subject:    [Datatype-cvs] common/vidrend vidrend.cpp,1.97.2.16,1.97.2.17
From:       stanb () helixcommunity ! org
Date:       2011-10-28 23:14:34
[Download RAW message or body]

Update of /cvsroot/datatype/common/vidrend
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv10000

Modified Files:
      Tag: hxclient_3_1_0_atlas
	vidrend.cpp 
Log Message:
Modified by: stanb@real.com
Date: 10/21/2011
Project: WinAgent (UNIFI), RealPlayer 15

Bug Number: 404918

Synopsis: For each FLV file processing a new MiniContext/CHXMediaPlatform is created \
and never destroyed.

Overview:  And the final fix in this group is caused by the fact that we \
create/initialize IHXMultiInstanceSiteUserSupplier* m_pMISUS in \
CVideoRenderer::StartStream() and terminate/release it in \
CVideoRenderer::DetachSite(). In case IHXSite is never attached or detached we have a \
leak. 

This fix makes sure we terminate/release inside CVideoRenderer::EndStream() in case \
IHXSite was never attached.


Files Modified:
datatype/common/vidrend/vidrend.cpp



Index: vidrend.cpp
===================================================================
RCS file: /cvsroot/datatype/common/vidrend/vidrend.cpp,v
retrieving revision 1.97.2.16
retrieving revision 1.97.2.17
diff -u -d -r1.97.2.16 -r1.97.2.17
--- vidrend.cpp	4 Oct 2011 17:39:14 -0000	1.97.2.16
+++ vidrend.cpp	28 Oct 2011 23:14:31 -0000	1.97.2.17
@@ -828,6 +828,16 @@
     DumpInputEntries();
 #endif  // ENABLE_INPUT_TRACE
 
+    if(!m_bSiteAttached)
+    {
+        // No one will call DetachSite, so release m_pMISUS
+        if (m_pMISUS)
+        {
+            m_pMISUS->ReleaseSingleSiteUser();
+        }
+        HX_RELEASE(m_pMISUS);
+    }
+
     return HXR_OK;
 }
 


_______________________________________________
Datatype-cvs mailing list
Datatype-cvs@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/datatype-cvs


[prev in list] [next in list] [prev in thread] [next in thread]