From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) (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 5BFB081DC7 for ; Sun, 13 Nov 2016 22:24:50 -0800 (PST) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga103.jf.intel.com with ESMTP; 13 Nov 2016 22:24:54 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,488,1473145200"; d="scan'208";a="1059127545" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga001.jf.intel.com with ESMTP; 13 Nov 2016 22:24:54 -0800 Received: from fmsmsx153.amr.corp.intel.com (10.18.125.6) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.248.2; Sun, 13 Nov 2016 22:24:53 -0800 Received: from shsmsx104.ccr.corp.intel.com (10.239.4.70) by FMSMSX153.amr.corp.intel.com (10.18.125.6) with Microsoft SMTP Server (TLS) id 14.3.248.2; Sun, 13 Nov 2016 22:24:53 -0800 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.239]) by SHSMSX104.ccr.corp.intel.com ([169.254.5.142]) with mapi id 14.03.0248.002; Mon, 14 Nov 2016 14:24:51 +0800 From: "Gao, Liming" To: Gary Lin , "Zhang, Chao B" CC: "edk2-devel@lists.01.org" Thread-Topic: [edk2] [PATCH 29/33] SecurityPkg: Fix typos in comments Thread-Index: AQHSKdb73TJH2gj810yy2mZQZecPyqDX9WbA//+I/YCAAKqIIA== Date: Mon, 14 Nov 2016 06:24:50 +0000 Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14B4B1691@shsmsx102.ccr.corp.intel.com> References: <20161019070138.16424-1-glin@suse.com> <20161019070138.16424-30-glin@suse.com> <20161114040343.dtiw7wzqybrvgt4s@GaryWorkstation> In-Reply-To: <20161114040343.dtiw7wzqybrvgt4s@GaryWorkstation> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH 29/33] SecurityPkg: Fix typos in comments X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 14 Nov 2016 06:24:50 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Lin: We don't plan to clean up all existing codes with EDKII DEBUG_ macros. We= expect DEBUG_ macro will be applied with other change together. So, I prop= ose to add this check in PatchCheck.py script. You only need to update your= changed lines. =20 Thanks Liming > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Gary Lin > Sent: Monday, November 14, 2016 12:04 PM > To: Zhang, Chao B > Cc: edk2-devel@lists.01.org > Subject: Re: [edk2] [PATCH 29/33] SecurityPkg: Fix typos in comments >=20 > On Mon, Nov 14, 2016 at 03:12:16AM +0000, Zhang, Chao B wrote: > > Hi Gary: > > I see some warnings while running PatchChecy.py with this patch. > > Please help to update the patch, run PatchCheck.py(BaseTools/Scripts) & > re-send the patch. > > >=20 > Hi Chao, >=20 > PatchCheck.py showed this: >=20 > The commit message format passed all checks. > Code format is not valid: > * EFI_D_ERROR was used, but DEBUG_ERROR is now recommended > File: SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c > Line: DEBUG ((EFI_D_ERROR, "Separator Event not Measured. > Error!\n")); > * EFI_D_ERROR was used, but DEBUG_ERROR is now recommended > File: SecurityPkg/Tcg/TcgDxe/TcgDxe.c > Line: DEBUG ((EFI_D_ERROR, "Separator Event not Measured. > Error!\n")); > * EFI_D_ERROR was used, but DEBUG_ERROR is now recommended > File: SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c > Line: DEBUG ((EFI_D_ERROR, "Separator Event not Measured. > Error!\n")); >=20 > My patch only fixes typos (Seperator -> Separator) and I tried to avoid a= ny > functional change. I found that EFI_D_ERROR is widely used in SecurityPkg > and > it'd be better to convert all EFI_D_ERROR into DEBUG_ERROR in another > patch. > What do you think? >=20 > Cheers, >=20 > Gary Lin >=20 > > > > > > > > > > Thanks & Best regards > > Chao Zhang > > > > > > -----Original Message----- > > From: Gary Lin [mailto:glin@suse.com] > > Sent: Wednesday, October 19, 2016 3:02 PM > > To: edk2-devel@lists.01.org > > Cc: Zhang, Chao B > > Subject: [PATCH 29/33] SecurityPkg: Fix typos in comments > > > > - intialized -> initialized > > - TURE -> TRUE > > - successull -> successfully > > - hanlder -> handler > > - funciton -> function > > - Seperator -> Separator > > - Sumbit -> Submit > > > > Cc: Chao Zhang > > Contributed-under: TianoCore Contribution Agreement 1.0 > > Signed-off-by: Gary Lin > > --- > > SecurityPkg/Hash2DxeCrypto/Hash2DxeCrypto.c > | 2 +- > > SecurityPkg/Include/Library/TcgPpVendorLib.h > | 4 ++-- > > SecurityPkg/Include/Library/TcgStorageOpalLib.h > | 2 +- > > SecurityPkg/Include/Library/TrEEPpVendorLib.h > | 4 ++-- > > SecurityPkg/Library/DxeDeferImageLoadLib/DxeDeferImageLoadLib.c > | 6 +++--- > > > SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048 > Sha256GuidedSectionExtractLib.c | 2 +- > > > SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048S > ha256GuidedSectionExtractLib.c | 2 +- > > SecurityPkg/Library/TcgStorageCoreLib/TcgStorageUtil.c > | 2 +- > > SecurityPkg/Library/TcgStorageOpalLib/TcgStorageOpalCore.c > | 2 +- > > SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalDriver.c > | 2 +- > > SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c = | 2 > +- > > SecurityPkg/Tcg/TcgDxe/TcgDxe.c = | 2 +- > > SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c = | 2 > +- > > > SecurityPkg/UserIdentification/UserProfileManagerDxe/ModifyIdentityPolic > y.c | 2 +- > > > SecurityPkg/UserIdentification/UserProfileManagerDxe/UserProfileManage > r.h | 2 +- > > 15 files changed, 19 insertions(+), 19 deletions(-) > > > > diff --git a/SecurityPkg/Hash2DxeCrypto/Hash2DxeCrypto.c > b/SecurityPkg/Hash2DxeCrypto/Hash2DxeCrypto.c > > index ab34de7..93e3273 100644 > > --- a/SecurityPkg/Hash2DxeCrypto/Hash2DxeCrypto.c > > +++ b/SecurityPkg/Hash2DxeCrypto/Hash2DxeCrypto.c > > @@ -64,7 +64,7 @@ BOOLEAN > > > > This function performs Hash digest on a data buffer of the specified= size. > > It can be called multiple times to compute the digest of long or > discontinuous data streams. > > - Hash context should be already correctly intialized by HashInit(), a= nd > should not be finalized > > + Hash context should be already correctly initialized by HashInit(), = and > should not be finalized > > by HashFinal(). Behavior with invalid context is undefined. > > > > If HashContext is NULL, then return FALSE. > > diff --git a/SecurityPkg/Include/Library/TcgPpVendorLib.h > b/SecurityPkg/Include/Library/TcgPpVendorLib.h > > index 08b3690..284aa9e 100644 > > --- a/SecurityPkg/Include/Library/TcgPpVendorLib.h > > +++ b/SecurityPkg/Include/Library/TcgPpVendorLib.h > > @@ -42,8 +42,8 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF > ANY KIND, EITHER EXPRESS OR IMPLIED. > > #define TCG_PP_OPERATION_RESPONSE_BIOS_FAILURE 0xFFFFFFF1 > > > > // > > -// The return code for Sumbit TPM Request to Pre-OS Environment > > -// and Sumbit TPM Request to Pre-OS Environment 2 > > +// The return code for Submit TPM Request to Pre-OS Environment > > +// and Submit TPM Request to Pre-OS Environment 2 > > // > > #define TCG_PP_SUBMIT_REQUEST_TO_PREOS_SUCCESS > 0 > > #define TCG_PP_SUBMIT_REQUEST_TO_PREOS_NOT_IMPLEMENTED > 1 > > diff --git a/SecurityPkg/Include/Library/TcgStorageOpalLib.h > b/SecurityPkg/Include/Library/TcgStorageOpalLib.h > > index 108affc..9b64a8e 100644 > > --- a/SecurityPkg/Include/Library/TcgStorageOpalLib.h > > +++ b/SecurityPkg/Include/Library/TcgStorageOpalLib.h > > @@ -290,7 +290,7 @@ OpalGlobalLockingRangeGenKey( > > /** > > > > The function updates the ReadLocked and WriteLocked columns of the > Global Locking Range. > > - This funciton is required for a user1 authority, since a user1 autho= rity shall > only have access to ReadLocked and WriteLocked columns > > + This function is required for a user1 authority, since a user1 autho= rity > shall only have access to ReadLocked and WriteLocked columns > > (not ReadLockEnabled and WriteLockEnabled columns). > > > > @param[in] LockingSpSession OPAL_SESSION with > OPAL_UID_LOCKING_SP to generate key > > diff --git a/SecurityPkg/Include/Library/TrEEPpVendorLib.h > b/SecurityPkg/Include/Library/TrEEPpVendorLib.h > > index d3a0767..f0dcfd9 100644 > > --- a/SecurityPkg/Include/Library/TrEEPpVendorLib.h > > +++ b/SecurityPkg/Include/Library/TrEEPpVendorLib.h > > @@ -45,8 +45,8 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF > ANY KIND, EITHER EXPRESS OR IMPLIED. > > #define TREE_PP_OPERATION_RESPONSE_BIOS_FAILURE 0xFFFFFFF1 > > > > // > > -// The return code for Sumbit TPM Request to Pre-OS Environment > > -// and Sumbit TPM Request to Pre-OS Environment 2 > > +// The return code for Submit TPM Request to Pre-OS Environment > > +// and Submit TPM Request to Pre-OS Environment 2 > > // > > #define TREE_PP_SUBMIT_REQUEST_TO_PREOS_SUCCESS > 0 > > #define TREE_PP_SUBMIT_REQUEST_TO_PREOS_NOT_IMPLEMENTED > 1 > > diff --git > a/SecurityPkg/Library/DxeDeferImageLoadLib/DxeDeferImageLoadLib.c > b/SecurityPkg/Library/DxeDeferImageLoadLib/DxeDeferImageLoadLib.c > > index 142b546..02a87f9 100644 > > --- a/SecurityPkg/Library/DxeDeferImageLoadLib/DxeDeferImageLoadLib.c > > +++ > b/SecurityPkg/Library/DxeDeferImageLoadLib/DxeDeferImageLoadLib.c > > @@ -469,7 +469,7 @@ Done: > > @param[in] DevicePath Points to device path. > > @param[in] AccessType The type of user access control. > > > > - @retval TURE The DevicePath is in the specified List. > > + @retval TRUE The DevicePath is in the specified List. > > @retval FALSE The DevicePath is not in the specified List. > > > > **/ > > @@ -512,7 +512,7 @@ IsDevicePathInList ( > > > > @param[in] DevicePath Points to device path > > > > - @retval TURE The image pointed by DevicePath is permitted = to load. > > + @retval TRUE The image pointed by DevicePath is permitted = to load. > > @retval FALSE The image pointed by DevicePath is forbidden = to load. > > > > **/ > > @@ -545,7 +545,7 @@ VerifyDevicePath ( > > > > @param[in] DevicePath Points to device path. > > > > - @retval TURE The image pointed by DevicePath is a boot opt= ion. > > + @retval TRUE The image pointed by DevicePath is a boot opt= ion. > > @retval FALSE The image pointed by DevicePath is not a boot= option. > > > > **/ > > diff --git > a/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa20 > 48Sha256GuidedSectionExtractLib.c > b/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa20 > 48Sha256GuidedSectionExtractLib.c > > index 5f5d242..39768fb 100644 > > --- > a/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa20 > 48Sha256GuidedSectionExtractLib.c > > +++ > b/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa20 > 48Sha256GuidedSectionExtractLib.c > > @@ -59,7 +59,7 @@ CONST UINT8 mRsaE[] =3D { 0x01, 0x00, 0x01 }; > > @param SectionAttribute The attribute of the input guided section. > > > > @retval EFI_SUCCESS The size of destination buffer, the s= ize of > scratch buffer and > > - the attribute of the input section ar= e successull retrieved. > > + the attribute of the input section ar= e successfully > retrieved. > > @retval EFI_INVALID_PARAMETER The GUID in InputSection does not > match this instance guid. > > > > **/ > > diff --git > a/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa204 > 8Sha256GuidedSectionExtractLib.c > b/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa204 > 8Sha256GuidedSectionExtractLib.c > > index 4733703..ba1c700 100644 > > --- > a/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa204 > 8Sha256GuidedSectionExtractLib.c > > +++ > b/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa204 > 8Sha256GuidedSectionExtractLib.c > > @@ -57,7 +57,7 @@ CONST UINT8 mRsaE[] =3D { 0x01, 0x00, 0x01 }; > > @param SectionAttribute The attribute of the input guided section. > > > > @retval EFI_SUCCESS The size of destination buffer, the s= ize of > scratch buffer and > > - the attribute of the input section ar= e successull retrieved. > > + the attribute of the input section ar= e successfully > retrieved. > > @retval EFI_INVALID_PARAMETER The GUID in InputSection does not > match this instance guid. > > > > **/ > > diff --git a/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageUtil.c > b/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageUtil.c > > index 30808cd..286f5b6 100644 > > --- a/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageUtil.c > > +++ b/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageUtil.c > > @@ -881,7 +881,7 @@ TcgIsProtocolSupported( > > > > @param Discovery > > > > - @retval TURE if lock, FALSE if not lock. > > + @retval TRUE if lock, FALSE if not lock. > > **/ > > BOOLEAN > > EFIAPI > > diff --git a/SecurityPkg/Library/TcgStorageOpalLib/TcgStorageOpalCore.c > b/SecurityPkg/Library/TcgStorageOpalLib/TcgStorageOpalCore.c > > index a0eac33..90cc51a 100644 > > --- a/SecurityPkg/Library/TcgStorageOpalLib/TcgStorageOpalCore.c > > +++ b/SecurityPkg/Library/TcgStorageOpalLib/TcgStorageOpalCore.c > > @@ -1055,7 +1055,7 @@ OpalGlobalLockingRangeGenKey( > > /** > > > > The function updates the ReadLocked and WriteLocked columns of the > Global Locking Range. > > - This funciton is required for a user1 authority, since a user1 autho= rity shall > only have access to ReadLocked and WriteLocked columns > > + This function is required for a user1 authority, since a user1 autho= rity > shall only have access to ReadLocked and WriteLocked columns > > (not ReadLockEnabled and WriteLockEnabled columns). > > > > @param[in] LockingSpSession OPAL_SESSION with > OPAL_UID_LOCKING_SP to generate key > > diff --git a/SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalDriver.c > b/SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalDriver.c > > index 9bf1ced..bd12d5a 100644 > > --- a/SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalDriver.c > > +++ b/SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalDriver.c > > @@ -406,7 +406,7 @@ OpalDriverGetDeviceList( > > ReadyToBoot callback to send BlockSid command. > > > > @param Event Pointer to this event > > - @param Context Event hanlder private Data > > + @param Context Event handler private Data > > > > **/ > > VOID > > diff --git a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c > b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c > > index db8d662..804929e 100644 > > --- a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c > > +++ b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c > > @@ -2073,7 +2073,7 @@ OnReadyToBoot ( > > for (PcrIndex =3D 0; PcrIndex < 7; PcrIndex++) { > > Status =3D MeasureSeparatorEvent (PcrIndex); > > if (EFI_ERROR (Status)) { > > - DEBUG ((EFI_D_ERROR, "Seperator Event not Measured. Error!\n")= ); > > + DEBUG ((EFI_D_ERROR, "Separator Event not Measured. Error!\n")= ); > > } > > } > > > > diff --git a/SecurityPkg/Tcg/TcgDxe/TcgDxe.c > b/SecurityPkg/Tcg/TcgDxe/TcgDxe.c > > index 0d7e32a..3e6d87b 100644 > > --- a/SecurityPkg/Tcg/TcgDxe/TcgDxe.c > > +++ b/SecurityPkg/Tcg/TcgDxe/TcgDxe.c > > @@ -1160,7 +1160,7 @@ OnReadyToBoot ( > > for (PcrIndex =3D 0; PcrIndex < 8; PcrIndex++) { > > Status =3D MeasureSeparatorEvent (PcrIndex); > > if (EFI_ERROR (Status)) { > > - DEBUG ((EFI_D_ERROR, "Seperator Event not Measured. Error!\n")= ); > > + DEBUG ((EFI_D_ERROR, "Separator Event not Measured. Error!\n")= ); > > } > > } > > > > diff --git a/SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c > b/SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c > > index fb69fa1..fa51b64 100644 > > --- a/SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c > > +++ b/SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c > > @@ -1500,7 +1500,7 @@ OnReadyToBoot ( > > for (PcrIndex =3D 0; PcrIndex < 7; PcrIndex++) { > > Status =3D MeasureSeparatorEvent (PcrIndex); > > if (EFI_ERROR (Status)) { > > - DEBUG ((EFI_D_ERROR, "Seperator Event not Measured. Error!\n")= ); > > + DEBUG ((EFI_D_ERROR, "Separator Event not Measured. Error!\n")= ); > > } > > } > > > > diff --git > a/SecurityPkg/UserIdentification/UserProfileManagerDxe/ModifyIdentityPo > licy.c > b/SecurityPkg/UserIdentification/UserProfileManagerDxe/ModifyIdentityPo > licy.c > > index 9f157f8..8382744 100644 > > --- > a/SecurityPkg/UserIdentification/UserProfileManagerDxe/ModifyIdentityPo > licy.c > > +++ > b/SecurityPkg/UserIdentification/UserProfileManagerDxe/ModifyIdentityPo > licy.c > > @@ -409,7 +409,7 @@ CheckNewIdentityPolicy ( > > /** > > Save the identity policy and update UI with it. > > > > - This funciton will verify the new identity policy, in current implem= entation, > > + This function will verify the new identity policy, in current > implementation, > > the identity policy can be: T, P & P & P & ..., P | P | P | ... > > Here, "T" means "True", "P" means "Credential Provider", "&" means > "and", "|" means "or". > > Other identity policies are not supported. > > diff --git > a/SecurityPkg/UserIdentification/UserProfileManagerDxe/UserProfileMana > ger.h > b/SecurityPkg/UserIdentification/UserProfileManagerDxe/UserProfileMana > ger.h > > index afa420e..6491dc2 100644 > > --- > a/SecurityPkg/UserIdentification/UserProfileManagerDxe/UserProfileMana > ger.h > > +++ > b/SecurityPkg/UserIdentification/UserProfileManagerDxe/UserProfileMana > ger.h > > @@ -254,7 +254,7 @@ AddIdentityPolicyItem ( > > /** > > Save the identity policy and update UI with it. > > > > - This funciton will verify the new identity policy, in current implem= entation, > > + This function will verify the new identity policy, in current > implementation, > > the identity policy can be: T, P & P & P & ..., P | P | P | ... > > Here, "T" means "True", "P" means "Credential Provider", "&" means > "and", "|" means "or". > > Other identity policies are not supported. > > -- > > 2.10.1 > > > > _______________________________________________ > > edk2-devel mailing list > > edk2-devel@lists.01.org > > https://lists.01.org/mailman/listinfo/edk2-devel > > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel