[MPlayer-dev-eng] [PATCH] segfault fix when parsing fuzzed ogg files

Attila Kinali attila at kinali.ch
Mon Jul 9 22:19:14 CEST 2007


On Sun, 8 Jul 2007 22:46:25 +0200
Pierre Lombard <p_l at gmx.fr> wrote:

> ===================================================================
> --- tremor/codebook.c	(revision 23744)
> +++ tremor/codebook.c	(working copy)
> @@ -155,6 +155,9 @@
>    long lok = oggpack_look(b,book->dec_firsttablen);
>   
>    if (lok >= 0) {
> +    if (lok >= book->dec_firsttablen) {
> +      return -1;
> +    }
>      long entry = book->dec_firsttable[lok];

I don't understand anything about how tremor works, 
but at least the patch looks ok.

Anyone against commiting?

			Attila Kinali

-- 
Linux ist... wenn man einfache Dinge auch mit einer kryptischen
post-fix Sprache loesen kann
                        -- Daniel Hottinger



More information about the MPlayer-dev-eng mailing list