pgsql/contrib/pgcrypto blf.c crypt-blowfish.c ... - Mailing list pgsql-committers

From momjian@postgresql.org
Subject pgsql/contrib/pgcrypto blf.c crypt-blowfish.c ...
Date
Msg-id 200111291940.fATJecR22558@postgresql.org
Whole thread Raw
List pgsql-committers
CVSROOT:    /cvsroot
Module name:    pgsql
Changes by:    momjian@postgresql.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


pgsql-committers by date:

Previous
From: petere@postgresql.org
Date:
Subject: pgsql/src nls-global.mk backend/po/nls.mk back ...
Next
From: momjian@postgresql.org
Date:
Subject: pgsql/doc/src/sgml monitoring.sgml