From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:400c:c09::233; helo=mail-wm0-x233.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wm0-x233.google.com (mail-wm0-x233.google.com [IPv6:2a00:1450:400c:c09::233]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id D547921E781F7 for ; Tue, 3 Oct 2017 10:14:17 -0700 (PDT) Received: by mail-wm0-x233.google.com with SMTP id u138so18392155wmu.4 for ; Tue, 03 Oct 2017 10:17:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=p/rUDtfzdW65N3N7UX+wrQsfUmL00nE5w93y1T0h4AY=; b=eZ5iabbol/iUVMNUlwuWnk303uKgFT9v+jx6YWYjHEqS3H+QZyvAMKxXq5fVJR0Ofe Fc6KR0jLQS5gdfoPUtzqBYGf5CXQI6oNj7X6N0aJcSa9xj6wc+lv0sbrvf1sAw+j9iuS gTORRygH6XclvD9zLJ7ap1Ucr6A//T2MrQbp4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=p/rUDtfzdW65N3N7UX+wrQsfUmL00nE5w93y1T0h4AY=; b=Zz351IQItmygInw8X/AZjWdylBL/mYuHckvzaHdP5abB/4YPKSp6iL3K3YKEyE0Z+K 0HJpAvw3B+v+SoInDpPir52Z4YUXnnQALqD0UziveIY7bYoY6EAiDNIuvBDw5O7gNZGm X1XGl9fM3rP/ao/qmv6gCjXPp6gwzUg1laSVk5YQ+l5cLYSGv46MAVqCRhusha6J9yBs orYtmnD1CVBrNRFdIw6G9R9mLiUebsgPaVIu0ne+PKx1/FrzoJsk1P8qkSZ2gPN66U3I 1GltJh2/8WOWu3arIuk1QXfHxMeI1pugQm/HUI0tJc8/sIt2eafsTA/dfyAWF2BZXlgq Z3Bw== X-Gm-Message-State: AMCzsaXULUAPPlBNFe2ab0fz1dt2R5sHsOz1y/ZSxdTwuO24dngt/ZDB QUXRWV3Oire2En2rshTLmWqQhjr3dkQ= X-Google-Smtp-Source: AOwi7QB7N+W6NuGmJgmBqD3fFn95ZbYaiTguEhm+dIM5KN7pL2U8W2Fz4S8fRXiY92DvuH2SWjxzoQ== X-Received: by 10.28.191.215 with SMTP id o84mr15341770wmi.153.1507051056483; Tue, 03 Oct 2017 10:17:36 -0700 (PDT) Received: from localhost.localdomain ([160.90.203.54]) by smtp.gmail.com with ESMTPSA id 31sm4748440wrm.0.2017.10.03.10.17.32 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 03 Oct 2017 10:17:33 -0700 (PDT) From: Ard Biesheuvel To: edk2-devel@lists.01.org, star.zeng@intel.com, eric.dong@intel.com Cc: leif.lindholm@linaro.org, Ard Biesheuvel Date: Tue, 3 Oct 2017 18:17:27 +0100 Message-Id: <20171003171727.5641-1-ard.biesheuvel@linaro.org> X-Mailer: git-send-email 2.11.0 Subject: [PATCH] MdeModulePkg/UefiBootManagerLib: don't ASSERT on 'BootNext' varname X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 03 Oct 2017 17:14:18 -0000 When invoking EfiBootManagerIsValidLoadOptionVariableName() with the variable name 'BootNext', we should simply return FALSE, given that it is not an indexed Boot#### load option. However, in DEBUG mode, we will hit an assert in BmCharToUint() and crash. So remove the assert. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel --- MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c b/MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c index 11ab86792a52..a3fa25424592 100644 --- a/MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c @@ -420,7 +420,6 @@ BmCharToUint ( return (Char - L'A' + 0xA); } - ASSERT (FALSE); return (UINTN) -1; } -- 2.11.0