]> git.proxmox.com Git - mirror_edk2.git/commit
MdeModulePkg/Core/Dxe: Add lock protection in CoreLocateHandleBuffer()
authorHua Ma <hua.ma@intel.com>
Wed, 29 Sep 2021 05:08:14 +0000 (13:08 +0800)
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Fri, 8 Oct 2021 06:57:55 +0000 (06:57 +0000)
commit5ece2ad36caa7ddc62d4954559b2cdd0d8a40a14
tree94f08d46876e700f423403ffe08c3b5963c17c36
parentb4da6c29f1d36031e04212f53277ce0dcba309f1
MdeModulePkg/Core/Dxe: Add lock protection in CoreLocateHandleBuffer()

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3666

Currently, CoreLocateHandleBuffer() follows three steps:
1) get the size of protocol database firstly
2) allocate the buffer based on the size
3) get the protocol database into the buffer
There is no lock protection for the whole three steps. If a new protocol
added in step 2) by other task, e.g. (event timer handle USB device
hotplug). The size of protocol database may be increased and cannot fit
into the previous buffer in step 3). The protocol database cannot be
returned successfully, EFI_BUFFER_TOO_SMALL error will be returned.

This patch adds the lock to protect the whole three steps.
It can make sure the correct protocol database be returned.

Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Dandan Bi <dandan.bi@intel.com>
Signed-off-by: Hua Ma <hua.ma@intel.com>
Reviewed-by: Dandan Bi dandan.bi@intel.com
Reviewed-by: Liming Gao gaoliming@byosoft.com.cn
MdeModulePkg/Core/Dxe/Hand/Locate.c