New GSS-preauth plugin available for testing

Alejandro Perez Mendez alex at um.es
Wed Sep 5 08:58:48 EDT 2012


On 05/09/12 13:24, Luke Howard wrote:
> Hi Alejandro,
>
>>> * for use_default_gss_cred/federated, Greg would probably prefer an 
>>> int (I've been picked up for using bitfields before)
>>
>> Do you mean using a single int variable, and then using flags for the 
>> options? Why is that better? To maintain somehow struct size when 
>> adding new options?
>
> I meant use separate ints for each variable, but actually maybe using 
> krb5_boolean is a cleaner idea. Using char directly for a boolean type 
> seems a little 1970s. ;-)

I wasn't even born yet then :). How can my coding style be so 
old-fashined :)

>
> (Of course using flags is another option, so to speak.)
>
>>> * there are a bunch of lines with trailing spaces that "git diff" flags
>>
>> How do I know what are these files, and how do I put them into their 
>> original shape? I'm quite new using git, sorry :)
>
> I just meant removing lines in the files you have added or modified 
> where lines contain trailing spaces (i.e. the last character(s) of a 
> line is a space). It has nothing to do with git /per se/, just git 
> flags it as it's nice to avoid.
>
>> I don't think that's really necessary, since I'm comparing KRB names 
>> with GSS names. The common point is actually the text format.
>
> Maybe gss_compare_name() would be nicer. Ask Nico. (The code is on the 
> KDC side and appears to be comparing the initiator name with 
> WELLKNOWN/FEDERATED.)

Yeah, I know. Maybe gss_compare_name would be better.

>
> -- Luke



More information about the krbdev mailing list