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

List:       helix-server-cvs
Subject:    [Server-cvs] engine/session clientsession.cpp, 1.174.2.10,
From:       svaidhya () helixcommunity ! org
Date:       2010-02-26 23:18:34
Message-ID: 201002262318.o1QNIW7T020921 () mailer ! progressive-comp ! com
[Download RAW message or body]

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

Modified Files:
      Tag: SERVER_14_0
	clientsession.cpp 
Log Message:
Synopsis
========
PR 258447: Access log for Log Style 7 and 8 has incorrect value for Connected Time

Branch : SERVER_14_0_RN, HEAD
Suggested Reviewer: Anyone

Description
===========

While Playing SMIL Files through proxy, two sessions are created on the server, one \
for accounting connection and one for cached. If the file is already cached on the \
server, Begin() is not called on ClientSession, and hence the session start time is \
not initialized. On ClipDone we compute the difference between current time and \
session start time and compute connected_time. In this case, this leads to a bizarre \
                value.
Fix: For Cached connections we initialize the Session start time on setup_tansport.

Files Changed:
============
./server/engine/session/clientsession.cpp


Testing Performed
=================
Unit Tests:
- None

Integration Tests:
- Verified connected_time shows reasonable values.

Leak Tests:
- None.

Performance Tests:
- None

Build verified: win32-i386-vc7
Platform tested : win32-i386-vc7




Index: clientsession.cpp
===================================================================
RCS file: /cvsroot/server/engine/session/clientsession.cpp,v
retrieving revision 1.174.2.10
retrieving revision 1.174.2.11
diff -u -d -r1.174.2.10 -r1.174.2.11
--- clientsession.cpp	15 Feb 2010 23:30:39 -0000	1.174.2.10
+++ clientsession.cpp	26 Feb 2010 23:18:31 -0000	1.174.2.11
@@ -3689,6 +3689,13 @@
     }
 
     m_bInitialized = TRUE;
+    //For cached connectiond , if file already cached, then play is not called and \
this values +    //remains uninitialized, and is logging incorrect values for \
Sending_Time +    //Set Clip Start Time
+    if (m_bBlockTransfer)
+    {
+        m_ulClipStartTicks = HX_GET_TICKCOUNT();
+    }
 
 #ifdef HELIX_FEATURE_SERVER_FCS
 


_______________________________________________
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]