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