diff options
author | lgao4 <lgao4@6f19259b-4bc3-4df7-8a09-765794883524> | 2009-07-15 06:57:39 +0000 |
---|---|---|
committer | lgao4 <lgao4@6f19259b-4bc3-4df7-8a09-765794883524> | 2009-07-15 06:57:39 +0000 |
commit | e0e51f622aba7751dd7a8d52bcb735a0f4f03240 (patch) | |
tree | e5dab7e04fb3c9f0703414ec2b228c2a9f6e5207 /EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk | |
parent | 6bfa178ccaa1ec0fa2bfddf0a5ebbb931744e7a4 (diff) | |
download | edk2-platforms-e0e51f622aba7751dd7a8d52bcb735a0f4f03240.tar.xz |
Remove the auto conversion for the first varstore name to "Setup".
And fix a bug that return status is not set.
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@8949 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk')
-rw-r--r-- | EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/ConfigAccess.c | 8 | ||||
-rw-r--r-- | EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/Utility.c | 14 |
2 files changed, 5 insertions, 17 deletions
diff --git a/EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/ConfigAccess.c b/EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/ConfigAccess.c index 3efda6703a..a96ca84272 100644 --- a/EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/ConfigAccess.c +++ b/EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/ConfigAccess.c @@ -343,11 +343,11 @@ CallFormCallBack ( }
*Data = AllocateZeroPool (*DataSize);
- if (Data == NULL) {
+ if (*Data == NULL) {
return EFI_OUT_OF_RESOURCES;
}
- FwFormCallBack->NvRead (
+ Status = FwFormCallBack->NvRead (
FwFormCallBack,
BufferStorage->Name,
&BufferStorage->Guid,
@@ -403,7 +403,7 @@ GetUefiVariable ( }
*Data = AllocateZeroPool (*DataSize);
- if (Data == NULL) {
+ if (*Data == NULL) {
return EFI_OUT_OF_RESOURCES;
}
@@ -611,7 +611,7 @@ ThunkRouteConfig ( if (EFI_ERROR (Status)) {
goto Done;
}
-
+
if (ConfigAccess->ThunkContext->NvMapOverride == NULL) {
if (ConfigAccess->FormCallbackProtocol == NULL ||
ConfigAccess->FormCallbackProtocol->NvWrite == NULL) {
diff --git a/EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/Utility.c b/EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/Utility.c index f05bf0c3fa..9e4679cb9e 100644 --- a/EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/Utility.c +++ b/EdkCompatibilityPkg/Compatibility/FrameworkHiiOnUefiHiiThunk/Utility.c @@ -665,19 +665,7 @@ GetFormsetDefaultVarstoreId ( if (FormSet->DefaultVarStoreId == 0) {
DEBUG ((EFI_D_INFO, "FormSet %g: No Varstore Found\n", &FormSet->Guid));
- } else {
- // The name of default VARSTORE with a Explicit declaration statement will be updated to L"Setup" to make sure
- // the Framework HII Setup module will run correctly. Framework HII Setup module always assumed that default
- // VARSTORE to have L"Setup" as name, Formset GUID as GUID.
-
- DEBUG ((EFI_D_INFO, "FormSet %g: Default Varstore ID (0x%x) N(%s) G(%g)\n", &FormSet->Guid, FormSet->DefaultVarStoreId, DefaultStorage->Name, &DefaultStorage->Guid));
-
- if (StrCmp (DefaultStorage->Name, FrameworkReservedVarstoreName) != 0) {
- DEBUG ((EFI_D_INFO, " : Name is updated from %s to %s.\n", DefaultStorage->Name, FrameworkReservedVarstoreName));
- FormSet->OriginalDefaultVarStoreName = DefaultStorage->Name;
- DefaultStorage->Name = AllocateCopyPool (StrSize (FrameworkReservedVarstoreName), FrameworkReservedVarstoreName);
- }
- }
+ }
return;
}
|