Sean O'Malley wrote:
> im looking at struct ktc_token in auth/auth.h or afs/auth.h
>
> struct ktc_token {
> afs_int32 startTime;
> afs_int32 endTime;
> short kvno; /* XXX UNALIGNED */
> int ticketLen;
> char ticket[MAXKTCTICKETLEN];
> };
>
> Should the types of afs_int32's be changes to time_t's since they are time
> values?
>
> I am -assuming- since solaris is defining time_t as a long in part of the
> code clean-up to get some of the 64-bit stuff working it was improperly
> defined as an afs_int32 or it was declared that way for platforms that
> don't have a time.h file defining time_t.Absolutely not. time_t is a 64-bit value on many systems. time within AFS is currently 32-bit and must remain that way because times are part of data structures that must remain platform agnostic. afs_int32 is used intentionally. At some point OpenAFS and Kerberos will need to address 64-bit times. But that day is not today. Jeffrey Altman
smime.p7s
Description: S/MIME Cryptographic Signature
