Best Regards
Zhao Liang
Tel:86-10-65646158
-----Original Message-----
From: Lu Mingjun-E7978C
Sent: Thursday, April 19, 2007 4:59 PM
To: ehyche real.com
Cc: Qu Yongzhi-W20664; Zhao Liang-E3423C; Wang
JinJian-E12714
Subject: RE: [Protocol-dev] CR-Client: live streaming
pause/replay will
lead to coredump issue (ezx)
Hi Eric
How about this issue? We need your further information about
this issue
before checking related code in.
Thanks
Lu Mingjun
-----Original Message-----
From: Lu Mingjun-E7978C
Sent: Tuesday, April 10, 2007 11:44 AM
To: Wang JinJian-E12714
Cc: Qu Yongzhi-W20664; Zhao Liang-E3423C
Subject: [Protocol-dev] CR-Client: live streaming
pause/replay will lead
to coredump issue (ezx)
Hi Eric
I think the description about the CR is enough. What
actually made you
puzzled is that this diff result is not based on the latest
code. I
regenerate the result as in the attached file.
Best regards
Lu Mingjun
-----Original Message-----
From: Eric Hyche [mailto:ehyche real.com]
Sent: Monday, April 09, 2007 8:49 PM
To: Lu Mingjun-E7978C; protocol-dev helixcommunity.org
Cc: Wang JinJian-E12714; Qu Yongzhi-W20664; Zhao
Liang-E3423C
Subject: RE: [Protocol-dev] RE: CR-Client: protocol project
For future reference, a lot of my questions may be able to
be answered
if you could add more description in the CR email itself.
- if (IsQueueEmpty())
+ if (m_bQueueIsEmpty)
Why are all the calls to IsQueueEmpty() being replaced by
m_bQueueIsEmpty?
 -1103,19
+1111,10 
{
if (!m_bFirstPacketGet)
{
- if(!pPacket->IsLostPacket()) {
m_bFirstPacketGet = TRUE;
HXLOGL1(HXLOG_TRAN,
"RTSPTransportBuffer[%p]: First
Packet Retrieved
%i",
this, m_uStreamNumber);
- } else {
- //Return HXR_NO_DATA, if the first packet
is a lost
packet.
- //This is to avoid the situation of the
base time
being
set to 0,
- //which could be far from real time-range
of the
packets
and cause the player
- //to "freeze"
- HX_RELEASE(pPacket);
- rc = HXR_NO_DATA;
- }
}
}
Why is the first-packet-lost code being removed here?
-RTSPTransportBuffer::GetCurrentBuffering(UINT32&
ulLowestTimestamp,
- UINT32&
ulHighestTimestamp,
+RTSPTransportBuffer::GetCurrentBuffering(INT64&
llLowestTimestamp,
+ INT64&
llHighestTimestamp,
Why are these being changed to 64-bit?
Eric
=============================================
Eric Hyche (ehyche real.com)
Technical Lead
RealNetworks, Inc.
> -----Original Message-----
> From: protocol-dev-bounces helixcommunity.org
> [mailto:protocol-dev-bounces helixcommunity.org] On
Behalf Of Lu
> Mingjun-E7978C
> Sent: Wednesday, April 04, 2007 11:20 PM
> To: protocol-dev helixcommunity.org
> Cc: Wang JinJian-E12714; Qu Yongzhi-W20664; Zhao
Liang-E3423C
> Subject: [Protocol-dev] RE: CR-Client: protocol
project
>
> Sorry for missing diff file.
>
> ________________________________
>
> From: Lu Mingjun-E7978C
> Sent: Thursday, April 05, 2007 11:17 AM
> To: 'protocol-dev helixcommunity.org'
> Cc: Wang JinJian-E12714; Zhao Liang-E3423C; Rishi
Mathew; Qu
> Yongzhi-W20664
> Subject: CR-Client: protocol project
>
>
> Modified by: e12714 motorola.com
<mailto:e12714 motorola.com>
>
> Date: 04:04:2007
>
> Project: mediaplayer on motorola phone Bug Number:
> Bug URL:
>
> Synopsis: live streaming pause/replay will lead to
coredump issue
> Overview: <detailed description>
> Description:
> 1. Connect to live streaming RTSP server (Huawei's
server).
>
> CMCC live streaming
> 1.Go to CMCC streaming video test server:
> 211.136.165.53:8001/test.jsp password:54shlmt.
> 2.Select live streaming test site
>
> 2. Click live streaming case.
>
> 3. press pause, after success, then press play!
>
> 4. helix will segfault at
> protocol/transport/common/system/transbuf.cpp!
>
> Analysis:
>
> 1. For Huawei's server, after pasue command, the seq
number of RTP
> packet will reset to start from 1, but fro real's
server, seq number
> will increase forever!
> Such case will lead to the buffer disorder!
>
> 2. Pause/replay will not flush pervious buffer
immediately, in currect
> code, we need to wait for a long time to get old buffer
be flushed!
> That means the replay delay is very very long!
>
> Temp fix solution is:
> When work in live streaming mode, just flush all remain
data in to
> buffer if pause occurred!
>
>
> Files Added:
> [File 1] - <purpose synopsis>
> [File 2] - <purpose synopsis>
>
>
> Files Modified:
> [File 1] -
protocol/transport/common/system/transbuf.cpp
> Image Size and Heap Use impact (Client -Only):
> <Description of image size and heaps size impact.
>
>
> Platforms and Profiles Affected:
> <List of profiles & platforms affected and
comments on how they are
> affected if they are different>
>
> Distribution Libraries Affected:
>
> Distribution library impact and planned action:
> <Is an update required and if so when will it
occur>
>
> Platforms and Profiles Build Verified:
> build for arm11 version
>
> Platforms and Profiles Functionality verified:
> <List of profiles & platforms for which the
functionality was
> verified>
>
> Branch: <code branch(es) change is for>
>
> Copyright assignment: <3. >
>
> 1. In consideration for RealNetworks' hosting
and maintenance
> of my modification, I agree to assign to
RealNetworks full
> copyright ownership of the code included in
the attached
> patch, and agree that RealNetworks has no
duty of
> accounting
> to me for it. I warrant that this code is
entirely original
> to and owned by me, that I can legally grant
the copyright
> assignment, and that my contribution does
not violate any
> other
> person's rights, and laws or breach any
contract.
> I understand
> that RealNetworks may license this code
under RPSL, RCSL,
> and/or any other license at RealNetworks'
> discretion, and use
> the code in any way.
>
> 2. I have signed and delivered a Joint
Copyright Assignment
> to RealNetworks, and received acknowledgment
that the
> agreement was received.
>
> 3. My company (Motorola) submits this code
under the terms
> of a commercial contribution agreement with
RealNetworks,
> and I am authorized to contribute this code
under said
> agreement.
>
> 4. I am a RealNetworks employee or contractor
>
> QA Instructions:
> <Description of functionality affected &
suggestions on verification
> areas>
>
>
>
>
>
> <http://www.realnetworks.com/products/support/dev
support.html>
>
>
_______________________________________________
Protocol-dev mailing list
Protocol-dev helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/pr
otocol-dev
|