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

Roman Shaposhnik rvs
Tue May 15 08:06:29 CEST 2007


On Sat, 2007-05-12 at 16:53 +0200, Reimar D?ffinger wrote:
> Hello,
> 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.

Thanks,
Roman.





More information about the ffmpeg-devel mailing list