diff options
author | Laszlo Ersek <lersek@redhat.com> | 2019-09-07 00:58:19 +0200 |
---|---|---|
committer | Laszlo Ersek <lersek@redhat.com> | 2019-10-09 09:40:10 +0200 |
commit | fcf8bdcd5313ae076506cbbf8474bb660fa47a93 (patch) | |
tree | 2a30ff6c7405feac3e647c48dd843d5a6ad8d2f4 | |
parent | a629d5b9ef039e8ca3541602b3129c5cb9ab8ee8 (diff) | |
download | edk2-fcf8bdcd5313ae076506cbbf8474bb660fa47a93.tar.gz edk2-fcf8bdcd5313ae076506cbbf8474bb660fa47a93.tar.bz2 edk2-fcf8bdcd5313ae076506cbbf8474bb660fa47a93.zip |
SecurityPkg: stop abusing EFI_EVENT for protocol notify registration
EfiCreateProtocolNotifyEvent() takes a (VOID**) for "Registration",
similarly to gBS->RegisterProtocolNotify(). We should pass the address of
an actual pointer-to-VOID, and not the address of an EFI_EVENT. EFI_EVENT
just happens to be specified as (VOID*), and has nothing to do with the
registration.
This change is a no-op in practice; it's a semantic improvement.
Cc: Chao Zhang <chao.b.zhang@intel.com>
Cc: Jian Wang <jian.j.wang@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
Reviewed-by: Chao Zhang <chao.b.zhang@intel.com>
-rw-r--r-- | SecurityPkg/HddPassword/HddPasswordDxe.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/SecurityPkg/HddPassword/HddPasswordDxe.c b/SecurityPkg/HddPassword/HddPasswordDxe.c index b0d795b659..051e64091d 100644 --- a/SecurityPkg/HddPassword/HddPasswordDxe.c +++ b/SecurityPkg/HddPassword/HddPasswordDxe.c @@ -2770,7 +2770,7 @@ HddPasswordDxeInit ( {
EFI_STATUS Status;
HDD_PASSWORD_DXE_PRIVATE_DATA *Private;
- EFI_EVENT Registration;
+ VOID *Registration;
EFI_EVENT EndOfDxeEvent;
EDKII_VARIABLE_LOCK_PROTOCOL *VariableLock;
|