diff options
| author | Michael Kubacki <michael.kubacki@microsoft.com> | 2022-11-08 15:15:31 -0500 |
|---|---|---|
| committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2023-04-03 15:29:08 +0000 |
| commit | 84d77d9bf5dfc99159b2736d9f16661141ee5cb9 (patch) | |
| tree | 2930b01364f4c0de7846033b1a934f2024dc74ca /MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootOption.c | |
| parent | dbe820d5fa158f3bb04d2a9b335bf6e0ca0e0bb0 (diff) | |
| download | edk2-84d77d9bf5dfc99159b2736d9f16661141ee5cb9.tar.gz edk2-84d77d9bf5dfc99159b2736d9f16661141ee5cb9.tar.bz2 edk2-84d77d9bf5dfc99159b2736d9f16661141ee5cb9.zip | |
CryptoPkg: Fix conditionally uninitialized variable
Fixes CodeQL alerts for CWE-457:
https://cwe.mitre.org/data/definitions/457.html
Checks the return value from `ASN1_get_object()` to verify values
set by the function are valid.
Note that the function returns literal `0x80`:
`return (0x80);`
That is used to check the return value is as the case in other areas
of the code.
Cc: Erich McMillan <emcmillan@microsoft.com>
Cc: Guomin Jiang <guomin.jiang@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Michael Kubacki <mikuback@linux.microsoft.com>
Cc: Xiaoyu Lu <xiaoyu1.lu@intel.com>
Co-authored-by: Erich McMillan <emcmillan@microsoft.com>
Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
Reviewed-by: Oliver Smith-Denny <osd@smith-denny.com>
Diffstat (limited to 'MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootOption.c')
0 files changed, 0 insertions, 0 deletions