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

List:       helix-server-cvs
Subject:    [Server-cvs] engine/session clientsession.cpp, 1.72.2.41.2.12,
From:       jzeng () helixcommunity ! org
Date:       2008-11-12 19:41:08
Message-ID: 200811121948.mACJmdPH023406 () mailer ! progressive-comp ! com
[Download RAW message or body]

Update of /cvsroot/server/engine/session
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv13340/server/engine/session

Modified Files:
      Tag: SERVER_12_1
	clientsession.cpp 
Log Message:
Synopsis
========
fix leaks found in testing.
Branches:  head, server_12_1
Suggested Reviewer: Shantha

Description
===========
Leaks found in testing.

Files Affected
==============

server/engine/session/clientsession.cpp,v
server-restricted/datatype/playlist/playlist_control.cpp,v
server-restricted/datatype/playlist/plmgr.cpp,v
server-restricted/protocol/http-ctrl/ssplgethandler.cpp,v
server-restricted/protocol/http-ctrl/ssplutil.cpp,v

Testing Performed
=================

Unit Tests:
None.

Integration Tests:
None.

Leak Tests:
yes.

Performance Tests:
- None

Platforms Tested: linux-rhel4-i686
Build verified: linux-rhel4-i686



Index: clientsession.cpp
===================================================================
RCS file: /cvsroot/server/engine/session/clientsession.cpp,v
retrieving revision 1.72.2.41.2.12
retrieving revision 1.72.2.41.2.13
diff -u -d -r1.72.2.41.2.12 -r1.72.2.41.2.13
--- clientsession.cpp	7 Nov 2008 04:53:52 -0000	1.72.2.41.2.12
+++ clientsession.cpp	12 Nov 2008 19:41:06 -0000	1.72.2.41.2.13
@@ -7167,6 +7167,7 @@
     IHXURL *pTempURL = new CHXURLParser((const char*)pUrl, strlen(pUrl));
     pTempURL->AddRef();
     OnURL(pTempURL, pRequest);
+    pRequest->Release();
     pTempURL->Release();
 
     return HXR_OK;


_______________________________________________
Server-cvs mailing list
Server-cvs@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/server-cvs
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic