[edk2-devel] [PATCH 26/35] SecurityPkg: fix UninstallMultipleProtocolInterfaces() calls
Laszlo Ersek
lersek at redhat.com
Mon Oct 7 18:14:25 UTC 2019
On 10/05/19 16:28, Zhang, Chao B wrote:
> Reviewed-by : Chao Zhang <chao.b.zhang at intel.com>
Thanks!
Laszlo
>
> -----Original Message-----
> From: Zhang, Chao B
> Sent: 2019年10月4日 21:14
> To: edk2-devel-groups-io <devel at edk2.groups.io>; 'lersek at redhat.com' <lersek at redhat.com>; Wang, Jian J <jian.j.wang at intel.com>; Yao, Jiewen <jiewen.yao at intel.com>
> Subject: RE: [edk2-devel] [PATCH 26/35] SecurityPkg: fix UninstallMultipleProtocolInterfaces() calls
>
> Hi Laszlo:
> Sorry for late response. The fix is good to me. I am also interested in how you find this issue, can you share it?
>
> -----Original Message-----
> From: Laszlo Ersek [mailto:lersek at redhat.com]
> Sent: 2019年10月3日 19:07
> To: Zhang, Chao B <chao.b.zhang at intel.com>; Wang, Jian J <jian.j.wang at intel.com>; Yao, Jiewen <jiewen.yao at intel.com>
> Cc: edk2-devel-groups-io <devel at edk2.groups.io>
> Subject: Re: [edk2-devel] [PATCH 26/35] SecurityPkg: fix UninstallMultipleProtocolInterfaces() calls
>
> Pinging SecurityPkg maintainers again, for reviewing this patch.
>
> Thanks
> Laszlo
>
> On 09/26/19 14:45, Laszlo Ersek wrote:
>> Chao, Jian, Jiewen,
>>
>> can you please review this patch?
>>
>> Thanks,
>> Laszlo
>>
>> On 09/17/19 21:49, Laszlo Ersek wrote:
>>> Unlike the InstallMultipleProtocolInterfaces() boot service, which
>>> takes an (EFI_HANDLE*) as first parameter, the
>>> UninstallMultipleProtocolInterfaces() boot service takes an
>>> EFI_HANDLE as first parameter.
>>>
>>> These are actual bugs. They must have remained hidden until now
>>> because they are all in Unload() functions, which are probably
>>> exercised infrequently. Fix the UninstallMultipleProtocolInterfaces() calls.
>>>
>>> Cc: Chao Zhang <chao.b.zhang at intel.com>
>>> Cc: Jian Wang <jian.j.wang at intel.com>
>>> Cc: Jiewen Yao <jiewen.yao at intel.com>
>>> Signed-off-by: Laszlo Ersek <lersek at redhat.com>
>>> ---
>>>
>>> Notes:
>>> build-tested only
>>>
>>> SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c | 2 +-
>>> SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c | 2 +-
>>>
>>> SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfi
>>> gDriver.c | 2 +-
>>> 3 files changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c
>>> b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c
>>> index 54155a338100..9052eced757d 100644
>>> --- a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c
>>> +++ b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c
>>> @@ -443,7 +443,7 @@ Tcg2ConfigDriverUnload (
>>> ASSERT (PrivateData->Signature ==
>>> TCG2_CONFIG_PRIVATE_DATA_SIGNATURE);
>>>
>>> gBS->UninstallMultipleProtocolInterfaces (
>>> - &ImageHandle,
>>> + ImageHandle,
>>> &gEfiCallerIdGuid,
>>> PrivateData,
>>> NULL
>>> diff --git a/SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c
>>> b/SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c
>>> index 341879e4c4ba..fb06624fdb8f 100644
>>> --- a/SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c
>>> +++ b/SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c
>>> @@ -138,7 +138,7 @@ TcgConfigDriverUnload (
>>> ASSERT (PrivateData->Signature ==
>>> TCG_CONFIG_PRIVATE_DATA_SIGNATURE);
>>>
>>> gBS->UninstallMultipleProtocolInterfaces (
>>> - &ImageHandle,
>>> + ImageHandle,
>>> &gEfiCallerIdGuid,
>>> PrivateData,
>>> NULL
>>> diff --git
>>> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootCon
>>> figDriver.c
>>> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootCon
>>> figDriver.c index 798ef9cfbc01..6c0294151e6c 100644
>>> ---
>>> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootCon
>>> figDriver.c
>>> +++ b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBoo
>>> +++ tConfigDriver.c
>>> @@ -115,7 +115,7 @@ SecureBootConfigDriverUnload (
>>> ASSERT (PrivateData->Signature ==
>>> SECUREBOOT_CONFIG_PRIVATE_DATA_SIGNATURE);
>>>
>>> gBS->UninstallMultipleProtocolInterfaces (
>>> - &ImageHandle,
>>> + ImageHandle,
>>> &gEfiCallerIdGuid,
>>> PrivateData,
>>> NULL
>>>
>>
>>
>>
>>
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#48504): https://edk2.groups.io/g/devel/message/48504
Mute This Topic: https://groups.io/mt/34180228/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list