diff options
| author | Laszlo Ersek <lersek@redhat.com> | 2018-07-18 19:38:40 +0200 |
|---|---|---|
| committer | Laszlo Ersek <lersek@redhat.com> | 2018-08-16 20:02:50 +0200 |
| commit | 1bf5007428f741857eaefbacfb6762ce5ea8120d (patch) | |
| tree | 2cd7ce2b59a2787666851fde5dfa16d449940dc9 /IntelFspPkg/Tools/GenCfgOpt.py | |
| parent | 9f5d1f7c316065fcc7eee86d11f9015728b0ca85 (diff) | |
| download | edk2-1bf5007428f741857eaefbacfb6762ce5ea8120d.tar.gz edk2-1bf5007428f741857eaefbacfb6762ce5ea8120d.tar.bz2 edk2-1bf5007428f741857eaefbacfb6762ce5ea8120d.zip | |
SecurityPkg/SecureBootConfigDxe: replace OpenFileByDevicePath() with UefiLib API
Replace the OpenFileByDevicePath() function with EfiOpenFileByDevicePath()
from UefiLib, correcting the following issues:
- imprecise comments on OpenFileByDevicePath(),
- code duplication between this module and other modules,
- local variable name "EfiSimpleFileSystemProtocol" starting with "Efi"
prefix,
- bogus "FileHandle = NULL" assignments,
- leaking "Handle1" when the device path type/subtype check or the
realignment-motivated AllocateCopyPool() fails in the loop,
- stale SHELL_FILE_HANDLE reference in a comment.
Cc: Chao Zhang <chao.b.zhang@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Roman Bacik <roman.bacik@broadcom.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1008
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Chao Zhang <chao.b.zhang@intel.com>
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>
Diffstat (limited to 'IntelFspPkg/Tools/GenCfgOpt.py')
0 files changed, 0 insertions, 0 deletions