* Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib [not found] <15D6A215430F1F2B.28343@groups.io> @ 2019-11-13 8:43 ` Wang, Jian J 2019-11-13 9:28 ` Laszlo Ersek 0 siblings, 1 reply; 4+ messages in thread From: Wang, Jian J @ 2019-11-13 8:43 UTC (permalink / raw) To: devel@edk2.groups.io, Wang, Jian J Cc: Kinney, Michael D, Gao, Liming, Laszlo Ersek, Ard Biesheuvel, Ni, Ray As requested by Ray, I'd like to mention more usage about this library instance. To be more specific, if following components or functionalities are used in a platform, the BaseRngLibNull should *not* be used. Instead, a non-Null version of RngLib must be used (like BaseRngLib for IA32/X64, or coming RngDxeLib for all ARCHs). - SecurityPkg/HddPassword/HddPasswordDxe.c - AES, TLS (NetworkPkg/TlsDxe/TlsDxe.inf, CryptoPkg/Library/TlsLib/TlsLib.inf) RSA_OAEP, RSA_PK1 - (If BaseRngLibNull interface ASSERTed at boot time) Regards, Jian > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Wang, Jian > J > Sent: Wednesday, November 13, 2019 1:35 PM > To: devel@edk2.groups.io > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming > <liming.gao@intel.com>; Laszlo Ersek <lersek@redhat.com>; Ard Biesheuvel > <ard.biesheuvel@linaro.org>; Ni, Ray <ray.ni@intel.com> > Subject: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib > > >v2: > > - Change the name from RngLibNull to BaseRngLibNull according to Laszlo's > > comments > > - Move the module from SecurityPkg to MdePkg according to Laszlo's > comments > > - Update commit message according to Laszlo and Ray's comments > > This is null version of RngLib which should be used with modules that > inherit an (indirect) dependency on the RngLib class, but never actually > call RngLib APIs for consuming randomness. > > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 > Cc: Michael D Kinney <michael.d.kinney@intel.com> > Cc: Liming Gao <liming.gao@intel.com> > Cc: Laszlo Ersek <lersek@redhat.com> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > Cc: Ray Ni <ray.ni@intel.com> > Signed-off-by: Jian J Wang <jian.j.wang@intel.com> > --- > .../Library/BaseRngLibNull/BaseRngLibNull.c | 95 +++++++++++++++++++ > .../Library/BaseRngLibNull/BaseRngLibNull.inf | 31 ++++++ > .../Library/BaseRngLibNull/BaseRngLibNull.uni | 14 +++ > MdePkg/MdePkg.dsc | 1 + > 4 files changed, 141 insertions(+) > create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c > create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni > > diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c > b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c > new file mode 100644 > index 0000000000..13677abc84 > --- /dev/null > +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c > @@ -0,0 +1,95 @@ > +/** @file > + Null version of Random number generator services. > + > +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/RngLib.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 > + ) > +{ > + ASSERT (FALSE); > + return FALSE; > +} > + > +/** > + 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 > + ) > +{ > + ASSERT (FALSE); > + return FALSE; > +} > + > +/** > + 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 > + ) > +{ > + ASSERT (FALSE); > + return FALSE; > +} > + > +/** > + 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 (FALSE); > + return FALSE; > +} > diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > new file mode 100644 > index 0000000000..f456df1dae > --- /dev/null > +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > @@ -0,0 +1,31 @@ > +## @file > +# Null instance of RNG (Random Number Generator) Library. > +# > +# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > +# > +# SPDX-License-Identifier: BSD-2-Clause-Patent > +# > +## > + > +[Defines] > + INF_VERSION = 0x00010005 > + BASE_NAME = BaseRngLibNull > + MODULE_UNI_FILE = BaseRngLibNull.uni > + FILE_GUID = CD8991F8-2061-4084-8C9E-9C6F352DC58D > + MODULE_TYPE = BASE > + VERSION_STRING = 1.0 > + LIBRARY_CLASS = RngLib > + > +# > +# VALID_ARCHITECTURES = IA32 X64 ARM AARCH64 > +# > + > +[Sources] > + BaseRngLibNull.c > + > +[Packages] > + MdePkg/MdePkg.dec > + > +[LibraryClasses] > + BaseLib > + DebugLib > diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni > b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni > new file mode 100644 > index 0000000000..f32be6a617 > --- /dev/null > +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni > @@ -0,0 +1,14 @@ > +// /** @file > +// Null Instance of RNG (Random Number Generator) Library. > +// > +// Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > +// > +// SPDX-License-Identifier: BSD-2-Clause-Patent > +// > +// **/ > + > + > +#string STR_MODULE_ABSTRACT #language en-US "Null Instance of > RNG Library" > + > +#string STR_MODULE_DESCRIPTION #language en-US "This library > instance should be used with modules that inherit an (indirect) dependency on > the RngLib class, but never actually call RngLib APIs for consuming randomness." > + > diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc > index c1278e7907..0aeafaaacc 100644 > --- a/MdePkg/MdePkg.dsc > +++ b/MdePkg/MdePkg.dsc > @@ -61,6 +61,7 @@ > > MdePkg/Library/BaseUefiDecompressLib/BaseUefiTianoCustomDecompressLib. > inf > MdePkg/Library/BaseSmbusLibNull/BaseSmbusLibNull.inf > MdePkg/Library/BaseSafeIntLib/BaseSafeIntLib.inf > + MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > > MdePkg/Library/DxeCoreEntryPoint/DxeCoreEntryPoint.inf > MdePkg/Library/DxeCoreHobLib/DxeCoreHobLib.inf > -- > 2.17.1.windows.2 > > > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib 2019-11-13 8:43 ` [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib Wang, Jian J @ 2019-11-13 9:28 ` Laszlo Ersek 2019-11-13 14:21 ` Wang, Jian J 0 siblings, 1 reply; 4+ messages in thread From: Laszlo Ersek @ 2019-11-13 9:28 UTC (permalink / raw) To: Wang, Jian J, devel@edk2.groups.io Cc: Kinney, Michael D, Gao, Liming, Ard Biesheuvel, Ni, Ray On 11/13/19 09:43, Wang, Jian J wrote: > As requested by Ray, I'd like to mention more usage about this library > instance. > > To be more specific, if following components or functionalities are used in a > platform, the BaseRngLibNull should *not* be used. Instead, a non-Null version > of RngLib must be used (like BaseRngLib for IA32/X64, or coming RngDxeLib > for all ARCHs). > > - SecurityPkg/HddPassword/HddPasswordDxe.c > - AES, TLS (NetworkPkg/TlsDxe/TlsDxe.inf, CryptoPkg/Library/TlsLib/TlsLib.inf) > RSA_OAEP, RSA_PK1 > - (If BaseRngLibNull interface ASSERTed at boot time) Just a naming suggestion for "RngDxeLib": I think it should be called "DxeRngLibXxxx", where "Xxxx" should stand for the core idea in the implementation. For example, if the library is based on EFI_RNG_PROTOCOL, then it likely should be called DxeRngLibRngProtocol This scheme seems consistent with some other library instance names under MdePkg/Library: MdePkg/Library/UefiDebugLibDebugPortProtocol MdePkg/Library/UefiDevicePathLibDevicePathProtocol and maybe even with MdePkg/Library/SmmPciLibPciRootBridgeIo MdePkg/Library/UefiPciLibPciRootBridgeIo Thanks Laszlo > > Regards, > Jian > >> -----Original Message----- >> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Wang, Jian >> J >> Sent: Wednesday, November 13, 2019 1:35 PM >> To: devel@edk2.groups.io >> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming >> <liming.gao@intel.com>; Laszlo Ersek <lersek@redhat.com>; Ard Biesheuvel >> <ard.biesheuvel@linaro.org>; Ni, Ray <ray.ni@intel.com> >> Subject: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib >> >>> v2: >>> - Change the name from RngLibNull to BaseRngLibNull according to Laszlo's >>> comments >>> - Move the module from SecurityPkg to MdePkg according to Laszlo's >> comments >>> - Update commit message according to Laszlo and Ray's comments >> >> This is null version of RngLib which should be used with modules that >> inherit an (indirect) dependency on the RngLib class, but never actually >> call RngLib APIs for consuming randomness. >> >> >> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 >> Cc: Michael D Kinney <michael.d.kinney@intel.com> >> Cc: Liming Gao <liming.gao@intel.com> >> Cc: Laszlo Ersek <lersek@redhat.com> >> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> >> Cc: Ray Ni <ray.ni@intel.com> >> Signed-off-by: Jian J Wang <jian.j.wang@intel.com> >> --- >> .../Library/BaseRngLibNull/BaseRngLibNull.c | 95 +++++++++++++++++++ >> .../Library/BaseRngLibNull/BaseRngLibNull.inf | 31 ++++++ >> .../Library/BaseRngLibNull/BaseRngLibNull.uni | 14 +++ >> MdePkg/MdePkg.dsc | 1 + >> 4 files changed, 141 insertions(+) >> create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c >> create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf >> create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni >> >> diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c >> b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c >> new file mode 100644 >> index 0000000000..13677abc84 >> --- /dev/null >> +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c >> @@ -0,0 +1,95 @@ >> +/** @file >> + Null version of Random number generator services. >> + >> +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/RngLib.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 >> + ) >> +{ >> + ASSERT (FALSE); >> + return FALSE; >> +} >> + >> +/** >> + 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 >> + ) >> +{ >> + ASSERT (FALSE); >> + return FALSE; >> +} >> + >> +/** >> + 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 >> + ) >> +{ >> + ASSERT (FALSE); >> + return FALSE; >> +} >> + >> +/** >> + 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 (FALSE); >> + return FALSE; >> +} >> diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf >> b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf >> new file mode 100644 >> index 0000000000..f456df1dae >> --- /dev/null >> +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf >> @@ -0,0 +1,31 @@ >> +## @file >> +# Null instance of RNG (Random Number Generator) Library. >> +# >> +# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> >> +# >> +# SPDX-License-Identifier: BSD-2-Clause-Patent >> +# >> +## >> + >> +[Defines] >> + INF_VERSION = 0x00010005 >> + BASE_NAME = BaseRngLibNull >> + MODULE_UNI_FILE = BaseRngLibNull.uni >> + FILE_GUID = CD8991F8-2061-4084-8C9E-9C6F352DC58D >> + MODULE_TYPE = BASE >> + VERSION_STRING = 1.0 >> + LIBRARY_CLASS = RngLib >> + >> +# >> +# VALID_ARCHITECTURES = IA32 X64 ARM AARCH64 >> +# >> + >> +[Sources] >> + BaseRngLibNull.c >> + >> +[Packages] >> + MdePkg/MdePkg.dec >> + >> +[LibraryClasses] >> + BaseLib >> + DebugLib >> diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni >> b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni >> new file mode 100644 >> index 0000000000..f32be6a617 >> --- /dev/null >> +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni >> @@ -0,0 +1,14 @@ >> +// /** @file >> +// Null Instance of RNG (Random Number Generator) Library. >> +// >> +// Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> >> +// >> +// SPDX-License-Identifier: BSD-2-Clause-Patent >> +// >> +// **/ >> + >> + >> +#string STR_MODULE_ABSTRACT #language en-US "Null Instance of >> RNG Library" >> + >> +#string STR_MODULE_DESCRIPTION #language en-US "This library >> instance should be used with modules that inherit an (indirect) dependency on >> the RngLib class, but never actually call RngLib APIs for consuming randomness." >> + >> diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc >> index c1278e7907..0aeafaaacc 100644 >> --- a/MdePkg/MdePkg.dsc >> +++ b/MdePkg/MdePkg.dsc >> @@ -61,6 +61,7 @@ >> >> MdePkg/Library/BaseUefiDecompressLib/BaseUefiTianoCustomDecompressLib. >> inf >> MdePkg/Library/BaseSmbusLibNull/BaseSmbusLibNull.inf >> MdePkg/Library/BaseSafeIntLib/BaseSafeIntLib.inf >> + MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf >> >> MdePkg/Library/DxeCoreEntryPoint/DxeCoreEntryPoint.inf >> MdePkg/Library/DxeCoreHobLib/DxeCoreHobLib.inf >> -- >> 2.17.1.windows.2 >> >> >> > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib 2019-11-13 9:28 ` Laszlo Ersek @ 2019-11-13 14:21 ` Wang, Jian J 0 siblings, 0 replies; 4+ messages in thread From: Wang, Jian J @ 2019-11-13 14:21 UTC (permalink / raw) To: Laszlo Ersek, devel@edk2.groups.io Cc: Kinney, Michael D, Gao, Liming, Ard Biesheuvel, Ni, Ray Laszlo, > -----Original Message----- > From: Laszlo Ersek <lersek@redhat.com> > Sent: Wednesday, November 13, 2019 5:28 PM > To: Wang, Jian J <jian.j.wang@intel.com>; devel@edk2.groups.io > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming > <liming.gao@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>; Ni, Ray > <ray.ni@intel.com> > Subject: Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib > > On 11/13/19 09:43, Wang, Jian J wrote: > > As requested by Ray, I'd like to mention more usage about this library > > instance. > > > > To be more specific, if following components or functionalities are used in a > > platform, the BaseRngLibNull should *not* be used. Instead, a non-Null > version > > of RngLib must be used (like BaseRngLib for IA32/X64, or coming RngDxeLib > > for all ARCHs). > > > > - SecurityPkg/HddPassword/HddPasswordDxe.c > > - AES, TLS (NetworkPkg/TlsDxe/TlsDxe.inf, CryptoPkg/Library/TlsLib/TlsLib.inf) > > RSA_OAEP, RSA_PK1 > > - (If BaseRngLibNull interface ASSERTed at boot time) > > Just a naming suggestion for "RngDxeLib": I think it should be called > "DxeRngLibXxxx", where "Xxxx" should stand for the core idea in the > implementation. For example, if the library is based on > EFI_RNG_PROTOCOL, then it likely should be called > > DxeRngLibRngProtocol > > This scheme seems consistent with some other library instance names > under MdePkg/Library: > > MdePkg/Library/UefiDebugLibDebugPortProtocol > MdePkg/Library/UefiDevicePathLibDevicePathProtocol > > and maybe even with > > MdePkg/Library/SmmPciLibPciRootBridgeIo > MdePkg/Library/UefiPciLibPciRootBridgeIo > Good suggestion. Thanks. Regards, Jian > Thanks > Laszlo > > > > > Regards, > > Jian > > > >> -----Original Message----- > >> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Wang, > Jian > >> J > >> Sent: Wednesday, November 13, 2019 1:35 PM > >> To: devel@edk2.groups.io > >> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming > >> <liming.gao@intel.com>; Laszlo Ersek <lersek@redhat.com>; Ard Biesheuvel > >> <ard.biesheuvel@linaro.org>; Ni, Ray <ray.ni@intel.com> > >> Subject: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib > >> > >>> v2: > >>> - Change the name from RngLibNull to BaseRngLibNull according to Laszlo's > >>> comments > >>> - Move the module from SecurityPkg to MdePkg according to Laszlo's > >> comments > >>> - Update commit message according to Laszlo and Ray's comments > >> > >> This is null version of RngLib which should be used with modules that > >> inherit an (indirect) dependency on the RngLib class, but never actually > >> call RngLib APIs for consuming randomness. > >> > >> > >> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 > >> Cc: Michael D Kinney <michael.d.kinney@intel.com> > >> Cc: Liming Gao <liming.gao@intel.com> > >> Cc: Laszlo Ersek <lersek@redhat.com> > >> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > >> Cc: Ray Ni <ray.ni@intel.com> > >> Signed-off-by: Jian J Wang <jian.j.wang@intel.com> > >> --- > >> .../Library/BaseRngLibNull/BaseRngLibNull.c | 95 +++++++++++++++++++ > >> .../Library/BaseRngLibNull/BaseRngLibNull.inf | 31 ++++++ > >> .../Library/BaseRngLibNull/BaseRngLibNull.uni | 14 +++ > >> MdePkg/MdePkg.dsc | 1 + > >> 4 files changed, 141 insertions(+) > >> create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c > >> create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > >> create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni > >> > >> diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c > >> b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c > >> new file mode 100644 > >> index 0000000000..13677abc84 > >> --- /dev/null > >> +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c > >> @@ -0,0 +1,95 @@ > >> +/** @file > >> + Null version of Random number generator services. > >> + > >> +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/RngLib.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 > >> + ) > >> +{ > >> + ASSERT (FALSE); > >> + return FALSE; > >> +} > >> + > >> +/** > >> + 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 > >> + ) > >> +{ > >> + ASSERT (FALSE); > >> + return FALSE; > >> +} > >> + > >> +/** > >> + 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 > >> + ) > >> +{ > >> + ASSERT (FALSE); > >> + return FALSE; > >> +} > >> + > >> +/** > >> + 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 (FALSE); > >> + return FALSE; > >> +} > >> diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > >> b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > >> new file mode 100644 > >> index 0000000000..f456df1dae > >> --- /dev/null > >> +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > >> @@ -0,0 +1,31 @@ > >> +## @file > >> +# Null instance of RNG (Random Number Generator) Library. > >> +# > >> +# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > >> +# > >> +# SPDX-License-Identifier: BSD-2-Clause-Patent > >> +# > >> +## > >> + > >> +[Defines] > >> + INF_VERSION = 0x00010005 > >> + BASE_NAME = BaseRngLibNull > >> + MODULE_UNI_FILE = BaseRngLibNull.uni > >> + FILE_GUID = CD8991F8-2061-4084-8C9E-9C6F352DC58D > >> + MODULE_TYPE = BASE > >> + VERSION_STRING = 1.0 > >> + LIBRARY_CLASS = RngLib > >> + > >> +# > >> +# VALID_ARCHITECTURES = IA32 X64 ARM AARCH64 > >> +# > >> + > >> +[Sources] > >> + BaseRngLibNull.c > >> + > >> +[Packages] > >> + MdePkg/MdePkg.dec > >> + > >> +[LibraryClasses] > >> + BaseLib > >> + DebugLib > >> diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni > >> b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni > >> new file mode 100644 > >> index 0000000000..f32be6a617 > >> --- /dev/null > >> +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni > >> @@ -0,0 +1,14 @@ > >> +// /** @file > >> +// Null Instance of RNG (Random Number Generator) Library. > >> +// > >> +// Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> > >> +// > >> +// SPDX-License-Identifier: BSD-2-Clause-Patent > >> +// > >> +// **/ > >> + > >> + > >> +#string STR_MODULE_ABSTRACT #language en-US "Null Instance of > >> RNG Library" > >> + > >> +#string STR_MODULE_DESCRIPTION #language en-US "This library > >> instance should be used with modules that inherit an (indirect) dependency on > >> the RngLib class, but never actually call RngLib APIs for consuming > randomness." > >> + > >> diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc > >> index c1278e7907..0aeafaaacc 100644 > >> --- a/MdePkg/MdePkg.dsc > >> +++ b/MdePkg/MdePkg.dsc > >> @@ -61,6 +61,7 @@ > >> > >> > MdePkg/Library/BaseUefiDecompressLib/BaseUefiTianoCustomDecompressLib. > >> inf > >> MdePkg/Library/BaseSmbusLibNull/BaseSmbusLibNull.inf > >> MdePkg/Library/BaseSafeIntLib/BaseSafeIntLib.inf > >> + MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > >> > >> MdePkg/Library/DxeCoreEntryPoint/DxeCoreEntryPoint.inf > >> MdePkg/Library/DxeCoreHobLib/DxeCoreHobLib.inf > >> -- > >> 2.17.1.windows.2 > >> > >> > >> > > ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v2] MdePkg: add null version of RngLib @ 2019-11-13 5:35 Wang, Jian J 2019-11-13 8:17 ` [edk2-devel] " Liming Gao 0 siblings, 1 reply; 4+ messages in thread From: Wang, Jian J @ 2019-11-13 5:35 UTC (permalink / raw) To: devel; +Cc: Michael D Kinney, Liming Gao, Laszlo Ersek, Ard Biesheuvel, Ray Ni >v2: > - Change the name from RngLibNull to BaseRngLibNull according to Laszlo's > comments > - Move the module from SecurityPkg to MdePkg according to Laszlo's comments > - Update commit message according to Laszlo and Ray's comments This is null version of RngLib which should be used with modules that inherit an (indirect) dependency on the RngLib class, but never actually call RngLib APIs for consuming randomness. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 Cc: Michael D Kinney <michael.d.kinney@intel.com> Cc: Liming Gao <liming.gao@intel.com> Cc: Laszlo Ersek <lersek@redhat.com> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> Cc: Ray Ni <ray.ni@intel.com> Signed-off-by: Jian J Wang <jian.j.wang@intel.com> --- .../Library/BaseRngLibNull/BaseRngLibNull.c | 95 +++++++++++++++++++ .../Library/BaseRngLibNull/BaseRngLibNull.inf | 31 ++++++ .../Library/BaseRngLibNull/BaseRngLibNull.uni | 14 +++ MdePkg/MdePkg.dsc | 1 + 4 files changed, 141 insertions(+) create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c new file mode 100644 index 0000000000..13677abc84 --- /dev/null +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c @@ -0,0 +1,95 @@ +/** @file + Null version of Random number generator services. + +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/RngLib.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 + ) +{ + ASSERT (FALSE); + return FALSE; +} + +/** + 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 + ) +{ + ASSERT (FALSE); + return FALSE; +} + +/** + 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 + ) +{ + ASSERT (FALSE); + return FALSE; +} + +/** + 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 (FALSE); + return FALSE; +} diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf new file mode 100644 index 0000000000..f456df1dae --- /dev/null +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf @@ -0,0 +1,31 @@ +## @file +# Null instance of RNG (Random Number Generator) Library. +# +# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> +# +# SPDX-License-Identifier: BSD-2-Clause-Patent +# +## + +[Defines] + INF_VERSION = 0x00010005 + BASE_NAME = BaseRngLibNull + MODULE_UNI_FILE = BaseRngLibNull.uni + FILE_GUID = CD8991F8-2061-4084-8C9E-9C6F352DC58D + MODULE_TYPE = BASE + VERSION_STRING = 1.0 + LIBRARY_CLASS = RngLib + +# +# VALID_ARCHITECTURES = IA32 X64 ARM AARCH64 +# + +[Sources] + BaseRngLibNull.c + +[Packages] + MdePkg/MdePkg.dec + +[LibraryClasses] + BaseLib + DebugLib diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni new file mode 100644 index 0000000000..f32be6a617 --- /dev/null +++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni @@ -0,0 +1,14 @@ +// /** @file +// Null Instance of RNG (Random Number Generator) Library. +// +// Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> +// +// SPDX-License-Identifier: BSD-2-Clause-Patent +// +// **/ + + +#string STR_MODULE_ABSTRACT #language en-US "Null Instance of RNG Library" + +#string STR_MODULE_DESCRIPTION #language en-US "This library instance should be used with modules that inherit an (indirect) dependency on the RngLib class, but never actually call RngLib APIs for consuming randomness." + diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc index c1278e7907..0aeafaaacc 100644 --- a/MdePkg/MdePkg.dsc +++ b/MdePkg/MdePkg.dsc @@ -61,6 +61,7 @@ MdePkg/Library/BaseUefiDecompressLib/BaseUefiTianoCustomDecompressLib.inf MdePkg/Library/BaseSmbusLibNull/BaseSmbusLibNull.inf MdePkg/Library/BaseSafeIntLib/BaseSafeIntLib.inf + MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf MdePkg/Library/DxeCoreEntryPoint/DxeCoreEntryPoint.inf MdePkg/Library/DxeCoreHobLib/DxeCoreHobLib.inf -- 2.17.1.windows.2 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib 2019-11-13 5:35 Wang, Jian J @ 2019-11-13 8:17 ` Liming Gao 0 siblings, 0 replies; 4+ messages in thread From: Liming Gao @ 2019-11-13 8:17 UTC (permalink / raw) To: devel@edk2.groups.io, Wang, Jian J Cc: Kinney, Michael D, Laszlo Ersek, Ard Biesheuvel, Ni, Ray The change is good. Reviewed-by: Liming Gao <liming.gao@intel.com> >-----Original Message----- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >Wang, Jian J >Sent: Wednesday, November 13, 2019 1:35 PM >To: devel@edk2.groups.io >Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming ><liming.gao@intel.com>; Laszlo Ersek <lersek@redhat.com>; Ard Biesheuvel ><ard.biesheuvel@linaro.org>; Ni, Ray <ray.ni@intel.com> >Subject: [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib > >>v2: >> - Change the name from RngLibNull to BaseRngLibNull according to Laszlo's >> comments >> - Move the module from SecurityPkg to MdePkg according to Laszlo's >comments >> - Update commit message according to Laszlo and Ray's comments > >This is null version of RngLib which should be used with modules that >inherit an (indirect) dependency on the RngLib class, but never actually >call RngLib APIs for consuming randomness. > > >Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 >Cc: Michael D Kinney <michael.d.kinney@intel.com> >Cc: Liming Gao <liming.gao@intel.com> >Cc: Laszlo Ersek <lersek@redhat.com> >Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> >Cc: Ray Ni <ray.ni@intel.com> >Signed-off-by: Jian J Wang <jian.j.wang@intel.com> >--- > .../Library/BaseRngLibNull/BaseRngLibNull.c | 95 +++++++++++++++++++ > .../Library/BaseRngLibNull/BaseRngLibNull.inf | 31 ++++++ > .../Library/BaseRngLibNull/BaseRngLibNull.uni | 14 +++ > MdePkg/MdePkg.dsc | 1 + > 4 files changed, 141 insertions(+) > create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c > create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > create mode 100644 MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni > >diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c >b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c >new file mode 100644 >index 0000000000..13677abc84 >--- /dev/null >+++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.c >@@ -0,0 +1,95 @@ >+/** @file >+ Null version of Random number generator services. >+ >+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/RngLib.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 >+ ) >+{ >+ ASSERT (FALSE); >+ return FALSE; >+} >+ >+/** >+ 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 >+ ) >+{ >+ ASSERT (FALSE); >+ return FALSE; >+} >+ >+/** >+ 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 >+ ) >+{ >+ ASSERT (FALSE); >+ return FALSE; >+} >+ >+/** >+ 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 (FALSE); >+ return FALSE; >+} >diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf >b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf >new file mode 100644 >index 0000000000..f456df1dae >--- /dev/null >+++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf >@@ -0,0 +1,31 @@ >+## @file >+# Null instance of RNG (Random Number Generator) Library. >+# >+# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> >+# >+# SPDX-License-Identifier: BSD-2-Clause-Patent >+# >+## >+ >+[Defines] >+ INF_VERSION = 0x00010005 >+ BASE_NAME = BaseRngLibNull >+ MODULE_UNI_FILE = BaseRngLibNull.uni >+ FILE_GUID = CD8991F8-2061-4084-8C9E-9C6F352DC58D >+ MODULE_TYPE = BASE >+ VERSION_STRING = 1.0 >+ LIBRARY_CLASS = RngLib >+ >+# >+# VALID_ARCHITECTURES = IA32 X64 ARM AARCH64 >+# >+ >+[Sources] >+ BaseRngLibNull.c >+ >+[Packages] >+ MdePkg/MdePkg.dec >+ >+[LibraryClasses] >+ BaseLib >+ DebugLib >diff --git a/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni >b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni >new file mode 100644 >index 0000000000..f32be6a617 >--- /dev/null >+++ b/MdePkg/Library/BaseRngLibNull/BaseRngLibNull.uni >@@ -0,0 +1,14 @@ >+// /** @file >+// Null Instance of RNG (Random Number Generator) Library. >+// >+// Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> >+// >+// SPDX-License-Identifier: BSD-2-Clause-Patent >+// >+// **/ >+ >+ >+#string STR_MODULE_ABSTRACT #language en-US "Null Instance of >RNG Library" >+ >+#string STR_MODULE_DESCRIPTION #language en-US "This library >instance should be used with modules that inherit an (indirect) dependency >on the RngLib class, but never actually call RngLib APIs for consuming >randomness." >+ >diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc >index c1278e7907..0aeafaaacc 100644 >--- a/MdePkg/MdePkg.dsc >+++ b/MdePkg/MdePkg.dsc >@@ -61,6 +61,7 @@ > >MdePkg/Library/BaseUefiDecompressLib/BaseUefiTianoCustomDecompressL >ib.inf > MdePkg/Library/BaseSmbusLibNull/BaseSmbusLibNull.inf > MdePkg/Library/BaseSafeIntLib/BaseSafeIntLib.inf >+ MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf > > MdePkg/Library/DxeCoreEntryPoint/DxeCoreEntryPoint.inf > MdePkg/Library/DxeCoreHobLib/DxeCoreHobLib.inf >-- >2.17.1.windows.2 > > > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-11-13 14:21 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <15D6A215430F1F2B.28343@groups.io>
2019-11-13 8:43 ` [edk2-devel] [PATCH v2] MdePkg: add null version of RngLib Wang, Jian J
2019-11-13 9:28 ` Laszlo Ersek
2019-11-13 14:21 ` Wang, Jian J
2019-11-13 5:35 Wang, Jian J
2019-11-13 8:17 ` [edk2-devel] " Liming Gao
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox