On Thursday 19 January 2006 13:43, Ian Romanick wrote:
> Adam Jackson wrote:
> > Log message:
> >   Avoid a crash when the user has enabled glVertex{,Attrib}Pointer but
> > hasn't filled in a valid pointer for that attribute or vertex array.
>
> NAK.  Any app that is hitting a crash because of this is wrong.  It is
> likely that such apps will tank on other implementations, so working
> around this app bug in Mesa is a disservice to the developer.

The offender is glxcompmgr, for the record.  See screen.c around line 770.

My concern with backing this out is how it affects the indirect rendering 
case, if at all.  I would prefer that clients not be able to crash the server 
by enabling client state and then not filling it in; although it appears to 
"work" in indirect mode as it is, so maybe I'm misguided.

- ajax

Attachment: pgpxINyJxDuMB.pgp
Description: PGP signature

Reply via email to