[MPlayer-dev-eng] PATCH for demux_ts

[Marcus Metzler]

>>>>> "Nico" == Nico Sabbi <nsabbi at tiscali.it> writes:

    Nico> Marcus Metzler wrote:
    >> Hi,
    >> 
    >> I have a patch for demux_ts.c that checks for and allows TS
    >> packets larger than 188 bytes. Those packages exits if the FEC
    >> information is not stripped from the packets received by a DVB
    >> card (204 bytes), or if some cards add time stamp information
    >> to the packets (192 bytes).
    >> 
    >> Here is a diff -u against the latest CVS:
    >> 
    >> 
    >> 
    >> 

    Nico> [snip]

    >> I tested it with streams that have 188 and 192 byte packets.
    >> 
    >> Marcus
    >> 
    >> 
    >> 


    Nico> it seems you don't

    Nico> stream_skip(stream, junk);

    Nico> at the end of the loop; in this case, unlikely as it sounds,
    Nico> if there is a 0x47 in the final junk bytes we'll have some
    Nico> error, so the next iteration will skip some bytes and we'll
    Nico> miss something.  For the rest the patch is good. Do you know

Ok, that could happen. Should I send another patch or can you add that?


    Nico> of any other fancy packet size TS ? I read of 127 and 136
    Nico> bytes, but I could never find a sample.


I have only got samples for 192 bytes, which come directly from a
prototype card. I guess 204 byte samples can be gotton from some
Twinhan cards, but I don't have a sample for that.
127 and 136 seem unlikely because they would not conform to MPEG
standards, maybe some encrypted non DVB standard channels are using them.
I am currently uploading a sample file called ZDF.ts

Marcus

-- 
/--------------------------------------------------------------------\
| Dr. Marcus O.C. Metzler        |                                   |
| mocm at metzlerbros.de            | http://www.metzlerbros.de/        |
\--------------------------------------------------------------------/
 |>>>             Quis custodiet ipsos custodies                 <<<|