[FFmpeg-devel] BUG in mpeg video decoding? ...

[Michael Niedermayer]

On Mon, Jun 08, 2009 at 02:12:06PM +0200, Luca Abeni wrote:
> Hi Michael,
>
> Michael Niedermayer wrote:
> [...]
>>>  mpegvideo.c |   15 ++++++++++++---
>>>  1 file changed, 12 insertions(+), 3 deletions(-)
>>> a7f886697c278bfae0b1501bb5e0e5239e4abcb7  maybe_fix_decoder_pts-1.diff
>>> Index: libavcodec/mpegvideo.c
>>> ===================================================================
>>> --- libavcodec/mpegvideo.c	(revision 19128)
>>> +++ libavcodec/mpegvideo.c	(working copy)
>>> @@ -819,14 +819,23 @@
>>>       if(shared){
>>>          for(i=0; i<MAX_PICTURE_COUNT; i++){
>>> -            if(s->picture[i].data[0]==NULL && s->picture[i].type==0) 
>>> return i;
>>> +            if(s->picture[i].data[0]==NULL && s->picture[i].type==0) {
>>> +                avcodec_get_frame_defaults((AVFrame *)&s->picture[i]);
>>> +                return i;
>>> +            }
>>>          }
>> can it also be done during codec init?
>
> Ah, I knew I was missing something! :) New patch attached (this looks 
> cleaner,
> and this time I also ran "make test": no regressions detected).
>
>
> 				Luca

>  mpegvideo.c |    3 +++
>  1 file changed, 3 insertions(+)
> cfc2d71e758a6b1813839f6834cf53989f6b0178  maybe_fix_decoder_pts-2.diff
> diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c

looks ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090608/df3d5d00/attachment.pgp>