pgsql/contrib/pgcrypto blf.c crypt-blowfish.c ...

From: momjian(at)postgresql(dot)org
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql/contrib/pgcrypto blf.c crypt-blowfish.c ...
Date: 2001-11-29 19:40:38
Message-ID: 200111291940.fATJecR22558@postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

CVSROOT: /cvsroot
Module name: pgsql
Changes by: momjian(at)postgresql(dot)org 01/11/29 14:40:37

Modified files:
contrib/pgcrypto: blf.c crypt-blowfish.c crypt-des.c
crypt-gensalt.c md5.c px.h rijndael.c sha1.c

Log message:
* When postgres.h does not define BYTE_ENDIAN pgcrypto
produces garbage.

I learned the hard way that

#if UNDEFINED_1 == UNDEFINED_2
#error "gcc is idiot"
#endif

prints "gcc is idiot" ...

Affected are MD5/SHA1 in internal library, and also HMAC-MD5/HMAC-SHA1/
crypt-md5 which use them. Blowfish is ok, also Rijndael on at
least x86.

Big thanks to Daniel Holtzman who send me a build log which
contained warning:

md5.c:246: warning: `X' defined but not used

Yes, gcc is that helpful...

Please apply this.

--
marko

Browse pgsql-committers by date

  From Date Subject
Next Message momjian 2001-11-29 20:26:55 pgsql/doc/src/sgml monitoring.sgml
Previous Message petere 2001-11-29 18:59:28 pgsql/src nls-global.mk backend/po/nls.mk back ...