commit 18c60aaca100f265bed82dd3fe49d98f3cd0f207
author Andrzej Kurek <andrzej.kurek@arm.com> Tue Nov 17 12:16:42 2020 +0100
committer Andrzej Kurek <andrzej.kurek@arm.com> Wed Nov 18 12:37:41 2020 +0100
tree 7929233adaf01f30f45ac7e86eece4e9b2b1a159
parent 0fa427b0279da33e6b725429925437b5c0bdb794

ccm: use random_in_range instead of duplicating its functionality

Signed-off-by: Andrzej Kurek <andrzej.kurek@arm.com>

library/ccm.c

diff --git a/library/ccm.c b/library/ccm.c
index d785866..6d37760 100644
--- a/library/ccm.c
+++ b/library/ccm.c
@@ -131,7 +131,7 @@
     for( i = size - 1; i > 0; i-- )
     {
         unsigned char tmp;
-        j = mbedtls_platform_random_uint32() % ( i + 1 );
+        j = mbedtls_platform_random_in_range( (uint32_t) i + 1 );
         tmp = table[i];
         table[i] = table[j];
         table[j] = tmp;
@@ -144,7 +144,7 @@
 
     for( i = 0; i < size; i++ )
     {
-        table[i] = mbedtls_platform_random_uint32() % ( 256 );
+        table[i] = mbedtls_platform_random_in_range( 256 );
     }
 }
 
@@ -174,7 +174,7 @@
 #define COPY_MASK( dst, src, mask, len_src, len_dst )                   \
     do                                                                  \
     {                                                                   \
-        unsigned j, offset = mbedtls_platform_random_uint32() % 256;    \
+        unsigned j, offset = mbedtls_platform_random_in_range( 256 );   \
         for( i = 0; i < len_src; i++ )                                  \
         {                                                               \
             j = (i + offset) % len_src;                                 \