Hi Pierre, Thank you for this patch.
Reviewed-by: Sami Mujawar <sami.muja...@arm.com> Regards, Sami Mujawar On 10/10/2022 10:20 am, pierre.gond...@arm.com wrote:
From: Pierre Gondois <pierre.gond...@arm.com> Add a PrintString to print strings in the CmObjParser. String must be NULL terminated and no buffer overrun check is done by this function. Signed-off-by: Pierre Gondois <pierre.gond...@arm.com> --- .../ConfigurationManagerObjectParser.c | 26 +++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.c b/DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.c index 13e6c0e321c4..b46f19693bb5 100644 --- a/DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.c +++ b/DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.c @@ -19,6 +19,14 @@ PrintOemId ( UINT8 *Ptr );+STATIC+VOID +EFIAPI +PrintString ( + CONST CHAR8 *Format, + UINT8 *Ptr + ); + /** A parser for EArmObjBootArchInfo. */ STATIC CONST CM_OBJ_PARSER CmArmBootArchInfoParser[] = { @@ -646,6 +654,24 @@ PrintOemId ( )); }+/** Print string.+ + The string must be NULL terminated. + + @param [in] Format Format to print the Ptr. + @param [in] Ptr Pointer to the string. +**/ +STATIC +VOID +EFIAPI +PrintString ( + CONST CHAR8 *Format, + UINT8 *Ptr + ) +{ + DEBUG ((DEBUG_ERROR, "%a", Ptr)); +} + /** Print fields of the objects.@param [in] Data Pointer to the object to print.
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#95601): https://edk2.groups.io/g/devel/message/95601 Mute This Topic: https://groups.io/mt/94232521/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
