From cfffccd9df1ed205721dd21038880a4fa72a5102 Mon Sep 17 00:00:00 2001 From: Tapan Shah Date: Fri, 12 Sep 2014 16:46:23 +0000 Subject: [PATCH] ShellPkg: Fix ASSERT when bcfg moving boot option with source location = total number of boot options Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Tapan Shah Reviewed-by: Jaben Carsey git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16102 6f19259b-4bc3-4df7-8a09-765794883524 --- .../UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c b/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c index d62b569ab3..a0cb2da0fe 100644 --- a/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c +++ b/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c @@ -1,6 +1,7 @@ /** @file Main file for BCFG command. + Copyright (c) 2014, Hewlett-Packard Development Company, L.P.
Copyright (c) 2010 - 2014, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License @@ -1353,7 +1354,7 @@ ShellCommandRunBcfg ( } else { Status = ShellConvertStringToUint64(CurrentParam, &Intermediate, TRUE, FALSE); CurrentOperation.Number1 = (UINT16)Intermediate; - if (CurrentOperation.Number1 > Count){ + if (CurrentOperation.Number1 >= Count){ ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_NUMB_RANGE), gShellBcfgHiiHandle, Count); ShellStatus = SHELL_INVALID_PARAMETER; } @@ -1371,7 +1372,7 @@ ShellCommandRunBcfg ( } else { Status = ShellConvertStringToUint64(CurrentParam, &Intermediate, TRUE, FALSE); CurrentOperation.Number1 = (UINT16)Intermediate; - if (CurrentOperation.Number1 > Count){ + if (CurrentOperation.Number1 >= Count){ ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_BCFG_NUMB_RANGE), gShellBcfgHiiHandle, Count); ShellStatus = SHELL_INVALID_PARAMETER; } else { -- 2.39.2