]> git.ipfire.org Git - thirdparty/shadow.git/commitdiff
Use a more precise name for a CSPRNG API with an interval
authorAlejandro Colomar <alx@kernel.org>
Fri, 30 Dec 2022 17:46:22 +0000 (18:46 +0100)
committerSerge Hallyn <serge@hallyn.com>
Sat, 28 Jan 2023 03:48:37 +0000 (21:48 -0600)
I have plans to split this function in smaller functions that implement
bits of this functionallity, to simplify the implementation.  So, let's
use names that distinguish them.

This one produces a number within an interval, so make that clear.  Also
make clear that the function produces cryptographically-secure numbers.

Signed-off-by: Alejandro Colomar <alx@kernel.org>
libmisc/salt.c

index 9c5ca006f4405d18518617660ce3a298ea599a89..58b0879f6f6a83e78603f9c777214bf7d09b0e5c 100644 (file)
@@ -94,7 +94,7 @@ static long read_random_bytes (void);
 static /*@observer@*/const char *gensalt (size_t salt_size);
 #endif /* !USE_XCRYPT_GENSALT */
 #if defined(USE_SHA_CRYPT) || defined(USE_BCRYPT)
-static long shadow_random (long min, long max);
+static long csrand_interval (long min, long max);
 #endif /* USE_SHA_CRYPT || USE_BCRYPT */
 #ifdef USE_SHA_CRYPT
 static /*@observer@*/unsigned long SHA_get_salt_rounds (/*@null@*/const int *prefered_rounds);
@@ -163,7 +163,7 @@ end:
  *
  * It favors slightly the higher numbers.
  */
-static long shadow_random (long min, long max)
+static long csrand_interval (long min, long max)
 {
        double drand;
        long ret;
@@ -207,7 +207,7 @@ static /*@observer@*/unsigned long SHA_get_salt_rounds (/*@null@*/const int *pre
                                max_rounds = min_rounds;
                        }
 
-                       rounds = (unsigned long) shadow_random (min_rounds, max_rounds);
+                       rounds = (unsigned long) csrand_interval (min_rounds, max_rounds);
                }
        } else if (0 == *prefered_rounds) {
                rounds = SHA_ROUNDS_DEFAULT;
@@ -280,7 +280,7 @@ static /*@observer@*/unsigned long BCRYPT_get_salt_rounds (/*@null@*/const int *
                                max_rounds = min_rounds;
                        }
 
-                       rounds = (unsigned long) shadow_random (min_rounds, max_rounds);
+                       rounds = (unsigned long) csrand_interval (min_rounds, max_rounds);
                }
        } else if (0 == *prefered_rounds) {
                rounds = B_ROUNDS_DEFAULT;