[FFmpeg-devel] [PATCH 7/7] Add check in av_write_header() which validates the rawvideo codec tag.

Stefano Sabatini stefano.sabatini-lala
Sun Jun 6 10:52:35 CEST 2010


On date Sunday 2010-06-06 05:03:57 +0200, Michael Niedermayer encoded:
> On Sun, Jun 06, 2010 at 02:02:21AM +0200, Stefano Sabatini wrote:
> > On date Sunday 2010-06-06 01:40:55 +0200, Michael Niedermayer encoded:
> > > On Sat, Jun 05, 2010 at 09:46:07PM +0200, Stefano Sabatini wrote:
> > > [...]
> > > >  const AVCodecTag ff_nut_video_tags[] = {
> > > > +    { CODEC_ID_RAWVIDEO, 0                         },
> > > 
> > > rejected
> > > 
> > > i keep saying the same thing and you keep doing the exact opposit.
> > > i fixed it and ill add some reg test, 
> > 
> > It didn't fix neither NUT nor AVI. Please try this:
> 
> avi works, we have a regression test for it now
> nut was what you worked on, if it doesnt work dont blame me.
> and rgb444 was what you just added, so if that is the only format
> not working in avi ...
> 
> avi worked in the past, and one of the recent patches broke it, and
> exactly the way i repeatly told you you must be carefull with, namely codec_tag
> must be 0 in avi for rgb you totally ignored me and ffmpeg generated
> invalid files as a result. Now this patch might make invalid nut files
> and you dont even attempt to explain why you would want add such a
> incorrect codec_tag to nut.

I already posted the failing command, problem is that NUT/AVI muxers
are creating files which cannot be played (rather than just fail). It
is not a regression as I said multiple times, is a problem I noticed
when running the lavfi_pix_fmts regression test, which generates many
files with weird/exotic pixel formats such as rgb444.

> The only problem i see is mov, and this could be a real issue that might
> need some changes but here with avi and nut iam totally puzzled what problems
> you have.
> I guess i will have to look into it myself

Regards.
-- 
FFmpeg = Fierce and Free Magical Pitiful Educated Guru



More information about the ffmpeg-devel mailing list