fix
[oweals/gnunet.git] / src / util / crypto_random.c
index 14d87b1cd94412d1195de40544e832f935bc36d5..7fb2d14281a7d26e5c5f61ee7f416199d8dd041e 100644 (file)
 #include "gnunet_os_lib.h"
 #include <gcrypt.h>
 
+#define LOG(kind,...) GNUNET_log_from (kind, "util", __VA_ARGS__)
+
+#define LOG_STRERROR(kind,syscall) GNUNET_log_from_strerror (kind, "util", syscall)
+
 /**
  * Create a cryptographically weak pseudo-random number in the interval of 0 to 1.
- * 
+ *
  * @return number between 0 and 1.
  */
 static double
@@ -60,40 +64,39 @@ GNUNET_CRYPTO_random_u32 (enum GNUNET_CRYPTO_Quality mode, uint32_t i)
 
   GNUNET_assert (i > 0);
 
-  if (mode == GNUNET_CRYPTO_QUALITY_STRONG)
+  switch (mode)
     {
+    case GNUNET_CRYPTO_QUALITY_STRONG:
       /* see http://lists.gnupg.org/pipermail/gcrypt-devel/2004-May/000613.html */
 #ifdef gcry_fast_random_poll
       if ((invokeCount++ % 256) == 0)
-        gcry_fast_random_poll ();
+       gcry_fast_random_poll ();
 #endif
       ul = UINT32_MAX - (UINT32_MAX % i);
       do
        {
-         gcry_randomize ((unsigned char *) &ret,
-                         sizeof (uint32_t), GCRY_STRONG_RANDOM);
+         gcry_randomize ((unsigned char *) &ret, sizeof (uint32_t),
+                         GCRY_STRONG_RANDOM);
        }
       while (ret >= ul);
       return ret % i;
-    }
-  else if (mode == GNUNET_CRYPTO_QUALITY_NONCE)
-    {
+    case GNUNET_CRYPTO_QUALITY_NONCE:
       ul = UINT32_MAX - (UINT32_MAX % i);
       do
-        {
-          gcry_create_nonce(&ret, sizeof(ret));
-        }
+       {
+         gcry_create_nonce (&ret, sizeof (ret));
+       }
       while (ret >= ul);
-
       return ret % i;
-    }
-  else
-    {
+    case GNUNET_CRYPTO_QUALITY_WEAK:
       ret = i * weak_random ();
       if (ret >= i)
-        ret = i - 1;
+       ret = i - 1;
       return ret;
+    default:
+      GNUNET_assert (0);
     }
+  return 0;
 }
 
 
@@ -119,7 +122,7 @@ GNUNET_CRYPTO_random_permute (enum GNUNET_CRYPTO_Quality mode, unsigned int n)
     ret[i] = i;
   for (i = n - 1; i > 0; i--)
     {
-      x = GNUNET_CRYPTO_random_u32 (mode, i+1);
+      x = GNUNET_CRYPTO_random_u32 (mode, i + 1);
       tmp = ret[x];
       ret[x] = ret[i];
       ret[i] = tmp;
@@ -142,35 +145,35 @@ GNUNET_CRYPTO_random_u64 (enum GNUNET_CRYPTO_Quality mode, uint64_t max)
   uint64_t ul;
 
   GNUNET_assert (max > 0);
-  if (mode == GNUNET_CRYPTO_QUALITY_STRONG)
+  switch (mode)
     {
+    case GNUNET_CRYPTO_QUALITY_STRONG:
       ul = UINT64_MAX - (UINT64_MAX % max);
       do
        {
-         gcry_randomize ((unsigned char *) &ret,
-                         sizeof (uint64_t), GCRY_STRONG_RANDOM);
+         gcry_randomize ((unsigned char *) &ret, sizeof (uint64_t),
+                         GCRY_STRONG_RANDOM);
        }
       while (ret >= ul);
       return ret % max;
-    }
-  else if (mode == GNUNET_CRYPTO_QUALITY_NONCE)
-    {
+    case GNUNET_CRYPTO_QUALITY_NONCE:
       ul = UINT64_MAX - (UINT64_MAX % max);
       do
-        {
-          gcry_create_nonce(&ret, sizeof(ret));
-        }
+       {
+         gcry_create_nonce (&ret, sizeof (ret));
+       }
       while (ret >= ul);
 
       return ret % max;
-    }
-  else
-    {
+    case GNUNET_CRYPTO_QUALITY_WEAK:
       ret = max * weak_random ();
       if (ret >= max)
-        ret = max - 1;
+       ret = max - 1;
       return ret;
+    default:
+      GNUNET_assert (0);
     }
+  return 0;
 }
 
 /**
@@ -189,15 +192,15 @@ GNUNET_CRYPTO_random_disable_entropy_gathering ()
  * Process ID of the "find" process that we use for
  * entropy gathering.
  */
-static pid_t genproc;
+static struct GNUNET_OS_Process *genproc;
 
 /**
  * Function called by libgcrypt whenever we are
  * blocked gathering entropy.
  */
 static void
-entropy_generator (void *cls,
-                   const char *what, int printchar, int current, int total)
+entropy_generator (void *cls, const char *what, int printchar, int current,
+                  int total)
 {
   unsigned long code;
   enum GNUNET_OS_ProcessStatusType type;
@@ -207,61 +210,62 @@ entropy_generator (void *cls,
     return;
   if (current == total)
     {
-      if (genproc != 0)
-        {
-          if (0 != PLIBC_KILL (genproc, SIGTERM))
-            GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "kill");
-          GNUNET_break (GNUNET_OK == GNUNET_OS_process_wait (genproc));
-          genproc = 0;
-        }
+      if (genproc != NULL)
+       {
+         if (0 != GNUNET_OS_process_kill (genproc, SIGTERM))
+           LOG_STRERROR (GNUNET_ERROR_TYPE_ERROR, "kill");
+         GNUNET_break (GNUNET_OK == GNUNET_OS_process_wait (genproc));
+         GNUNET_OS_process_close (genproc);
+         genproc = NULL;
+       }
       return;
     }
-  if (genproc != 0)
+  if (genproc != NULL)
     {
       ret = GNUNET_OS_process_status (genproc, &type, &code);
       if (ret == GNUNET_NO)
-        return;                 /* still running */
+       return;                 /* still running */
       if (ret == GNUNET_SYSERR)
-        {
-          GNUNET_break (0);
-          return;
-        }
-      if (0 != PLIBC_KILL (genproc, SIGTERM))
-        GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "kill");
+       {
+         GNUNET_break (0);
+         return;
+       }
+      if (0 != GNUNET_OS_process_kill (genproc, SIGTERM))
+       LOG_STRERROR (GNUNET_ERROR_TYPE_ERROR, "kill");
       GNUNET_break (GNUNET_OK == GNUNET_OS_process_wait (genproc));
-      genproc = 0;
+      GNUNET_OS_process_close (genproc);
+      genproc = NULL;
     }
-  GNUNET_log (GNUNET_ERROR_TYPE_INFO,
-              _("Starting `%s' process to generate entropy\n"), "find");
-  genproc = GNUNET_OS_start_process (NULL, NULL, "sh",
-                                     "sh",
-                                     "-c",
-                                     "exec find / -mount -type f -exec cp {} /dev/null \\; 2>/dev/null",
-                                     NULL);
+  LOG (GNUNET_ERROR_TYPE_INFO,
+       _("Starting `%s' process to generate entropy\n"), "find");
+  genproc =
+    GNUNET_OS_start_process (NULL, NULL, "sh", "sh", "-c",
+                            "exec find / -mount -type f -exec cp {} /dev/null \\; 2>/dev/null",
+                            NULL);
 }
 
 
 static void
 killfind ()
 {
-  if (genproc != 0)
+  if (genproc != NULL)
     {
-      PLIBC_KILL (genproc, SIGKILL);
-      genproc = 0;
+      GNUNET_OS_process_kill (genproc, SIGKILL);
+      GNUNET_OS_process_close (genproc);
+      genproc = NULL;
     }
 }
 
 
 void __attribute__ ((constructor)) GNUNET_CRYPTO_random_init ()
 {
-  SRANDOM (time (NULL));
   gcry_control (GCRYCTL_DISABLE_SECMEM, 0);
   if (!gcry_check_version (GCRYPT_VERSION))
     {
       fprintf (stderr,
-               _
-               ("libgcrypt has not the expected version (version %s is required).\n"),
-               GCRYPT_VERSION);
+              _
+              ("libgcrypt has not the expected version (version %s is required).\n"),
+              GCRYPT_VERSION);
       abort ();
     }
 #ifdef gcry_fast_random_poll
@@ -269,6 +273,9 @@ void __attribute__ ((constructor)) GNUNET_CRYPTO_random_init ()
 #endif
   gcry_set_progress_handler (&entropy_generator, NULL);
   atexit (&killfind);
+  SRANDOM (time (NULL) ^
+          GNUNET_CRYPTO_random_u32 (GNUNET_CRYPTO_QUALITY_NONCE,
+                                    UINT32_MAX));
 }