On 9/17/19 9:49 PM, Laszlo Ersek wrote: > The HandleProtocol() boot service takes an EFI_HANDLE, not an > (EFI_HANDLE*). Remove the bogus cast in the > InternalImageHandleToFvHandle() function. > > This is a semantic cleanup; there is no change in behavior. > > Cc: Liming Gao <liming....@intel.com> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > Signed-off-by: Laszlo Ersek <ler...@redhat.com> > --- > > Notes: > lightly tested, most probably: it's practically impossible to build a > platform without consuming DxeServicesLib > > MdePkg/Library/DxeServicesLib/DxeServicesLib.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c > b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c > index c416b2dd8c65..0735b2f80400 100644 > --- a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c > +++ b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c > @@ -49,7 +49,7 @@ InternalImageHandleToFvHandle ( > ASSERT (ImageHandle != NULL); > > Status = gBS->HandleProtocol ( > - (EFI_HANDLE *) ImageHandle, > + ImageHandle, > &gEfiLoadedImageProtocolGuid, > (VOID **) &LoadedImage > ); >
Reviewed-by: Philippe Mathieu-Daude <phi...@redhat.com> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#47963): https://edk2.groups.io/g/devel/message/47963 Mute This Topic: https://groups.io/mt/34180218/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-