On Tue, May 19, 2015 at 10:33:52AM +0200, Carl Eugen Hoyos wrote: > On Thursday 14 May 2015 01:47:54 pm Michael Niedermayer wrote: > > On Thu, May 14, 2015 at 12:28:13PM +0200, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes reading valid pict files with non-zero user data. > > > > https://developer.apple.com/legacy/library/documentation/mac/QuickDraw/QuickDraw-334.html > > > > > > Please comment, Carl Eugen > > > > > > qdrw.c | 5 ++--- > > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > edf25620818c022655ed281d4e7d614459d56df3 patchqdrw.diff > > > diff --git a/libavcodec/qdrw.c b/libavcodec/qdrw.c > > > index 1c3244b..6ec35c6 100644 > > > --- a/libavcodec/qdrw.c > > > +++ b/libavcodec/qdrw.c > > > @@ -124,9 +124,8 @@ static int decode_frame(AVCodecContext *avctx, > > > int w, h, ret; > > > > > > bytestream2_init(&gbc, avpkt->data, avpkt->size); > > > - while ( bytestream2_get_bytes_left(&gbc) >= 552 > > > - && ( !AV_RB16(&avpkt->data[bytestream2_tell(&gbc)+6]) > > > - || !AV_RB16(&avpkt->data[bytestream2_tell(&gbc)+8]))) > > > + if ( bytestream2_get_bytes_left(&gbc) >= 552 > > > + && AV_RB32(&avpkt->data[522]) == 0x001102FF) > > > > This wont work with files that have a different version > > New patch attached. > > Carl Eugen
> qdrw.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > a6709bfc86552bd386e843fd6c32c61d28e98f24 patchqdrw.diff do you have a file this fixes ? [...] -- 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
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
