[edk2-devel] [PATCH v2] MdeModulePkg/DxeCorePerformanceLib: Switch to UnicodeStrnToAsciiStrS
Jeff Brasen
jbrasen at nvidia.com
Fri Jul 17 20:46:29 UTC 2020
Do we need any additional review on this before it getting submitted?
-Jeff
-----Original Message-----
From: Bi, Dandan <dandan.bi at intel.com>
Sent: Tuesday, June 23, 2020 7:36 PM
To: Jeff Brasen <jbrasen at nvidia.com>; devel at edk2.groups.io
Cc: Gao, Liming <liming.gao at intel.com>
Subject: RE: [PATCH v2] MdeModulePkg/DxeCorePerformanceLib: Switch to UnicodeStrnToAsciiStrS
External email: Use caution opening links or attachments
Reviewed-by: Dandan Bi <dandan.bi at intel.com>
Thanks,
Dandan
> -----Original Message-----
> From: Jeff Brasen <jbrasen at nvidia.com>
> Sent: Tuesday, June 23, 2020 2:20 AM
> To: devel at edk2.groups.io
> Cc: Gao, Liming <liming.gao at intel.com>; Bi, Dandan
> <dandan.bi at intel.com>; Jeff Brasen <jbrasen at nvidia.com>
> Subject: [PATCH v2] MdeModulePkg/DxeCorePerformanceLib: Switch to
> UnicodeStrnToAsciiStrS
>
> UnicodeStrToAsciiStrS requires that the source string is shorter than
> the destination buffer and will ASSERT if this is not true. Switch to
> UnicodeStrnToAsciiStrS as there are cases where the source string is
> longer than the buffer allocated for the device path.
>
> Signed-off-by: Jeff Brasen <jbrasen at nvidia.com>
> ---
> .../Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git
> a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLi
> b.c
> b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLi
> b.c
> index f500e20b32..d378c59dd9 100644
> ---
> a/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLi
> b.c
> +++
> b/MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLi
> b.c
> @@ -837,7 +837,7 @@ GetDeviceInfoFromHandleAndUpdateLength (
> ControllerNameStringSize = FPDT_MAX_PERF_RECORD_SIZE -
> (*Length) - 1;
> }
>
> - UnicodeStrToAsciiStrS(StringPtr, ComponentNameString,
> ControllerNameStringSize);
> + UnicodeStrnToAsciiStrS(StringPtr, ControllerNameStringSize - 1,
> + ComponentNameString, ControllerNameStringSize,
> + &ControllerNameStringSize);
>
> //
> // Add a space in the end of the ControllerName @@ -879,7 +879,7
> @@ GetDeviceInfoFromHandleAndUpdateLength (
> AsciiStringPtr = ComponentNameString;
> }
>
> - UnicodeStrToAsciiStrS(StringPtr, AsciiStringPtr, DevicePathStringSize);
> + UnicodeStrnToAsciiStrS(StringPtr, DevicePathStringSize - 1,
> + AsciiStringPtr, DevicePathStringSize, &DevicePathStringSize);
> *Length += (UINT8)DevicePathStringSize;
> return EFI_SUCCESS;
> }
> --
> 2.17.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#62782): https://edk2.groups.io/g/devel/message/62782
Mute This Topic: https://groups.io/mt/75044750/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list