Re: [edk2-devel] [PATCH] OvmfPkg/SmbiosPlatformDxe: tweak fallback release date again

["joeyli via groups.io" <jlee=suse.com@groups.io>]

Hi experts,

On Sat, Feb 03, 2024 at 12:32:01AM +0800, Lee, Chun-Yi wrote:
> In case PcdFirmwareReleaseDateString is not set use a valid date
> as fallback. But the default valid date can _NOT_ pass the Microsoft
> SVVP test "Check SMBIOS Table Specific Requirements". The test emitted
> the error message:
> 
> BIOS Release Date string is unexpected length: 8. This string must be in
> MM/DD/YYYY format. No other format is allowed and no additional information
> may be included. See field description in the SMBIOS specification.
> 
> Base on SMBIOS spec v3.7.0:
> 
> 08h     2.0+    BIOS Release Date       BYTE    STRING
> String number of the BIOS release date. The date
> string, if supplied, is in either mm/dd/yy or
> mm/dd/yyyy format. If the year portion of the string
> is two digits, the year is assumed to be 19yy.
> NOTE: The mm/dd/yyyy format is required for SMBIOS
> version 2.3 and later.
> 
> So, let's tweek the fallback release date again.
> 
> Fixes: a0f9628705e3 ("OvmfPkg/SmbiosPlatformDxe: tweak fallback release date") [edk2-stable202305~327]
> Signed-off-by: "Lee, Chun-Yi" <jlee@suse.com>
> ---
>  CryptoPkg/Library/OpensslLib/openssl          | 2 +-
>  OvmfPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/CryptoPkg/Library/OpensslLib/openssl b/CryptoPkg/Library/OpensslLib/openssl
> index de90e54bbe..830bf8e1e4 160000
> --- a/CryptoPkg/Library/OpensslLib/openssl
> +++ b/CryptoPkg/Library/OpensslLib/openssl
> @@ -1 +1 @@
> -Subproject commit de90e54bbe82e5be4fb9608b6f5c308bb837d355
> +Subproject commit 830bf8e1e4749ad65c51b6a1d0d769ae689404ba

Sorry for I touched openssl commit when generating this patch.
I will re-send patch again.

Joey Lee

> diff --git a/OvmfPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c b/OvmfPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> index 0ca3776045..e929da6b81 100644
> --- a/OvmfPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> +++ b/OvmfPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.c
> @@ -160,7 +160,7 @@ InstallAllStructures (
>      DateStr = (CHAR16 *)FixedPcdGetPtr (PcdFirmwareReleaseDateString);
>      DateLen = StrLen (DateStr);
>      if (DateLen < 3) {
> -      DateStr = L"2/2/2022";
> +      DateStr = L"02/02/2022";
>        DateLen = StrLen (DateStr);
>      }
>  
> -- 
> 2.12.3


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#115082): https://edk2.groups.io/g/devel/message/115082
Mute This Topic: https://groups.io/mt/104153758/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-