]> git.ipfire.org Git - thirdparty/shadow.git/commitdiff
Use naming consistent with other common functions
authorAlejandro Colomar <alx@kernel.org>
Fri, 30 Dec 2022 18:42:17 +0000 (19:42 +0100)
committerSerge Hallyn <serge@hallyn.com>
Sat, 28 Jan 2023 03:48:37 +0000 (21:48 -0600)
arc4random(3) returns a number.
arc4random_buf(3) fills a buffer.
arc4random_uniform(3) returns a number less than a bound.

and I'd add a hypothetical one which we use:

*_interval() should return a number within the interval [min, max].

In reality, the function being called csrand() in this patch is not
really cryptographically secure, since it had a bias, but a subsequent
patch will fix that.

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

index fd28145c7cbb5e5301ca391f698545fc8bc53b17..90ff702fbe9179df55ea83f4237eee680c15310d 100644 (file)
@@ -89,7 +89,7 @@
 #define GENSALT_SETTING_SIZE 100
 
 /* local function prototypes */
-static long read_random_bytes (void);
+static long csrand (void);
 #if !USE_XCRYPT_GENSALT
 static /*@observer@*/const char *gensalt (size_t salt_size);
 #endif /* !USE_XCRYPT_GENSALT */
@@ -110,7 +110,7 @@ static /*@observer@*/void YESCRYPT_salt_cost_to_buf (char *buf, unsigned long co
 #endif /* USE_YESCRYPT */
 
 /* Read sizeof (long) random bytes from /dev/urandom. */
-static long read_random_bytes (void)
+static long csrand (void)
 {
        long randval = 0;
 
@@ -168,7 +168,7 @@ static unsigned long csrand_interval (unsigned long min, unsigned long max)
        double drand;
        long ret;
 
-       drand = (double) (read_random_bytes () & RAND_MAX) / (double) RAND_MAX;
+       drand = (double) (csrand () & RAND_MAX) / (double) RAND_MAX;
        drand *= (double) (max - min + 1);
        /* On systems were this is not random() range is lower, we favor
         * higher numbers of salt. */
@@ -401,9 +401,9 @@ static /*@observer@*/const char *gensalt (size_t salt_size)
 
        assert (salt_size >= MIN_SALT_SIZE &&
                salt_size <= MAX_SALT_SIZE);
-       strcat (salt, l64a (read_random_bytes ()));
+       strcat (salt, l64a (csrand ()));
        do {
-               strcat (salt, l64a (read_random_bytes ()));
+               strcat (salt, l64a (csrand ()));
        } while (strlen (salt) < salt_size);
 
        salt[salt_size] = '\0';