Portability fixes.
authorWerner Koch <wk@gnupg.org>
Mon, 16 Feb 2009 21:05:37 +0000 (21:05 +0000)
committerWerner Koch <wk@gnupg.org>
Mon, 16 Feb 2009 21:05:37 +0000 (21:05 +0000)
20 files changed:
THANKS
cipher/ChangeLog
cipher/crc.c
cipher/md4.c
cipher/md5.c
cipher/rmd160.c
cipher/rsa.c
cipher/sha1.c
cipher/sha256.c
cipher/tiger.c
cipher/whirlpool.c
mpi/ChangeLog
mpi/mpicoder.c
mpi/mpiutil.c
src/ChangeLog
src/ath.h
src/mpi.h
src/sexp.c
tests/ChangeLog
tests/fipsdrv.c

diff --git a/THANKS b/THANKS
index f34974d..cf99a6d 100644 (file)
--- a/THANKS
+++ b/THANKS
@@ -21,6 +21,7 @@ Christian Grothoff         grothoff@cs.purdue.edu
 Christian von Roques      roques@pond.sub.org
 Christopher Oliver        oliver@fritz.traverse.net
 Christian Recktenwald     chris@citecs.de
+Dan Fandrich               dan at coneharvesters com
 Daniel Eisenbud           eisenbud@cs.swarthmore.edu
 Daniel Koening            dan@mail.isis.de
 David Ellement            ellement@sdd.hp.com
@@ -148,7 +149,8 @@ Wim Vandeputte                 wim@kd85.com
                           nbecker@hns.com
 
 
- Copyright 1998, 1999, 2000, 2001, 2002, 2003 Free Software Foundation, Inc.
+ Copyright 1998, 1999, 2000, 2001, 2002, 2003, 
+           2009 Free Software Foundation, Inc.
 
  This file is free software; as a special exception the author gives
  unlimited permission to copy and/or distribute it, with or without
index 8924f17..3efd490 100644 (file)
@@ -1,3 +1,11 @@
+2009-02-16  Werner Koch  <wk@g10code.com>
+
+       * rsa.c (generate_x931): Do not initialize TBL with automatic
+       variables.
+       * whirlpool.c, tiger.c, sha256.c, sha1.c, rmd160.c, md5.c
+       * md4.c, crc.c: Remove memory.h.  This is garbage from gnupg.
+       Reported by Dan Fandrich.
+
 2009-01-22  Werner Koch  <wk@g10code.com>
 
        * ecc.c (compute_keygrip): Remove superfluous const.
index d04fff8..9e406f1 100644 (file)
@@ -25,7 +25,6 @@
 #include <string.h>
 
 #include "g10lib.h"
-#include "memory.h"
 #include "cipher.h"
 
 #include "bithelp.h"
index 680cf87..a13c45c 100644 (file)
@@ -53,7 +53,6 @@
 #include <string.h>
 
 #include "g10lib.h"
-#include "memory.h"
 #include "cipher.h"
 
 #include "bithelp.h"
index 899dce8..051fc81 100644 (file)
@@ -37,7 +37,6 @@
 #include <string.h>
 
 #include "g10lib.h"
-#include "memory.h"
 #include "cipher.h"
 
 #include "bithelp.h"
index 7805bf5..3fdc41c 100644 (file)
@@ -24,7 +24,6 @@
 #include <string.h>
 
 #include "g10lib.h"
-#include "memory.h"
 #include "rmd.h"
 #include "cipher.h" /* Only used for the rmd160_hash_buffer() prototype. */
 
index cf278c2..8ee6018 100644 (file)
@@ -444,18 +444,28 @@ generate_x931 (RSA_secret_key *sk, unsigned int nbits, unsigned long e_value,
     else
       {
         /* Parameters to derive the key are given.  */
+        /* Note that we explicitly need to setup the values of tbl
+           because some compilers (e.g. OpenWatcom, IRIX) don't allow
+           to initialize a structure with automatic variables.  */
         struct { const char *name; gcry_mpi_t *value; } tbl[] = {
-          { "Xp1", &xp1 },
-          { "Xp2", &xp2 },
-          { "Xp",  &xp  },
-          { "Xq1", &xq1 },
-          { "Xq2", &xq2 },
-          { "Xq",  &xq  },
-          { NULL,  NULL }
+          { "Xp1" },
+          { "Xp2" },
+          { "Xp"  },
+          { "Xq1" },
+          { "Xq2" },
+          { "Xq"  },
+          { NULL }
         };
         int idx;
         gcry_sexp_t oneparm;
         
+        tbl[0].value = &xp1;
+        tbl[1].value = &xp2;
+        tbl[2].value = &xp;
+        tbl[3].value = &xq1;
+        tbl[4].value = &xq2;
+        tbl[5].value = &xq;
+
         for (idx=0; tbl[idx].name; idx++)
           {
             oneparm = gcry_sexp_find_token (deriveparms, tbl[idx].name, 0);
index 8862c64..0b5dc43 100644 (file)
@@ -37,7 +37,6 @@
 #endif
 
 #include "g10lib.h"
-#include "memory.h"
 #include "bithelp.h"
 #include "cipher.h"
 #include "hash-common.h"
index 5d61d2f..e0148da 100644 (file)
@@ -41,7 +41,6 @@
 #include <string.h>
 
 #include "g10lib.h"
-#include "memory.h"
 #include "bithelp.h"
 #include "cipher.h"
 #include "hash-common.h"
index a620045..320cdb1 100644 (file)
@@ -24,7 +24,6 @@
 #include <string.h>
 
 #include "g10lib.h"
-#include "memory.h"
 #include "cipher.h"
 
 #ifdef HAVE_U64_TYPEDEF
index 9b029ee..e6c226c 100644 (file)
@@ -36,7 +36,6 @@
 
 #include "types.h"
 #include "g10lib.h"
-#include "memory.h"
 #include "cipher.h"
 
 #include "bithelp.h"
index 2488e58..235a369 100644 (file)
@@ -1,3 +1,7 @@
+2009-02-16  Werner Koch  <wk@g10code.com>
+
+       * mpiutil.c: Remove memory.h.
+
 2008-12-05  Werner Koch  <wk@g10code.com>
 
        * mpicoder.c (mpi_read_from_buffer): Do not bail out if the mpi is
index 8f0c76f..4c76189 100644 (file)
@@ -456,6 +456,10 @@ gcry_mpi_scan (struct gcry_mpi **ret_mpi, enum gcry_mpi_format format,
       const unsigned char *s = buffer;
       size_t n;
       
+      /* This test is not strictly necessary and an assert (!len)
+         would be sufficient.  We keep this test in case we later
+         allow the BUFLEN argument to act as a sanitiy check.  Same
+         below. */
       if (len && len < 4)
         return gcry_error (GPG_ERR_TOO_SHORT);
 
index 4dc5211..950e4ea 100644 (file)
@@ -25,7 +25,6 @@
 
 #include "g10lib.h"
 #include "mpi-internal.h"
-#include "memory.h"
 #include "mod-source-info.h"
 
 
index 649324d..1b784e4 100644 (file)
@@ -1,3 +1,13 @@
+2009-02-16  Werner Koch  <wk@g10code.com>
+
+       * ath.h [HAVE_SYS_SELECT_H]: Include <sys/select.h> for fd_set.
+       [!HAVE_SYS_SELECT_H]: Include <sys/time.h>.  Move inclusion of
+       config.h to the top.  The actual configure check was already
+       there.
+
+       * sexp.c: Remove memory.h.
+       * mpi.h: Remove memory.h.  Add string.h.
+
 2009-02-02  Werner Koch  <wk@g10code.com>
 
        * ath.h: Include sys/time.h.  Fixes bug#993.
index 0813a4a..be7444d 100644 (file)
--- a/src/ath.h
+++ b/src/ath.h
 #ifndef ATH_H
 #define ATH_H
 
+#include <config.h>
+
 #ifdef _WIN32
-#include <windows.h>
-#else
-#include <sys/types.h>
-#include <sys/socket.h>
-#endif
+# include <windows.h>
+#else /* !_WIN32 */
+# ifdef HAVE_SYS_SELECT_H
+#  include <sys/select.h>
+# else
+#  include <sys/time.h>
+# endif
+# include <sys/types.h>
+# include <sys/socket.h>
+#endif /* !_WIN32 */
 #include <gpg-error.h>
 
-#include <sys/time.h>  /* Required by Interix.  */
-
-#include <config.h>
 
 \f
 /* Define _ATH_EXT_SYM_PREFIX if you want to give all external symbols
index f630c3f..eaac738 100644 (file)
--- a/src/mpi.h
+++ b/src/mpi.h
@@ -30,8 +30,9 @@
 
 #include <config.h>
 #include <stdio.h>
+#include <string.h>
+
 #include "types.h"
-#include "memory.h"
 #include "../mpi/mpi-asm-defs.h"
 
 #include "g10lib.h"
index 59a4e7e..81500b1 100644 (file)
@@ -30,7 +30,6 @@
 
 #define GCRYPT_NO_MPI_MACROS 1
 #include "g10lib.h"
-#include "memory.h"
 
 typedef struct gcry_sexp *NODE;
 typedef unsigned short DATALEN;
index bce3e8d..4ff10a8 100644 (file)
@@ -1,3 +1,8 @@
+2009-02-16  Werner Koch  <wk@g10code.com>
+
+       * fipsdrv.c (print_buffer): Remove parens from initializer for
+       better portability.  Reported by Dan Fandrich.
+
 2009-02-13  Werner Koch  <wk@g10code.com>
 
        * rsacvt.c (compute_missing): Fix dqm1 computation.  Take care of
index f80e30c..29ceedd 100644 (file)
@@ -797,9 +797,9 @@ print_buffer (const void *buffer, size_t length)
   if (base64_output)
     {
       static const unsigned char bintoasc[64+1] = 
-        ("ABCDEFGHIJKLMNOPQRSTUVWXYZ" 
-         "abcdefghijklmnopqrstuvwxyz" 
-         "0123456789+/")
+        "ABCDEFGHIJKLMNOPQRSTUVWXYZ" 
+        "abcdefghijklmnopqrstuvwxyz" 
+        "0123456789+/"
       const unsigned char *p;
       unsigned char inbuf[4];
       char outbuf[4];