[FFmpeg-devel] BUG in mpeg video decoding? ...
Michael Niedermayer
michaelni
Mon Jun 8 16:45:56 CEST 2009
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>
More information about the ffmpeg-devel
mailing list