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

List:       helix-filesystem-dev
Subject:    [Filesystem-dev] CN: merge bug 10090 fix to 310 and head
From:       Qiang Luo <qluo () real ! com>
Date:       2010-04-23 21:20:48
Message-ID: 4BD20F30.6080104 () real ! com
[Download RAW message or body]

This check-in merges Kinson's m_ulHTTPResponseContentRead fix from 361 
back to 310 and head.

-----------------------------------
Modified by: kliu@real.com
Date: 03/26/2010
Project: RealPlayer for Android Smartphones
Bug Number: 10090
Bug URL: https://bugs.helixcommunity.org/show_bug.cgi?id=10090

Synopsis: Fixing non-encoded http packages mistaken as encoded http packages

Overview: In httpfsys.cpp, the assumption that http responses were gzip 
encoded when m_ulHTTPResponseContentRead >= m_ulHTTPResponseContentSize 
is not always right.  The videos on ifeng.com are not gzip encoded, but 
m_ulHTTPResponseContentRead >= m_ulHTTPResponseContentSize becomes true 
from time to time after multiple seekings when doing video streaming on 
ifeng.com.  The reason is that m_ulHTTPResponseContentRead and 
m_nContentRead should be always updated with the same values, but it's 
not at one place.  We will fill that loophole.

Files Modified:
filesystem/http/httpfsys.cpp

diff:
Index: httpfsys.cpp
===================================================================
RCS file: /cvsroot/filesystem/http/httpfsys.cpp,v
retrieving revision 1.109.2.21
diff -u -r1.109.2.21 httpfsys.cpp
--- httpfsys.cpp        7 Apr 2010 20:00:13 -0000       1.109.2.21
+++ httpfsys.cpp        23 Apr 2010 21:16:55 -0000
@@ -3946,6 +3946,7 @@
                              // But for now there are other places that 
will get
  confused if
                              // we don't fake it out like this.
                              m_nContentRead = m_uByteRangeSeekOffset;
+                            m_ulHTTPResponseContentRead = 
m_uByteRangeSeekOffset;

                              // If we are attempting a byte offset seek,
------------------------

_______________________________________________
Filesystem-dev mailing list
Filesystem-dev@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/filesystem-dev
[prev in list] [next in list] [prev in thread] [next in thread]