Re: [edk2] [Patch 3/4] MdeModulePkg/Bds: Support short-form URI boot.

2016-03-02 Thread Wang, Sunny (HPS SW)
, February 26, 2016 8:28 PM To: Wang, Sunny (HPS SW); edk2-devel@lists.01.org Cc: Fu, Siyuan; El-Haj-Mahmoud, Samer Subject: RE: [edk2] [Patch 3/4] MdeModulePkg/Bds: Support short-form URI boot. Importance: High Sunny, For a short-form URI device path, it's like Uri(...)\End. We cannot get the p

Re: [edk2] [Patch 3/4] MdeModulePkg/Bds: Support short-form URI boot.

2016-02-26 Thread Ni, Ruiyu
t;Sent: Friday, February 26, 2016 5:28 PM >To: Ni, Ruiyu ; edk2-devel@lists.01.org >Cc: Fu, Siyuan ; Wang, Sunny (HPS SW) >; El-Haj-Mahmoud, Samer > >Subject: RE: [edk2] [Patch 3/4] MdeModulePkg/Bds: Support short-form URI boot. > >Hi Ray, > For this patch, I just have a

Re: [edk2] [Patch 3/4] MdeModulePkg/Bds: Support short-form URI boot.

2016-02-26 Thread Wang, Sunny (HPS SW)
ePath instead. Regards, Sunny Wang -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ruiyu Ni Sent: Wednesday, February 24, 2016 3:40 PM To: edk2-devel@lists.01.org Cc: Ruiyu Ni; Siyuan Fu Subject: [edk2] [Patch 3/4] MdeModulePkg/Bds: Support short

Re: [edk2] [Patch 3/4] MdeModulePkg/Bds: Support short-form URI boot.

2016-02-24 Thread Fu, Siyuan
Hi, Ray You'd better to set the FileBuffer pointer to NULL after free it, otherwise the function will return a freed pointer if the second LoadFile() is falied. Other part is good to me. +if (FileBuffer != NULL) { + FreePool (FileBuffer); +} Reviewed-by: Fu Siyuan > -Ori

[edk2] [Patch 3/4] MdeModulePkg/Bds: Support short-form URI boot.

2016-02-23 Thread Ruiyu Ni
The patch adds short-form URI boot support to follow UEFI Spec. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Siyuan Fu --- MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 72 1 file changed, 72 insertions(+) diff --git a/MdeMod