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

Roman Shaposhnik rvs
Thu May 17 01:32:59 CEST 2007


On Tue, 2007-05-15 at 11:23 +0200, Reimar Doeffinger wrote:
> 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 

  No ;-) But you're welcome to give it a try.

Thanks,
Roman.





More information about the ffmpeg-devel mailing list