[libglnx] dirfd: Use better and faster random algorithm for gen_temp_name()

[Colin Walters <walters src gnome org>]

commit 597f03b4058a4d495252ee5479371c0b428fe40f
Author: Colin Walters <walters verbum org>
Date:   Thu Jan 26 08:49:03 2017 -0500

    dirfd: Use better and faster random algorithm for gen_temp_name()
    
    I was looking at ostree performance, and a surprising amount of
    time was spent in `glnx_gen_temp_name()`.  We end up calling it
    from the main loop, and the iteration here shows up in my perf
    profiles.
    
    The glibc algorithm here that we adopted is *very* dated; let's
    switch to use `GRand`, which gives us a better algorithm.
    
    It'd be even better of course to use `getrandom()`, but we should do that in
    glib at some point.
    
    While I had the patient open, I extended the charset with lowercase, to better
    avoid collisions.

 glnx-dirfd.c |   30 ++++--------------------------
 1 files changed, 4 insertions(+), 26 deletions(-)
---
diff --git a/glnx-dirfd.c b/glnx-dirfd.c
index 3a02bb0..11388c1 100644
--- a/glnx-dirfd.c
+++ b/glnx-dirfd.c
@@ -293,13 +293,10 @@ glnx_gen_temp_name (gchar *tmpl)
 {
   size_t len;
   char *XXXXXX;
-  int count;
+  int i;
   static const char letters[] =
-    "ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789";
+    "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
   static const int NLETTERS = sizeof (letters) - 1;
-  glong value;
-  GTimeVal tv;
-  static int counter = 0;
 
   g_return_if_fail (tmpl != NULL);
   len = strlen (tmpl);
@@ -307,27 +304,8 @@ glnx_gen_temp_name (gchar *tmpl)
 
   XXXXXX = tmpl + (len - 6);
 
-  /* Get some more or less random data.  */
-  g_get_current_time (&tv);
-  value = (tv.tv_usec ^ tv.tv_sec) + counter++;
-
-  for (count = 0; count < 100; value += 7777, ++count)
-    {
-      glong v = value;
-
-      /* Fill in the random bits.  */
-      XXXXXX[0] = letters[v % NLETTERS];
-      v /= NLETTERS;
-      XXXXXX[1] = letters[v % NLETTERS];
-      v /= NLETTERS;
-      XXXXXX[2] = letters[v % NLETTERS];
-      v /= NLETTERS;
-      XXXXXX[3] = letters[v % NLETTERS];
-      v /= NLETTERS;
-      XXXXXX[4] = letters[v % NLETTERS];
-      v /= NLETTERS;
-      XXXXXX[5] = letters[v % NLETTERS];
-    }
+  for (i = 0; i < 6; i++)
+    XXXXXX[i] = letters[g_random_int_range(0, NLETTERS)];
 }
 
 /**