MdePkg/Library/UefiDebugLibStdErr/DebugLib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
From: Marvin Haeuser <mhaeuser@outlook.de>
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2302
The second argument of "UnicodeVSPrintAsciiFormat" is "BufferSize",
which takes the size of the buffer in bytes. Replace the currently
used MAX_DEBUG_MESSAGE_LENGTH usage, which is the buffer's length,
with the actual buffer size.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Signed-off-by: Marvin Haeuser <mhaeuser@outlook.de>
---
MdePkg/Library/UefiDebugLibStdErr/DebugLib.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c b/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c
index 40eb697e7e2c..fcfdafede08f 100644
--- a/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c
+++ b/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c
@@ -106,9 +106,9 @@ DebugPrintMarker (
// Convert the DEBUG() message to a Unicode String
//
if (BaseListMarker == NULL) {
- UnicodeVSPrintAsciiFormat (Buffer, MAX_DEBUG_MESSAGE_LENGTH, Format, VaListMarker);
+ UnicodeVSPrintAsciiFormat (Buffer, sizeof (Buffer), Format, VaListMarker);
} else {
- UnicodeBSPrintAsciiFormat (Buffer, MAX_DEBUG_MESSAGE_LENGTH, Format, BaseListMarker);
+ UnicodeBSPrintAsciiFormat (Buffer, sizeof (Buffer), Format, BaseListMarker);
}
//
--
2.23.0.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#49310): https://edk2.groups.io/g/devel/message/49310
Mute This Topic: https://groups.io/mt/36272499/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Reviewed-by: Liming Gao <liming.gao@intel.com> >-----Original Message----- >From: Marvin Häuser [mailto:Marvin.Haeuser@outlook.com] >Sent: Monday, October 21, 2019 1:28 AM >To: devel@edk2.groups.io >Cc: vit9696@protonmail.com; Kinney, Michael D ><michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com> >Subject: [PATCH] MdePkg/UefiDebugLibStdErr: Pass the correct buffer size > >From: Marvin Haeuser <mhaeuser@outlook.de> > >REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2302 > >The second argument of "UnicodeVSPrintAsciiFormat" is "BufferSize", >which takes the size of the buffer in bytes. Replace the currently >used MAX_DEBUG_MESSAGE_LENGTH usage, which is the buffer's length, >with the actual buffer size. > >Cc: Michael D Kinney <michael.d.kinney@intel.com> >Cc: Liming Gao <liming.gao@intel.com> >Signed-off-by: Marvin Haeuser <mhaeuser@outlook.de> >--- > MdePkg/Library/UefiDebugLibStdErr/DebugLib.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c >b/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c >index 40eb697e7e2c..fcfdafede08f 100644 >--- a/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c >+++ b/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c >@@ -106,9 +106,9 @@ DebugPrintMarker ( > // Convert the DEBUG() message to a Unicode String > > // > > if (BaseListMarker == NULL) { > >- UnicodeVSPrintAsciiFormat (Buffer, MAX_DEBUG_MESSAGE_LENGTH, >Format, VaListMarker); > >+ UnicodeVSPrintAsciiFormat (Buffer, sizeof (Buffer), Format, >VaListMarker); > > } else { > >- UnicodeBSPrintAsciiFormat (Buffer, MAX_DEBUG_MESSAGE_LENGTH, >Format, BaseListMarker); > >+ UnicodeBSPrintAsciiFormat (Buffer, sizeof (Buffer), Format, >BaseListMarker); > > } > > > > // > >-- >2.23.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#49272): https://edk2.groups.io/g/devel/message/49272 Mute This Topic: https://groups.io/mt/36272499/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.