Patch for use of RT_NONE busted by an update to resource.c

Julien Cristau jcristau at debian.org
Wed Dec 24 16:11:00 PST 2008


On Wed, Aug  6, 2008 at 15:30:09 -0700, Keith Packard wrote:

> On Wed, 2008-08-06 at 14:25 -0600, Pete Zaitcev wrote:
> > Hi All:
> > 
> > What would it take to include the attached patch for dixfonts.c?
> 
> Uh, you've uncovered a serious bug in DIX. The XSELinux patches tried to
> automatically detect the difference between looking up a resource by
> class and looking up a resource by type. Turns out, it's hard to tell
> just by looking at the type/class parameter:
> 
>  RT_NONE has no bits set and is a Type lookup
>  RT_WINDOW has lots of bits set, including some class bits and is a Type
> lookup
>  RC_ANY has all bits set and is a class lookup
> 
> I suggest that the attempt to automatically separate these lookup types
> was misguided and that we should have dixLookupResourceByType and
> dixLookupResourceByClass as separate functions once again.
> 
Hi Keith,

as far as I can tell your patch was never committed.  Was this issue
fixed somehow, or is this still relevant?

Cheers,
Julien



More information about the xorg mailing list