mirror of https://github.com/acidanthera/audk.git
Before the "Done" label at the end of DxeImageVerificationHandler(), we now have a single access to "Status": we set "Status" to EFI_ACCESS_DENIED at the top of the function. Therefore, the (Status != EFI_SUCCESS) condition is always true under the "Done" label. Accordingly, unnest the AddImageExeInfo() call dependent on that condition, remove the condition, and also rename the "Done" label to "Failed". Functionally, this patch is a no-op. It's easier to review with: git show -b -W Cc: Chao Zhang <chao.b.zhang@intel.com> Cc: Jian J Wang <jian.j.wang@intel.com> Cc: Jiewen Yao <jiewen.yao@intel.com> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2129 Signed-off-by: Laszlo Ersek <lersek@redhat.com> Message-Id: <20200116190705.18816-8-lersek@redhat.com> Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com> [lersek@redhat.com: replace EFI_D_INFO w/ DEBUG_INFO for PatchCheck.py] [lersek@redhat.com: push with Mike's R-b due to Chinese New Year Holiday: <https://edk2.groups.io/g/devel/message/53429>; msgid <d3fbb76dabed4e1987c512c328c82810@intel.com>] |
||
---|---|---|
.. | ||
FvReportPei | ||
Hash2DxeCrypto | ||
HddPassword | ||
Include | ||
Library | ||
Pkcs7Verify/Pkcs7VerifyDxe | ||
RandomNumberGenerator/RngDxe | ||
Tcg | ||
VariableAuthenticated/SecureBootConfigDxe | ||
SecurityPkg.ci.yaml | ||
SecurityPkg.dec | ||
SecurityPkg.dsc | ||
SecurityPkg.uni | ||
SecurityPkgExtra.uni |