Hello David,

Monday, December 10, 2001, 10:12:59 AM, you wrote:

DM> Hi,

DM> I have found a bug in asffmt.cpp while reading windows media packets.

DM> When the segment contain more than one fragment it is not necessary to increase 
the offset after segment reading :

DM> void packet::init()
DM> {
DM> [...]
DM> offset+=segments.back().length;    // <- don't do this
DM> //  if the offset have been increased while segment reading
DM> [...]
DM> }

DM> The right way is to remember the offset before the reading of the segment, if the 
offset haven't been changed, you can increase it with "offset+=segments.back().length" 
but if the offset have
DM> been increased by the fragment reading process, don't increase it another time.

The offset cannot be increased by the fragment reading process. Offset
is a local variable of packet::init and there are no other functions that
have access to it. Maybe I don't understand what you mean?



-- 
Best regards,
 Eugene                            mailto:[EMAIL PROTECTED]


_______________________________________________
Avifile mailing list
[EMAIL PROTECTED]
http://prak.org/mailman/listinfo/avifile

Reply via email to