[MPlayer-dev-eng] Help Wanted: MPEG Demuxer Parse_PSM corrputs video frames, caused by nicodvb's patch on 4/6/2005

[Nico Sabbi]

Jeff Doozan wrote:

> I've been noticing some corruption (garbled video frames) in videos 
> that I'm transcoding from vob to avi.  I would estimate that this 
> happens between 1 and 3 times per 90 minute video.  I've finally had a 
> chance to track down the source and it seems to be the result of a 
> patch applied by nicodvb to libmpdemux/demux_mpg.c on 4/6/2005.
>
> His patch, revision 1.55, is stated to 'add support for other codecs 
> (mpeg4/h264/aac) in mpeg-ps parsing the PSM'.  I don't understand 
> enough about what his patch is doing to make any useful changes so I'm 
> hoping to find someone more experienced with this code to help find a 
> solution.
>
>
> I don't know who nicodvb is, and I can't find his email address, or 
> else I would just email him directly.
>
it's me.
You had particular bad luck, since you encountered a particular corner 
case: there's some
corruption in the stream that leads to an invalid PSM (len > 1018).

Should be fixed in cvs, please report.

    Nico