pgsql: Fix some warnings (probably actual bugs) generated by new GSSAPI
Log Message:
-----------
Fix some warnings (probably actual bugs) generated by new GSSAPI code
when built on a 64-bit machine. Per buildfarm results extracted by Stefan.
Modified Files:
--------------
pgsql/src/backend/libpq:
auth.c (r1.152 -> r1.153)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/libpq/auth.c.diff?r1=1.152&r2=1.153)
On Thu, 2007-07-12 at 17:36 -0300, Tom Lane wrote:
Fix some warnings (probably actual bugs) generated by new GSSAPI code
when built on a 64-bit machine. Per buildfarm results extracted by Stefan.
Wouldn't it be more correct to cast size_t to unsigned long, rather than
unsigned int? Obviously getting GSS tokens larger than an unsigned int
is unlikely to say the least, so it's probably not a problem in
practice.
-Neil
Neil Conway <neilc@samurai.com> writes:
On Thu, 2007-07-12 at 17:36 -0300, Tom Lane wrote:
Fix some warnings (probably actual bugs) generated by new GSSAPI code
when built on a 64-bit machine. Per buildfarm results extracted by Stefan.
Wouldn't it be more correct to cast size_t to unsigned long, rather than
unsigned int? Obviously getting GSS tokens larger than an unsigned int
is unlikely to say the least, so it's probably not a problem in
practice.
Feel free to change it if you want; I don't think it's an issue as long
as the cast matches the sprintf format.
regards, tom lane