[prev in list] [next in list] [prev in thread] [next in thread]
List: helix-server-cvs
Subject: [Server-cvs] fs/sdpgenfsys sdpgenfobj.h,1.1.2.2,1.1.2.3
From: rasher () helixcommunity ! org
Date: 2007-10-29 12:32:07
Message-ID: 200710291232.l9TCWEmH031621 () mailer ! progressive-comp ! com
[Download RAW message or body]
Update of /cvsroot/server/fs/sdpgenfsys
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv18138
Modified Files:
Tag: SERVER_BORABORA
sdpgenfobj.h
Log Message:
Synopsis
==========
This CR is a fix for Bug 206362
Branches : Borabora
Suggested Reviewer : Jamie
Description
============
Sdpgen returns invalid urls when Spaces encoded in the url.When server
returns a sdp file in response to http request, it decodes the url and
sends the response.So after sdp returned to client,client sends a setup
request with the decoded spaces and the parsing after the set up request
fails due to the decoded spaces.
Fix
===========
In the sdp returned, the url returned is the encoded url.So the player
returns the encoded url in the setup so that server can successfully
parse the setup request.
Files Affected
================
server\engine\dataflow\fsmanager.cpp
server\engine\dataflow\http.cpp
server\protocol\http\httpprot.cpp
server\fs\sdpgenfsys\sdpgen.cpp
server\fs\sdpgenfsys\sdpgen.h
Testing Performed
=================
Functional-Testing:
Checked that a http request with spaces is played and we get no error.
Leak-Tests:
Build verified : win32-i386-vc7,linux-rhel14-i686,sunos-5.10-sparc-server
Platforms tested: win32-i386-vc7
QA Hints
===========
Thanks
Raj Asher
Index: sdpgenfobj.h
===================================================================
RCS file: /cvsroot/server/fs/sdpgenfsys/sdpgenfobj.h,v
retrieving revision 1.1.2.2
retrieving revision 1.1.2.3
diff -u -d -r1.1.2.2 -r1.1.2.3
--- sdpgenfobj.h 6 Aug 2007 04:30:43 -0000 1.1.2.2
+++ sdpgenfobj.h 29 Oct 2007 12:32:04 -0000 1.1.2.3
@@ -231,6 +231,7 @@
HXBOOL m_bUseHostName;
IHXSessionStats* m_pSessionStats;
HXBOOL m_bDoesExistDoneCalled;
+ char* m_pEncURL;
// Content file timestamps
UINT32 m_ulFileCTime;
_______________________________________________
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