Including Jaben for review. Send from mobile phone
> 在 2017年6月6日，下午4:43，Ni, Ruiyu <[email protected]> 写道： > > Including Tapan. > > Thanks/Ray > >> -----Original Message----- >> From: edk2-devel [mailto:[email protected]] On Behalf Of Ruiyu >> Ni >> Sent: Tuesday, June 6, 2017 4:29 PM >> To: [email protected] >> Subject: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: Ruiyu Ni <[email protected]> >> Cc: Tapan Shah <[email protected]> >> --- >> ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c | 11 ++++++++++- >> 1 file changed, 10 insertions(+), 1 deletion(-) >> >> diff --git a/ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c >> b/ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c >> index 5a3b7e5913..7277bd4b90 100644 >> --- a/ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c >> +++ b/ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c >> @@ -2,7 +2,7 @@ >> Main file for Alias shell level 3 function. >> >> (C) Copyright 2015 Hewlett-Packard Development Company, L.P.<BR> >> - Copyright (c) 2009 - 2016, Intel Corporation. All rights reserved. <BR> >> + Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. >> + <BR> >> This program and the accompanying materials >> are licensed and made available under the terms and conditions of the BSD >> License >> which accompanies this distribution. The full text of the license may be >> found >> at @@ -151,6 +151,15 @@ ShellCommandRunAlias ( >> // delete an alias >> // >> Status = gEfiShellProtocol->SetAlias(Param1, NULL, TRUE, FALSE); >> + if (EFI_ERROR(Status)) { >> + if (Status == EFI_ACCESS_DENIED) { >> + ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_ERR_AD), >> gShellLevel3HiiHandle, L"alias"); >> + ShellStatus = SHELL_ACCESS_DENIED; >> + } else { >> + ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_ERR_UK), >> gShellLevel3HiiHandle, L"alias", Status); >> + ShellStatus = SHELL_DEVICE_ERROR; >> + } >> + } >> } else if (ShellCommandLineGetCount(Package) == 3) { >> // >> // must be adding an alias >> -- >> 2.12.2.windows.2 >> >> _______________________________________________ >> edk2-devel mailing list >> [email protected] >> https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
