TRANS_NOXAUTH

Jeremy Huddleston jeremyhu at freedesktop.org
Wed Aug 6 10:04:56 PDT 2008


On Aug 6, 2008, at 09:09, Adam Jackson wrote:

> On Wed, 2008-08-06 at 09:13 +0300, Daniel Stone wrote:
>> On Tue, Aug 05, 2008 at 07:09:21PM -0700, Jeremy Huddleston wrote:
>>> +    if(trans_conn->flags | TRANS_NOXAUTH) {
>>                            ^
>>                            & ?

 >< ... yes, thank you Daniel... that would've been bad.

>>
>>
>>> +        auth_id = (XID) 0L;
>>> +    } else {
>>> +        auth_id = CheckAuthorization (proto_n, auth_proto,  
>>> string_n,
>>> auth_string, client, &reason);
>>> +    }
>
> Yes, definitely that.
>
> There's no new xtrans release with NOXAUTH yet, so if you'd also add a
>
> #ifndef TRANS_NOXAUTH
> #define TRANS_NOXAUTH 0
> #endif
>
> in the appropriate place, then this looks fine for master, and we can
> just cull the ifdef once new xtrans goes out.

Ok ... but I think I'll do (1<<6) or whatever it was in xtrans unless  
there's some reason you really want the 0.

--Jeremy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3040 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg/attachments/20080806/38fc062b/attachment.bin>


More information about the xorg mailing list