* [PATCH v1 0/2] Use RngLib instead of TimerLib for OpensslLib
@ 2020-07-28 1:53 Matthew Carlson
2020-07-28 1:53 ` [PATCH v1 1/2] CryptoPkg: OpensslLib: Use RngLib to generate entropy in rand_pool Matthew Carlson
2020-07-28 1:53 ` [PATCH v1 2/2] MdePkg: TimerRngLib: Added RngLib that uses TimerLib Matthew Carlson
0 siblings, 2 replies; 6+ messages in thread
From: Matthew Carlson @ 2020-07-28 1:53 UTC (permalink / raw)
To: devel
From: Matthew Carlson <macarl@microsoft.com>
This fixes bugzilla 1871.
Matthew Carlson (2):
CryptoPkg: OpensslLib: Use RngLib to generate entropy in rand_pool
MdePkg: TimerRngLib: Added RngLib that uses TimerLib
CryptoPkg/Library/OpensslLib/rand_pool.c | 200 ++------------------
CryptoPkg/Library/OpensslLib/rand_pool_noise.c | 29 ---
CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c | 43 -----
MdePkg/Library/TimerRngLib/TimerRng.c | 153 +++++++++++++++
CryptoPkg/Library/OpensslLib/OpensslLib.inf | 15 +-
CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf | 15 +-
CryptoPkg/Library/OpensslLib/rand_pool_noise.h | 29 ---
MdePkg/Library/TimerRngLib/TimerRngLib.inf | 37 ++++
MdePkg/MdePkg.dsc | 2 +
9 files changed, 212 insertions(+), 311 deletions(-)
delete mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise.c
delete mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c
create mode 100644 MdePkg/Library/TimerRngLib/TimerRng.c
delete mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise.h
create mode 100644 MdePkg/Library/TimerRngLib/TimerRngLib.inf
--
2.27.0.windows.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v1 1/2] CryptoPkg: OpensslLib: Use RngLib to generate entropy in rand_pool
2020-07-28 1:53 [PATCH v1 0/2] Use RngLib instead of TimerLib for OpensslLib Matthew Carlson
@ 2020-07-28 1:53 ` Matthew Carlson
2020-07-29 1:08 ` [edk2-devel] " Guomin Jiang
2020-07-28 1:53 ` [PATCH v1 2/2] MdePkg: TimerRngLib: Added RngLib that uses TimerLib Matthew Carlson
1 sibling, 1 reply; 6+ messages in thread
From: Matthew Carlson @ 2020-07-28 1:53 UTC (permalink / raw)
To: devel; +Cc: Jiewen Yao, Jian J Wang, Xiaoyu Lu, Matthew Carlson
From: Matthew Carlson <macarl@microsoft.com>
Changes OpenSSL to no longer depend on TimerLib and instead use RngLib.
This allows platforms to decide for themsevles what sort of entropy source
they provide to OpenSSL and TlsLib.
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Xiaoyu Lu <xiaoyux.lu@intel.com>
Signed-off-by: Matthew Carlson <matthewfcarlson@gmail.com>
---
CryptoPkg/Library/OpensslLib/rand_pool.c | 200 ++------------------
CryptoPkg/Library/OpensslLib/rand_pool_noise.c | 29 ---
CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c | 43 -----
CryptoPkg/Library/OpensslLib/OpensslLib.inf | 15 +-
CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf | 15 +-
CryptoPkg/Library/OpensslLib/rand_pool_noise.h | 29 ---
6 files changed, 20 insertions(+), 311 deletions(-)
diff --git a/CryptoPkg/Library/OpensslLib/rand_pool.c b/CryptoPkg/Library/OpensslLib/rand_pool.c
index 9e0179b03490..55bf6c9c6950 100644
--- a/CryptoPkg/Library/OpensslLib/rand_pool.c
+++ b/CryptoPkg/Library/OpensslLib/rand_pool.c
@@ -11,44 +11,10 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
#include <openssl/aes.h>
#include <Uefi.h>
-#include <Library/TimerLib.h>
+#include <Library/RngLib.h>
#include "rand_pool_noise.h"
-/**
- Get some randomness from low-order bits of GetPerformanceCounter results.
- And combine them to the 64-bit value
-
- @param[out] Rand Buffer pointer to store the 64-bit random value.
-
- @retval TRUE Random number generated successfully.
- @retval FALSE Failed to generate.
-**/
-STATIC
-BOOLEAN
-EFIAPI
-GetRandNoise64FromPerformanceCounter(
- OUT UINT64 *Rand
- )
-{
- UINT32 Index;
- UINT32 *RandPtr;
-
- if (NULL == Rand) {
- return FALSE;
- }
-
- RandPtr = (UINT32 *) Rand;
-
- for (Index = 0; Index < 2; Index ++) {
- *RandPtr = (UINT32) (GetPerformanceCounter () & 0xFF);
- MicroSecondDelay (10);
- RandPtr++;
- }
-
- return TRUE;
-}
-
/**
Calls RandomNumber64 to fill
a buffer of arbitrary size with random bytes.
@@ -56,8 +22,8 @@ GetRandNoise64FromPerformanceCounter(
@param[in] Length Size of the buffer, in bytes, to fill with.
@param[out] RandBuffer Pointer to the buffer to store the random result.
- @retval EFI_SUCCESS Random bytes generation succeeded.
- @retval EFI_NOT_READY Failed to request random bytes.
+ @retval True Random bytes generation succeeded.
+ @retval False Failed to request random bytes.
**/
STATIC
@@ -73,17 +39,17 @@ RandGetBytes (
Ret = FALSE;
+ if (RandBuffer == NULL) {
+ DEBUG((DEBUG_ERROR, "[OPENSSL_RAND_POOL] NULL RandBuffer. No random numbers are generated and your system is not secure\n"));
+ ASSERT(FALSE); // Since we can't generate random numbers, we should assert. Otherwise we will just blow up later.
+ return Ret;
+ }
+
+
while (Length > 0) {
- //
- // Get random noise from platform.
- // If it failed, fallback to PerformanceCounter
- // If you really care about security, you must override
- // GetRandomNoise64FromPlatform.
- //
- Ret = GetRandomNoise64 (&TempRand);
- if (Ret == FALSE) {
- Ret = GetRandNoise64FromPerformanceCounter (&TempRand);
- }
+ // Use RngLib to get random number
+ Ret = GetRandomNumber64(&TempRand);
+
if (!Ret) {
return Ret;
}
@@ -100,125 +66,6 @@ RandGetBytes (
return Ret;
}
-/**
- Creates a 128bit random value that is fully forward and backward prediction resistant,
- suitable for seeding a NIST SP800-90 Compliant.
- This function takes multiple random numbers from PerformanceCounter to ensure reseeding
- and performs AES-CBC-MAC over the data to compute the seed value.
-
- @param[out] SeedBuffer Pointer to a 128bit buffer to store the random seed.
-
- @retval TRUE Random seed generation succeeded.
- @retval FALSE Failed to request random bytes.
-
-**/
-STATIC
-BOOLEAN
-EFIAPI
-RandGetSeed128 (
- OUT UINT8 *SeedBuffer
- )
-{
- BOOLEAN Ret;
- UINT8 RandByte[16];
- UINT8 Key[16];
- UINT8 Ffv[16];
- UINT8 Xored[16];
- UINT32 Index;
- UINT32 Index2;
- AES_KEY AESKey;
-
- //
- // Chose an arbitrary key and zero the feed_forward_value (FFV)
- //
- for (Index = 0; Index < 16; Index++) {
- Key[Index] = (UINT8) Index;
- Ffv[Index] = 0;
- }
-
- AES_set_encrypt_key (Key, 16 * 8, &AESKey);
-
- //
- // Perform CBC_MAC over 32 * 128 bit values, with 10us gaps between 128 bit value
- // The 10us gaps will ensure multiple reseeds within the system time with a large
- // design margin.
- //
- for (Index = 0; Index < 32; Index++) {
- MicroSecondDelay (10);
- Ret = RandGetBytes (16, RandByte);
- if (!Ret) {
- return Ret;
- }
-
- //
- // Perform XOR operations on two 128-bit value.
- //
- for (Index2 = 0; Index2 < 16; Index2++) {
- Xored[Index2] = RandByte[Index2] ^ Ffv[Index2];
- }
-
- AES_encrypt (Xored, Ffv, &AESKey);
- }
-
- for (Index = 0; Index < 16; Index++) {
- SeedBuffer[Index] = Ffv[Index];
- }
-
- return Ret;
-}
-
-/**
- Generate high-quality entropy source.
-
- @param[in] Length Size of the buffer, in bytes, to fill with.
- @param[out] Entropy Pointer to the buffer to store the entropy data.
-
- @retval EFI_SUCCESS Entropy generation succeeded.
- @retval EFI_NOT_READY Failed to request random data.
-
-**/
-STATIC
-BOOLEAN
-EFIAPI
-RandGenerateEntropy (
- IN UINTN Length,
- OUT UINT8 *Entropy
- )
-{
- BOOLEAN Ret;
- UINTN BlockCount;
- UINT8 Seed[16];
- UINT8 *Ptr;
-
- BlockCount = Length / 16;
- Ptr = (UINT8 *) Entropy;
-
- //
- // Generate high-quality seed for DRBG Entropy
- //
- while (BlockCount > 0) {
- Ret = RandGetSeed128 (Seed);
- if (!Ret) {
- return Ret;
- }
- CopyMem (Ptr, Seed, 16);
-
- BlockCount--;
- Ptr = Ptr + 16;
- }
-
- //
- // Populate the remained data as request.
- //
- Ret = RandGetSeed128 (Seed);
- if (!Ret) {
- return Ret;
- }
- CopyMem (Ptr, Seed, (Length % 16));
-
- return Ret;
-}
-
/*
* Add random bytes to the pool to acquire requested amount of entropy
*
@@ -238,7 +85,7 @@ size_t rand_pool_acquire_entropy(RAND_POOL *pool)
buffer = rand_pool_add_begin(pool, bytes_needed);
if (buffer != NULL) {
- Ret = RandGenerateEntropy(bytes_needed, buffer);
+ Ret = RandGetBytes(bytes_needed, buffer);
if (FALSE == Ret) {
rand_pool_add_end(pool, 0, 0);
} else {
@@ -257,13 +104,8 @@ size_t rand_pool_acquire_entropy(RAND_POOL *pool)
*/
int rand_pool_add_nonce_data(RAND_POOL *pool)
{
- struct {
- UINT64 Rand;
- UINT64 TimerValue;
- } data = { 0 };
-
- RandGetBytes(8, (UINT8 *)&(data.Rand));
- data.TimerValue = GetPerformanceCounter();
+ UINT8 data[16];
+ RandGetBytes(sizeof(data), data);
return rand_pool_add(pool, (unsigned char*)&data, sizeof(data), 0);
}
@@ -275,13 +117,8 @@ int rand_pool_add_nonce_data(RAND_POOL *pool)
*/
int rand_pool_add_additional_data(RAND_POOL *pool)
{
- struct {
- UINT64 Rand;
- UINT64 TimerValue;
- } data = { 0 };
-
- RandGetBytes(8, (UINT8 *)&(data.Rand));
- data.TimerValue = GetPerformanceCounter();
+ UINT8 data[16];
+ RandGetBytes(sizeof(data), data);
return rand_pool_add(pool, (unsigned char*)&data, sizeof(data), 0);
}
@@ -313,4 +150,3 @@ void rand_pool_cleanup(void)
void rand_pool_keep_random_devices_open(int keep)
{
}
-
diff --git a/CryptoPkg/Library/OpensslLib/rand_pool_noise.c b/CryptoPkg/Library/OpensslLib/rand_pool_noise.c
deleted file mode 100644
index 212834e27acc..000000000000
--- a/CryptoPkg/Library/OpensslLib/rand_pool_noise.c
+++ /dev/null
@@ -1,29 +0,0 @@
-/** @file
- Provide rand noise source.
-
-Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
-SPDX-License-Identifier: BSD-2-Clause-Patent
-
-**/
-
-#include <Library/BaseLib.h>
-
-/**
- Get 64-bit noise source
-
- @param[out] Rand Buffer pointer to store 64-bit noise source
-
- @retval FALSE Failed to generate
-**/
-BOOLEAN
-EFIAPI
-GetRandomNoise64 (
- OUT UINT64 *Rand
- )
-{
- //
- // Return FALSE will fallback to use PerformanceCounter to
- // generate noise.
- //
- return FALSE;
-}
diff --git a/CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c b/CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c
deleted file mode 100644
index 4158106231fd..000000000000
--- a/CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c
+++ /dev/null
@@ -1,43 +0,0 @@
-/** @file
- Provide rand noise source.
-
-Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
-SPDX-License-Identifier: BSD-2-Clause-Patent
-
-**/
-
-#include <Library/BaseLib.h>
-#include <Library/DebugLib.h>
-#include <Library/TimerLib.h>
-
-/**
- Get 64-bit noise source
-
- @param[out] Rand Buffer pointer to store 64-bit noise source
-
- @retval TRUE Get randomness successfully.
- @retval FALSE Failed to generate
-**/
-BOOLEAN
-EFIAPI
-GetRandomNoise64 (
- OUT UINT64 *Rand
- )
-{
- UINT32 Index;
- UINT32 *RandPtr;
-
- if (NULL == Rand) {
- return FALSE;
- }
-
- RandPtr = (UINT32 *)Rand;
-
- for (Index = 0; Index < 2; Index ++) {
- *RandPtr = (UINT32) ((AsmReadTsc ()) & 0xFF);
- RandPtr++;
- MicroSecondDelay (10);
- }
-
- return TRUE;
-}
diff --git a/CryptoPkg/Library/OpensslLib/OpensslLib.inf b/CryptoPkg/Library/OpensslLib/OpensslLib.inf
index dbbe5386a10c..4baad565564c 100644
--- a/CryptoPkg/Library/OpensslLib/OpensslLib.inf
+++ b/CryptoPkg/Library/OpensslLib/OpensslLib.inf
@@ -571,22 +571,9 @@
$(OPENSSL_PATH)/ssl/statem/statem_local.h
# Autogenerated files list ends here
buildinf.h
- rand_pool_noise.h
ossl_store.c
rand_pool.c
-[Sources.Ia32]
- rand_pool_noise_tsc.c
-
-[Sources.X64]
- rand_pool_noise_tsc.c
-
-[Sources.ARM]
- rand_pool_noise.c
-
-[Sources.AARCH64]
- rand_pool_noise.c
-
[Packages]
MdePkg/MdePkg.dec
CryptoPkg/CryptoPkg.dec
@@ -594,7 +581,7 @@
[LibraryClasses]
BaseLib
DebugLib
- TimerLib
+ RngLib
PrintLib
[LibraryClasses.ARM]
diff --git a/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf b/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
index 616ccd9f62d1..3557711bd85a 100644
--- a/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
+++ b/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
@@ -520,22 +520,9 @@
$(OPENSSL_PATH)/crypto/x509v3/v3_admis.h
# Autogenerated files list ends here
buildinf.h
- rand_pool_noise.h
ossl_store.c
rand_pool.c
-[Sources.Ia32]
- rand_pool_noise_tsc.c
-
-[Sources.X64]
- rand_pool_noise_tsc.c
-
-[Sources.ARM]
- rand_pool_noise.c
-
-[Sources.AARCH64]
- rand_pool_noise.c
-
[Packages]
MdePkg/MdePkg.dec
CryptoPkg/CryptoPkg.dec
@@ -543,7 +530,7 @@
[LibraryClasses]
BaseLib
DebugLib
- TimerLib
+ RngLib
PrintLib
[LibraryClasses.ARM]
diff --git a/CryptoPkg/Library/OpensslLib/rand_pool_noise.h b/CryptoPkg/Library/OpensslLib/rand_pool_noise.h
deleted file mode 100644
index 75acc686a9f1..000000000000
--- a/CryptoPkg/Library/OpensslLib/rand_pool_noise.h
+++ /dev/null
@@ -1,29 +0,0 @@
-/** @file
- Provide rand noise source.
-
-Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
-SPDX-License-Identifier: BSD-2-Clause-Patent
-
-**/
-
-#ifndef __RAND_POOL_NOISE_H__
-#define __RAND_POOL_NOISE_H__
-
-#include <Uefi/UefiBaseType.h>
-
-/**
- Get 64-bit noise source.
-
- @param[out] Rand Buffer pointer to store 64-bit noise source
-
- @retval TRUE Get randomness successfully.
- @retval FALSE Failed to generate
-**/
-BOOLEAN
-EFIAPI
-GetRandomNoise64 (
- OUT UINT64 *Rand
- );
-
-
-#endif // __RAND_POOL_NOISE_H__
--
2.27.0.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v1 2/2] MdePkg: TimerRngLib: Added RngLib that uses TimerLib
2020-07-28 1:53 [PATCH v1 0/2] Use RngLib instead of TimerLib for OpensslLib Matthew Carlson
2020-07-28 1:53 ` [PATCH v1 1/2] CryptoPkg: OpensslLib: Use RngLib to generate entropy in rand_pool Matthew Carlson
@ 2020-07-28 1:53 ` Matthew Carlson
2020-07-28 2:06 ` [edk2-devel] " Yao, Jiewen
1 sibling, 1 reply; 6+ messages in thread
From: Matthew Carlson @ 2020-07-28 1:53 UTC (permalink / raw)
To: devel; +Cc: Michael D Kinney, Liming Gao, Zhiguang Liu, Matthew Carlson
From: Matthew Carlson <macarl@microsoft.com>
Added a new RngLib that provides random numbers from the TimerLib
using the performance counter. This is meant to be used for OpenSSL
to replicate past behavior. This should not be used in production as
a real source of entropy.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Signed-off-by: Matthew Carlson <matthewfcarlson@gmail.com>
---
MdePkg/Library/TimerRngLib/TimerRng.c | 153 ++++++++++++++++++++
MdePkg/Library/TimerRngLib/TimerRngLib.inf | 37 +++++
MdePkg/MdePkg.dsc | 2 +
3 files changed, 192 insertions(+)
diff --git a/MdePkg/Library/TimerRngLib/TimerRng.c b/MdePkg/Library/TimerRngLib/TimerRng.c
new file mode 100644
index 000000000000..1b0f7f04c01d
--- /dev/null
+++ b/MdePkg/Library/TimerRngLib/TimerRng.c
@@ -0,0 +1,153 @@
+/** @file
+ BaseRng Library that uses the TimerLib to provide reasonably random numbers.
+ Do not use this on a production system.
+
+ Copyright (c) Microsoft Corporation.
+ SPDX-License-Identifier: BSD-2-Clause-Patent
+**/
+
+#include <Library/BaseLib.h>
+#include <Library/DebugLib.h>
+#include <Library/TimerLib.h>
+#include <Uefi.h>
+
+/**
+ Generates a 16-bit random number.
+
+ if Rand is NULL, then ASSERT().
+
+ @param[out] Rand Buffer pointer to store the 16-bit random value.
+
+ @retval TRUE Random number generated successfully.
+ @retval FALSE Failed to generate the random number.
+
+**/
+BOOLEAN
+EFIAPI
+GetRandomNumber16 (
+ OUT UINT16 *Rand
+ )
+{
+ UINT32 Index;
+ UINT8* RandPtr;
+
+ ASSERT (Rand != NULL);
+
+ if (NULL == Rand) {
+ return FALSE;
+ }
+
+ RandPtr = (UINT8 *) Rand;
+ // Get 2 bytes of random ish data
+ for (Index = 0; Index < 2; Index ++) {
+ *RandPtr = (UINT8) (GetPerformanceCounter () & 0xFF);
+ MicroSecondDelay (4);
+ RandPtr++;
+ }
+ return TRUE;
+}
+
+/**
+ Generates a 32-bit random number.
+
+ if Rand is NULL, then ASSERT().
+
+ @param[out] Rand Buffer pointer to store the 32-bit random value.
+
+ @retval TRUE Random number generated successfully.
+ @retval FALSE Failed to generate the random number.
+
+**/
+BOOLEAN
+EFIAPI
+GetRandomNumber32 (
+ OUT UINT32 *Rand
+ )
+{
+ UINT32 Index;
+ UINT8* RandPtr;
+
+ ASSERT (Rand != NULL);
+
+ if (NULL == Rand) {
+ return FALSE;
+ }
+
+ RandPtr = (UINT8 *) Rand;
+ // Get 4 bytes of random ish data
+ for (Index = 0; Index < 4; Index ++) {
+ *RandPtr = (UINT8) (GetPerformanceCounter () & 0xFF);
+ MicroSecondDelay (2);
+ RandPtr++;
+ }
+ return TRUE;
+}
+
+/**
+ Generates a 64-bit random number.
+
+ if Rand is NULL, then ASSERT().
+
+ @param[out] Rand Buffer pointer to store the 64-bit random value.
+
+ @retval TRUE Random number generated successfully.
+ @retval FALSE Failed to generate the random number.
+
+**/
+BOOLEAN
+EFIAPI
+GetRandomNumber64 (
+ OUT UINT64 *Rand
+ )
+{
+ UINT32 Index;
+ UINT8* RandPtr;
+
+ ASSERT (Rand != NULL);
+
+ if (NULL == Rand) {
+ return FALSE;
+ }
+
+ RandPtr = (UINT8 *) Rand;
+ // Get 8 bytes of random ish data
+ for (Index = 0; Index < 8; Index ++) {
+ *RandPtr = (UINT8) (GetPerformanceCounter () & 0xFF);
+ MicroSecondDelay (1);
+ RandPtr++;
+ }
+
+ return TRUE;
+}
+
+/**
+ Generates a 128-bit random number.
+
+ if Rand is NULL, then ASSERT().
+
+ @param[out] Rand Buffer pointer to store the 128-bit random value.
+
+ @retval TRUE Random number generated successfully.
+ @retval FALSE Failed to generate the random number.
+
+**/
+BOOLEAN
+EFIAPI
+GetRandomNumber128 (
+ OUT UINT64 *Rand
+ )
+{
+ ASSERT (Rand != NULL);
+
+ //
+ // Read first 64 bits
+ //
+ if (!GetRandomNumber64 (Rand)) {
+ return FALSE;
+ }
+
+ //
+ // Read second 64 bits
+ //
+ return GetRandomNumber64 (++Rand);
+}
diff --git a/MdePkg/Library/TimerRngLib/TimerRngLib.inf b/MdePkg/Library/TimerRngLib/TimerRngLib.inf
new file mode 100644
index 000000000000..a80a89b77e72
--- /dev/null
+++ b/MdePkg/Library/TimerRngLib/TimerRngLib.inf
@@ -0,0 +1,37 @@
+## @file
+# Instance of RNG (Random Number Generator) Library.
+#
+# BaseRng Library that uses the TimerLib to provide reasonably random numbers.
+# Do not use this on a production system.
+#
+# Copyright (c) Microsoft Corporation. All rights reserved.<BR>
+#
+# SPDX-License-Identifier: BSD-2-Clause-Patent
+#
+#
+##
+
+[Defines]
+ INF_VERSION = 0x00010005
+ BASE_NAME = BaseRngLib
+ MODULE_UNI_FILE = BaseRngLib.uni
+ FILE_GUID = 74950C45-10FC-4AB5-B114-49C87C17409B
+ MODULE_TYPE = BASE
+ VERSION_STRING = 1.0
+ LIBRARY_CLASS = RngLib
+ CONSTRUCTOR = BaseRngLibConstructor
+
+#
+# VALID_ARCHITECTURES = IA32 X64
+#
+
+[Sources]
+ TimerRng.c
+
+[Packages]
+ MdePkg/MdePkg.dec
+
+[LibraryClasses]
+ BaseLib
+ DebugLib
+ TimerLib
diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc
index 472fa3777412..c46a95e52e15 100644
--- a/MdePkg/MdePkg.dsc
+++ b/MdePkg/MdePkg.dsc
@@ -120,6 +120,8 @@
MdePkg/Library/SmmLibNull/SmmLibNull.inf
MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.inf
+ MdePkg/Library/TimerRngLib/TimerRngLib.inf
+
MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoint.inf
MdePkg/Library/StandaloneMmServicesTableLib/StandaloneMmServicesTableLib.inf
--
2.27.0.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH v1 2/2] MdePkg: TimerRngLib: Added RngLib that uses TimerLib
2020-07-28 1:53 ` [PATCH v1 2/2] MdePkg: TimerRngLib: Added RngLib that uses TimerLib Matthew Carlson
@ 2020-07-28 2:06 ` Yao, Jiewen
2020-07-28 2:42 ` Ni, Ray
0 siblings, 1 reply; 6+ messages in thread
From: Yao, Jiewen @ 2020-07-28 2:06 UTC (permalink / raw)
To: devel@edk2.groups.io, matthewfcarlson@gmail.com
Cc: Kinney, Michael D, Gao, Liming, Liu, Zhiguang
Hi Carlson
The naming conversion for a lib instance is <Phase><Class>Lib<Instance>. See examples in MdePkg\Library.
I feel that we had better name it to be BaseRngLibTimer
Thank you
Yao Jiewen
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Matthew
> Carlson
> Sent: Tuesday, July 28, 2020 9:53 AM
> To: devel@edk2.groups.io
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Liu, Zhiguang <zhiguang.liu@intel.com>; Matthew
> Carlson <matthewfcarlson@gmail.com>
> Subject: [edk2-devel] [PATCH v1 2/2] MdePkg: TimerRngLib: Added RngLib that
> uses TimerLib
>
> From: Matthew Carlson <macarl@microsoft.com>
>
> Added a new RngLib that provides random numbers from the TimerLib
> using the performance counter. This is meant to be used for OpenSSL
> to replicate past behavior. This should not be used in production as
> a real source of entropy.
>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Signed-off-by: Matthew Carlson <matthewfcarlson@gmail.com>
> ---
> MdePkg/Library/TimerRngLib/TimerRng.c | 153 ++++++++++++++++++++
> MdePkg/Library/TimerRngLib/TimerRngLib.inf | 37 +++++
> MdePkg/MdePkg.dsc | 2 +
> 3 files changed, 192 insertions(+)
>
> diff --git a/MdePkg/Library/TimerRngLib/TimerRng.c
> b/MdePkg/Library/TimerRngLib/TimerRng.c
> new file mode 100644
> index 000000000000..1b0f7f04c01d
> --- /dev/null
> +++ b/MdePkg/Library/TimerRngLib/TimerRng.c
> @@ -0,0 +1,153 @@
> +/** @file
>
> + BaseRng Library that uses the TimerLib to provide reasonably random
> numbers.
>
> + Do not use this on a production system.
>
> +
>
> + Copyright (c) Microsoft Corporation.
>
> + SPDX-License-Identifier: BSD-2-Clause-Patent
>
> +**/
>
> +
>
> +#include <Library/BaseLib.h>
>
> +#include <Library/DebugLib.h>
>
> +#include <Library/TimerLib.h>
>
> +#include <Uefi.h>
>
> +
>
> +/**
>
> + Generates a 16-bit random number.
>
> +
>
> + if Rand is NULL, then ASSERT().
>
> +
>
> + @param[out] Rand Buffer pointer to store the 16-bit random value.
>
> +
>
> + @retval TRUE Random number generated successfully.
>
> + @retval FALSE Failed to generate the random number.
>
> +
>
> +**/
>
> +BOOLEAN
>
> +EFIAPI
>
> +GetRandomNumber16 (
>
> + OUT UINT16 *Rand
>
> + )
>
> +{
>
> + UINT32 Index;
>
> + UINT8* RandPtr;
>
> +
>
> + ASSERT (Rand != NULL);
>
> +
>
> + if (NULL == Rand) {
>
> + return FALSE;
>
> + }
>
> +
>
> + RandPtr = (UINT8 *) Rand;
>
> + // Get 2 bytes of random ish data
>
> + for (Index = 0; Index < 2; Index ++) {
>
> + *RandPtr = (UINT8) (GetPerformanceCounter () & 0xFF);
>
> + MicroSecondDelay (4);
>
> + RandPtr++;
>
> + }
>
> + return TRUE;
>
> +}
>
> +
>
> +/**
>
> + Generates a 32-bit random number.
>
> +
>
> + if Rand is NULL, then ASSERT().
>
> +
>
> + @param[out] Rand Buffer pointer to store the 32-bit random value.
>
> +
>
> + @retval TRUE Random number generated successfully.
>
> + @retval FALSE Failed to generate the random number.
>
> +
>
> +**/
>
> +BOOLEAN
>
> +EFIAPI
>
> +GetRandomNumber32 (
>
> + OUT UINT32 *Rand
>
> + )
>
> +{
>
> + UINT32 Index;
>
> + UINT8* RandPtr;
>
> +
>
> + ASSERT (Rand != NULL);
>
> +
>
> + if (NULL == Rand) {
>
> + return FALSE;
>
> + }
>
> +
>
> + RandPtr = (UINT8 *) Rand;
>
> + // Get 4 bytes of random ish data
>
> + for (Index = 0; Index < 4; Index ++) {
>
> + *RandPtr = (UINT8) (GetPerformanceCounter () & 0xFF);
>
> + MicroSecondDelay (2);
>
> + RandPtr++;
>
> + }
>
> + return TRUE;
>
> +}
>
> +
>
> +/**
>
> + Generates a 64-bit random number.
>
> +
>
> + if Rand is NULL, then ASSERT().
>
> +
>
> + @param[out] Rand Buffer pointer to store the 64-bit random value.
>
> +
>
> + @retval TRUE Random number generated successfully.
>
> + @retval FALSE Failed to generate the random number.
>
> +
>
> +**/
>
> +BOOLEAN
>
> +EFIAPI
>
> +GetRandomNumber64 (
>
> + OUT UINT64 *Rand
>
> + )
>
> +{
>
> + UINT32 Index;
>
> + UINT8* RandPtr;
>
> +
>
> + ASSERT (Rand != NULL);
>
> +
>
> + if (NULL == Rand) {
>
> + return FALSE;
>
> + }
>
> +
>
> + RandPtr = (UINT8 *) Rand;
>
> + // Get 8 bytes of random ish data
>
> + for (Index = 0; Index < 8; Index ++) {
>
> + *RandPtr = (UINT8) (GetPerformanceCounter () & 0xFF);
>
> + MicroSecondDelay (1);
>
> + RandPtr++;
>
> + }
>
> +
>
> + return TRUE;
>
> +}
>
> +
>
> +/**
>
> + Generates a 128-bit random number.
>
> +
>
> + if Rand is NULL, then ASSERT().
>
> +
>
> + @param[out] Rand Buffer pointer to store the 128-bit random value.
>
> +
>
> + @retval TRUE Random number generated successfully.
>
> + @retval FALSE Failed to generate the random number.
>
> +
>
> +**/
>
> +BOOLEAN
>
> +EFIAPI
>
> +GetRandomNumber128 (
>
> + OUT UINT64 *Rand
>
> + )
>
> +{
>
> + ASSERT (Rand != NULL);
>
> +
>
> + //
>
> + // Read first 64 bits
>
> + //
>
> + if (!GetRandomNumber64 (Rand)) {
>
> + return FALSE;
>
> + }
>
> +
>
> + //
>
> + // Read second 64 bits
>
> + //
>
> + return GetRandomNumber64 (++Rand);
>
> +}
>
> diff --git a/MdePkg/Library/TimerRngLib/TimerRngLib.inf
> b/MdePkg/Library/TimerRngLib/TimerRngLib.inf
> new file mode 100644
> index 000000000000..a80a89b77e72
> --- /dev/null
> +++ b/MdePkg/Library/TimerRngLib/TimerRngLib.inf
> @@ -0,0 +1,37 @@
> +## @file
>
> +# Instance of RNG (Random Number Generator) Library.
>
> +#
>
> +# BaseRng Library that uses the TimerLib to provide reasonably random
> numbers.
>
> +# Do not use this on a production system.
>
> +#
>
> +# Copyright (c) Microsoft Corporation. All rights reserved.<BR>
>
> +#
>
> +# SPDX-License-Identifier: BSD-2-Clause-Patent
>
> +#
>
> +#
>
> +##
>
> +
>
> +[Defines]
>
> + INF_VERSION = 0x00010005
>
> + BASE_NAME = BaseRngLib
>
> + MODULE_UNI_FILE = BaseRngLib.uni
>
> + FILE_GUID = 74950C45-10FC-4AB5-B114-49C87C17409B
>
> + MODULE_TYPE = BASE
>
> + VERSION_STRING = 1.0
>
> + LIBRARY_CLASS = RngLib
>
> + CONSTRUCTOR = BaseRngLibConstructor
>
> +
>
> +#
>
> +# VALID_ARCHITECTURES = IA32 X64
>
> +#
>
> +
>
> +[Sources]
>
> + TimerRng.c
>
> +
>
> +[Packages]
>
> + MdePkg/MdePkg.dec
>
> +
>
> +[LibraryClasses]
>
> + BaseLib
>
> + DebugLib
>
> + TimerLib
>
> diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc
> index 472fa3777412..c46a95e52e15 100644
> --- a/MdePkg/MdePkg.dsc
> +++ b/MdePkg/MdePkg.dsc
> @@ -120,6 +120,8 @@
> MdePkg/Library/SmmLibNull/SmmLibNull.inf
>
>
> MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.inf
>
>
>
> + MdePkg/Library/TimerRngLib/TimerRngLib.inf
>
> +
>
>
> MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoin
> t.inf
>
>
> MdePkg/Library/StandaloneMmServicesTableLib/StandaloneMmServicesTableLi
> b.inf
>
>
>
> --
> 2.27.0.windows.1
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
>
> View/Reply Online (#63373): https://edk2.groups.io/g/devel/message/63373
> Mute This Topic: https://groups.io/mt/75836598/1772286
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [jiewen.yao@intel.com]
> -=-=-=-=-=-=
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH v1 2/2] MdePkg: TimerRngLib: Added RngLib that uses TimerLib
2020-07-28 2:06 ` [edk2-devel] " Yao, Jiewen
@ 2020-07-28 2:42 ` Ni, Ray
0 siblings, 0 replies; 6+ messages in thread
From: Ni, Ray @ 2020-07-28 2:42 UTC (permalink / raw)
To: devel@edk2.groups.io, Yao, Jiewen, matthewfcarlson@gmail.com
Cc: Kinney, Michael D, Gao, Liming, Liu, Zhiguang
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yao, Jiewen
> Sent: Tuesday, July 28, 2020 10:06 AM
> To: devel@edk2.groups.io; matthewfcarlson@gmail.com
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>; Liu, Zhiguang
> <zhiguang.liu@intel.com>
> Subject: Re: [edk2-devel] [PATCH v1 2/2] MdePkg: TimerRngLib: Added RngLib that uses TimerLib
>
> Hi Carlson
> The naming conversion for a lib instance is <Phase><Class>Lib<Instance>. See examples in MdePkg\Library.
Nice summary:)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH v1 1/2] CryptoPkg: OpensslLib: Use RngLib to generate entropy in rand_pool
2020-07-28 1:53 ` [PATCH v1 1/2] CryptoPkg: OpensslLib: Use RngLib to generate entropy in rand_pool Matthew Carlson
@ 2020-07-29 1:08 ` Guomin Jiang
0 siblings, 0 replies; 6+ messages in thread
From: Guomin Jiang @ 2020-07-29 1:08 UTC (permalink / raw)
To: devel@edk2.groups.io, matthewfcarlson@gmail.com
Cc: Yao, Jiewen, Wang, Jian J, Lu, XiaoyuX
Add comments inline
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> Matthew Carlson
> Sent: Tuesday, July 28, 2020 9:53 AM
> To: devel@edk2.groups.io
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>; Lu, XiaoyuX <xiaoyux.lu@intel.com>; Matthew
> Carlson <matthewfcarlson@gmail.com>
> Subject: [edk2-devel] [PATCH v1 1/2] CryptoPkg: OpensslLib: Use RngLib to
> generate entropy in rand_pool
>
> From: Matthew Carlson <macarl@microsoft.com>
>
> Changes OpenSSL to no longer depend on TimerLib and instead use RngLib.
> This allows platforms to decide for themsevles what sort of entropy source
> they provide to OpenSSL and TlsLib.
>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Xiaoyu Lu <xiaoyux.lu@intel.com>
> Signed-off-by: Matthew Carlson <matthewfcarlson@gmail.com>
> ---
> CryptoPkg/Library/OpensslLib/rand_pool.c | 200 ++------------------
> CryptoPkg/Library/OpensslLib/rand_pool_noise.c | 29 ---
> CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c | 43 -----
> CryptoPkg/Library/OpensslLib/OpensslLib.inf | 15 +-
> CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf | 15 +-
> CryptoPkg/Library/OpensslLib/rand_pool_noise.h | 29 ---
> 6 files changed, 20 insertions(+), 311 deletions(-)
>
> diff --git a/CryptoPkg/Library/OpensslLib/rand_pool.c
> b/CryptoPkg/Library/OpensslLib/rand_pool.c
> index 9e0179b03490..55bf6c9c6950 100644
> --- a/CryptoPkg/Library/OpensslLib/rand_pool.c
> +++ b/CryptoPkg/Library/OpensslLib/rand_pool.c
> @@ -11,44 +11,10 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> #include <openssl/aes.h>
>
>
>
> #include <Uefi.h>
>
> -#include <Library/TimerLib.h>
>
> +#include <Library/RngLib.h>
>
>
>
> #include "rand_pool_noise.h"
It seem that you delete the rand_pool_noise.h file, but forget to remove the header?
>
>
>
> -/**
>
> - Get some randomness from low-order bits of GetPerformanceCounter
> results.
>
> - And combine them to the 64-bit value
>
> -
>
> - @param[out] Rand Buffer pointer to store the 64-bit random value.
>
> -
>
> - @retval TRUE Random number generated successfully.
>
> - @retval FALSE Failed to generate.
>
> -**/
>
> -STATIC
>
> -BOOLEAN
>
> -EFIAPI
>
> -GetRandNoise64FromPerformanceCounter(
>
> - OUT UINT64 *Rand
>
> - )
>
> -{
>
> - UINT32 Index;
>
> - UINT32 *RandPtr;
>
> -
>
> - if (NULL == Rand) {
>
> - return FALSE;
>
> - }
>
> -
>
> - RandPtr = (UINT32 *) Rand;
>
> -
>
> - for (Index = 0; Index < 2; Index ++) {
>
> - *RandPtr = (UINT32) (GetPerformanceCounter () & 0xFF);
>
> - MicroSecondDelay (10);
>
> - RandPtr++;
>
> - }
>
> -
>
> - return TRUE;
>
> -}
>
> -
>
> /**
>
> Calls RandomNumber64 to fill
>
> a buffer of arbitrary size with random bytes.
>
> @@ -56,8 +22,8 @@ GetRandNoise64FromPerformanceCounter(
> @param[in] Length Size of the buffer, in bytes, to fill with.
>
> @param[out] RandBuffer Pointer to the buffer to store the random
> result.
>
>
>
> - @retval EFI_SUCCESS Random bytes generation succeeded.
>
> - @retval EFI_NOT_READY Failed to request random bytes.
>
> + @retval True Random bytes generation succeeded.
>
> + @retval False Failed to request random bytes.
>
>
>
> **/
>
> STATIC
>
> @@ -73,17 +39,17 @@ RandGetBytes (
>
>
> Ret = FALSE;
>
>
>
> + if (RandBuffer == NULL) {
>
> + DEBUG((DEBUG_ERROR, "[OPENSSL_RAND_POOL] NULL RandBuffer. No
> random numbers are generated and your system is not secure\n"));
>
> + ASSERT(FALSE); // Since we can't generate random numbers, we should
> assert. Otherwise we will just blow up later.
>
> + return Ret;
>
> + }
>
> +
>
> +
>
> while (Length > 0) {
>
> - //
>
> - // Get random noise from platform.
>
> - // If it failed, fallback to PerformanceCounter
>
> - // If you really care about security, you must override
>
> - // GetRandomNoise64FromPlatform.
>
> - //
>
> - Ret = GetRandomNoise64 (&TempRand);
>
> - if (Ret == FALSE) {
>
> - Ret = GetRandNoise64FromPerformanceCounter (&TempRand);
>
> - }
>
> + // Use RngLib to get random number
>
> + Ret = GetRandomNumber64(&TempRand);
>
> +
>
> if (!Ret) {
>
> return Ret;
>
> }
>
> @@ -100,125 +66,6 @@ RandGetBytes (
> return Ret;
>
> }
>
>
>
> -/**
>
> - Creates a 128bit random value that is fully forward and backward prediction
> resistant,
>
> - suitable for seeding a NIST SP800-90 Compliant.
>
> - This function takes multiple random numbers from PerformanceCounter to
> ensure reseeding
>
> - and performs AES-CBC-MAC over the data to compute the seed value.
>
> -
>
> - @param[out] SeedBuffer Pointer to a 128bit buffer to store the random
> seed.
>
> -
>
> - @retval TRUE Random seed generation succeeded.
>
> - @retval FALSE Failed to request random bytes.
>
> -
>
> -**/
>
> -STATIC
>
> -BOOLEAN
>
> -EFIAPI
>
> -RandGetSeed128 (
>
> - OUT UINT8 *SeedBuffer
>
> - )
>
> -{
>
> - BOOLEAN Ret;
>
> - UINT8 RandByte[16];
>
> - UINT8 Key[16];
>
> - UINT8 Ffv[16];
>
> - UINT8 Xored[16];
>
> - UINT32 Index;
>
> - UINT32 Index2;
>
> - AES_KEY AESKey;
>
> -
>
> - //
>
> - // Chose an arbitrary key and zero the feed_forward_value (FFV)
>
> - //
>
> - for (Index = 0; Index < 16; Index++) {
>
> - Key[Index] = (UINT8) Index;
>
> - Ffv[Index] = 0;
>
> - }
>
> -
>
> - AES_set_encrypt_key (Key, 16 * 8, &AESKey);
>
> -
>
> - //
>
> - // Perform CBC_MAC over 32 * 128 bit values, with 10us gaps between 128
> bit value
>
> - // The 10us gaps will ensure multiple reseeds within the system time with a
> large
>
> - // design margin.
>
> - //
>
> - for (Index = 0; Index < 32; Index++) {
>
> - MicroSecondDelay (10);
>
> - Ret = RandGetBytes (16, RandByte);
>
> - if (!Ret) {
>
> - return Ret;
>
> - }
>
> -
>
> - //
>
> - // Perform XOR operations on two 128-bit value.
>
> - //
>
> - for (Index2 = 0; Index2 < 16; Index2++) {
>
> - Xored[Index2] = RandByte[Index2] ^ Ffv[Index2];
>
> - }
>
> -
>
> - AES_encrypt (Xored, Ffv, &AESKey);
>
> - }
>
> -
>
> - for (Index = 0; Index < 16; Index++) {
>
> - SeedBuffer[Index] = Ffv[Index];
>
> - }
>
> -
>
> - return Ret;
>
> -}
>
> -
>
> -/**
>
> - Generate high-quality entropy source.
>
> -
>
> - @param[in] Length Size of the buffer, in bytes, to fill with.
>
> - @param[out] Entropy Pointer to the buffer to store the entropy data.
>
> -
>
> - @retval EFI_SUCCESS Entropy generation succeeded.
>
> - @retval EFI_NOT_READY Failed to request random data.
>
> -
>
> -**/
>
> -STATIC
>
> -BOOLEAN
>
> -EFIAPI
>
> -RandGenerateEntropy (
>
> - IN UINTN Length,
>
> - OUT UINT8 *Entropy
>
> - )
>
> -{
>
> - BOOLEAN Ret;
>
> - UINTN BlockCount;
>
> - UINT8 Seed[16];
>
> - UINT8 *Ptr;
>
> -
>
> - BlockCount = Length / 16;
>
> - Ptr = (UINT8 *) Entropy;
>
> -
>
> - //
>
> - // Generate high-quality seed for DRBG Entropy
>
> - //
>
> - while (BlockCount > 0) {
>
> - Ret = RandGetSeed128 (Seed);
>
> - if (!Ret) {
>
> - return Ret;
>
> - }
>
> - CopyMem (Ptr, Seed, 16);
>
> -
>
> - BlockCount--;
>
> - Ptr = Ptr + 16;
>
> - }
>
> -
>
> - //
>
> - // Populate the remained data as request.
>
> - //
>
> - Ret = RandGetSeed128 (Seed);
>
> - if (!Ret) {
>
> - return Ret;
>
> - }
>
> - CopyMem (Ptr, Seed, (Length % 16));
>
> -
>
> - return Ret;
>
> -}
>
> -
>
> /*
>
> * Add random bytes to the pool to acquire requested amount of entropy
>
> *
>
> @@ -238,7 +85,7 @@ size_t rand_pool_acquire_entropy(RAND_POOL
> *pool)
> buffer = rand_pool_add_begin(pool, bytes_needed);
>
>
>
> if (buffer != NULL) {
>
> - Ret = RandGenerateEntropy(bytes_needed, buffer);
>
> + Ret = RandGetBytes(bytes_needed, buffer);
>
> if (FALSE == Ret) {
>
> rand_pool_add_end(pool, 0, 0);
>
> } else {
>
> @@ -257,13 +104,8 @@ size_t rand_pool_acquire_entropy(RAND_POOL
> *pool)
> */
>
> int rand_pool_add_nonce_data(RAND_POOL *pool)
>
> {
>
> - struct {
>
> - UINT64 Rand;
>
> - UINT64 TimerValue;
>
> - } data = { 0 };
>
> -
>
> - RandGetBytes(8, (UINT8 *)&(data.Rand));
>
> - data.TimerValue = GetPerformanceCounter();
>
> + UINT8 data[16];
>
> + RandGetBytes(sizeof(data), data);
>
>
>
> return rand_pool_add(pool, (unsigned char*)&data, sizeof(data), 0);
>
> }
>
> @@ -275,13 +117,8 @@ int rand_pool_add_nonce_data(RAND_POOL *pool)
> */
>
> int rand_pool_add_additional_data(RAND_POOL *pool)
>
> {
>
> - struct {
>
> - UINT64 Rand;
>
> - UINT64 TimerValue;
>
> - } data = { 0 };
>
> -
>
> - RandGetBytes(8, (UINT8 *)&(data.Rand));
>
> - data.TimerValue = GetPerformanceCounter();
>
> + UINT8 data[16];
>
> + RandGetBytes(sizeof(data), data);
>
>
>
> return rand_pool_add(pool, (unsigned char*)&data, sizeof(data), 0);
>
> }
>
> @@ -313,4 +150,3 @@ void rand_pool_cleanup(void)
> void rand_pool_keep_random_devices_open(int keep)
>
> {
>
> }
>
> -
>
> diff --git a/CryptoPkg/Library/OpensslLib/rand_pool_noise.c
> b/CryptoPkg/Library/OpensslLib/rand_pool_noise.c
> deleted file mode 100644
> index 212834e27acc..000000000000
> --- a/CryptoPkg/Library/OpensslLib/rand_pool_noise.c
> +++ /dev/null
> @@ -1,29 +0,0 @@
> -/** @file
>
> - Provide rand noise source.
>
> -
>
> -Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
>
> -SPDX-License-Identifier: BSD-2-Clause-Patent
>
> -
>
> -**/
>
> -
>
> -#include <Library/BaseLib.h>
>
> -
>
> -/**
>
> - Get 64-bit noise source
>
> -
>
> - @param[out] Rand Buffer pointer to store 64-bit noise source
>
> -
>
> - @retval FALSE Failed to generate
>
> -**/
>
> -BOOLEAN
>
> -EFIAPI
>
> -GetRandomNoise64 (
>
> - OUT UINT64 *Rand
>
> - )
>
> -{
>
> - //
>
> - // Return FALSE will fallback to use PerformanceCounter to
>
> - // generate noise.
>
> - //
>
> - return FALSE;
>
> -}
>
> diff --git a/CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c
> b/CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c
> deleted file mode 100644
> index 4158106231fd..000000000000
> --- a/CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c
> +++ /dev/null
> @@ -1,43 +0,0 @@
> -/** @file
>
> - Provide rand noise source.
>
> -
>
> -Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
>
> -SPDX-License-Identifier: BSD-2-Clause-Patent
>
> -
>
> -**/
>
> -
>
> -#include <Library/BaseLib.h>
>
> -#include <Library/DebugLib.h>
>
> -#include <Library/TimerLib.h>
>
> -
>
> -/**
>
> - Get 64-bit noise source
>
> -
>
> - @param[out] Rand Buffer pointer to store 64-bit noise source
>
> -
>
> - @retval TRUE Get randomness successfully.
>
> - @retval FALSE Failed to generate
>
> -**/
>
> -BOOLEAN
>
> -EFIAPI
>
> -GetRandomNoise64 (
>
> - OUT UINT64 *Rand
>
> - )
>
> -{
>
> - UINT32 Index;
>
> - UINT32 *RandPtr;
>
> -
>
> - if (NULL == Rand) {
>
> - return FALSE;
>
> - }
>
> -
>
> - RandPtr = (UINT32 *)Rand;
>
> -
>
> - for (Index = 0; Index < 2; Index ++) {
>
> - *RandPtr = (UINT32) ((AsmReadTsc ()) & 0xFF);
>
> - RandPtr++;
>
> - MicroSecondDelay (10);
>
> - }
>
> -
>
> - return TRUE;
>
> -}
>
> diff --git a/CryptoPkg/Library/OpensslLib/OpensslLib.inf
> b/CryptoPkg/Library/OpensslLib/OpensslLib.inf
> index dbbe5386a10c..4baad565564c 100644
> --- a/CryptoPkg/Library/OpensslLib/OpensslLib.inf
> +++ b/CryptoPkg/Library/OpensslLib/OpensslLib.inf
> @@ -571,22 +571,9 @@
> $(OPENSSL_PATH)/ssl/statem/statem_local.h
>
> # Autogenerated files list ends here
>
> buildinf.h
>
> - rand_pool_noise.h
>
> ossl_store.c
>
> rand_pool.c
>
>
>
> -[Sources.Ia32]
>
> - rand_pool_noise_tsc.c
>
> -
>
> -[Sources.X64]
>
> - rand_pool_noise_tsc.c
>
> -
>
> -[Sources.ARM]
>
> - rand_pool_noise.c
>
> -
>
> -[Sources.AARCH64]
>
> - rand_pool_noise.c
>
> -
>
> [Packages]
>
> MdePkg/MdePkg.dec
>
> CryptoPkg/CryptoPkg.dec
>
> @@ -594,7 +581,7 @@
> [LibraryClasses]
>
> BaseLib
>
> DebugLib
>
> - TimerLib
>
> + RngLib
>
> PrintLib
>
>
>
> [LibraryClasses.ARM]
>
> diff --git a/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
> b/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
> index 616ccd9f62d1..3557711bd85a 100644
> --- a/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
> +++ b/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
> @@ -520,22 +520,9 @@
> $(OPENSSL_PATH)/crypto/x509v3/v3_admis.h
>
> # Autogenerated files list ends here
>
> buildinf.h
>
> - rand_pool_noise.h
>
> ossl_store.c
>
> rand_pool.c
>
>
>
> -[Sources.Ia32]
>
> - rand_pool_noise_tsc.c
>
> -
>
> -[Sources.X64]
>
> - rand_pool_noise_tsc.c
>
> -
>
> -[Sources.ARM]
>
> - rand_pool_noise.c
>
> -
>
> -[Sources.AARCH64]
>
> - rand_pool_noise.c
>
> -
>
> [Packages]
>
> MdePkg/MdePkg.dec
>
> CryptoPkg/CryptoPkg.dec
>
> @@ -543,7 +530,7 @@
> [LibraryClasses]
>
> BaseLib
>
> DebugLib
>
> - TimerLib
>
> + RngLib
>
> PrintLib
>
>
>
> [LibraryClasses.ARM]
>
> diff --git a/CryptoPkg/Library/OpensslLib/rand_pool_noise.h
> b/CryptoPkg/Library/OpensslLib/rand_pool_noise.h
> deleted file mode 100644
> index 75acc686a9f1..000000000000
> --- a/CryptoPkg/Library/OpensslLib/rand_pool_noise.h
> +++ /dev/null
> @@ -1,29 +0,0 @@
> -/** @file
>
> - Provide rand noise source.
>
> -
>
> -Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
>
> -SPDX-License-Identifier: BSD-2-Clause-Patent
>
> -
>
> -**/
>
> -
>
> -#ifndef __RAND_POOL_NOISE_H__
>
> -#define __RAND_POOL_NOISE_H__
>
> -
>
> -#include <Uefi/UefiBaseType.h>
>
> -
>
> -/**
>
> - Get 64-bit noise source.
>
> -
>
> - @param[out] Rand Buffer pointer to store 64-bit noise source
>
> -
>
> - @retval TRUE Get randomness successfully.
>
> - @retval FALSE Failed to generate
>
> -**/
>
> -BOOLEAN
>
> -EFIAPI
>
> -GetRandomNoise64 (
>
> - OUT UINT64 *Rand
>
> - );
>
> -
>
> -
>
> -#endif // __RAND_POOL_NOISE_H__
>
> --
> 2.27.0.windows.1
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
>
> View/Reply Online (#63372): https://edk2.groups.io/g/devel/message/63372
> Mute This Topic: https://groups.io/mt/75836597/4399222
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub
> [guomin.jiang@intel.com]
> -=-=-=-=-=-=
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-07-29 1:09 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-07-28 1:53 [PATCH v1 0/2] Use RngLib instead of TimerLib for OpensslLib Matthew Carlson
2020-07-28 1:53 ` [PATCH v1 1/2] CryptoPkg: OpensslLib: Use RngLib to generate entropy in rand_pool Matthew Carlson
2020-07-29 1:08 ` [edk2-devel] " Guomin Jiang
2020-07-28 1:53 ` [PATCH v1 2/2] MdePkg: TimerRngLib: Added RngLib that uses TimerLib Matthew Carlson
2020-07-28 2:06 ` [edk2-devel] " Yao, Jiewen
2020-07-28 2:42 ` Ni, Ray
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox