A RetroSearch Logo

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Search Query:

Showing content from https://ffmpeg.org/pipermail/ffmpeg-devel/2007-January/027370.html below:

[Ffmpeg-devel] [PATCH] Musepack out of bounds accesses

[Ffmpeg-devel] [PATCH] Musepack out of bounds accessesReimar Döffinger Reimar.Doeffinger
Sun Jan 14 17:15:23 CET 2007
Hello,
On Sun, Jan 14, 2007 at 07:55:20AM +0200, Kostya wrote:
> On Sat, Jan 13, 2007 at 04:43:28PM +0100, Reimar D?ffinger wrote:
> > attached patch fixes two such.
> > First is in the demuxer during playback-index-building, there is no check that the
> > index array is large enough. It would be possible to realloc it, but I'm
> > not sure if that's such a good idea, thinking e.g. of doing streaming
> > (esp. since I think it does not respect noidx?).
> 
> I don't think it's right solution - maybe that frame should not be passed at all.
> Try to change condition at line 121 and check what happens.

Well, that condition was obviously wrong, I allowed myself to change it
without sending a patch first.
Though I do not like treating the frame count in some header as the ultimate
truth, esp. as it makes it impossible to play files that are in the process
of being created...

> > The other one is because the decoder does not provide the padding the
> > bitstream reader needs.
> 
> This patch is fine to me, apply anytime.

applied.

Greetings,
Reimar D?ffinger


More information about the ffmpeg-devel mailing list

RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4