diff options
| author | Laszlo Ersek <lersek@redhat.com> | 2019-04-30 23:33:39 +0200 |
|---|---|---|
| committer | Ard Biesheuvel <ard.biesheuvel@linaro.org> | 2019-05-01 08:30:35 +0200 |
| commit | 727d7ebaa9f3dab8822d264fbc8104aee8f08867 (patch) | |
| tree | 7dd999cd9f3433166cde5789908ad2c20bc7e197 /OvmfPkg/EnrollDefaultKeys | |
| parent | 273190e189755d652de886b653d2c14bfceeb3ed (diff) | |
| download | edk2-727d7ebaa9f3dab8822d264fbc8104aee8f08867.tar.gz edk2-727d7ebaa9f3dab8822d264fbc8104aee8f08867.tar.bz2 edk2-727d7ebaa9f3dab8822d264fbc8104aee8f08867.zip | |
OvmfPkg/EnrollDefaultKeys: suppress incorrect compiler warning
When building OvmfPkg/EnrollDefaultKeys for IA32 with gcc-4.8 for the
DEBUG target (and possibly under other build configurations too), the
compiler incorrectly reports,
> OvmfPkg/EnrollDefaultKeys/EnrollDefaultKeys.c: In function
> 'ShellAppMain':
> OvmfPkg/EnrollDefaultKeys/EnrollDefaultKeys.c:631:10: error:
> 'SizeOfPkKek1' may be used uninitialized in this function
> [-Werror=maybe-uninitialized]
> Status = EnrollListOfCerts (
> ^
> OvmfPkg/EnrollDefaultKeys/EnrollDefaultKeys.c:703:12: error: 'PkKek1'
> may be used uninitialized in this function [-Werror=maybe-uninitialized]
> FreePool (PkKek1);
> ^
> cc1: all warnings being treated as errors
Suppress this warning, in the style suggested under
<https://bugzilla.tianocore.org/show_bug.cgi?id=607>.
Reported-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Fixes: be9470b3c91fc50436c15a76c85cdde940355b9f
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Diffstat (limited to 'OvmfPkg/EnrollDefaultKeys')
| -rw-r--r-- | OvmfPkg/EnrollDefaultKeys/EnrollDefaultKeys.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/OvmfPkg/EnrollDefaultKeys/EnrollDefaultKeys.c b/OvmfPkg/EnrollDefaultKeys/EnrollDefaultKeys.c index b7b2e424c5..75f2749dc8 100644 --- a/OvmfPkg/EnrollDefaultKeys/EnrollDefaultKeys.c +++ b/OvmfPkg/EnrollDefaultKeys/EnrollDefaultKeys.c @@ -559,6 +559,13 @@ ShellAppMain ( }
//
+ // Set PkKek1 and SizeOfPkKek1 to suppress incorrect compiler/analyzer
+ // warnings.
+ //
+ PkKek1 = NULL;
+ SizeOfPkKek1 = 0;
+
+ //
// Fetch the X509 certificate (to be used as Platform Key and first Key
// Exchange Key) from SMBIOS.
//
|