From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 84AAB21A143F5 for ; Mon, 26 Jun 2017 02:19:27 -0700 (PDT) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 26 Jun 2017 02:20:56 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.39,395,1493708400"; d="scan'208";a="871752092" Received: from shwdeopenpsi068.ccr.corp.intel.com ([10.239.9.2]) by FMSMGA003.fm.intel.com with ESMTP; 26 Jun 2017 02:20:55 -0700 From: Star Zeng To: edk2-devel@lists.01.org Cc: Star Zeng Date: Mon, 26 Jun 2017 17:20:50 +0800 Message-Id: <1498468853-175036-1-git-send-email-star.zeng@intel.com> X-Mailer: git-send-email 2.7.0.windows.1 Subject: [PATCH V3 0/3] Update comments and code for GetNextVariableName to follow UEFI 2.7 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: Mon, 26 Jun 2017 09:19:27 -0000 V3: Remove the ((VariableName[MaxLen - 1] != 0) check and enhance the code comments based on Ruiyu's feedback. V2: Add changes for EmuVariable and FsVariable. "The size must be large enough to fit input string supplied in VariableName buffer" is added in the description for VariableNameSize. And two cases of EFI_INVALID_PARAMETER are added. 1. The input values of VariableName and VendorGuid are not a name and GUID of an existing variable. 2. Null-terminator is not found in the first VariableNameSize bytes of the input VariableName buffer. Star Zeng (3): MdePkg: Update comments for GetNextVariableName to follow UEFI 2.7 MdeModulePkg Variable: Update GetNextVariableName to follow UEFI 2.7 DuetPkg FsVariable: Update GetNextVariableName to follow UEFI 2.7 DuetPkg/FSVariable/FSVariable.c | 31 ++++++++++++- .../Universal/Variable/EmuRuntimeDxe/EmuVariable.c | 41 ++++++++++++++--- .../Variable/EmuRuntimeDxe/InitVariable.c | 24 ++++++---- .../Universal/Variable/EmuRuntimeDxe/Variable.h | 15 ++++--- .../Universal/Variable/RuntimeDxe/Variable.c | 51 +++++++++++++++++++--- .../Universal/Variable/RuntimeDxe/Variable.h | 25 ++++++++--- MdePkg/Include/Uefi/UefiSpec.h | 8 +++- 7 files changed, 160 insertions(+), 35 deletions(-) -- 2.7.0.windows.1