[mythtv-commits] Ticket #1392: invalid read in PESPacket::PESPacket()

MythTV mythtv at cvs.mythtv.org
Sun Mar 12 00:21:57 UTC 2006


#1392: invalid read in PESPacket::PESPacket()
------------------------------------+---------------------------------------
 Reporter:  janne-mythtv at grunau.be  |        Owner:  danielk
     Type:  defect                  |       Status:  closed 
 Priority:  minor                   |    Milestone:  0.20   
Component:  dvb                     |      Version:  head   
 Severity:  low                     |   Resolution:  fixed  
------------------------------------+---------------------------------------
Changes (by danielk):

  * milestone:  => 0.20
  * version:  => head

Comment:

 That 0x00 is for the start of field pointer, but since I moved the offset
 of the PES packets to match the DVB drivers a few weeks ago it should now
 be considered part of the TS packet rather than the PES packet so I moved
 it. This also makes more sense in the case of multiple PSIP sections per
 TS packet, which is allowed by the DVB standard (but not by the ATSC for
 which this parser was originally written.)

 The actual fix in the patch was in the length setting in those CreateBlank
 functions. We set the length so that exactly 188 bytes of the 188 byte TS
 Packet are copied just before the packet copy, and then set it to the
 proper length right afterward. This means we copy just the right number of
 bytes, but the length is still set properly by the end of the function.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/1392>
MythTV <http://www.mythtv.org/>
MythTV


More information about the mythtv-commits mailing list