* [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it @ 2017-02-07 2:53 Ruiyu Ni 2017-02-08 8:13 ` Wang, Sunny (HPS SW) 0 siblings, 1 reply; 6+ messages in thread From: Ruiyu Ni @ 2017-02-07 2:53 UTC (permalink / raw) To: edk2-devel; +Cc: Jeff Fan BmExpandMediaDevicePath contains a bug that it uses the uninitialized Handle. Since the function is called when the Handle supports BlockIo or SimpleFileSystem, when there is no SimpleFileSystem installed on the Handle, BlockIo should be installed on the Handle. The fix initializes the Handle by locating the BlockIo protocol from the device path. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com> Cc: Jeff Fan <jeff.fan@intel.com> --- MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c index 75bd5dc..9f99122 100644 --- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c @@ -991,9 +991,13 @@ BmExpandMediaDevicePath ( return FileBuffer; } + Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, &TempDevicePath, &Handle); + ASSERT_EFI_ERROR (Status); + // // For device boot option only pointing to the removable device handle, - // should make sure all its children handles (its child partion or media handles) are created and connected. + // should make sure all its children handles (its child partion or media handles) + // are created and connected. // gBS->ConnectController (Handle, NULL, NULL, TRUE); @@ -1004,8 +1008,6 @@ BmExpandMediaDevicePath ( // returned. After the Block IO protocol is reinstalled, subsequent // Block IO read/write will success. // - Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, &TempDevicePath, &Handle); - ASSERT_EFI_ERROR (Status); Status = gBS->HandleProtocol (Handle, &gEfiBlockIoProtocolGuid, (VOID **) &BlockIo); ASSERT_EFI_ERROR (Status); Buffer = AllocatePool (BlockIo->Media->BlockSize); -- 2.9.0.windows.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it 2017-02-07 2:53 [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it Ruiyu Ni @ 2017-02-08 8:13 ` Wang, Sunny (HPS SW) 2017-02-08 9:17 ` Laszlo Ersek 2017-02-08 9:18 ` Ni, Ruiyu 0 siblings, 2 replies; 6+ messages in thread From: Wang, Sunny (HPS SW) @ 2017-02-08 8:13 UTC (permalink / raw) To: Ruiyu Ni, edk2-devel@lists.01.org; +Cc: Jeff Fan, Wang, Sunny (HPS SW) Looks good to me. Reviewed-by: Sunny Wang <sunnywang@hpe.com> However, I saw the other potential issue below. If you also think the potential issue is valid, you can fix this in the other patch. We need to consider the case where MDEPKG_NDEBUG is defined. We're using ASSERT_EFI_ERROR to handle the error status from LocateDevicePath() in the following code block, which contains potential bug that we still use uninitialized Handle for the case where LocateDevicePath returns error status. Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, &TempDevicePath, &Handle); ASSERT_EFI_ERROR (Status); -----Original Message----- From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Ruiyu Ni Sent: Tuesday, February 07, 2017 10:54 AM To: edk2-devel@lists.01.org Cc: Jeff Fan <jeff.fan@intel.com> Subject: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it BmExpandMediaDevicePath contains a bug that it uses the uninitialized Handle. Since the function is called when the Handle supports BlockIo or SimpleFileSystem, when there is no SimpleFileSystem installed on the Handle, BlockIo should be installed on the Handle. The fix initializes the Handle by locating the BlockIo protocol from the device path. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com> Cc: Jeff Fan <jeff.fan@intel.com> --- MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c index 75bd5dc..9f99122 100644 --- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c @@ -991,9 +991,13 @@ BmExpandMediaDevicePath ( return FileBuffer; } + Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, + &TempDevicePath, &Handle); ASSERT_EFI_ERROR (Status); + // // For device boot option only pointing to the removable device handle, - // should make sure all its children handles (its child partion or media handles) are created and connected. + // should make sure all its children handles (its child partion or + media handles) // are created and connected. // gBS->ConnectController (Handle, NULL, NULL, TRUE); @@ -1004,8 +1008,6 @@ BmExpandMediaDevicePath ( // returned. After the Block IO protocol is reinstalled, subsequent // Block IO read/write will success. // - Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, &TempDevicePath, &Handle); - ASSERT_EFI_ERROR (Status); Status = gBS->HandleProtocol (Handle, &gEfiBlockIoProtocolGuid, (VOID **) &BlockIo); ASSERT_EFI_ERROR (Status); Buffer = AllocatePool (BlockIo->Media->BlockSize); -- 2.9.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it 2017-02-08 8:13 ` Wang, Sunny (HPS SW) @ 2017-02-08 9:17 ` Laszlo Ersek 2017-02-08 9:38 ` Wang, Sunny (HPS SW) 2017-02-08 9:18 ` Ni, Ruiyu 1 sibling, 1 reply; 6+ messages in thread From: Laszlo Ersek @ 2017-02-08 9:17 UTC (permalink / raw) To: Wang, Sunny (HPS SW), Ruiyu Ni, edk2-devel@lists.01.org; +Cc: Jeff Fan On 02/08/17 09:13, Wang, Sunny (HPS SW) wrote: > Looks good to me. > Reviewed-by: Sunny Wang <sunnywang@hpe.com> > > However, I saw the other potential issue below. If you also think the > potential issue is valid, you can fix this in the other patch. > We need to consider the case where MDEPKG_NDEBUG is defined. We're > using ASSERT_EFI_ERROR to handle the error status from > LocateDevicePath() in the following code block, which contains > potential bug that we still use uninitialized Handle for the case > where LocateDevicePath returns error status. > > Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, &TempDevicePath, &Handle); > ASSERT_EFI_ERROR (Status); I think that Ray made a sufficient argument in the commit message why the assertion was correct -- it is indeed an assertion, and not error checking. Since the function is called when the Handle supports BlockIo or SimpleFileSystem, when there is no SimpleFileSystem installed on the Handle, BlockIo should be installed on the Handle. Perhaps the wording could be clarified, as in: ... BlockIo is *guaranteed* to be installed on the Handle. Just my two cents anyway. Laszlo > > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Ruiyu Ni > Sent: Tuesday, February 07, 2017 10:54 AM > To: edk2-devel@lists.01.org > Cc: Jeff Fan <jeff.fan@intel.com> > Subject: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it > > BmExpandMediaDevicePath contains a bug that it uses the uninitialized Handle. > > Since the function is called when the Handle supports BlockIo or SimpleFileSystem, when there is no SimpleFileSystem installed on the Handle, BlockIo should be installed on the Handle. > The fix initializes the Handle by locating the BlockIo protocol from the device path. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com> > Cc: Jeff Fan <jeff.fan@intel.com> > --- > MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > index 75bd5dc..9f99122 100644 > --- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > @@ -991,9 +991,13 @@ BmExpandMediaDevicePath ( > return FileBuffer; > } > > + Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > + &TempDevicePath, &Handle); ASSERT_EFI_ERROR (Status); > + > // > // For device boot option only pointing to the removable device handle, > - // should make sure all its children handles (its child partion or media handles) are created and connected. > + // should make sure all its children handles (its child partion or > + media handles) // are created and connected. > // > gBS->ConnectController (Handle, NULL, NULL, TRUE); > > @@ -1004,8 +1008,6 @@ BmExpandMediaDevicePath ( > // returned. After the Block IO protocol is reinstalled, subsequent > // Block IO read/write will success. > // > - Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, &TempDevicePath, &Handle); > - ASSERT_EFI_ERROR (Status); > Status = gBS->HandleProtocol (Handle, &gEfiBlockIoProtocolGuid, (VOID **) &BlockIo); > ASSERT_EFI_ERROR (Status); > Buffer = AllocatePool (BlockIo->Media->BlockSize); > -- > 2.9.0.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 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it 2017-02-08 9:17 ` Laszlo Ersek @ 2017-02-08 9:38 ` Wang, Sunny (HPS SW) 2017-02-08 9:41 ` Ni, Ruiyu 0 siblings, 1 reply; 6+ messages in thread From: Wang, Sunny (HPS SW) @ 2017-02-08 9:38 UTC (permalink / raw) To: Laszlo Ersek, Ruiyu Ni, edk2-devel@lists.01.org Cc: Jeff Fan, Wang, Sunny (HPS SW) Yeah, I misread the commit message and had a misunderstanding on what situation BmExpandMediaDevicePath() is called . Ray and Lazlo, thanks for both of you clarifying this. :) Regards, Sunny Wang -----Original Message----- From: Laszlo Ersek [mailto:lersek@redhat.com] Sent: Wednesday, February 08, 2017 5:18 PM To: Wang, Sunny (HPS SW) <sunnywang@hpe.com>; Ruiyu Ni <ruiyu.ni@intel.com>; edk2-devel@lists.01.org <edk2-devel@ml01.01.org> Cc: Jeff Fan <jeff.fan@intel.com> Subject: Re: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it Importance: High On 02/08/17 09:13, Wang, Sunny (HPS SW) wrote: > Looks good to me. > Reviewed-by: Sunny Wang <sunnywang@hpe.com> > > However, I saw the other potential issue below. If you also think the > potential issue is valid, you can fix this in the other patch. > We need to consider the case where MDEPKG_NDEBUG is defined. We're > using ASSERT_EFI_ERROR to handle the error status from > LocateDevicePath() in the following code block, which contains > potential bug that we still use uninitialized Handle for the case > where LocateDevicePath returns error status. > > Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > &TempDevicePath, &Handle); ASSERT_EFI_ERROR (Status); I think that Ray made a sufficient argument in the commit message why the assertion was correct -- it is indeed an assertion, and not error checking. Since the function is called when the Handle supports BlockIo or SimpleFileSystem, when there is no SimpleFileSystem installed on the Handle, BlockIo should be installed on the Handle. Perhaps the wording could be clarified, as in: ... BlockIo is *guaranteed* to be installed on the Handle. Just my two cents anyway. Laszlo > > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Ruiyu Ni > Sent: Tuesday, February 07, 2017 10:54 AM > To: edk2-devel@lists.01.org > Cc: Jeff Fan <jeff.fan@intel.com> > Subject: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize > Handle before using it > > BmExpandMediaDevicePath contains a bug that it uses the uninitialized Handle. > > Since the function is called when the Handle supports BlockIo or SimpleFileSystem, when there is no SimpleFileSystem installed on the Handle, BlockIo should be installed on the Handle. > The fix initializes the Handle by locating the BlockIo protocol from the device path. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com> > Cc: Jeff Fan <jeff.fan@intel.com> > --- > MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > index 75bd5dc..9f99122 100644 > --- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > @@ -991,9 +991,13 @@ BmExpandMediaDevicePath ( > return FileBuffer; > } > > + Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > + &TempDevicePath, &Handle); ASSERT_EFI_ERROR (Status); > + > // > // For device boot option only pointing to the removable device > handle, > - // should make sure all its children handles (its child partion or media handles) are created and connected. > + // should make sure all its children handles (its child partion or > + media handles) // are created and connected. > // > gBS->ConnectController (Handle, NULL, NULL, TRUE); > > @@ -1004,8 +1008,6 @@ BmExpandMediaDevicePath ( > // returned. After the Block IO protocol is reinstalled, subsequent > // Block IO read/write will success. > // > - Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > &TempDevicePath, &Handle); > - ASSERT_EFI_ERROR (Status); > Status = gBS->HandleProtocol (Handle, &gEfiBlockIoProtocolGuid, (VOID **) &BlockIo); > ASSERT_EFI_ERROR (Status); > Buffer = AllocatePool (BlockIo->Media->BlockSize); > -- > 2.9.0.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 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it 2017-02-08 9:38 ` Wang, Sunny (HPS SW) @ 2017-02-08 9:41 ` Ni, Ruiyu 0 siblings, 0 replies; 6+ messages in thread From: Ni, Ruiyu @ 2017-02-08 9:41 UTC (permalink / raw) To: Wang, Sunny (HPS SW), Laszlo Ersek, edk2-devel@lists.01.org; +Cc: Fan, Jeff I will change the commit message as Laszlo suggested. Thanks/Ray > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Wang, Sunny (HPS SW) > Sent: Wednesday, February 8, 2017 5:38 PM > To: Laszlo Ersek <lersek@redhat.com>; Ni, Ruiyu <ruiyu.ni@intel.com>; > edk2-devel@lists.01.org <edk2-devel@ml01.01.org> > Cc: Fan, Jeff <jeff.fan@intel.com> > Subject: Re: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize > Handle before using it > > Yeah, I misread the commit message and had a misunderstanding on what > situation BmExpandMediaDevicePath() is called . > Ray and Lazlo, thanks for both of you clarifying this. :) > > Regards, > Sunny Wang > > -----Original Message----- > From: Laszlo Ersek [mailto:lersek@redhat.com] > Sent: Wednesday, February 08, 2017 5:18 PM > To: Wang, Sunny (HPS SW) <sunnywang@hpe.com>; Ruiyu Ni > <ruiyu.ni@intel.com>; edk2-devel@lists.01.org <edk2-devel@ml01.01.org> > Cc: Jeff Fan <jeff.fan@intel.com> > Subject: Re: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize > Handle before using it > Importance: High > > On 02/08/17 09:13, Wang, Sunny (HPS SW) wrote: > > Looks good to me. > > Reviewed-by: Sunny Wang <sunnywang@hpe.com> > > > > However, I saw the other potential issue below. If you also think the > > potential issue is valid, you can fix this in the other patch. > > We need to consider the case where MDEPKG_NDEBUG is defined. We're > > using ASSERT_EFI_ERROR to handle the error status from > > LocateDevicePath() in the following code block, which contains > > potential bug that we still use uninitialized Handle for the case > > where LocateDevicePath returns error status. > > > > Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > > &TempDevicePath, &Handle); ASSERT_EFI_ERROR (Status); > > I think that Ray made a sufficient argument in the commit message why the > assertion was correct -- it is indeed an assertion, and not error checking. > > Since the function is called when the Handle supports BlockIo or > SimpleFileSystem, when there is no SimpleFileSystem installed on > the Handle, BlockIo should be installed on the Handle. > > Perhaps the wording could be clarified, as in: > > ... BlockIo is *guaranteed* to be installed on the Handle. > > Just my two cents anyway. > Laszlo > > > > > -----Original Message----- > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > > Ruiyu Ni > > Sent: Tuesday, February 07, 2017 10:54 AM > > To: edk2-devel@lists.01.org > > Cc: Jeff Fan <jeff.fan@intel.com> > > Subject: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize > > Handle before using it > > > > BmExpandMediaDevicePath contains a bug that it uses the uninitialized > Handle. > > > > Since the function is called when the Handle supports BlockIo or > SimpleFileSystem, when there is no SimpleFileSystem installed on the Handle, > BlockIo should be installed on the Handle. > > The fix initializes the Handle by locating the BlockIo protocol from the > device path. > > > > Contributed-under: TianoCore Contribution Agreement 1.0 > > Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com> > > Cc: Jeff Fan <jeff.fan@intel.com> > > --- > > MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 8 +++++--- > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > > b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > > index 75bd5dc..9f99122 100644 > > --- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > > +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > > @@ -991,9 +991,13 @@ BmExpandMediaDevicePath ( > > return FileBuffer; > > } > > > > + Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > > + &TempDevicePath, &Handle); ASSERT_EFI_ERROR (Status); > > + > > // > > // For device boot option only pointing to the removable device > > handle, > > - // should make sure all its children handles (its child partion or media > handles) are created and connected. > > + // should make sure all its children handles (its child partion or > > + media handles) // are created and connected. > > // > > gBS->ConnectController (Handle, NULL, NULL, TRUE); > > > > @@ -1004,8 +1008,6 @@ BmExpandMediaDevicePath ( > > // returned. After the Block IO protocol is reinstalled, subsequent > > // Block IO read/write will success. > > // > > - Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > > &TempDevicePath, &Handle); > > - ASSERT_EFI_ERROR (Status); > > Status = gBS->HandleProtocol (Handle, &gEfiBlockIoProtocolGuid, (VOID > **) &BlockIo); > > ASSERT_EFI_ERROR (Status); > > Buffer = AllocatePool (BlockIo->Media->BlockSize); > > -- > > 2.9.0.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 > > > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it 2017-02-08 8:13 ` Wang, Sunny (HPS SW) 2017-02-08 9:17 ` Laszlo Ersek @ 2017-02-08 9:18 ` Ni, Ruiyu 1 sibling, 0 replies; 6+ messages in thread From: Ni, Ruiyu @ 2017-02-08 9:18 UTC (permalink / raw) To: Wang, Sunny (HPS SW), edk2-devel@lists.01.org; +Cc: Fan, Jeff Sunny, It's impossible for the Handle which doesn't have SimpleFileSystem or BlockIo installed. Thanks/Ray > -----Original Message----- > From: Wang, Sunny (HPS SW) [mailto:sunnywang@hpe.com] > Sent: Wednesday, February 8, 2017 4:13 PM > To: Ni, Ruiyu <ruiyu.ni@intel.com>; edk2-devel@lists.01.org > Cc: Fan, Jeff <jeff.fan@intel.com>; Wang, Sunny (HPS SW) > <sunnywang@hpe.com> > Subject: RE: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize > Handle before using it > > Looks good to me. > Reviewed-by: Sunny Wang <sunnywang@hpe.com> > > However, I saw the other potential issue below. If you also think the > potential issue is valid, you can fix this in the other patch. > We need to consider the case where MDEPKG_NDEBUG is defined. We're > using ASSERT_EFI_ERROR to handle the error status from LocateDevicePath() > in the following code block, which contains potential bug that we still use > uninitialized Handle for the case where LocateDevicePath returns error > status. > > Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > &TempDevicePath, &Handle); ASSERT_EFI_ERROR (Status); > > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Ruiyu Ni > Sent: Tuesday, February 07, 2017 10:54 AM > To: edk2-devel@lists.01.org > Cc: Jeff Fan <jeff.fan@intel.com> > Subject: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize > Handle before using it > > BmExpandMediaDevicePath contains a bug that it uses the uninitialized > Handle. > > Since the function is called when the Handle supports BlockIo or > SimpleFileSystem, when there is no SimpleFileSystem installed on the Handle, > BlockIo should be installed on the Handle. > The fix initializes the Handle by locating the BlockIo protocol from the device > path. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com> > Cc: Jeff Fan <jeff.fan@intel.com> > --- > MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > index 75bd5dc..9f99122 100644 > --- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > @@ -991,9 +991,13 @@ BmExpandMediaDevicePath ( > return FileBuffer; > } > > + Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > + &TempDevicePath, &Handle); ASSERT_EFI_ERROR (Status); > + > // > // For device boot option only pointing to the removable device handle, > - // should make sure all its children handles (its child partion or media > handles) are created and connected. > + // should make sure all its children handles (its child partion or > + media handles) // are created and connected. > // > gBS->ConnectController (Handle, NULL, NULL, TRUE); > > @@ -1004,8 +1008,6 @@ BmExpandMediaDevicePath ( > // returned. After the Block IO protocol is reinstalled, subsequent > // Block IO read/write will success. > // > - Status = gBS->LocateDevicePath (&gEfiBlockIoProtocolGuid, > &TempDevicePath, &Handle); > - ASSERT_EFI_ERROR (Status); > Status = gBS->HandleProtocol (Handle, &gEfiBlockIoProtocolGuid, (VOID **) > &BlockIo); > ASSERT_EFI_ERROR (Status); > Buffer = AllocatePool (BlockIo->Media->BlockSize); > -- > 2.9.0.windows.1 > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-02-08 9:41 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-02-07 2:53 [PATCH] MdeModulePkg/UefiBootManagerLib: Initialize Handle before using it Ruiyu Ni 2017-02-08 8:13 ` Wang, Sunny (HPS SW) 2017-02-08 9:17 ` Laszlo Ersek 2017-02-08 9:38 ` Wang, Sunny (HPS SW) 2017-02-08 9:41 ` Ni, Ruiyu 2017-02-08 9:18 ` Ni, Ruiyu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox