[MPlayer-dev-eng] [PATCH] stupid bug in parser-cfg.c causes crash

Alex Sisson alex_sisson at yahoo.co.uk
Thu Jun 5 09:54:22 CEST 2003


 --- Diego Biurrun <diego at biurrun.de> wrote: > D
Richard Felker III writes:
>  > On Wed, Jun 04, 2003 at 09:51:04PM +0100, Alex
> Sisson wrote:
>  > > a missing parameter on mp_msg meant it crashed
> instead
>  > > of printing a nice message if you put entries
> in the
>  > > config with no parameter.
>  > > 
>  > > also did a few english-tidies on error messages
> in
>  > > there.
>  > 
>  > applied
>  > 
>  > > ps: this is what a 10[0[0]]L bug is right?
>  > 
>  > yes, i'd say 1000l :)
> 
> This applies cleanly to 0_90 and fixes a segfault
> there.  I think you
> should port it.
> 
> Diego
> 

ok. (just take the parser-cfg.c from the 0.90 tarball
right ?)

another thing though - i thought afterwards that it
might be better to not print an error message, but
assume a parameter of one. (eg I have fixed-vo=1 in my
config but it would be nice to be able to just put
fixed-vo)

alexs

__________________________________________________
Yahoo! Plus - For a better Internet experience
http://uk.promotions.yahoo.com/yplus/yoffer.html



More information about the MPlayer-dev-eng mailing list