[gnutls-dev] size_t, int and 64 bit machines
deanna at sdf.lonestar.org
Wed Mar 21 16:57:25 CET 2007
Simon Josefsson writes:
>> typedef struct
>> unsigned char *data;
>> - unsigned int size;
>> + size_t size;
>> } gnutls_datum_t;
> I am aware of this one, but the problem is: it is part of gnutls.h and
> thus part of the external API. This is really problematic to change,
> since it won't be backwards compatible.
Oops. That changes everything, doesn't it. :-)
> However, can we get away by considering that GnuTLS doesn't
> work at all on platforms with 8-byte size_t and just make the
> change? We have already some supported 64-bit platforms,
> e.g. amd64 and ia64, but do they have 8-byte size_t? I doubt
> this is a reasonable way forward...
On OpenBSD, all 64-bit hardware platforms have an 8-byte size_t.
I just checked NetBSD/alpha, and it's the same there. I don't
know about Linux or any other systems. Now, the testsuite does
pass on two of those, OpenBSD/amd64 and OpenBSD/alpha, but fails
If supported means compiles and runs, then I guess things are
ok. Often, when some error like this is found on OpenBSD, it's
because one of the memory protection features caught it. Since
sparc64 is the only 64-bit platform that actually produced
incorrect results, I'm thinking about what's unique there: 1)
strict alignment and 2) StackGhost. It's a bit over my head to
say for sure, but I'd be happy to perform any tests you might
want to try.
More information about the Gnutls-devel