* random.c (gcry_random_add_bytes): Return if buflen is zero to
authorWerner Koch <wk@gnupg.org>
Mon, 27 Oct 2003 17:40:28 +0000 (17:40 +0000)
committerWerner Koch <wk@gnupg.org>
Mon, 27 Oct 2003 17:40:28 +0000 (17:40 +0000)
avoid gcc warning about unsed parameter.
(MASK_LEVEL): Simplified; does not work for signed and unsigned
w/o warnings.

* md.c (md_start_debug): Removed the const from SUFFIX, because
this function is called from the control fucntion which does not
require const.

Added some comments.

cipher/ChangeLog
cipher/ac.c
cipher/md.c
cipher/pubkey.c
cipher/random.c

index d6eb1e3..aef7c57 100644 (file)
@@ -4,7 +4,16 @@
 
 2003-10-27  Werner Koch  <wk@gnupg.org>
 
-       Prefixed all (pubkey,digest,cipher}_spec_* globale varaibles with
+       * random.c (gcry_random_add_bytes): Return if buflen is zero to
+       avoid gcc warning about unsed parameter.
+       (MASK_LEVEL): Simplified; does not work for signed and unsigned
+       w/o warnings.
+
+       * md.c (md_start_debug): Removed the const from SUFFIX, because
+       this function is called from the control fucntion which does not
+       require const.
+
+       Prefixed all (pubkey,digest,cipher}_spec_* globale variables with
        _gcry_.
 
        * ac.c (ac_key_identifiers): Made static.
index a723958..a4b4bf5 100644 (file)
@@ -688,8 +688,7 @@ gcry_ac_open (gcry_ac_handle_t *handle,
   if (! *algorithm_name)
     err = GPG_ERR_PUBKEY_ALGO;
 
-  if (! err)
-    /* Acquire reference to the pubkey module.  */
+  if (! err)  /* Acquire reference to the pubkey module.  */
     err = _gcry_pk_module_lookup (algorithm, &module);
   
   if (! err)
index 3491485..6ed6323 100644 (file)
@@ -232,7 +232,7 @@ static byte *md_read( gcry_md_hd_t a, int algo );
 static int md_get_algo( gcry_md_hd_t a );
 static int md_digest_length( int algo );
 static const byte *md_asn_oid( int algo, size_t *asnlen, size_t *mdlen );
-static void md_start_debug( gcry_md_hd_t a, const char *suffix );
+static void md_start_debug( gcry_md_hd_t a, char *suffix );
 static void md_stop_debug( gcry_md_hd_t a );
 
 static int 
@@ -1090,7 +1090,7 @@ gcry_md_algo_info (int algo, int what, void *buffer, size_t *nbytes)
 
 
 static void
-md_start_debug( gcry_md_hd_t md, const char *suffix )
+md_start_debug( gcry_md_hd_t md, char *suffix )
 {
   static int idx=0;
   char buf[25];
index d3289ab..657e4c1 100644 (file)
@@ -52,7 +52,7 @@ static struct pubkey_table_entry
 #if USE_DSA
     { &_gcry_pubkey_spec_dsa, GCRY_PK_DSA },
 #endif
-    { NULL },
+    { NULL, 0 },
   };
 
 /* List of registered ciphers.  */
index c86ad9b..e34e1ad 100644 (file)
@@ -86,7 +86,7 @@
 
 
 static int is_initialized;
-#define MASK_LEVEL(a) do {if( a > 2 ) a = 2; else if( a < 0 ) a = 0; } while(0)
+#define MASK_LEVEL(a) do { (a) &= 3; } while(0)
 static char *rndpool;  /* allocated size is POOLSIZE+BLOCKLEN */
 static char *keypool;  /* allocated size is POOLSIZE+BLOCKLEN */
 static size_t pool_readpos;
@@ -288,10 +288,9 @@ gcry_random_add_bytes (const void * buf, size_t buflen, int quality)
 
   if (!buf || quality < -1 || quality > 100)
     err = GPG_ERR_INV_ARG;
-  /* FIXME */
-#if 0
   if (!buflen)
     return 0; /* Shortcut this dummy case. */
+#if 0
   /* Before we actuall enbale this code, we need to lock the pool,
      have a look at the quality and find a way to add them without
      disturbing the real entropy (we have estimated). */
@@ -469,7 +468,7 @@ _gcry_set_random_seed_file( const char *name )
  * and return true if this was successful
  */
 static int
-read_seed_file()
+read_seed_file (void)
 {
     int fd;
     struct stat sb;
@@ -520,7 +519,7 @@ read_seed_file()
     } while( n == -1 && errno == EINTR );
     if( n != POOLSIZE ) {
        log_fatal(_("can't read `%s': %s\n"), seed_file_name,strerror(errno) );
-       close(fd);
+       close(fd);/*NOTREACHED*/
        return 0;
     }
 
@@ -775,7 +774,7 @@ getfnc_gather_random (void))(void (*)(const void*, size_t, int), int,
 
   log_fatal (_("no entropy gathering module detected\n"));
 
-  return NULL;
+  return NULL; /*NOTREACHED*/
 }
 
 static void (*