diff options
| author | Mike Maslenkin <mike.maslenkin@gmail.com> | 2025-01-09 21:04:53 +0300 |
|---|---|---|
| committer | Ard Biesheuvel <workofard@gmail.com> | 2025-01-29 17:20:57 +0100 |
| commit | a5cb67fb8d443c5a0ee25440c0c686a198543bc3 (patch) | |
| tree | 1e1f3589bac59e630612f6b4f6da67eebd68c924 | |
| parent | bba72ffbe1d1a8f6f94b0c14489bfcf08617fe1f (diff) | |
| download | edk2-a5cb67fb8d443c5a0ee25440c0c686a198543bc3.tar.gz | |
MdeModulePkg: PeiMain: fix warning about uninitialized variable
Dispatcher.c:1639:34: error: 'FvMigrationFlags' may be used uninitialized
in this function [-Werror=maybe-uninitialized]
(((FvMigrationFlags & FLAGS_FV_MIGRATE_BEFORE_PEI_CORE_REENTRY) == 0) ||
~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
Signed-off-by: Mike Maslenkin <mike.maslenkin@gmail.com>
| -rw-r--r-- | MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c b/MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c index 5e8d5b8e77..4073bad82c 100644 --- a/MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c +++ b/MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c @@ -1564,6 +1564,11 @@ EvacuateTempRam ( DEBUG ((DEBUG_VERBOSE, "Beginning evacuation of content in temporary RAM.\n"));
//
+ // By default migrate all FVs and copy raw data
+ //
+ FvMigrationFlags = FLAGS_FV_RAW_DATA_COPY;
+
+ //
// Migrate PPI Pointers of PEI_CORE from temporary memory to newly loaded PEI_CORE in permanent memory.
//
Status = PeiLocatePpi ((CONST EFI_PEI_SERVICES **)&Private->Ps, &gEfiPeiCoreFvLocationPpiGuid, 0, NULL, (VOID **)&PeiCoreFvLocationPpi);
@@ -1617,11 +1622,6 @@ EvacuateTempRam ( //
return EFI_SUCCESS;
}
-
- //
- // Migrate all FVs and copy raw data
- //
- FvMigrationFlags = FLAGS_FV_RAW_DATA_COPY;
} else {
for (Index = 0; Index < MigrationInfo->ToMigrateFvCount; Index++) {
ToMigrateFvInfo = ((TO_MIGRATE_FV_INFO *)(MigrationInfo + 1)) + Index;
|