crypto: jitterentropy - Put constants on the right side of the expression
authorMilan Djurovic <mdjurovic@zohomail.com>
Wed, 17 Mar 2021 01:44:03 +0000 (18:44 -0700)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 26 Mar 2021 09:02:35 +0000 (20:02 +1100)
This patch fixes the following checkpatch.pl warnings:

crypto/jitterentropy.c:600: WARNING: Comparisons should place the constant on the right side of the test
crypto/jitterentropy.c:681: WARNING: Comparisons should place the constant on the right side of the test
crypto/jitterentropy.c:772: WARNING: Comparisons should place the constant on the right side of the test
crypto/jitterentropy.c:829: WARNING: Comparisons should place the constant on the right side of the test

Signed-off-by: Milan Djurovic <mdjurovic@zohomail.com>
Reviewed-by: Stephan Mueller <smueller@chronox.de>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
crypto/jitterentropy.c

index 6e147c4..a11b320 100644 (file)
@@ -597,7 +597,7 @@ int jent_read_entropy(struct rand_data *ec, unsigned char *data,
        if (!ec)
                return -1;
 
-       while (0 < len) {
+       while (len > 0) {
                unsigned int tocopy;
 
                jent_gen_entropy(ec);
@@ -678,7 +678,7 @@ struct rand_data *jent_entropy_collector_alloc(unsigned int osr,
        }
 
        /* verify and set the oversampling rate */
-       if (0 == osr)
+       if (osr == 0)
                osr = 1; /* minimum sampling rate is 1 */
        entropy_collector->osr = osr;
 
@@ -769,7 +769,7 @@ int jent_entropy_init(void)
                 * etc. with the goal to clear it to get the worst case
                 * measurements.
                 */
-               if (CLEARCACHE > i)
+               if (i < CLEARCACHE)
                        continue;
 
                if (stuck)
@@ -826,7 +826,7 @@ int jent_entropy_init(void)
         * should not fail. The value of 3 should cover the NTP case being
         * performed during our test run.
         */
-       if (3 < time_backwards)
+       if (time_backwards > 3)
                return JENT_ENOMONOTONIC;
 
        /*