Please remove the ending back-slash in following lines. They are not needed.
+  while (MaxSize >= sizeof (EFI_DEVICE_PATH_PROTOCOL) && \
+        (MaxSize - sizeof (EFI_DEVICE_PATH_PROTOCOL) >= Size) && \
+        !IsDevicePathEnd (DevicePath)) {

With the above changes, Reviewed-by: Ruiyu Ni <ruiyu...@intel.com>

>-----Original Message-----
>From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Eric 
>Dong
>Sent: Wednesday, October 12, 2016 8:39 PM
>To: edk2-devel@lists.01.org; Kinney, Michael D <michael.d.kin...@intel.com>; 
>Gao, Liming <liming....@intel.com>
>Subject: [edk2] [Patch 0/2] MdePkg DevicePathLib: Check buffer length before 
>use.
>
>In IsDevicePathValid API, code should validate the device path
>buffer not exceed the input MaxSize before reference the path
>info. This patched series add this check.
>
>Eric Dong (2):
>  MdePkg UefiDevicePathLib: Validate buffer length before use buffer.
>  MdePkg UefiDevicePathLibDevicePathProtocol: Validate before use.
>
> .../UefiDevicePathLib/DevicePathUtilities.c        | 25 ++++++++++++++++------
> .../UefiDevicePathLib.c                            | 24 +++++++++++++++------
> 2 files changed, 35 insertions(+), 14 deletions(-)
>
>--
>2.6.4.windows.1
>
>_______________________________________________
>edk2-devel mailing list
>edk2-devel@lists.01.org
>https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to