diff options
| author | jcarsey <jcarsey@6f19259b-4bc3-4df7-8a09-765794883524> | 2012-03-05 17:51:52 +0000 |
|---|---|---|
| committer | jcarsey <jcarsey@6f19259b-4bc3-4df7-8a09-765794883524> | 2012-03-05 17:51:52 +0000 |
| commit | 50b43b1506f927f94f7e655e6f2c2a27413f8914 (patch) | |
| tree | e3194ae1290d1f17be86a360c5a4cf65ef9ab37c /ShellPkg | |
| parent | 81c0d6e9a7f78bb43e9a1045ea084e728fa76983 (diff) | |
| download | edk2-platforms-50b43b1506f927f94f7e655e6f2c2a27413f8914.tar.xz | |
ShellPkg: Fix ICC11 build failure.
Eliminate SHELL_STATUS and EFI_STATUS mismatch.
Signed-off-by: jcarsey
Reviewed-by: lgao4
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@13075 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'ShellPkg')
| -rw-r--r-- | ShellPkg/Library/UefiShellDebug1CommandsLib/Bcfg.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/Bcfg.c b/ShellPkg/Library/UefiShellDebug1CommandsLib/Bcfg.c index d6c3ef4e15..4d2fbfa843 100644 --- a/ShellPkg/Library/UefiShellDebug1CommandsLib/Bcfg.c +++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/Bcfg.c @@ -934,33 +934,37 @@ BcfgAddOptDebug1( //
// Open the file and populate the data buffer.
//
- ShellStatus = ShellOpenFileByName(
+ Status = ShellOpenFileByName(
FileName,
&FileHandle,
EFI_FILE_MODE_READ,
0);
- if (ShellStatus == SHELL_SUCCESS) {
- ShellStatus = ShellGetFileSize(FileHandle, &Intermediate);
+ if (!EFI_ERROR(Status)) {
+ Status = ShellGetFileSize(FileHandle, &Intermediate);
}
Data = AllocateZeroPool((UINTN)Intermediate);
if (Data == NULL) {
ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_NO_MEM), gShellDebug1HiiHandle);
ShellStatus = SHELL_OUT_OF_RESOURCES;
}
- if (ShellStatus == SHELL_SUCCESS) {
- ShellStatus = ShellReadFile(FileHandle, (UINTN *)&Intermediate, Data);
+ if (!EFI_ERROR(Status)) {
+ Status = ShellReadFile(FileHandle, (UINTN *)&Intermediate, Data);
}
} else {
Intermediate = StrSize(Data);
}
- if (ShellStatus == SHELL_SUCCESS && Data != NULL) {
+ if (!EFI_ERROR(Status) && ShellStatus == SHELL_SUCCESS && Data != NULL) {
Status = UpdateOptionalDataDebug1(CurrentOrder[OptionIndex], (UINTN)Intermediate, (UINT8*)Data, Target);
if (EFI_ERROR(Status)) {
ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_SET_VAR_FAIL), gShellDebug1HiiHandle, VariableName, Status);
ShellStatus = SHELL_INVALID_PARAMETER;
}
}
+ if (EFI_ERROR(Status) && ShellStatus == SHELL_SUCCESS) {
+ ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_SET_VAR_FAIL), gShellDebug1HiiHandle, VariableName, Status);
+ ShellStatus = SHELL_INVALID_PARAMETER;
+ }
}
SHELL_FREE_NON_NULL(Data);
|