Reviewed-by: Liming Gao <[email protected]>

>-----Original Message-----
>From: Wu, Hao A
>Sent: Friday, January 06, 2017 4:50 PM
>To: [email protected]
>Cc: Wu, Hao A <[email protected]>; Gao, Liming <[email protected]>;
>Kinney, Michael D <[email protected]>
>Subject: [PATCH] MdePkg/UefiFileHandleLib: Refine the check for valid Ascii
>character
>
>The commit will check if bit 7 is set of an Ascii character to judge its
>validity.
>
>Cc: Liming Gao <[email protected]>
>Cc: Michael Kinney <[email protected]>
>Contributed-under: TianoCore Contribution Agreement 1.0
>Signed-off-by: Hao Wu <[email protected]>
>---
> MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
>b/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
>index daed0f4..57aad77 100644
>--- a/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
>+++ b/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
>@@ -1,7 +1,7 @@
> /** @file
>   Provides interface to EFI_FILE_HANDLE functionality.
>
>-  Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved. <BR>
>+  Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved. <BR>
>   This program and the accompanying materials
>   are licensed and made available under the terms and conditions of the BSD
>License
>   which accompanies this distribution.  The full text of the license may be
>found at
>@@ -1138,7 +1138,7 @@ FileHandleWriteLine(
>     }
>     UnicodeStrToAsciiStrS (Buffer, AsciiBuffer, Size);
>     for (Index = 0; Index < Size; Index++) {
>-      if (!((AsciiBuffer[Index] >= 0) && (AsciiBuffer[Index] < 128))){
>+      if ((AsciiBuffer[Index] & BIT7) != 0) {
>         FreePool(AsciiBuffer);
>         return EFI_INVALID_PARAMETER;
>       }
>--
>1.9.5.msysgit.0

_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to