From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <supreeth.venkatesh@arm.com>
Received-SPF: Pass (sender SPF authorized) identity=mailfrom;
 client-ip=217.140.101.70; helo=foss.arm.com;
 envelope-from=supreeth.venkatesh@arm.com; receiver=edk2-devel@lists.01.org 
Received: from foss.arm.com (usa-sjc-mx-foss1.foss.arm.com [217.140.101.70])
 by ml01.01.org (Postfix) with ESMTP id A5A1D211799DC
 for <edk2-devel@lists.01.org>; Tue, 16 Oct 2018 02:48:38 -0700 (PDT)
Received: by usa-sjc-mx-foss1.foss.arm.com (Postfix, from userid 105)
 id 1D35932CF; Tue, 16 Oct 2018 02:58:34 -0700 (PDT)
Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249])
 by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id ED2D734E9;
 Sun, 14 Oct 2018 19:23:50 -0700 (PDT)
Received: from [10.6.43.238] (bc-c3-3-14.eu.iaas.arm.com [10.6.43.238])
 by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 8EB233F5B1;
 Sun, 14 Oct 2018 19:23:33 -0700 (PDT)
To: Eric Jin <eric.jin@intel.com>, edk2-devel@lists.01.org
Cc: Jiaxin Wu <jiaxin.wu@intel.com>
References: <20181014013127.16664-1-eric.jin@intel.com>
From: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
Message-ID: <0f380833-2719-3cd0-b4ee-9d2ab416d698@arm.com>
Date: Mon, 15 Oct 2018 03:23:31 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.9.1
MIME-Version: 1.0
In-Reply-To: <20181014013127.16664-1-eric.jin@intel.com>
Subject: Re: [PATCH] uefi-sct/SctPkg:Assign 0 to the tail of HwErrRecVariableName
X-BeenThere: edk2-devel@lists.01.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: EDK II Development  <edk2-devel.lists.01.org>
List-Unsubscribe: <https://lists.01.org/mailman/options/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=unsubscribe>
List-Archive: <http://lists.01.org/pipermail/edk2-devel/>
List-Post: <mailto:edk2-devel@lists.01.org>
List-Help: <mailto:edk2-devel-request@lists.01.org?subject=help>
List-Subscribe: <https://lists.01.org/mailman/listinfo/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=subscribe>
X-List-Received-Date: Tue, 16 Oct 2018 09:48:39 -0000
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Language: en-US



On 10/14/2018 02:31 AM, Eric Jin wrote:
> Ensure the HwErrRecVariable could be deleted before the test exit.
>
> Cc: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
> Cc: Jiaxin Wu <jiaxin.wu@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Eric Jin <eric.jin@intel.com>
> ---
>   .../BlackBoxTest/VariableServicesBBTestFunction.c              | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/uefi-sct/SctPkg/TestCase/UEFI/EFI/RuntimeServices/VariableServices/BlackBoxTest/VariableServicesBBTestFunction.c b/uefi-sct/SctPkg/TestCase/UEFI/EFI/RuntimeServices/VariableServices/BlackBoxTest/VariableServicesBBTestFunction.c
> index d1064cec..1be1720a 100644
> --- a/uefi-sct/SctPkg/TestCase/UEFI/EFI/RuntimeServices/VariableServices/BlackBoxTest/VariableServicesBBTestFunction.c
> +++ b/uefi-sct/SctPkg/TestCase/UEFI/EFI/RuntimeServices/VariableServices/BlackBoxTest/VariableServicesBBTestFunction.c
> @@ -1,7 +1,7 @@
>   /** @file
>   
>     Copyright 2006 - 2012 Unified EFI, Inc.<BR>
> -  Copyright (c) 2010 - 2012, Intel Corporation. All rights reserved.<BR>
> +  Copyright (c) 2010 - 2018, 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
> @@ -3052,6 +3052,7 @@ HardwareErrorRecordFuncTest (
>     //
>   step2:
>     DataSize = 255;
> +  HwErrRecVariableName[12] = L'\0';
Magic Number 12.
>     SctStrnCpy ( HwErrRecVariableName, (CHAR16*)(RecoveryData+2), 12 );
>     Status = RT->GetVariable (
>                           HwErrRecVariableName,