[FFmpeg-devel] [PATCH] fix DV muxer stream count check

Reimar Doeffinger Reimar.Doeffinger
Tue May 15 11:23:34 CEST 2007


Hello,
On Mon, May 14, 2007 at 11:06:29PM -0700, Roman Shaposhnik wrote:
> On Sat, 2007-05-12 at 16:53 +0200, Reimar D?ffinger wrote:
> > currently it is possible to pass the DV demuxer three audio streams
> > causing an access beyond a buffer or to pass it two video streams in
> > which case it will just ignore one.
> > Attached patch fixes this.
> > In addition I would also suggest replacing all those "goto bail_out;" by
> > "return NULL" since they both do the same - while I gotos have their
> > use, I think in this case they are useless and confusing.
> 
>   Barring Michael's comments the patch looks reasonable.

Could you fix the indentation so I know what I am supposed to use? *g*

Greetings,
Reimar Doeffinger




More information about the ffmpeg-devel mailing list