[FFmpeg-devel] [PATCH] API changes in ffmpeg.c
Fri Apr 10 20:38:54 CEST 2009
On Fri, Apr 10, 2009 at 08:29:25PM +0200, Thilo Borgmann wrote:
> Michael Niedermayer schrieb:
>>>>> &picture, &got_picture, ptr, len);
>>>>> + ret = avcodec_decode_video2(ist->st->codec,
>>>>> + &picture, &got_picture,
>>>> Crash here.
>>> This is revision 1 which fixes the crash during make test.
>> probably ok if tested
> Although applied, I tested it again using ffmpeg for transcoding the
> corepng.avi into yuv format.
> This revealed a bug in my ffmpeg.c patch, but this bug breaks CorePNG
> decoding only, since this is the only one using more information of the
> provided packet than .data and .size attributes.
> FFplay is not broken as it uses no local AVPackets but the "original" one.
> I attached a little patch for the time being, but I would propose to add a
> "av_copy_packet" function into libavcodec/avpacket.c as future codecs will
> also use more attributes of the avpacket and even others than just the
> .flags attribute.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel